aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Remove unnecessary args from test cases.Gravatar Numerlor2020-03-11-6/+6
| | | | | Needless call args which were constant were kept in the test cases, resulting in redundant code, the args were moved directly into the function call.
* Add alert with silenced channels on `cog_unload`.Gravatar Numerlor2020-03-10-0/+11
|
* Remove `channel` arg from commands.Gravatar Numerlor2020-03-10-20/+10
|
* Retain previous channel overwrites.Gravatar Numerlor2020-03-10-4/+12
| | | | | | | Previously silencing a channel reset all overwrites excluding `send_messages` and unsilencing them removed all overwrites. This is prevented by getting the current overwrite and applying it with only send_messages changed.
* Use `patch.object` instead of patch with direct `return_value`.Gravatar Numerlor2020-03-09-9/+2
|
* Move notifier to separate class.Gravatar Numerlor2020-03-08-40/+46
| | | | | | | Separating the notifier allows us to keep the Silence class and its methods to be more focused on the class' purpose, handling the logic of adding/removing channels and the loop itself behind `SilenceNotifier`'s interface.
* Remove "Channel" from output string for consistency.Gravatar Numerlor2020-03-08-1/+1
|
* Use `Context.invoke` instead of calling `unsilence` directly.Gravatar Numerlor2020-03-08-1/+1
| | | | | Calling the command coro directly did unnecessary checks and made tests for the method harder to realize.
* Add test for `unsilence` discord output.Gravatar Numerlor2020-03-08-0/+9
|
* Add test for `silence` discord output.Gravatar Numerlor2020-03-08-0/+24
|
* Add test case for Silence cog.Gravatar Numerlor2020-03-08-1/+10
|
* Add tests for FirstHash class.Gravatar Numerlor2020-03-08-0/+25
|
* Add handling for shh/unshh for `CommandNotFound`.Gravatar Numerlor2020-03-08-4/+32
|
* Restrict cog to moderators.Gravatar Numerlor2020-03-08-1/+7
|
* Add logging to loop start and loop end.Gravatar Numerlor2020-03-02-0/+8
|
* Add Silence cog.Gravatar Numerlor2020-03-02-0/+143
| | | | FirstHash is used for handling channels in `loop_alert_channels` set as tuples without considering other elements.
* Add HushDurationConverter.Gravatar Numerlor2020-03-02-0/+28
|
* Merge pull request #783 from python-discord/python38-migrationGravatar S. Co12020-03-01-661/+301
|\ | | | | Migrate bot to Python 3.8
| * Merge branch 'master' into python38-migrationGravatar Leon Sandøy2020-03-01-2/+29
| |\ | |/ |/|
* | Merge pull request #809 from python-discord/ban-grabify-urlsGravatar kwzrd2020-03-01-0/+24
|\ \ | | | | | | Add grabify (IP logger) domains to banned domains
| * | Add grabify (IP logger) domains to banned domainsGravatar Joseph2020-03-01-0/+24
|/ /
* | Merge pull request #808 from spyoungtech/masterGravatar scragly2020-03-02-2/+5
|\ \ | | | | | | Optimize Dockerfile for faster rebuilds
| * \ Merge branch 'master' into masterGravatar scragly2020-03-02-0/+0
| |\ \ | |/ / |/| |
| * | Optimize DockerfileGravatar Spencer Young2020-03-01-2/+5
| | |
| | * Modify log test regex to be non-os-specificGravatar S. Co12020-03-01-1/+1
| | | | | | | | | Previous regex utilized a `/`, which doesn't work for comparing against Windows paths, which use `\`
| | * Move syncer confirmation reaction check out of finally clauseGravatar S. Co12020-03-01-11/+11
| | | | | | | | | Returning directly out of a `finally` clause can cause any exceptions raised in the clause to be discarded, so we can remove the finally clause entirely and shift the control statements into the body of the function
| | * Bump Dependencies & RelockGravatar S. Co12020-03-01-78/+76
| | | | | | | | | * Remove explicit urllib3 pinning, CVE that caused its pinning has been resolved by 1.25+. This is a child dependency of requests.
| | * Merge branch 'master' into python38-migrationGravatar Sebastiaan Zeeff2020-03-01-29/+71
| | |\ | |_|/ |/| | | | | I have resolved a merge conflict in Pipfile.lock by relocking with the newly added dependencies of `coloredlogs` and `colorama`.
* | | Merge pull request #806 from python-discord/feat/frontend/b000/coloured-logsGravatar Eivind Teig2020-03-01-47/+89
|\| | | | | | | | Coloured log output
| * | Make logs coloured!Gravatar MarkKoz2020-02-29-7/+23
| | |
| * | Install the coloredlogs packageGravatar MarkKoz2020-02-29-40/+66
|/ / | | | | | | | | This makes it easy to add colour to the logs. Colorama is also installed if on a Windows system.
| * Migrate snekbox tests to Python 3.8's unittestGravatar Sebastiaan Zeeff2020-02-29-41/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've migrated the `tests/test_snekbox.py` file to use the new Python 3.8-style unittests instead of our old style using our custom Async mocks. In particular, I had to make a few changes: - Mocking the async post() context manager correctly Since `ClientSession.post` returns an async context manager when called, we need to make sure to assign the return value to the __aenter__ method of whatever `post()` returns, not of `post` itself (i.e.. when it's not called). - Use the new AsyncMock assert methods `assert_awaited_once` and `assert_awaited_once_with` Objects of the new `unittest.mock.AsyncMock` class have special methods to assert what they were called with that also assert that specific coroutine object was awaited. This means we test two things in one: Whether or not it was called with the right arguments and whether or not the returned coroutine object was then awaited. - Patch `functools.partial` as `partial` objects are compared by identity When you create two partial functions of the same function, you'll end up with two different `partial` objects. Since `partial` objects are compared by identity, you can't compare a `partial` created in a test method to that created in the callable you're trying to test. They will always compare as `False`. Since we're not interested in actually creating `partial` objects, I've just patched `functools.partial` in the namespace of the module we're testing to make sure we can compare them.
| * Merge branch 'master' into python38-migrationGravatar Sebastiaan Zeeff2020-02-29-403/+1005
| |\ | |/ |/| | | I've resolved the merge conflict by confirming the deleted part of tests/helpers.py
* | Merge pull request #755 from ↵Gravatar Sebastiaan Zeeff2020-02-29-44/+70
|\ \ | | | | | | | | | | | | python-discord/bug/backend/b754/scheduler-suppresses-errors Don't suppress all errors in scheduler
| * \ Merge branch 'master' into bug/backend/b754/scheduler-suppresses-errorsGravatar Sebastiaan Zeeff2020-02-29-353/+936
| |\ \ | |/ / |/| |
* | | Merge pull request #797 from Numerlor/fuzzy_zero_divGravatar Shirayuki Nekomata2020-02-29-26/+17
|\ \ \ | | | | | | | | Fix invalid tag names being passed to the tag cog.
| * \ \ Merge branch 'master' into fuzzy_zero_divGravatar Shirayuki Nekomata2020-02-29-32/+479
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #710 from python-discord/eval-enhancementsGravatar Mark2020-02-28-32/+479
|\ \ \ \ | | | | | | | | | | Eval cog improvements
| * | | | Snekbox: mention re-evaluation feature in the command's docstringGravatar MarkKoz2020-02-28-1/+4
| | | | |
| * | | | Merge remote-tracking branch 'origin/master' into eval-enhancementsGravatar MarkKoz2020-02-28-1440/+3084
| |\ \ \ \ | |/ / / / |/| | | |
| * | | | Ignore NotFound errors inside continue_evalGravatar Matteo2020-02-28-24/+25
| | | | | | | | | | | | | | | | | | | | It could have caused some errors if the user delete his own message
| * | | | Return only the new code in continue_eval and check for truthiness insteadGravatar Matteo2020-02-28-11/+10
| | | | |
| * | | | Use unicode code point instead of literal for the snekbox re-eval emojiGravatar Matteo2020-02-27-6/+8
| | | | | | | | | | | | | | | | | | | | Unicode literals aren't really safe compared to code points
| * | | | Split the eval command procedure into two functions.Gravatar Matteo2020-02-27-114/+148
| | | | | | | | | | | | | | | | | | | | Two functions were created: send_eval and continue_eval, in order to facilitate testing. The corresponding tests are also changed in this commit.
| * | | | Merge branch 'master' into eval-enhancementsGravatar Matteo Bertucci2020-02-18-379/+350
| |\ \ \ \
| * | | | | Use kwargs to set mock attributesGravatar Matteo2020-02-18-25/+11
| | | | | |
| * | | | | Assert return value of Snekbox.post_evalGravatar Matteo2020-02-18-1/+3
| | | | | |
| * | | | | Use a space instead of an empty string in test_get_status_emoji Gravatar Matteo Bertucci2020-02-18-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because of the stripping, it should still be considered as empty Co-Authored-By: Mark <[email protected]>
| * | | | | Delete additional informations from subtest Gravatar Matteo Bertucci2020-02-18-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce visual clutter Co-Authored-By: Mark <[email protected]>
| * | | | | Split assertions onto separate lines Gravatar Matteo Bertucci2020-02-18-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reads better as separate lines Co-Authored-By: Mark <[email protected]>
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-27 11:43:10 +0000