aboutsummaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeLines
...
| | | * | | | | | | | | 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
| | |\ \ \ \ \ \ \ \ | | | | |_|_|_|_|_|/ | | | |/| | | | | |
| | | * | | | | | | 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
| | | | | |\ \ \ \ \
| | | | | * | | | | | Infr. Tests: Add `apply_infraction` awaiting assertion with argsGravatar ks1292020-05-30-0/+6
| | | | | | | | | | |
| | | | | * | | | | | Simplify infraction reason truncation testsGravatar ks1292020-05-30-13/+7
| | | | | | | | | | |
| | | | | * | | | | | Infr Tests: Make `get_active_infraction` return `None`Gravatar ks1292020-05-20-1/+1
| | | | | | | | | | |
| | | | | * | | | | | Infr. Test: Replace `get_active_mock` return valueGravatar ks1292020-05-20-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace `{"foo": "bar"}` with `{"id": 1}`
| | | | | * | | | | | Infr. Tests: Replace `str` with `dict`Gravatar ks1292020-05-20-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To allow `.get`, I had to replace `str` return value with `dict`
| | | | | * | | | | | Infr. Tests: Add more content to await commentGravatar ks1292020-05-20-2/+2
| | | | | | | | | | |
| | | | | * | | | | | ModLog Tests: Fix embed description truncate testGravatar ks1292020-05-20-1/+1
| | | | | | | | | | |
| | | | | * | | | | | Merge branch 'master' into ban-kick-reason-lengthGravatar ks1292020-05-20-11/+183
| | | | | |\ \ \ \ \ \ | | | | | | | |/ / / / | | | | | | |/| | | |
| | | | | * | | | | | Infraction Tests: Small fixesGravatar ks1292020-05-20-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Remove unnecessary space from placeholder - Rename `has_active_infraction` to `get_active_infraction`
| | | | | * | | | | | ModLog Tests: Fix truncation tests docstringGravatar ks1292020-05-20-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Leon Sandøy <[email protected]>
| | | | | * | | | | | (Infractions and ModLog Tests): Replaced `shortening` with `truncation`, ↵Gravatar ks1292020-04-16-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | removed unnecessary type hint and added comment to kick truncation test about awaiting `kick`.
| | | | | * | | | | | (ModLog Tests): Created reason shortening tests for `send_log_message`.Gravatar ks1292020-04-16-0/+29
| | | | | | | | | | |
| | | | | * | | | | | (Infraction Tests): Created reason shortening tests for ban and kick.Gravatar ks1292020-04-16-0/+54
| | | | | | | | | | |
| | | | | * | | | | | (Test Helpers): Added `__ge__` function to `MockRole` for comparing.Gravatar ks1292020-04-15-0/+4
| | | | | | |_|_|/ / | | | | | |/| | | |
| | | | * | | | | | Fix busted test_information test.Gravatar Leon Sandøy2020-05-30-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I wish this test didn't exist.
| | | * | | | | | | Fix exception message assertions in converter testsGravatar MarkKoz2020-04-28-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `msg` arg is for displaying a message when the assertion fails. To match against the exception's message, `assertRaisesRegex` must be used. Since all of the messages are meant to be interpreted literally rather than as regex, `re.escape` is used.
| | | * | | | | | | Use await instead of asyncio.run in converter testsGravatar MarkKoz2020-04-28-28/+27
| | | | | | | | | |
| | | * | | | | | | Remove redundant parenthesis from test valuesGravatar MarkKoz2020-04-28-19/+19
| | | | | | | | | |
| | | * | | | | | | Test for out of range datetime in the Duration converterGravatar MarkKoz2020-04-28-0/+11
| | | | | | | | | |
| | * | | | | | | | Remove redis namespace collision testGravatar MarkKoz2020-05-31-10/+0
| | | |/ / / / / / | | |/| | | | | |
| * | | | | | | | Merge branch 'master' into truncate-internal-evalGravatar Numerlor2020-05-30-33/+390
| |\| | | | | | |
| * | | | | | | | Add tests for `send_to_paste_service`.Gravatar Numerlor2020-05-08-0/+74
| | | | | | | | |
| * | | | | | | | Adjust behaviour for new func usage.Gravatar Numerlor2020-05-07-14/+5
| | | | | | | | |
| * | | | | | | | Remove tests from moved function.Gravatar Numerlor2020-05-07-15/+0
| | |_|_|_|_|/ / | |/| | | | | |
* | | | | | | | Change ValidPythonIdentifier tests to PackageName.Gravatar Numerlor2020-07-22-11/+10
| |_|_|_|_|/ / |/| | | | | |
* | | | | | | Merge branch 'master' into test_antimalwareGravatar Sebastiaan Zeeff2020-05-30-35/+398
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | |
| * | | | | | Add custom exceptions for each error state.Gravatar Leon Sandøy2020-05-28-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The bot can get into trouble in three distinct ways: - It has no Bot instance - It has no namespace - It has no parent instance. These happen only if you're using it wrong. To make the test more precise, and to add a little bit more readability (RuntimeError could be anything!), we'll introduce some custom exceptions for these three states. This addresses a review comment by @aeros.
| * | | | | | Merge branch 'master' into redis_persistenceGravatar Leon Sandøy2020-05-27-10/+49
| |\ \ \ \ \ \
| | * | | | | | Add some tests for `in_whitelist_check`.Gravatar Leon Sandøy2020-05-27-0/+48
| | | | | | | |
| | * | | | | | Remove the now deprecated in_channel_check.Gravatar Leon Sandøy2020-05-27-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This check was no longer being used anywhere, having been replaced by in_whitelist_check.
| | * | | | | | Find + change all InWhitelistCheckFailure importsGravatar Leon Sandøy2020-05-27-2/+1
| | | | | | | |
| | * | | | | | Refactor the in_whitelist deco to a check.Gravatar Leon Sandøy2020-05-27-2/+2
| | | |_|_|/ / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're moving the actual predicate into the `utils.checks` folder, just like we're doing with most of the other decorators. This is to allow us the flexibility to use it as a pure check, not only as a decorator. This commit doesn't actually change any functionality, just moves it around.
| * | | | | | Floats are no longer permitted as RedisCache keys.Gravatar Leon Sandøy2020-05-27-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also added a test for this. This is the DRYest approach I could find. It's a little ugly, but I think it's probably good enough.
| * | | | | | Refactor .increment and add lock test.Gravatar Leon Sandøy2020-05-27-64/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The way we were doing the asyncio.Lock() stuff for increment was slightly problematic. @aeros has adviced us that it's better to just initialize the lock as None in __init__, and then initialize it inside the first coroutine that uses it instead. This ensures that the correct loop gets attached to the lock, so we don't end up getting errors like this one: RuntimeError: got Future <Future pending> attached to a different loop This happens because the lock and the actual calling coroutines aren't on the same loop. When creating a new test, test_increment_lock, we discovered that we needed a small refactor here and also in the test class to make this new test pass. So, now we're creating a DummyCog for every test method, and this will ensure the loop streams never cross. Cause we all know we must never cross the streams.
| * | | | | | Clear cache in asyncSetUp instead of tests.Gravatar Leon Sandøy2020-05-27-6/+1
| | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-29 13:10:04 +0000