aboutsummaryrefslogtreecommitdiffstats
path: root/tests/helpers.py (unfollow)
Commit message (Collapse)AuthorLines
2020-03-11Remove `_current_loop` from setup.Gravatar Numerlor-2/+1
2020-03-11Add tests for `add_channel`.Gravatar Numerlor-0/+21
2020-03-11Create test case for `SilenceNotifier`Gravatar Numerlor-1/+7
2020-03-11Add tests for `_get_instance_vars`.Gravatar Numerlor-1/+28
2020-03-11Pass created channel instead of new object.Gravatar Numerlor-1/+1
Creating a new object caused the assert to fail because different objects were used.
2020-03-11Reset `self.ctx` call history after every subtest.Gravatar Numerlor-0/+1
2020-03-11Fix erroneous `assert_called_once_with` calls.Gravatar Numerlor-8/+7
`assert_called_once_with` was being tested on call_args which always reported success.st.
2020-03-11Add test for `cog_check`.Gravatar Numerlor-0/+7
2020-03-11Add test for `cog_unload`.Gravatar Numerlor-1/+17
2020-03-11Use patch decorators and assign names from `with` patches.Gravatar Numerlor-11/+11
2020-03-11Separate tests for permissions and `muted_channels.add` on `_silence`.Gravatar Numerlor-5/+8
2020-03-11Add tests for `_unsilence`Gravatar Numerlor-1/+33
2020-03-11Use async test case.Gravatar Numerlor-14/+12
This allows us to use coroutines with await directly instead of asyncio.run
2020-03-11Add test for `_silence` method.Gravatar Numerlor-1/+34
2020-03-11Move adding of channel to `muted_channels` up.Gravatar Numerlor-1/+1
Before the channel was not added if `persistent` was `True`.
2020-03-11Remove channel mentions from output discord messages.Gravatar Numerlor-8/+8
With the removal of the channel args, it's no longer necessary to mention the channel in the command output. Tests adjusted accordingly
2020-03-11Remove unnecessary args from test cases.Gravatar Numerlor-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.
2020-03-10Add alert with silenced channels on `cog_unload`.Gravatar Numerlor-0/+11
2020-03-10Remove `channel` arg from commands.Gravatar Numerlor-20/+10
2020-03-10Retain previous channel overwrites.Gravatar Numerlor-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.
2020-03-09Use `patch.object` instead of patch with direct `return_value`.Gravatar Numerlor-9/+2
2020-03-08Move notifier to separate class.Gravatar Numerlor-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.
2020-03-08Remove "Channel" from output string for consistency.Gravatar Numerlor-1/+1
2020-03-08Use `Context.invoke` instead of calling `unsilence` directly.Gravatar Numerlor-1/+1
Calling the command coro directly did unnecessary checks and made tests for the method harder to realize.
2020-03-08Add test for `unsilence` discord output.Gravatar Numerlor-0/+9
2020-03-08Add test for `silence` discord output.Gravatar Numerlor-0/+24
2020-03-08Add test case for Silence cog.Gravatar Numerlor-1/+10
2020-03-08Add tests for FirstHash class.Gravatar Numerlor-0/+25
2020-03-08Add handling for shh/unshh for `CommandNotFound`.Gravatar Numerlor-4/+32
2020-03-08Restrict cog to moderators.Gravatar Numerlor-1/+7
2020-03-02Add logging to loop start and loop end.Gravatar Numerlor-0/+8
2020-03-02Add Silence cog.Gravatar Numerlor-0/+143
FirstHash is used for handling channels in `loop_alert_channels` set as tuples without considering other elements.
2020-03-02Add HushDurationConverter.Gravatar Numerlor-0/+28
2020-03-01Add grabify (IP logger) domains to banned domainsGravatar Joseph-0/+24
2020-03-01Modify log test regex to be non-os-specificGravatar S. Co1-1/+1
Previous regex utilized a `/`, which doesn't work for comparing against Windows paths, which use `\`
2020-03-01Move syncer confirmation reaction check out of finally clauseGravatar S. Co1-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
2020-03-01Bump Dependencies & RelockGravatar S. Co1-78/+76
* Remove explicit urllib3 pinning, CVE that caused its pinning has been resolved by 1.25+. This is a child dependency of requests.
2020-03-01Optimize DockerfileGravatar Spencer Young-2/+5
2020-02-29Make logs coloured!Gravatar MarkKoz-7/+23
2020-02-29Install the coloredlogs packageGravatar MarkKoz-40/+66
This makes it easy to add colour to the logs. Colorama is also installed if on a Windows system.
2020-02-29Migrate snekbox tests to Python 3.8's unittestGravatar Sebastiaan Zeeff-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.
2020-02-28Scheduler: correct type annotationsGravatar MarkKoz-6/+6
2020-02-28Snekbox: mention re-evaluation feature in the command's docstringGravatar MarkKoz-1/+4
2020-02-28Ignore NotFound errors inside continue_evalGravatar Matteo-24/+25
It could have caused some errors if the user delete his own message
2020-02-28Return only the new code in continue_eval and check for truthiness insteadGravatar Matteo-11/+10
2020-02-27Scheduler: log the exception instead of raisingGravatar MarkKoz-2/+5
Logging it ourselves has a cleaner traceback and gives more control over the output, such as including the task ID.
2020-02-27Remove unnecessary newlines from call.Gravatar Numerlor-4/+1
2020-02-27Moderation: fix task cancellation for permanent infraction when editingGravatar MarkKoz-1/+3
A task should not be cancelled if an infraction is permanent because tasks don't exist for permanent infractions. Fixes BOT-1V
2020-02-27Use unicode code point instead of literal for the snekbox re-eval emojiGravatar Matteo-6/+8
Unicode literals aren't really safe compared to code points
2020-02-27Split the eval command procedure into two functions.Gravatar Matteo-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.
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-27 05:38:19 +0000