aboutsummaryrefslogtreecommitdiffstats
path: root/tests/helpers.py (unfollow)
Commit message (Collapse)AuthorLines
2019-10-07minor fixGravatar kraktus-1/+1
2019-10-07Make sure that poor code does not contains tokenGravatar kraktus-14/+21
Added a new function `is_token_in_message` in `token_remover`. This function returns a `bool` and if the code contains a token then the embed message about the poorly formatted code is not displayed.
2019-10-03Improve readabilityGravatar Ben Diller-0/+1
Co-Authored-By: Mark <[email protected]>
2019-10-03Move asyncio.sleep() to avoid disturbing function signatures.Gravatar bendiller-16/+13
2019-10-02Add sleep(3) between retries, with bot indicating typing during sleepGravatar bendiller-4/+9
2019-10-02Add logging for invalid response (after all retries are exhausted)Gravatar bendiller-1/+3
2019-10-03Fix mutes not being re-applied on rejoins.Gravatar scragly-5/+42
2019-10-02Fix `cog_unload` bug in WatchChannel ABCGravatar Sebastiaan Zeeff-1/+1
https://github.com/python-discord/bot/issues/482 There was small bug in the `cog_unload` method of the WatchChannel ABC in `bot.cogs.watchchannels.watchchannel`. The problem was that it tries to check if the Task assigned to `self._consume_task` is done by accessing its `done` method. However, if a watch channel has not yet relayed messages after the bot has started, it will not have a consumption task yet, meaning this `_consume_task` attribute will be assigned to `None`. The solution is to change the `if` condition to: `if self._consume_task and not self._consume_task.done():` This commit closes #482
2019-10-01Fix missing type annotations and docstring errors.Gravatar Leon Sandøy-7/+3
2019-10-01Create !tools alias and split !site resources.Gravatar Leon Sandøy-7/+28
This splits the former !site resources into two separate commands, one for !site resources and one for !site tools. This makes sense now that we've split up the lists into two pages. It also adds a new alias, !tools, to call this command. This addresses #478. https://github.com/python-discord/bot/issues/478
2019-10-01Add checks for valid response and retries to fetch_posts()Gravatar bendiller-9/+13
2019-10-01Update docs cog docstring & fix URL converter attribute errorGravatar S. Co1-4/+4
2019-10-01Update bot/cogs/doc.pyGravatar Ayplo-1/+1
Co-Authored-By: S. Co1 <[email protected]>
2019-10-01Remove single-item listGravatar S. Co1-2/+1
It looks silly
2019-10-01Make `!tags edit` use PATCH to update tagsGravatar Sebastiaan Zeeff-2/+31
https://github.com/python-discord/bot/issues/474 This commit makes sure we're using the PATCH method to update tags, instead of the POST method. The problem was that the `!tags edit` command was an alias of `!tags set`, which uses the POST method to post new tags to the API. However, when a tag with a given name already exists, the API will refuse a POST request in an attempt to update it; we need to use the PATCH method to the `bot/tags/{tag_name}` endpoint instead. I fixed this by creating a separate subcommand, `!tags edit`, that uses the correct PATCH method to update an existing tag. This commit closes #474
2019-09-30Update contrib doc for new wiki linksGravatar sco1-3/+4
2019-09-30Forward arguments to getGravatar Tom Sputz-1/+1
2019-09-30Remove temp instructions, add link to contrib guide on website.Gravatar scragly-27/+1
2019-09-30Make `bot` wait for `web`, use `latest` web image.Gravatar scragly-2/+3
2019-09-29Do up some basic setup instructions for new setup.Gravatar scragly-0/+28
2019-09-28Update site to use new manage.py, add bot dev compose.Gravatar scragly-7/+20
2019-09-28Use pathlib, avoid inner `bot` directory for logs.Gravatar scragly-7/+6
2019-09-27Apply suggestions from code reviewGravatar S. Co1-2/+2
Co-Authored-By: Mark <[email protected]>
2019-09-28Lovely lemon let loose and forgot linting.Gravatar scragly-1/+1
2019-09-27Allow Rockstars and Partners to use !eval.Gravatar Leon Sandøy-7/+10
Anyone who is not a Rockstar, a Partner, or a member of staff will still be redirected to #bot-commands.
2019-09-27Ensure docker containers are published only for master branch builds.Gravatar scragly-1/+1
2019-09-26Remove invalid colorama markersGravatar S. Co1-2/+0
2019-09-26Update flake8-annotations & fix missing lintsGravatar S. Co1-10/+47
2019-09-26Use Command.cog_name attr instead of missing instance attrGravatar S. Co1-2/+2
2019-09-26Fix incorrect kwargs in watchchannel modlog sendGravatar S. Co1-2/+2
2019-09-26Parse timestamps with dateutil for watch channelsGravatar MarkKoz-5/+2
2019-09-26Use format_infaction datetime util in talentpoolGravatar Sebastiaan Zeeff-11/+5
The watchchannel ABC defined its own private utility function to format ISO datetime strings to something more human-readable. I have removed this private utility function and replaced the calls to it with calls to the new `format_infraction` utility function defined in bot.utils.time. In addition, I've changed the utility function to use `dateutil` to parse the datetime string, since `dateutil.parser.isoparse` supports the strings our API generates out of the box. With the built-in `datetime.datetime.fromisoformat`, we needed to prepare the string by slicing of the `Z` timezone indicator.
2019-09-26Add utility function to consistently format infraction timestampsGravatar MarkKoz-33/+18
The format used is %Y-%m-%d %H:%M.
2019-09-26Show when field is unchanged in infraction edit confirmation messageGravatar MarkKoz-0/+4
2019-09-25Display "Permanent" in mod log for edited infractionsGravatar MarkKoz-2/+2
2019-09-25Merge infraction edit commandsGravatar MarkKoz-108/+62
Both the duration and the reason can be edited with the new command. * Remove try-except; the default error handler is already adequate * Remove the new reason from the confirmation message * Simplify humanisation of the timestamp in the confirmation message * Add a converter to support permanent durations
2019-09-25Fix AttributeError with cog special methodsGravatar MarkKoz-4/+4
Discord.py's internals use the __func__ attribute of special methods (cog_command_error, cog_check, cog_before_invoke, cog_after_invoke). Therefore the methods must be bound methods rather than static so that the attribute exists.
2019-09-25Swapped Lambda for itemgetterGravatar Daniel Brown-1/+2
- For the sake of code style and consistency, the lambda has been swapped with operator.itemgetter Signed-off-by: Daniel Brown <[email protected]>
2019-09-25Free Command FixGravatar Daniel Brown-2/+2
- Moved the sorted function to its own line and instead passed the generated list for code clarity. Signed-off-by: Daniel Brown <[email protected]>
2019-09-25Free Command FixGravatar Daniel Brown-4/+4
- Fixed bug where if two channels had the same last message timestamp the command would error out. Signed-off-by: Daniel Brown <[email protected]>
2019-09-25Remove infraction reason from infraction confirmationGravatar S. Co1-53/+10
Closes #325
2019-09-25Unhide the tags commandGravatar S. Co1-1/+1
KAIZEN! Closes #385
2019-09-25Decouple warnings and notesGravatar S. Co1-1/+1
Closes #453
2019-09-25Remove permanent mutesGravatar S. Co1-89/+2
Add previous permanent mute invocations as aliases of their respective mute commands. Closes #318
2019-09-24Note Type CorrectionGravatar Daniel Brown-1/+1
- In the database, notes were being listed as "warnings" despite having a type specifically for them. Changed it so that notes are now listed as the proper type. Signed-off-by: Daniel Brown <[email protected]>
2019-09-25Add all the shieldsGravatar scragly-3/+7
2019-09-24Remove repeat logic for off-topic-name api callGravatar Sebastiaan Zeeff-6/+1
After a short discussion in the core-dev team, we decided to not use retry logic for a failed API call for new off-topic-names. We may introduce something similar in the future, but we're not sure on the direction we want to take yet.
2019-09-25Remove unnecessary sudo for pipenv install.Gravatar scragly-1/+1
2019-09-24Fix reminders filter queryGravatar S. Co1-2/+2
Closes #445
2019-09-25Remove regex wheel as dateparser is no longer used.Gravatar scragly-0/+0