aboutsummaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeLines
...
| | | * | | | | | | | | | Avoid instantiating the cog when testing static/class methodsGravatar MarkKoz2020-05-11-10/+5
| | | | | | | | | | | | |
| | | * | | | | | | | | | Correct the return type annotation for the autospec decoratorGravatar MarkKoz2020-05-11-2/+2
| | | | | | | | | | | | |
| | | * | | | | | | | | | Test token regex matches valid tokensGravatar MarkKoz2020-05-11-0/+21
| | | | | | | | | | | | |
| | | * | | | | | | | | | Test is_maybe_tokenGravatar MarkKoz2020-05-11-7/+24
| | | | | | | | | | | | |
| | | * | | | | | | | | | Test `is_maybe_token` returns False for missing partsGravatar MarkKoz2020-05-11-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In practice, this won't ever happen since the regex wouldn't match strings with missing parts. However, the function does check it so may as well test it. It's not necessarily bound to always use inputs from the regex either I suppose.
| | | * | | | | | | | | | Fix autospec decorator when used with multiple attributesGravatar MarkKoz2020-05-11-16/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original approach of messing with the `attribute_name` didn't work for reasons I won't discuss here (would require knowledge of patcher internals). The new approach doesn't use patch.multiple but mimics it by applying multiple patch decorators to the function. As a consequence, this can no longer be used as a context manager.
| | | * | | | | | | | | | Test token regex doesn't match invalid tokensGravatar MarkKoz2020-05-11-7/+25
| | | | | | | | | | | | |
| | | * | | | | | | | | | Test `find_token_in_message` returns the found tokenGravatar MarkKoz2020-05-11-0/+24
| | | | | | | | | | | | |
| | | * | | | | | | | | | Test `find_token_in_message` returns None if no matches foundGravatar MarkKoz2020-05-11-0/+14
| | | | | | | | | | | | |
| | | * | | | | | | | | | Fix test for token remover ignoring bot messagesGravatar MarkKoz2020-05-11-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's not possible to test this via asserting the return value of `on_message` since it never returns anything. Instead, the actual relevant unit, `find_token_in_message,` should be tested.
| | | * | | | | | | | | | Allow using arbitrary parameter names with the autospec decoratorGravatar MarkKoz2020-05-11-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This gives the caller more flexibility. Sometimes attribute names are too long or they don't follow a naming scheme accepted by the linter.
| | | * | | | | | | | | | Test token remover skips messages without tokensGravatar MarkKoz2020-05-11-0/+11
| | | | | | | | | | | | |
| | | * | | | | | | | | | Test token remover takes action if a token is foundGravatar MarkKoz2020-05-11-1/+13
| | | | | | | | | | | | |
| | | * | | | | | | | | | Add a test helper function to patch multiple attributes with autospecsGravatar MarkKoz2020-05-11-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This helper reduces redundancy/boilerplate by setting default values. It also has the consequence of shortening the length of the invocation, which makes it faster to use and easier to read.
| | | * | | | | | | | | | Test on_message_edit of token remover uses on_messageGravatar MarkKoz2020-05-11-2/+10
| | | | |_|_|_|_|_|_|/ / | | | |/| | | | | | | |
| | * | | | | | | | | | Merge pull request #978 from ItsDrike/unsilence-schedulerGravatar Mark2020-06-12-4/+14
| | |\ \ \ \ \ \ \ \ \ \ | | | |_|_|_|_|_|_|_|/ / | | |/| | | | | | | | | Use Scheduler instead of asyncio.sleep on silence cog
| | | * | | | | | | | | Test for channel not silenced messageGravatar ItsDrike2020-06-06-4/+14
| | | | |_|_|_|/ / / / | | | |/| | | | | | |
| | * | | | | | | | | Merge branch 'master' into bad-words-nickGravatar ks1292020-06-11-60/+317
| | |\ \ \ \ \ \ \ \ \ | | | | |_|_|_|_|_|/ / | | | |/| | | | | | |
| | * | | | | | | | | Remove redis namespace collision testGravatar MarkKoz2020-05-31-10/+0
| | | |_|_|_|_|_|_|/ | | |/| | | | | | |
| * | | | | | | | | In LinePaginator, truncate words that exceed scale_to_sizeGravatar Kyle Stanley2020-06-26-7/+5
| | | | | | | | | |
| * | | | | | | | | Update LinePaginator.add_line() testsGravatar Kyle Stanley2020-06-26-5/+5
| | | | | | | | | |
| * | | | | | | | | Merge branch 'master' into issue926-paginate-long-linesGravatar Kyle Stanley2020-06-04-40/+645
| |\ \ \ \ \ \ \ \ \ | | | |_|_|_|/ / / / | | |/| | | | | | |
| * | | | | | | | | Improve LinePaginator to support long linesGravatar Kyle Stanley2020-06-04-7/+34
| | |_|_|_|/ / / / | |/| | | | | | |
* | | | | | | | | Apply recent changes of notify infraction to testGravatar ks1292020-09-20-3/+2
| | | | | | | | |
* | | | | | | | | Try to fix location from where post infraction test get IDGravatar ks1292020-09-20-2/+2
| | | | | | | | |
* | | | | | | | | Restore newlines for `notify_infraction` embed descriptionGravatar Karlis S2020-07-06-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Truncate reason instead full content to avoid removing newlines
* | | | | | | | | Remove unnecessary if statement from send_private_embed testGravatar ks1292020-06-25-2/+1
| | | | | | | | |
* | | | | | | | | Reorder tests order to match with original fileGravatar ks1292020-06-24-65/+65
| | | | | | | | |
* | | | | | | | | Move infraction description template back to main file, apply it thereGravatar ks1292020-06-24-11/+5
| | | | | | | | |
* | | | | | | | | Fix test case key name in `test_notify_infraction`Gravatar ks1292020-06-24-1/+1
| | | | | | | | |
* | | | | | | | | Add other parameters to `test_post_user` `not_user` mockGravatar ks1292020-06-24-5/+5
| | | | | | | | |
* | | | | | | | | Remove unnecessary `log.debug` assert in `test_post_user`Gravatar ks1292020-06-24-8/+1
| | | | | | | | |
* | | | | | | | | Merge 2 if-else branches is `test_post_user`Gravatar ks1292020-06-24-5/+2
| | | | | | | | |
* | | | | | | | | Replace `True` test with `In` test on `test_post_user`Gravatar ks1292020-06-24-1/+1
| | | | | | | | |
* | | | | | | | | Sync keys, variable names and kwargs in `test_post_user`Gravatar ks1292020-06-24-12/+12
| | | | | | | | |
* | | | | | | | | Remove case variable definitions in `test_notify_infraction`Gravatar ks1292020-06-24-10/+6
| | | | | | | | |
* | | | | | | | | Mod Utils Tests: Remove unnecessary `user` from test nameGravatar ks1292020-06-24-1/+1
| | | | | | | | |
* | | | | | | | | Mod Utils Tests: Change `True` assert to `In` assert for message checkGravatar ks1292020-06-24-2/+3
| | | | | | | | |
* | | | | | | | | Mod Utils Tests: Move some test cases to `namedtuple`Gravatar ks1292020-06-24-66/+29
| | | | | | | | |
* | | | | | | | | Mod Utils Tests: Move INFRACTION_DESCRIPTION_TEMPLATE to tests fileGravatar ks1292020-06-24-5/+11
| | | | | | | | |
* | | | | | | | | Increase coverage of moderation utils testsGravatar ks1292020-06-12-5/+36
| | | | | | | | |
* | | | | | | | | Implement description shortening to infraction notify testsGravatar ks1292020-06-12-8/+27
| | | | | | | | |
* | | | | | | | | Remove deprecated avatar hash in `test_post_user`Gravatar ks1292020-06-12-3/+1
| | | | | | | | |
* | | | | | | | | Merge branch 'master' into mod-utils-testsGravatar ks1292020-06-11-95/+715
|\ \ \ \ \ \ \ \ \ | | |_|_|/ / / / / | |/| | | | | | |
| * | | | | | | | Merge branch 'master' into bug/backend/906/converter-year-rangeGravatar Joseph Banks2020-06-10-47/+824
| |\ \ \ \ \ \ \ \ | | | |_|_|_|_|_|/ | | |/| | | | | |
| | * | | | | | | Merge branch 'master' into staff_count_serverGravatar kwzrd2020-06-05-5/+0
| | |\ \ \ \ \ \ \ | | | | |/ / / / / | | | |/| | | | |
| | | * | | | | | Merge branch 'master' into stop_sending_avatar_hashGravatar Leon Sandøy2020-05-31-31/+638
| | | |\ \ \ \ \ \ | | | | | |_|/ / / | | | | |/| | | |
| | | * | | | | | Fix failing tests related to avatar_hashGravatar Leon Sandøy2020-05-27-5/+0
| | | | | | | | |
| | * | | | | | | Merge branch 'master' into staff_count_serverGravatar Leon Sandøy2020-05-31-0/+247
| | |\ \ \ \ \ \ \ | | | | |/ / / / / | | | |/| | | | |
| | | * | | | | | Merge branch 'master' into ban-kick-reason-lengthGravatar Mark2020-05-30-31/+550
| | | |\ \ \ \ \ \