aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | Add correct values for constants from production server.Gravatar Leon Sandøy2019-10-27-2/+2
| | | | | | | | | | | |
| * | | | | | | | | | | Appease the linterGravatar Leon Sandøy2019-10-27-6/+13
| | | | | | | | | | | |
| * | | | | | | | | | | Add duck_pond cog.Gravatar Leon Sandøy2019-10-27-0/+207
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This cog will listen for duck reactions on any message, and then: - If the reaction was added by a staff member - and the reaction was a duck - and the message has not already been added to the #duck-pond It will add the message to the #duck-pond and then add a green checkbox to the original message to indicate that the message has been ponded. Messages are added to the #duck-pond via webhook, so that they can retain the appearance of having their original authors. Once this checkmark has been added, the message will not be processed in the future. If the checkmark is removed and there are more than ducks_required ducks on the message, the bot will automatically add the checkmark back. However, if all reactions are removed, the bot does not have a countermeasure for this. In order to implement a countermeasure, it would be necessary to involve the API and the database.
| * | | | | | | | | | | Add duck-pond constants.Gravatar Leon Sandøy2019-10-27-29/+43
| | |_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the emojis, the channel, and the configuration needed for the duck-pond feature. This is added both to config-default.yml, and to the constants.py file.
| | | | | * | | | | | Make requested tweaks: Use URL constant from constants.py, re-add try/except ↵Gravatar Deniz2019-11-25-15/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | block and implement the changes requested by Ava
| | | | | * | | | | | Update PASTE_URL constant to be pydis instead of pythondiscordGravatar Deniz2019-11-25-1/+1
| | | | | | | | | | |
| | | | | * | | | | | Change order of importsGravatar Deniz2019-11-25-1/+1
| | | | | | | | | | |
| | | | | * | | | | | Forgot the word 'hey'Gravatar Deniz2019-11-25-1/+1
| | | | | | | | | | |
| | | | | * | | | | | Update antimalware.py to be more consistent with other information messages ↵Gravatar Deniz2019-11-25-24/+20
| |_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | (like the codeblock reminder) & improve code a slight bit
* | | | | | | | | | Relock to d.py 1.2.5 due to API breaking change for emoji.Gravatar scragly2019-11-19-80/+95
| | | | | | | | | |
* | | | | | | | | | Docs command improvements (#546)Gravatar scragly2019-11-15-37/+177
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docs command improvements Co-authored-by: Sebastiaan Zeeff <[email protected]>
| * \ \ \ \ \ \ \ \ \ Merge branch 'master' into doc-commandGravatar Sebastiaan Zeeff2019-11-15-2269/+4402
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #619 from python-discord/moderation-loggingGravatar Leon Sandøy2019-11-15-30/+102
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / |/| | | | | | | | | | Moderation logging
| * | | | | | | | | | Merge branch 'master' into moderation-loggingGravatar Leon Sandøy2019-11-15-193/+301
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Enhancements for `tests.helpers` and our test suite (#660)Gravatar scragly2019-11-14-157/+134
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enhancements for `tests.helpers` and our test suite Co-authored-by: null <[email protected]>
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into unittest-helpers-improvementsGravatar scragly2019-11-14-36/+167
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #656 from python-discord/checkpoint-changesGravatar Mark2019-11-10-22/+53
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Forward pings in checkpoint to mod-alerts and reword periodic checkpoint ping
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into checkpoint-changesGravatar Mark2019-11-10-540/+677
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #641 from kwzrd/unittest-linksGravatar Mark2019-11-08-0/+101
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add unit test for links antispam rule
| * \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into unittest-linksGravatar Mark2019-11-08-608/+1193
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | | | | | | |
* | | | | | | | | | | | | | Merge pull request #653 from python-discord/talent-poolingGravatar Sebastiaan Zeeff2019-11-08-14/+13
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow helpers to nominate people
| * | | | | | | | | | | | | | Refactor bigbrother to use constantsGravatar kosayoda2019-11-08-5/+5
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Allow helpers to nominate peopleGravatar kosayoda2019-11-08-9/+8
|/ / / / / / / / / / / / / /
| * | | | | | | | | | | | | Update docstring to use asterisks when referring to argument namesGravatar kwzrd2019-11-08-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Kyle Stanley <[email protected]>
| * | | | | | | | | | | | | Add whitespace for readability, consistency & allureGravatar kwzrd2019-11-08-0/+1
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Adjust case to only test a single aspectGravatar kwzrd2019-11-02-1/+1
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Annotate unclear test cases with inline commentsGravatar kwzrd2019-11-02-2/+2
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Add two more test cases for links rule unit testGravatar kwzrd2019-11-01-0/+7
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Add unit test for links antispam ruleGravatar kwzrd2019-11-01-0/+93
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | [kaizen] Remove now duplicate channel checkGravatar Derek2019-11-09-19/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unindent subsequent lines after check
| | * | | | | | | | | | | | Forward user/role pings in checkpoint to mod-alertsGravatar Derek2019-11-09-2/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [kaizen] limit on_message listener to verification channel
| | * | | | | | | | | | | | Reword periodic #checkpoint messageGravatar Derek2019-11-04-1/+1
| | | |_|_|_|_|_|_|/ / / / | | |/| | | | | | | | | |
| | | * | | | | | | | | | Prevent await warnings for MockBot's create_taskGravatar Sebastiaan Zeeff2019-11-13-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, the coroutine object passed to `MockBot.loop.create_task` would trigger a `RuntimeWarning` for not being awaited as we do not actually create a task for it. To prevent these warnings, coroutine objects passed will now automatically be closed.
| | | * | | | | | | | | | Prevent setting unknown attributes on d.py mocksGravatar Sebastiaan Zeeff2019-11-13-158/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Our custom `discord.py` now follow the specifications of the object they are mocking more strictly by using the `spec_set` instead of the `spec` kwarg to initialize the specifications. This means that trying to set an attribute that does not follow the specifications will now also result in an `AttributeError`. To make sure we are not trying to set illegal attributes during the default initialization of the mock objects, I've changed the way we handle default values of parameters. This does introduce a breaking change: Instead of passing a `suffix_id`, the `id` attribute should now be passed using the exact name. `id`. This commit also makes sure existing tests follow this change.
| | | * | | | | | | | | | Allow `name` attribute to be set during Mock initGravatar Sebastiaan Zeeff2019-11-13-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `name` keyword argument has a special meaning for the default mockobjects provided by `unittest.mock`. This means that by default, the common d.py `name` attribute can't be set during initalization of one of our custom Mock-objects by passing it to the constructor. Since it's unlikely for us to make use of the special `name` feature of mocks and more likely to want to set the d.py `name` attribute, I added special handling of the `name` kwarg.
| | | * | | | | | | | | | Prevent unwanted logging while running testsGravatar Sebastiaan Zeeff2019-11-13-1/+14
| |_|/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, logging messages would output to std.out. when running individual test files (instead of running the entire suite). To prevent this, I've added a `for`-loop to `tests.helpers` that sets the level of all registered loggers to `CRITICAL`. The reason for adding this to `tests.helpers` is simple: It's the most common file to be imported in individual tests, increasing the chance of the code being run for individual test files. A small downside of this way of handling logging is that when we are trying to assert logging messages are being emitted, we need to set the logger explicitly in the `self.assertLogs` context manager. This is a small downside, though, and probably good practice anyway. There was one test in `tests.bot.test_api` that did not do this, so I have changed this to make the test compatible with the new set-up.
| | | * | | | | | | | | Use trailing _ instead of leading for some variable namesGravatar MarkKoz2019-11-06-15/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PEP 8 states the convention is a trailing underscore when used to prevent name conflicts.
| | | * | | | | | | | | Moderation: use trailing _ instead of leading for variable namesGravatar MarkKoz2019-11-04-33/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PEP 8 states the convention is a trailing underscore when used to prevent name conflicts.
| | | * | | | | | | | | Add logging for moderation functionsGravatar MarkKoz2019-11-04-9/+81
| |_|/ / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #618 from python-discord/schedule-superstarifyGravatar S. Co12019-11-04-526/+563
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | | Schedule superstarify infractions for expiration
| * | | | | | | | | | Merge branch 'master' into schedule-superstarifyGravatar S. Co12019-11-04-260/+838
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | User command enhancements (#637)Gravatar scragly2019-11-04-62/+552
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | User command enhancements Co-authored-by: null <[email protected]>
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into user-command-enhancementsGravatar scragly2019-11-04-7/+66
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Change invite filter message to accurately reflect the new rulesGravatar kosayoda2019-11-04-1/+1
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #648 from python-discord/user-status-summaryGravatar Mark2019-11-03-1/+9
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / |/| | | | | | | | | | | Show custom statuses in !user output
| * | | | | | | | | | | Merge branch 'master' into user-status-summaryGravatar Mark2019-11-03-3/+53
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #640 from python-discord/bot-utils-init-testsGravatar S. Co12019-11-03-0/+52
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Write tests for `bot.utils`.
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into bot-utils-init-testsGravatar S. Co12019-11-03-5/+4
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #631 from manusaurio/masterGravatar S. Co12019-11-03-3/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / / / |/| | | | | | | | | | | | Remove redirection on rules command
| * | | | | | | | | | | | Merge branch 'master' into masterGravatar S. Co12019-11-03-194/+224
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |