aboutsummaryrefslogtreecommitdiffstats
path: root/tests/base.py (unfollow)
Commit message (Collapse)AuthorLines
2020-05-30Scheduler: Remove unnecessary `f` before stringGravatar ks129-1/+1
2020-05-30Scheduler: Add try-except to infraction deletionGravatar ks129-1/+6
2020-05-30Scheduler: Remove invalid commentGravatar ks129-1/+0
2020-05-30Scheduler: Move inline f-string if-else statement to normal if statementGravatar ks129-1/+4
2020-05-30Simplify infraction reason truncation testsGravatar ks129-13/+7
2020-05-20Infr Tests: Make `get_active_infraction` return `None`Gravatar ks129-1/+1
2020-05-20Infr. Test: Replace `get_active_mock` return valueGravatar ks129-1/+1
Replace `{"foo": "bar"}` with `{"id": 1}`
2020-05-20Infr. Tests: Replace `str` with `dict`Gravatar ks129-1/+1
To allow `.get`, I had to replace `str` return value with `dict`
2020-05-20Infr. Tests: Add more content to await commentGravatar ks129-2/+2
2020-05-20ModLog Tests: Fix embed description truncate testGravatar ks129-1/+1
2020-05-20ModLog: Fix embed description truncationGravatar ks129-1/+1
2020-05-20Infractions: Remove space from placeholderGravatar ks129-1/+1
2020-05-20Infraction Tests: Small fixesGravatar ks129-4/+4
- Remove unnecessary space from placeholder - Rename `has_active_infraction` to `get_active_infraction`
2020-05-20ModLog Tests: Fix truncation tests docstringGravatar ks129-1/+1
Co-authored-by: Leon Sandøy <[email protected]>
2020-05-18Add Steam gift card scam to domain blacklistGravatar S. Co1-0/+2
2020-05-19[bug] Adjustment to changes in #941, return message sent by webhook so ↵Gravatar Joseph Banks-1/+1
publish can take place
2020-05-17Fix incomplete variable renamingGravatar decorator-factory-1/+1
2020-05-17Rename `string` to `greeting`Gravatar decorator-factory-10/+10
2020-05-17Use `Command`-object for `send_help`Gravatar Sebastiaan Zeeff-21/+21
As @mathsman5133 pointed out, it's better to use the `Command`-instance we typically already have in the current context than to rely on parsing the qualified name again. The invocation is now done as: `await ctx.send_help(ctx.command)`
2020-05-17Add a note on user-defined classesGravatar decorator-factory-1/+1
2020-05-17Change standalone programs to interactive sessionsGravatar decorator-factory-8/+13
2020-05-17Apply language improvements proposed from kwzrdGravatar decorator-factory-3/+3
Co-authored-by: kwzrd <[email protected]>
2020-05-17Use `send_help` to invoke command helpGravatar Sebastiaan Zeeff-40/+32
After the refactoring of the help command, we need to use the built-in method of calling the help command: `Context.send_help`. As an argument, the qualified name (a string containing the full command path, including parents) of the command can be passed. Examples: - await ctx.send_help("reminders edit") This would send a help embed with information on `!reminders edit` to the Context. - await ctx.send_help(ctx.command.qualified_name) This would extract the qualified name of the command, which is the full command path, and send a help embed to Context. - await ctx.send_help() This will send the main "root" help embed to the Context.
2020-05-17added "solved" as a alias for "closed"Gravatar vivax3794-1/+1
2020-05-16fix redirect_output decorator; remove ninja codeGravatar mathsman5133-76/+66
- Lots of instance of `for c in ...` or `for a in ...` or `fmt` which are non-descriptive and sometimes cryptic. - Ves suggested running the command in an asyncio task for `@redirect_output`, rather than making a workaround which only applies to the help command. This fixes a fundamental flaw where the redirection message wouldn't be deleted until a further 60sec after the command has finished, which for `!help` could be up to 5min, meaning the invocation message could be sitting there for 6min, not the intended 60sec.
2020-05-15Change tests to use the new timeout constantGravatar Numerlor-1/+5
2020-05-15Move the re eval timeout to a module constantGravatar Numerlor-1/+2
2020-05-15Increase snekbox re eval timeout.Gravatar Numerlor-1/+1
2020-05-14Remove @Admins ping from the #verification messageGravatar Leon Sandøy-1/+1
This probably isn't necessary anymore. We get so many new users that someone is going to DM us very soon when something breaks. We've outgrown this, and it just adds noise to the #verification channel in the form of pings.
2020-05-14Remove everyone-ping from mentions alertGravatar Sebastiaan Zeeff-1/+0
The mentions alert that is sent out by the Verification cog currently pings `@everyone` despite being quite unactionable by most people receiving the ping. As it happens frequently, especially with the recent uptick in joins, I'm removing that ping to not bother our moderators as much.
2020-05-13Add message publishing to `Reddit` cogGravatar ks129-1/+7
2020-05-12Expand guild whitelistGravatar S. Co1-1/+7
2020-05-11Restructure `apply_ban()` logic Gravatar Daniel Brown-12/+7
Another refactor/cleaning to make the logic clearer and easier to understand. Also cleaned up the trace logs to be shorter and more concise. Thanks, @scragly! Co-authored-by: scragly <[email protected]>
2020-05-10Add remindme alias for the remind commandGravatar Suhail-1/+1
2020-05-07apply_ban() logic refinedGravatar Daniel Brown-15/+15
- Refined the logic for `apply_ban()` even further to be cleaner. (Thanks, @MarkKoz!) Signed-off-by: Daniel Brown <[email protected]>
2020-05-07Addressing Review ChangesGravatar Daniel Brown-25/+31
- Changed docstring explanation and function name of `get_active_infractions` to `get_active_infraction()` to better convey that only one infraction is returned. Also changed all relevant uses to reflect that change. - Added explanation of parameter `send_msg` to the doc strings of `pardon_infraction()` and `get_active_infraction()` - Adjusted placement of `log.trace()` in `pardon_infraction()` - Adjusted logic in `apply_ban()` to remove redundant check. - Adjusted logic in `apply_ban()` to be consistent with other checks. Signed-off-by: Daniel Brown <[email protected]>
2020-05-07Update Python News extension name in __main__.pyGravatar ks129-1/+1
Co-authored-by: Joseph Banks <[email protected]>
2020-05-06Remove `PythonNews.channel` because this is unnecessaryGravatar ks129-2/+0
2020-05-06Renamed `news.py` to `python_news.py` and `News` to `PythonNews` to avoid ↵Gravatar ks129-2/+2
confusion
2020-05-06Fix config Webhook IDs formattingGravatar ks129-6/+6
Co-authored-by: Sebastiaan Zeeff <[email protected]>
2020-05-04Update antimalware to filter txt files in cases where messages were longer ↵Gravatar Savant-Dev-5/+5
than 2000 chars
2020-05-04Perma Bans now Overwrite Temp BansGravatar Daniel Brown-18/+55
- Changed `has_active_infraction` to `get_active_infractions` in order to add additional logic in `apply_ban`. - Added `send_msg` parameters to `pardon_infraction` and `get_active_infractions` so that multi-step checks and actions don't need to send additional messages unless told to do so. Signed-off-by: Daniel Brown <[email protected]>
2020-05-03Fix `News` cog maillist news posting no threads check commentGravatar ks129-1/+3
Co-authored-by: Joseph Banks <[email protected]>
2020-05-02Remove mention command constantsGravatar MarkKoz-10/+0
2020-05-02Define encoding in `News` cog `await resp.text()` using Gravatar ks129-1/+1
In `News` cog PEP news posting, define `utf-8` as encoding on response parsing to avoid the error. Co-authored-by: Joseph Banks <[email protected]>
2020-05-01Remove the mention commandGravatar MarkKoz-46/+2
It was made obsolete by a new Discord feature. Users can be granted a permission to mention a role despite the role being set as non-mentionable.
2020-05-01Merge the two asynchronous tasks into one to prevent race conditionsGravatar Joseph Banks-6/+8
2020-05-01Added ignoring maillist when no recent threads (this month) in `News` cogGravatar ks129-0/+4
2020-05-01Added on cog unload news posting tasks canceling on `News` cogGravatar ks129-0/+5
2020-05-01Replace message ID storage to new specific ID storage in `News` cogGravatar ks129-38/+6
- Removed (now) unnecessary helper function `News.check_new_exist`. - Use thread IDs instead message IDs on maillists checking to avoid Discord API calls. - Use PEP number instead message IDs on PEP news checking to avoid Discord API calls.