aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | Change generation of child mocksGravatar Sebastiaan Zeeff2019-10-28-36/+67
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - https://docs.python.org/3/library/unittest.mock.html We previously used an override of the `__new__` method to prevent our custom mock types from instantiating their children with their own type instead of a general mock type like `MagicMock` or `Mock`. As it turns out, the Python documentation suggests another method of doing this that does not involve overriding `__new__`. This commit implements this new method to make sure we're using the idiomatic way of handling this. The suggested method is overriding the `_get_child_mock` method in the subclass. To make our code DRY, I've created a mixin that should come BEFORE the mock type we're subclassing in the MRO. --- In addition, I have also added this new mixin to our `AsyncMock` class to make sure that its `__call__` method returns a proper mock object after it has been awaited. This makes sure that subsequent attribute access on the returned object is mocked as expected.
| * | | | Superstarify: return None in _pardon_action for supported typesGravatar MarkKoz2019-10-25-1/+4
| | | | |
| * | | | Moderation: make infraction notification embed title lowercaseGravatar MarkKoz2019-10-25-1/+1
| | | | |
| * | | | Moderation: remove full stops from pardon embed titlesGravatar MarkKoz2019-10-25-2/+2
| | | | |
| * | | | Superstarify: make mod log title lowercase for consistencyGravatar MarkKoz2019-10-25-1/+1
| | | | |
| * | | | Superstarify: make the apply embed colour goldGravatar MarkKoz2019-10-25-9/+11
| | | | |
| * | | | Superstarify: fix incorrect infraction type check in _pardon_actionGravatar MarkKoz2019-10-25-1/+1
| | | | |
| * | | | InfractionScheduler: remove redundant loggingGravatar MarkKoz2019-10-25-3/+0
| | | | |
| * | | | Superstarify: save the old nickname before editing itGravatar MarkKoz2019-10-25-2/+3
| | | | |
| * | | | Superstarify: negate active infractions check in on_member_updateGravatar MarkKoz2019-10-25-1/+1
| | | | |
| * | | | Superstarify: fix mod log mentionGravatar MarkKoz2019-10-25-1/+1
| | | | |
| * | | | Superstarify: fit within 100 columns & use double quotesGravatar MarkKoz2019-10-25-7/+13
| | | | |
| * | | | Superstarify: refactor on_member_update eventGravatar MarkKoz2019-10-25-39/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Use notify_infraction to be more consistent * Add an audit log reason to the member edit * Simplify docstring * Simplify log message
| * | | | Superstarify: ignore member update event when applyingGravatar MarkKoz2019-10-25-0/+1
| | | | |
| * | | | InfractionScheduler: prevent duplicate rescheduling of infractionsGravatar MarkKoz2019-10-25-6/+9
| | | | |
| * | | | InfractionScheduler: create generic function for reapplying infractionsGravatar MarkKoz2019-10-25-68/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | reapply_infraction() is a generic function which reapplies a given infraction if it is still active or deactivates it if less than 60 seconds remain. Most of the code from Infractions.on_member_joined now lives in this new function. This simplifies the "fancier" DM and mod log that were previously sent for superstar in on_member_join.
| * | | | Superstarify: refactor superstarify commandGravatar MarkKoz2019-10-25-29/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Edit the nickname and schedule the infraction before sending messages * Reformat mod log description to be consistent with other infractions * Use INFRACTION_ICONS constant
| * | | | Superstarify: schedule infractions by subclassing InfractionSchedulerGravatar MarkKoz2019-10-25-42/+25
| | | | |
| * | | | InfractionScheduler: use fetched user for notify_infractionGravatar MarkKoz2019-10-25-1/+1
| | | | |
| * | | | Superstarify: add icons to constants and configGravatar MarkKoz2019-10-25-3/+10
| | | | |
| * | | | InfractionScheduler: delegate type-specific pardon code to subclassesGravatar MarkKoz2019-10-25-35/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Create an abstract method for performing type-specific infraction deactivation * Move infraction-specific pardon code to separate functions
| * | | | Moderation: create a class to handle scheduling of infractionsGravatar MarkKoz2019-10-25-353/+372
|/ / / / | | | | | | | | | | | | | | | | | | | | The class handles application, pardoning, and expiration of infractions. This will allow other cogs, such as superstarify, to later subclass it to receive the same functionality that the infractions cog had.
| * | | group and order constantsGravatar Numerlor2019-11-15-3/+5
| | | |
| * | | Add variable info after comment was deletedGravatar Numerlor2019-11-15-1/+1
| | | | | | | | | | | | | | | | Co-authored-by: scargly <[email protected]>
| * | | join extra newlineGravatar Numerlor2019-11-15-2/+1
| | | |
| * | | Adjust code style and commentsGravatar Numerlor2019-11-12-37/+40
| | | |
| * | | syntax highlight diff of reloaded inventoriesGravatar Numerlor2019-11-11-3/+3
| | | |
| * | | Do not cut off text arbitrarily but at last sentence to make sure no ↵Gravatar Numerlor2019-11-11-1/+1
| | | | | | | | | | | | | | | | unfinished markdown is left in
| * | | clear renamed symbols on inventory refreshGravatar Numerlor2019-11-11-0/+1
| | | |
| * | | Move signatures definitionGravatar Numerlor2019-11-10-1/+1
| | | |
| * | | Add a command for refreshing inventoriesGravatar Numerlor2019-11-10-0/+22
| | | |
| * | | Move paragraph search to not cut off long starting paragraphsGravatar Numerlor2019-11-10-1/+1
| | | | | | | | | | | | | | | | Co-authored-by: scargly <[email protected]>
| * | | Cut off description at 1000 chars if paragraph is not foundGravatar Numerlor2019-11-10-0/+2
| | | |
| * | | Add a newline after signatures for readabilityGravatar Numerlor2019-11-10-1/+1
| | | |
| * | | fix signatures and descriptions not being found when presentGravatar Numerlor2019-11-10-4/+4
| | | |
| * | | Make sure only signatures belonging to the symbol are fetchedGravatar Numerlor2019-11-03-3/+4
| | | |
| * | | Comment grammarGravatar Numerlor2019-11-03-2/+2
| | | |
| * | | Fix case for the python package name in `NO_OVERRIDE_PACKAGES`Gravatar Numerlor2019-11-03-1/+1
| | | |
| * | | Handle exceptions when fetching inventoriesGravatar Numerlor2019-11-03-31/+57
| | | |
| * | | Improve module description searchingGravatar Numerlor2019-11-02-7/+35
| | | |
| * | | Get up to 3 signatures of a symbolGravatar Numerlor2019-11-02-16/+14
| | | |
| * | | Get symbol description by searching for a dd tag instead of traversing the ↵Gravatar Numerlor2019-11-02-1/+1
| | | | | | | | | | | | | | | | siblings
| * | | Do not cut off description in code blocksGravatar Numerlor2019-11-02-1/+7
| | | |
| * | | Grammar check commentGravatar Numerlor2019-10-21-1/+1
| | | |
| * | | Allow embeds to not include signatures in case the symbol is a moduleGravatar Numerlor2019-10-21-11/+14
| | | |
| * | | Don't include a signature and only get first paragraphs when scraping when ↵Gravatar Numerlor2019-10-21-10/+18
| | | | | | | | | | | | | | | | symbol is a module
| * | | remove "function" from NO_OVERRIDE_GROUPSGravatar Numerlor2019-10-20-1/+0
| | | |
| * | | Auto delete messages when docs are not foundGravatar Numerlor2019-10-20-2/+8
| | | |
| * | | show renamed duplicates in embed footerGravatar Numerlor2019-10-20-8/+13
| | | |
| * | | add handling for duplicate symbols in docs inventoriesGravatar Numerlor2019-10-20-4/+35
| | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-11-07 12:41:41 +0000