| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | Fix broken antimalware tests. |  Leon Sandøy | 2020-07-29 | -1/+1 |
| | | ||||
| * | Restructure bot.filter_list_cache. | Leon Sandøy | 2020-07-29 | -5/+5 |
| | | | | | | | | This is an optimization designed to eliminate all the list comprehensions we were doing inside antimalware and filtering. The cache is now structured so that the content is the key and the metadata is the value. | |||
| * | Rename AllowDenyList to FilterLists | Leon Sandøy | 2020-07-27 | -1/+1 |
| | | ||||
| * | Remove AntiMalWare constants, use cache data. | Leon Sandøy | 2020-07-18 | -9/+15 |
| | | | | | Also updates the tests for this cog. | |||
| * | Merge branch 'master' into dm_relay |  Leon Sandøy | 2020-07-15 | -0/+12 |
| |\ | ||||
| | * | edit snekbox tests to work with filtering |  Slushie | 2020-07-13 | -0/+12 |
| | | | ||||
| * | | Fix DuckPond tests now that send_webhook is gone. | Leon Sandøy | 2020-07-12 | -39/+12 |
| |/ | | | | | | | | Some of the tests were failing because they were expecting send_webhook to be a method of the DuckPond cog, other tests simply were no longer applicable, and have been removed. https://github.com/python-discord/bot/issues/667 | |||
| * | Fix imports in slowmode tests |  Den4200 | 2020-07-06 | -3/+3 |
| | | ||||
| * | Improve set_slowmode tests by checking whether the channel was edited | Den4200 | 2020-07-06 | -8/+22 |
| | | ||||
| * | Add multiple test cases for set_slowmode tests | Den4200 | 2020-07-06 | -10/+34 |
| | | ||||
| * | Use local text_channel instead of instance attribute | Den4200 | 2020-07-06 | -13/+10 |
| | | ||||
| * | Add tests for set_slowmode | Den4200 | 2020-07-06 | -0/+20 |
| | | ||||
| * | Add tests for reset_slowmode | Den4200 | 2020-07-06 | -3/+22 |
| | | ||||
| * | Add tests for cog_check and get_slowmode | Den4200 | 2020-07-06 | -0/+37 |
| | | ||||
| * | Merge branch 'master' into issue926-paginate-long-lines |  Kyle Stanley | 2020-06-26 | -191/+514 |
| |\ | ||||
| | * | Merge branch 'master' into bug/backend/bot-50/sync-ignore-guilds |  kwzrd | 2020-06-22 | -10/+85 |
| | |\ | ||||
| | | * | Merge branch 'master' into logging-tests |  Kieran Siek | 2020-06-22 | -200/+1047 |
| | | |\ | ||||
| | | | * | Merge branch 'master' into bug/mod/bot-2a/webhook-clyde | kwzrd | 2020-06-20 | -10/+26 |
| | | | |\ | ||||
| | | | | * | Merge branch 'master' into bug/filters/bot-58/removers-ignore-dms | Dennis Pham | 2020-06-19 | -1/+3 |
| | | | | |\ | ||||
| | | | | | * | Merge branch 'master' into help_channel_rediscache | Kieran Siek | 2020-06-16 | -167/+361 |
| | | | | | |\ | ||||
| | | | | | * \ | Merge branch 'master' into help_channel_rediscache | Leon Sandøy | 2020-06-06 | -9/+8 |
| | | | | | |\ \ | ||||
| | | | | | * | | | Add support for bool values in RedisCache | Leon Sandøy | 2020-05-31 | -1/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're gonna need this for the help channel handling, and it seems like a reasonable type to support anyway. It requires a tiny bit of special handling, but nothing outrageous. | |||
| | | | | * | | | | Token remover: exit early if message already deleted |  MarkKoz | 2020-06-15 | -1/+14 |
| | | | | | | | | | ||||
| | | | | * | | | | Token remover: move bot check to on_message | MarkKoz | 2020-06-15 | -16/+7 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It just makes more sense to me to filter out messages at an earlier stage. | |||
| | | | | * | | | | Token remover: ignore DMs | MarkKoz | 2020-06-15 | -0/+10 |
| | | | | | |_|/ | | | | |/| | | | | | | | | | | | | | | | | | | | | | | | It's a private channel so there's no risk of a token "leaking". Furthermore, messages cannot be deleted in DMs. | |||
| | | | * / | | | Write unit test for `sub_clyde` |  kwzrd | 2020-06-20 | -0/+27 |
| | | | |/ / / | ||||
| | | * | | | | Logging Tests: Simplify `DEBUG_MODE` `False` test |  ks129 | 2020-05-22 | -11/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | - Remove embed attributes checks - Replace `self.dev_log.assert_awaited_once_with` with `self.dev_log.assert_awaited_once`. | |||
| | | * | | | | Merge branch 'master' into logging-tests | ks129 | 2020-05-17 | -10/+178 |
| | | |\ \ \ \ | ||||
| | | * | | | | | Created tests for `bot.cogs.logging` connected message. | ks129 | 2020-04-14 | -0/+42 |
| | | | | | | | | ||||
| | * | | | | | | Sync: fix guild ID check | MarkKoz | 2020-06-17 | -16/+19 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Need to compare the IDs against each other rather than the Guild object against the ID. | |||
| | * | | | | | | Sync: ignore 404s in on_user_update | MarkKoz | 2020-06-14 | -7/+10 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 404s probably mean the user is from another guild. | |||
| | * | | | | | | Sync tests: test listeners ignore events from other guilds | MarkKoz | 2020-06-14 | -10/+54 |
| | | |_|/ / / | |/| | | | | ||||
| | * | | | | | Merge pull request #937 from python-discord/bug/filters/928/non-ascii-token | Mark | 2020-06-12 | -102/+285 |
| | |\ \ \ \ \ | | | | | | | | | | | | | | | Make token detection more robust and completely rewrite its tests | |||
| | | * \ \ \ \ | Merge remote-tracking branch 'origin/master' into ↵ | MarkKoz | 2020-06-02 | -40/+645 |
| | | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | bug/filters/928/non-ascii-token | |||
| | | * | | | | | | Token remover: remove the `delete_message` function | MarkKoz | 2020-06-02 | -12/+7 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's redundant; there's no benefit here in abstracting two lines of code into a function. | |||
| | | * | | | | | | Test find_token_in_message returns None for invalid matches | MarkKoz | 2020-05-28 | -0/+15 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This covers the case when a token is matched, but its user ID and timestamp turn out to be invalid. | |||
| | | * | | | | | | Adjust find_token_in_message tests for the recent cog changes | MarkKoz | 2020-05-28 | -19/+20 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It now supports the changes that switched to finditer, added match groups, and added the Token NamedTuple. It also accounts for the is_maybe_token function being removed. For the sake of simplicity, call assertions on is_valid_user_id and is_valid_timestamp were not made. | |||
| | | * | | | | | | Remove is_maybe_token tests | MarkKoz | 2020-05-28 | -33/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The function was removed due to redundancy. Therefore, its tests are obsolete. | |||
| | | * | | | | | | Add more thorough and realistic inputs for token ID and timestamp tests | MarkKoz | 2020-05-27 | -18/+52 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tests for valid inputs and invalid inputs were split to make them more readable. | |||
| | | * | | | | | | Adjust token remover tests to use the Token NamedTuple | MarkKoz | 2020-05-27 | -6/+8 |
| | | | | | | | | | ||||
| | | * | | | | | | Switch findall to finditer in assertions | MarkKoz | 2020-05-27 | -4/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `find_token_in_message` now uses the latter so the tests should adjust accordingly. | |||
| | | * | | | | | | Use real token values for testing multiple matches in regex | MarkKoz | 2020-05-25 | -3/+4 |
| | | | | | | | | | ||||
| | | * | | | | | | Fix multiple match text for token regex | MarkKoz | 2020-05-25 | -2/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It has to account for the addition of groups. It's easiest to compare the entire string so `finditer` is used to return re.Match objects; the tuples of `findall` would be cumbersome. Also threw in a change to use `assertCountEqual` cause the order doesn't really matter. | |||
| | | * | | | | | | Fix valid token regex test | MarkKoz | 2020-05-25 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was broken due to the addition of groups. Rather than returning the full match, `findall` returns groups if any exist. The test was comparing a tuple of groups to the token string, which was of course failing. Now `fullmatch` is used cause it's simpler - just check for `None` and don't worry about iterating matches to search. | |||
| | | * | | | | | | Add more valid tokens to test the regex with | MarkKoz | 2020-05-25 | -3/+5 |
| | | | | | | | | | ||||
| | | * | | | | | | Test token regex won't match non-base64 characters | MarkKoz | 2020-05-25 | -4/+3 |
| | | | | | | | | | ||||
| | | * | | | | | | Fix a test needlessly being a coroutine | MarkKoz | 2020-05-13 | -1/+1 |
| | | | | | | | | | ||||
| | | * | | | | | | Use subtests for valid ID/timestamp tests and test non-ASCII inputs | MarkKoz | 2020-05-11 | -18/+25 |
| | | | | | | | | | ||||
| | | * | | | | | | Clean up token remover test imports | MarkKoz | 2020-05-11 | -16/+12 |
| | | | | | | | | | ||||
| | | * | | | | | | Replace deprecated assertion methods | MarkKoz | 2020-05-11 | -2/+2 |
| | | | | | | | | | ||||
 |
index : bot | |
| A mirrored copy of the python-discord/bot repository. |
| aboutsummaryrefslogtreecommitdiffstats |