| Commit message (Collapse) | Author | Age | Lines | ||
|---|---|---|---|---|---|
| ... | |||||
| * | | | Adjust tests for new calling behaviour. |  Numerlor | 2020-03-15 | -2/+2 | |
| | | | | | | | | | | | | | | | | `.set_permissions` calls were changed to use kwargs directly instead of an overwrite, this reflects the changes in tests. | ||||
| * | | | Use dict instead of a set and custom class. | Numerlor | 2020-03-15 | -25/+3 | |
| | | | | | | | | | | | | | | | | The FirstHash class is no longer necessary with only channels and the current loop in tuples. FirstHash was removed, along with its tests and tests were adjusted for new dict behaviour. | ||||
| * | | | Add tests for `_notifier`. | Numerlor | 2020-03-11 | -0/+19 | |
| | | | | |||||
| * | | | Add tests for `remove_channel`. | Numerlor | 2020-03-11 | -0/+18 | |
| | | | | |||||
| * | | | Remove `_current_loop` from setup. | Numerlor | 2020-03-11 | -2/+1 | |
| | | | | |||||
| * | | | Add tests for `add_channel`. | Numerlor | 2020-03-11 | -0/+21 | |
| | | | | |||||
| * | | | Create test case for `SilenceNotifier` | Numerlor | 2020-03-11 | -1/+7 | |
| | | | | |||||
| * | | | Add tests for `_get_instance_vars`. | Numerlor | 2020-03-11 | -1/+28 | |
| | | | | |||||
| * | | | Pass created channel instead of new object. | Numerlor | 2020-03-11 | -1/+1 | |
| | | | | | | | | | | | | | Creating a new object caused the assert to fail because different objects were used. | ||||
| * | | | Reset `self.ctx` call history after every subtest. | Numerlor | 2020-03-11 | -0/+1 | |
| | | | | |||||
| * | | | Fix erroneous `assert_called_once_with` calls. | Numerlor | 2020-03-11 | -8/+7 | |
| | | | | | | | | | | | | | `assert_called_once_with` was being tested on call_args which always reported success.st. | ||||
| * | | | Add test for `cog_check`. | Numerlor | 2020-03-11 | -0/+7 | |
| | | | | |||||
| * | | | Add test for `cog_unload`. | Numerlor | 2020-03-11 | -1/+17 | |
| | | | | |||||
| * | | | Use patch decorators and assign names from `with` patches. | Numerlor | 2020-03-11 | -11/+11 | |
| | | | | |||||
| * | | | Separate tests for permissions and `muted_channels.add` on `_silence`. | Numerlor | 2020-03-11 | -5/+8 | |
| | | | | |||||
| * | | | Add tests for `_unsilence` | Numerlor | 2020-03-11 | -1/+33 | |
| | | | | |||||
| * | | | Use async test case. | Numerlor | 2020-03-11 | -14/+12 | |
| | | | | | | | | | | | | | This allows us to use coroutines with await directly instead of asyncio.run | ||||
| * | | | Add test for `_silence` method. | Numerlor | 2020-03-11 | -1/+34 | |
| | | | | |||||
| * | | | Remove channel mentions from output discord messages. | Numerlor | 2020-03-11 | -4/+4 | |
| | | | | | | | | | | | | | | | | | | | With the removal of the channel args, it's no longer necessary to mention the channel in the command output. Tests adjusted accordingly | ||||
| * | | | Remove unnecessary args from test cases. | Numerlor | 2020-03-11 | -6/+6 | |
| | | | | | | | | | | | | | | | | Needless call args which were constant were kept in the test cases, resulting in redundant code, the args were moved directly into the function call. | ||||
| * | | | Use `patch.object` instead of patch with direct `return_value`. | Numerlor | 2020-03-09 | -9/+2 | |
| | | | | |||||
| * | | | Add test for `unsilence` discord output. | Numerlor | 2020-03-08 | -0/+9 | |
| | | | | |||||
| * | | | Add test for `silence` discord output. | Numerlor | 2020-03-08 | -0/+24 | |
| | | | | |||||
| * | | | Add test case for Silence cog. | Numerlor | 2020-03-08 | -1/+10 | |
| | | | | |||||
| * | | | Add tests for FirstHash class. | Numerlor | 2020-03-08 | -0/+25 | |
| | |/ |/| | |||||
| * | | Modify log test regex to be non-os-specific |  S. Co1 | 2020-03-01 | -1/+1 | |
| | | | | | | | Previous regex utilized a `/`, which doesn't work for comparing against Windows paths, which use `\` | ||||
| * | | Migrate snekbox tests to Python 3.8's unittest |  Sebastiaan Zeeff | 2020-02-29 | -41/+27 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've migrated the `tests/test_snekbox.py` file to use the new Python 3.8-style unittests instead of our old style using our custom Async mocks. In particular, I had to make a few changes: - Mocking the async post() context manager correctly Since `ClientSession.post` returns an async context manager when called, we need to make sure to assign the return value to the __aenter__ method of whatever `post()` returns, not of `post` itself (i.e.. when it's not called). - Use the new AsyncMock assert methods `assert_awaited_once` and `assert_awaited_once_with` Objects of the new `unittest.mock.AsyncMock` class have special methods to assert what they were called with that also assert that specific coroutine object was awaited. This means we test two things in one: Whether or not it was called with the right arguments and whether or not the returned coroutine object was then awaited. - Patch `functools.partial` as `partial` objects are compared by identity When you create two partial functions of the same function, you'll end up with two different `partial` objects. Since `partial` objects are compared by identity, you can't compare a `partial` created in a test method to that created in the callable you're trying to test. They will always compare as `False`. Since we're not interested in actually creating `partial` objects, I've just patched `functools.partial` in the namespace of the module we're testing to make sure we can compare them. | ||||
| * | | Merge branch 'master' into python38-migration | Sebastiaan Zeeff | 2020-02-29 | -23/+376 | |
| |\| | | | | | | | I've resolved the merge conflict by confirming the deleted part of tests/helpers.py | ||||
| | * | Merge branch 'master' into fuzzy_zero_div |  Shirayuki Nekomata | 2020-02-29 | -0/+380 | |
| | |\ | |||||
| | | * | Merge remote-tracking branch 'origin/master' into eval-enhancements |  MarkKoz | 2020-02-28 | -471/+1844 | |
| | | |\ | |||||
| | | * | | Return only the new code in continue_eval and check for truthiness instead |  Matteo | 2020-02-28 | -4/+4 | |
| | | | | | |||||
| | | * | | Use unicode code point instead of literal for the snekbox re-eval emoji | Matteo | 2020-02-27 | -3/+3 | |
| | | | | | | | | | | | | | | | | | Unicode literals aren't really safe compared to code points | ||||
| | | * | | Split the eval command procedure into two functions. | Matteo | 2020-02-27 | -69/+81 | |
| | | | | | | | | | | | | | | | | | Two functions were created: send_eval and continue_eval, in order to facilitate testing. The corresponding tests are also changed in this commit. | ||||
| | | * | | Merge branch 'master' into eval-enhancements | Matteo Bertucci | 2020-02-18 | -1/+8 | |
| | | |\ \ | |||||
| | | * | | | Use kwargs to set mock attributes | Matteo | 2020-02-18 | -25/+11 | |
| | | | | | | |||||
| | | * | | | Assert return value of Snekbox.post_eval | Matteo | 2020-02-18 | -1/+3 | |
| | | | | | | |||||
| | | * | | | Use a space instead of an empty string in test_get_status_emoji | Matteo Bertucci | 2020-02-18 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Because of the stripping, it should still be considered as empty Co-Authored-By: Mark <[email protected]> | ||||
| | | * | | | Delete additional informations from subtest | Matteo Bertucci | 2020-02-18 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce visual clutter Co-Authored-By: Mark <[email protected]> | ||||
| | | * | | | Split assertions onto separate lines | Matteo Bertucci | 2020-02-18 | -4/+8 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Reads better as separate lines Co-Authored-By: Mark <[email protected]> | ||||
| | | * | | | Split assertion onto separate lines | Matteo Bertucci | 2020-02-18 | -1/+2 | |
| | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]> | ||||
| | | * | | | Write tests for bot/cogs/test_snekbox.py |  Akarys42 | 2020-02-09 | -0/+363 | |
| | | | | | | |||||
| | | * | | | Create an AsyncContextManagerMock mock for testing asynchronous context managers | Akarys42 | 2020-02-09 | -0/+12 | |
| | | | | | | | | | | | | | | | | | | | | | It can be used to test aiohttp request functions, since they are async context managers | ||||
| | * | | | | Merge branch 'master' into fuzzy_zero_div | Numerlor | 2020-02-28 | -7/+7 | |
| | |\ \ \ \ | | | |_|/ | | |/| | | | | | | | | | | | | # Conflicts: # bot/cogs/error_handler.py | ||||
| | | * | | | Merge branch 'master' into feat/backend/b496/rename-constants |  Leon Sandøy | 2020-02-28 | -15/+0 | |
| | | |\ \ \ | |||||
| | | * | | | | Fix mismatched constant names in syncer tests | S. Co1 | 2020-02-25 | -2/+2 | |
| | | | | | | | |||||
| | | * | | | | Merge branch 'master' into feat/backend/b496/rename-constants | S. Co1 | 2020-02-25 | -205/+1197 | |
| | | |\ \ \ \ | |||||
| | | * | | | | | Tests: fix setting bot-commands ID in information tests | MarkKoz | 2020-02-23 | -4/+4 | |
| | | | | | | | | |||||
| | | * | | | | | Constants: rename roles to match their names in the guild | MarkKoz | 2020-02-23 | -1/+1 | |
| | | | | | | | | |||||
| | * | | | | | | Merge branch 'master' into fuzzy_zero_div | Sebastiaan Zeeff | 2020-02-27 | -15/+0 | |
| | |\ \ \ \ \ \ | | | |_|/ / / | | |/| | | | | |||||
| | | * | | | | | Merge branch 'master' into channel_constants_check |  Numerlor | 2020-02-25 | -205/+1197 | |
| | | |\ \ \ \ \ | | | | |/ / / | | | |/| | | | |||||
 |
index : bot | |
| A mirrored copy of the python-discord/bot repository. |
| aboutsummaryrefslogtreecommitdiffstats |