aboutsummaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeLines
...
| * | (Information Cog, !roles command test): Removed 's' at end of "Total 1 ↵Gravatar ks1232020-03-05-1/+1
| | | | | | | | | | | | role(s)" due changes in command.
| * | (Information Cog, !roles command test): Applied empty parameter change.Gravatar ks1232020-03-05-1/+1
| | |
| * | Merge branch 'master' into roles-command-paginationGravatar S. Co12020-03-03-512/+196
| |\|
| * | !roles Command Test: Applied !roles command changesGravatar Karlis. S2020-03-01-3/+2
| | |
* | | Add tests for `HushDurationConverter`.Gravatar Numerlor2020-03-17-0/+30
| | |
* | | Add tests ensuring permissions get preserved.Gravatar Numerlor2020-03-17-0/+31
| | |
* | | Use set `discard` instead of `remove`.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | | | | Discard ignores non present values, allowing us to skip the KeyError suppress.
* | | Add docstring to test.Gravatar Numerlor2020-03-17-0/+1
| | |
* | | Fix test name.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | `removed` was describing the opposite behaviour.
* | | Remove `alert_channel` mention from docstring.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | | | | After removing the optional channel arg and changing output message channels we're only testing `ctx`'s `send`.
* | | Remove one indentation level.Gravatar Numerlor2020-03-17-3/+3
| | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
* | | Rename `cog_unload` tests.Gravatar Numerlor2020-03-17-2/+2
| | | | | | | | | | | | Previous names were undescriptive from testing phases.
* | | Fix `test_cog_unload` passing tests with invalid values.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | | | | | | | | | | The first assert - `asyncio_mock.create_task.assert_called_once_with` called `alert_channel`'s send resulting in an extra call. `send` on `alert_channel` was not tested properly because of a typo and a missing assert in the method call.
* | | Set `_get_instance_vars_event` in test's `setUp`.Gravatar Numerlor2020-03-15-0/+2
| | |
* | | Adjust tests for new calling behaviour.Gravatar Numerlor2020-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.Gravatar Numerlor2020-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`.Gravatar Numerlor2020-03-11-0/+19
| | |
* | | Add tests for `remove_channel`.Gravatar Numerlor2020-03-11-0/+18
| | |
* | | Remove `_current_loop` from setup.Gravatar Numerlor2020-03-11-2/+1
| | |
* | | Add tests for `add_channel`.Gravatar Numerlor2020-03-11-0/+21
| | |
* | | Create test case for `SilenceNotifier`Gravatar Numerlor2020-03-11-1/+7
| | |
* | | Add tests for `_get_instance_vars`.Gravatar Numerlor2020-03-11-1/+28
| | |
* | | Pass created channel instead of new object.Gravatar Numerlor2020-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.Gravatar Numerlor2020-03-11-0/+1
| | |
* | | Fix erroneous `assert_called_once_with` calls.Gravatar Numerlor2020-03-11-8/+7
| | | | | | | | | | | | `assert_called_once_with` was being tested on call_args which always reported success.st.
* | | Add test for `cog_check`.Gravatar Numerlor2020-03-11-0/+7
| | |
* | | Add test for `cog_unload`.Gravatar Numerlor2020-03-11-1/+17
| | |
* | | Use patch decorators and assign names from `with` patches.Gravatar Numerlor2020-03-11-11/+11
| | |
* | | Separate tests for permissions and `muted_channels.add` on `_silence`.Gravatar Numerlor2020-03-11-5/+8
| | |
* | | Add tests for `_unsilence`Gravatar Numerlor2020-03-11-1/+33
| | |
* | | Use async test case.Gravatar Numerlor2020-03-11-14/+12
| | | | | | | | | | | | This allows us to use coroutines with await directly instead of asyncio.run
* | | Add test for `_silence` method.Gravatar Numerlor2020-03-11-1/+34
| | |
* | | Remove channel mentions from output discord messages.Gravatar Numerlor2020-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.Gravatar Numerlor2020-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`.Gravatar Numerlor2020-03-09-9/+2
| | |
* | | Add test for `unsilence` discord output.Gravatar Numerlor2020-03-08-0/+9
| | |
* | | Add test for `silence` discord output.Gravatar Numerlor2020-03-08-0/+24
| | |
* | | Add test case for Silence cog.Gravatar Numerlor2020-03-08-1/+10
| | |
* | | Add tests for FirstHash class.Gravatar Numerlor2020-03-08-0/+25
| |/ |/|
* | Modify log test regex to be non-os-specificGravatar S. Co12020-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 unittestGravatar Sebastiaan Zeeff2020-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-migrationGravatar Sebastiaan Zeeff2020-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_divGravatar Shirayuki Nekomata2020-02-29-0/+380
| |\
| | * Merge remote-tracking branch 'origin/master' into eval-enhancementsGravatar MarkKoz2020-02-28-471/+1844
| | |\
| | * | Return only the new code in continue_eval and check for truthiness insteadGravatar Matteo2020-02-28-4/+4
| | | |
| | * | Use unicode code point instead of literal for the snekbox re-eval emojiGravatar Matteo2020-02-27-3/+3
| | | | | | | | | | | | | | | | Unicode literals aren't really safe compared to code points
| | * | Split the eval command procedure into two functions.Gravatar Matteo2020-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-enhancementsGravatar Matteo Bertucci2020-02-18-1/+8
| | |\ \
| | * | | Use kwargs to set mock attributesGravatar Matteo2020-02-18-25/+11
| | | | |
| | * | | Assert return value of Snekbox.post_evalGravatar Matteo2020-02-18-1/+3
| | | | |