| Commit message (Collapse) | Author | Age | Lines | ||
|---|---|---|---|---|---|
| ... | |||||
| | | | * | | | | | | | | | | Avoid instantiating the cog when testing static/class methods | 2020-05-11 | -10/+5 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Correct the return type annotation for the autospec decorator | 2020-05-11 | -2/+2 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Test token regex matches valid tokens | 2020-05-11 | -0/+21 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Test is_maybe_token | 2020-05-11 | -7/+24 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Test `is_maybe_token` returns False for missing parts | 2020-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 attributes | 2020-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 tokens | 2020-05-11 | -7/+25 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Test `find_token_in_message` returns the found token | 2020-05-11 | -0/+24 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Test `find_token_in_message` returns None if no matches found | 2020-05-11 | -0/+14 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Fix test for token remover ignoring bot messages | 2020-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 decorator | 2020-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 tokens | 2020-05-11 | -0/+11 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Test token remover takes action if a token is found | 2020-05-11 | -1/+13 | ||
| | | | | | | | | | | | | | | |||||
| | | | * | | | | | | | | | | Add a test helper function to patch multiple attributes with autospecs | 2020-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_message | 2020-05-11 | -2/+10 | ||
| | | | | |_|_|_|_|_|_|/ / | | | |/| | | | | | | | | |||||
| | | * | | | | | | | | | | Merge pull request #978 from ItsDrike/unsilence-scheduler | 2020-06-12 | -4/+14 | ||
| | | |\ \ \ \ \ \ \ \ \ \ | | | |_|_|_|_|_|_|_|/ / | | |/| | | | | | | | | | Use Scheduler instead of asyncio.sleep on silence cog | ||||
| | | | * | | | | | | | | | Test for channel not silenced message | 2020-06-06 | -4/+14 | ||
| | | | | |_|_|_|/ / / / | | | |/| | | | | | | | |||||
| | | * | | | | | | | | | Merge branch 'master' into bad-words-nick | 2020-06-11 | -60/+317 | ||
| | | |\ \ \ \ \ \ \ \ \ | | | | |_|_|_|_|_|/ / | | | |/| | | | | | | | |||||
| | | * | | | | | | | | | Remove redis namespace collision test | 2020-05-31 | -10/+0 | ||
| | | | |_|_|_|_|_|_|/ | | |/| | | | | | | | |||||
| | * | | | | | | | | | In LinePaginator, truncate words that exceed scale_to_size | 2020-06-26 | -7/+5 | ||
| | | | | | | | | | | | |||||
| | * | | | | | | | | | Update LinePaginator.add_line() tests | 2020-06-26 | -5/+5 | ||
| | | | | | | | | | | | |||||
| | * | | | | | | | | | Merge branch 'master' into issue926-paginate-long-lines | 2020-06-04 | -40/+645 | ||
| | |\ \ \ \ \ \ \ \ \ | | | |_|_|_|/ / / / | | |/| | | | | | | | |||||
| | * | | | | | | | | | Improve LinePaginator to support long lines | 2020-06-04 | -7/+34 | ||
| | | |_|_|_|/ / / / | |/| | | | | | | | |||||
| * | | | | | | | | | Apply recent changes of notify infraction to test | 2020-09-20 | -3/+2 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Try to fix location from where post infraction test get ID | 2020-09-20 | -2/+2 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Restore newlines for `notify_infraction` embed description | 2020-07-06 | -11/+11 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Truncate reason instead full content to avoid removing newlines | ||||
| * | | | | | | | | | Remove unnecessary if statement from send_private_embed test | 2020-06-25 | -2/+1 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Reorder tests order to match with original file | 2020-06-24 | -65/+65 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Move infraction description template back to main file, apply it there | 2020-06-24 | -11/+5 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Fix test case key name in `test_notify_infraction` | 2020-06-24 | -1/+1 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Add other parameters to `test_post_user` `not_user` mock | 2020-06-24 | -5/+5 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Remove unnecessary `log.debug` assert in `test_post_user` | 2020-06-24 | -8/+1 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Merge 2 if-else branches is `test_post_user` | 2020-06-24 | -5/+2 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Replace `True` test with `In` test on `test_post_user` | 2020-06-24 | -1/+1 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Sync keys, variable names and kwargs in `test_post_user` | 2020-06-24 | -12/+12 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Remove case variable definitions in `test_notify_infraction` | 2020-06-24 | -10/+6 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Mod Utils Tests: Remove unnecessary `user` from test name | 2020-06-24 | -1/+1 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Mod Utils Tests: Change `True` assert to `In` assert for message check | 2020-06-24 | -2/+3 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Mod Utils Tests: Move some test cases to `namedtuple` | 2020-06-24 | -66/+29 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Mod Utils Tests: Move INFRACTION_DESCRIPTION_TEMPLATE to tests file | 2020-06-24 | -5/+11 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Increase coverage of moderation utils tests | 2020-06-12 | -5/+36 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Implement description shortening to infraction notify tests | 2020-06-12 | -8/+27 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Remove deprecated avatar hash in `test_post_user` | 2020-06-12 | -3/+1 | ||
| | | | | | | | | | | |||||
| * | | | | | | | | | Merge branch 'master' into mod-utils-tests | 2020-06-11 | -95/+715 | ||
| |\ \ \ \ \ \ \ \ \ | | |_|_|/ / / / / | |/| | | | | | | | |||||
| | * | | | | | | | | Merge branch 'master' into bug/backend/906/converter-year-range | 2020-06-10 | -47/+824 | ||
| | |\ \ \ \ \ \ \ \ | | | |_|_|_|_|_|/ | | |/| | | | | | | |||||
| | | * | | | | | | | Merge branch 'master' into staff_count_server | 2020-06-05 | -5/+0 | ||
| | | |\ \ \ \ \ \ \ | | | | |/ / / / / | | | |/| | | | | | |||||
| | | | * | | | | | | Merge branch 'master' into stop_sending_avatar_hash | 2020-05-31 | -31/+638 | ||
| | | | |\ \ \ \ \ \ | | | | | |_|/ / / | | | | |/| | | | | |||||
| | | | * | | | | | | Fix failing tests related to avatar_hash | 2020-05-27 | -5/+0 | ||
| | | | | | | | | | | |||||
| | | * | | | | | | | Merge branch 'master' into staff_count_server | 2020-05-31 | -0/+247 | ||
| | | |\ \ \ \ \ \ \ | | | | |/ / / / / | | | |/| | | | | | |||||
| | | | * | | | | | | Merge branch 'master' into ban-kick-reason-length | 2020-05-30 | -31/+550 | ||
| | | | |\ \ \ \ \ \ | |||||