| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | (Moderation Utils Tests): Move errors from booleans to actual errors in ↵ | 2020-03-12 | -7/+4 | |
| | | | | | `post_user` test. | |||
| * | (Moderation Utils Tests): Hard-coded `self.ctx` argument to `post_user` ↵ | 2020-03-12 | -5/+5 | |
| | | | | | test, renamed current `args` to `user`, applied this in code. | |||
| * | (Moderation Utils Tests): Fixed formatting in `notify_infraction` test. | 2020-03-12 | -3/+1 | |
| | | ||||
| * | (Moderation Utils Tests): Removed unnecessary symbols from ↵ | 2020-03-12 | -2/+2 | |
| | | | | | `has_active_infraction` test `infraction_nr` variable and changes this to more unique number. | |||
| * | (Moderation Utils Tests): Moved embed generating to test cases loop from ↵ | 2020-03-12 | -10/+10 | |
| | | | | | test cases listing, added icon to test cases in `notify_pardon` test | |||
| * | (Moderation Utils Tests): Added params to variable in ↵ | 2020-03-12 | -5/+7 | |
| | | | | | `has_active_infraction` test. | |||
| * | (Moderation Utils Tests): Create extra new tests set for `post_infraction` ↵ | 2020-03-12 | -103/+60 | |
| | | | | | testing, removed old. | |||
| * | (Moderation Utils Tests): Moved mock resetting to beginning of subtest in ↵ | 2020-03-08 | -4/+2 | |
| | | | | | `post_user` and `send_private_embed` test. | |||
| * | (Moderation Utils Tests): Removed unnecessary `if` check from ↵ | 2020-03-08 | -2/+1 | |
| | | | | | `send_private_embed` test | |||
| * | (Moderation Utils Tests): Hard-coded args for `send_private_embed` test. | 2020-03-08 | -10/+6 | |
| | | ||||
| * | (Moderation Utils Tests): Added new check to `post_user` test (`ctx.send` ↵ | 2020-03-08 | -35/+16 | |
| | | | | | content test), improved test cases. | |||
| * | (Moderation Utils Tests): Moved `expected_output` from `Dict` to ↵ | 2020-03-08 | -15/+9 | |
| | | | | | `discord.Embed` in `notify_pardon` test. | |||
| * | (Moderation Utils Tests): Removed unnecessary `ctx.send` `side_effect` from ↵ | 2020-03-08 | -11/+4 | |
| | | | | | `notify_pardon`, applied changes to test cases. | |||
| * | (Moderation Utils Tests): Moved `notify_infraction` embed check from dict to ↵ | 2020-03-08 | -28/+43 | |
| | | | | | `Embed`. | |||
| * | (Moderation Utils Tests): Removed unnecessary `ctx.send` mock resetting, ↵ | 2020-03-08 | -4/+1 | |
| | | | | | moved `send_private_embed` mock reset to beginning of subtest. | |||
| * | (Moderation Utils Tests): Removed unnecessary `ctx.send` `side_effect` and ↵ | 2020-03-08 | -12/+6 | |
| | | | | | removed these in test cases too in `notify_infraction` test. | |||
| * | (Moderation Utils Tests): Removed `Dict` unpacking in `notify_infraction` test. | 2020-03-08 | -20/+20 | |
| | | ||||
| * | (Moderation Utils Tests): Cleaned up `has_active_infraction` test cases, ↵ | 2020-03-08 | -30/+11 | |
| | | | | | hard-coded args, moved mocks resetting to beginning of subtest, added `ctx.send` check only is infraction nr and type in sent string. | |||
| * | (Moderation Utils Tests): Hard-coded API get request params for ↵ | 2020-03-08 | -13/+6 | |
| | | | | | `has_active_infraction` test. | |||
| * | (Moderation Utils Tests): Removed large `utils` parts import, use import ↵ | 2020-03-08 | -12/+10 | |
| | | | | | `utils` instead and added `utils` before variables and function that was imported directly before. | |||
| * | (Moderation Utils Tests): Fixed docstrings, added more information to these. | 2020-03-08 | -6/+8 | |
| | | ||||
| * | (Moderation Utils Tests): Fixed errors, added checks before assertions for ↵ | 2020-03-05 | -1/+4 | |
| | | | | | errors | |||
| * | (Moderation Utils Tests): Added additional assertions to `post_infraction` test. | 2020-03-05 | -8/+54 | |
| | | ||||
| * | (Moderation Utils Tests): Added `user.send` call assertion to ↵ | 2020-03-05 | -0/+2 | |
| | | | | | `test_send_private_embed`. | |||
| * | (Moderation Utils Tests): Added API POST call assertion to `test_post_user`. | 2020-03-05 | -5/+24 | |
| | | ||||
| * | (Moderation Utils Tests): Added new assertions to `notify_pardon`, added ↵ | 2020-03-05 | -6/+24 | |
| | | | | | `ctx.send` raising errors, added check for return values and `send_private_embed` call. | |||
| * | (Moderation Utils Tests): Added new assertions to `post_infraction`, added ↵ | 2020-03-05 | -11/+33 | |
| | | | | | `ctx.send` raising errors, added check for return values and `send_private_embed` call. | |||
| * | (Moderation Utils Tests): Added following new assertions to ↵ | 2020-03-05 | -3/+27 | |
| | | | | | `has_active_infraction` tests: `ctx.send` and `bot.api_client.get` calling. | |||
| * | (Moderation Utils Tests): Use `bot.cogs.moderation.utils`'s `RULES_URL` ↵ | 2020-03-05 | -2/+1 | |
| | | | | | instead creating new one | |||
| * | (Moderation Utils Tests): Removed unnecessary `AsyncMock()` from `__init__` ↵ | 2020-03-05 | -1/+0 | |
| | | | | | (`self.bot.api_client.get`) | |||
| * | (Moderation Utils Tests): Moved `has_active_infraction` tests to one test. | 2020-03-05 | -22/+35 | |
| | | ||||
| * | (Moderation Utils Tests): `send_private_embed` moved exception creating from ↵ | 2020-03-05 | -4/+6 | |
| | | | | | cases testing to test cases listing, added side_effect resetting. | |||
| * | (Moderation Utils Tests): Created tests for `post_infraction` function, ↵ | 2020-03-05 | -1/+68 | |
| | | | | | created __init__.py for moderation tests | |||
| * | (Moderation Utils Tests): Created tests for `send_private_embed` function + ↵ | 2020-03-04 | -3/+44 | |
| | | | | | Fixed errors. | |||
| * | (Moderation Utils Tests): Created tests for `post_user` function. | 2020-03-04 | -1/+59 | |
| | | ||||
| * | (Moderation Utils Tests): Created tests for `notify_pardon` function. | 2020-03-04 | -2/+39 | |
| | | ||||
| * | (Moderation Utils Tests): Created tests for `notify_infraction` function. | 2020-03-04 | -2/+91 | |
| | | ||||
| * | (Moderation Utils Tests): Created tests for `has_active_infraction` function | 2020-03-03 | -0/+24 | |
| | | ||||
| * | (Moderation Utils Tests): Added imports, modified tests class instance and ↵ | 2020-03-03 | -5/+7 | |
| | | | | | created new params for tests class | |||
| * | Created file for moderation utils tests + added setUp to this. | 2020-03-03 | -0/+12 | |
| | | ||||
| * | Modify log test regex to be non-os-specific | 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 | 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 | 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 | 2020-02-29 | -0/+380 | |
| | |\ | ||||
| | | * | Merge remote-tracking branch 'origin/master' into eval-enhancements | 2020-02-28 | -471/+1844 | |
| | | |\ | ||||
| | | * | | Return only the new code in continue_eval and check for truthiness instead | 2020-02-28 | -4/+4 | |
| | | | | | ||||
| | | * | | Use unicode code point instead of literal for the snekbox re-eval emoji | 2020-02-27 | -3/+3 | |
| | | | | | | | | | | | | | | | | | Unicode literals aren't really safe compared to code points | |||
| | | * | | Split the eval command procedure into two functions. | 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 | 2020-02-18 | -1/+8 | |
| | | |\ \ | ||||
| | | * | | | Use kwargs to set mock attributes | 2020-02-18 | -25/+11 | |
| | | | | | | ||||