aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Token remover: decode ID using URL-safe base64Gravatar MarkKoz2020-05-21-1/+1
| | | | | Though I've not seen an ID with neither + and \ nor - and _, given that the timestamp uses URL-safe encoding, the ID probably does too.
* Add a utility function to pad base64 dataGravatar MarkKoz2020-05-16-2/+8
|
* Token remover: use strict check for digits in token IDGravatar MarkKoz2020-05-15-2/+5
| | | | | | | `isnumeric` would be true for a wide range of characters in Unicode, but the ID must only consist of the characters 0-9 (ASCII digits). In fact, `isdigit` on its own would also match other Unicode characters too.
* Token remover: fix timestamp checkGravatar MarkKoz2020-05-14-10/+16
| | | | | | The timestamp calculation was incorrect. The bytes need to be interpreted as big-endian and the result is just a timestamp rather than a snowflake.
* Token remover: add logs to clarify why token is invalidGravatar MarkKoz2020-05-13-2/+4
|
* Add missing comma to token remover log messageGravatar MarkKoz2020-05-13-1/+1
|
* Fix a test needlessly being a coroutineGravatar MarkKoz2020-05-13-1/+1
|
* Use subtests for valid ID/timestamp tests and test non-ASCII inputsGravatar MarkKoz2020-05-11-18/+25
|
* Clean up token remover test importsGravatar MarkKoz2020-05-11-16/+12
|
* Replace deprecated assertion methodsGravatar MarkKoz2020-05-11-2/+2
|
* Refactor `TokenRemoverSetupTests` and add a more thorough testGravatar MarkKoz2020-05-11-4/+8
| | | | | The test now ensures the cog is instantiated and that the instance is passed as an argument to `add_cog`.
* Test TokenRemover.take_actionGravatar MarkKoz2020-05-11-43/+30
| | | | | | | | * Remove `bot.get_cog` mocks in `setUp` * Mock the logger cause it's easier to assert logs * Remove subtests * Assert helper functions were called * Create an autospec for ModLog
* Test TokenRemover.format_log_messageGravatar MarkKoz2020-05-11-0/+16
|
* Test token remover's message deletionGravatar MarkKoz2020-05-11-0/+9
|
* Simplify token remover's message mockGravatar MarkKoz2020-05-11-9/+3
| | | | | * Rely on default values for the author * Set the content to a non-empty string
* Avoid instantiating the cog when testing static/class methodsGravatar MarkKoz2020-05-11-10/+5
|
* Token remover: use a string template for the log messageGravatar MarkKoz2020-05-11-4/+11
|
* Token remover: split some of `take_action` into separate functionsGravatar MarkKoz2020-05-11-11/+21
|
* 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
|
* Token remover: fix `is_maybe_token` returning None instead of FalseGravatar MarkKoz2020-05-11-0/+2
| | | | | It's annotated as returning a bool and when the split fails it already returns False. To be consistent, it should always return a bool.
* 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
|
* Token remover: reduce duplicated code in `on_message_edit`Gravatar MarkKoz2020-05-11-3/+1
|
* Token remover: catch ValueError when non-ASCII chars are presentGravatar MarkKoz2020-05-11-2/+2
| | | | | | | | | | The token uses base64 and base64 only allows ASCII characters. Thus, if a match has non-ASCII characters, it's not a valid token. Catching the ValueError is simpler than trying to adjust the regex to only match valid base64. Fixes #928 Fixes BOT-3X
* Merge pull request #924 from ↵Gravatar Daniel Brown2020-05-11-20/+58
|\ | | | | | | | | python-discord/feature/hemlock/perma-ban-override-temp Perma Bans now Overwrite Temp Bans
| * Merge branch 'master' into feature/hemlock/perma-ban-override-tempGravatar Daniel Brown2020-05-11-58/+34
| |\ | |/ |/|
* | Merge pull request #938 from Suhail6inkling/remind_aliasGravatar Leon Sandøy2020-05-11-1/+1
|\ \ | | | | | | Add remindme alias for the remind command
| * | Add remindme alias for the remind commandGravatar Suhail2020-05-10-1/+1
|/ /
* | Merge pull request #923 from python-discord/feat/util/remove-mention-cmdGravatar Daniel Brown2020-05-08-56/+2
|\ \ | | | | | | Remove the mention command and configuration settings for it
| * \ Merge branch 'master' into feat/util/remove-mention-cmdGravatar Daniel Brown2020-05-08-1/+31
| |\ \ | |/ / |/| |
* | | Merge pull request #903 from python-discord/bug/backend/894/win-selector-loopGravatar Daniel Brown2020-05-07-1/+7
|\ \ \ | | | | | | | | Use selector event loop on Windows
| * \ \ Merge branch 'master' into bug/backend/894/win-selector-loopGravatar Daniel Brown2020-05-07-136/+405
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #919 from ↵Gravatar Sebastiaan Zeeff2020-05-05-0/+12
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/bug/backend/911/log-listener-exceptions Log unhandled errors from event listeners
| * \ \ \ Merge branch 'master' into bug/backend/911/log-listener-exceptionsGravatar Sebastiaan Zeeff2020-05-05-3/+16
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #925 from Savant-Dev/antimalwareGravatar Daniel Brown2020-05-05-0/+12
|\ \ \ \ \ | | | | | | | | | | | | Update to Antimalware Filter (.txt uploads)
| * \ \ \ \ Merge branch 'master' into antimalwareGravatar Daniel Brown2020-05-05-2/+2
| |\ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Update antimalware to filter txt files in cases where messages were longer ↵Gravatar Savant-Dev2020-05-04-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | than 2000 chars
| * | | | | Update extension filter to distinguish .txt in cases where messages are ↵Gravatar Savant-Dev2020-05-01-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | longer than 2000 characters
| * | | | | Update extension filter to distinguish .txt in cases where messages are ↵Gravatar Savant-Dev2020-05-01-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | longer than 2000 characters
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-29 23:07:36 +0000