Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | EH Tests: Merge test classes | 2020-05-19 | -11/+4 | |
| | ||||
* | EH Tests: Create test for `handle_check_failure` | 2020-05-19 | -0/+48 | |
| | ||||
* | EH Tests: Create test for `handle_input_error` | 2020-05-19 | -6/+36 | |
| | ||||
* | Error Handler: Changed way of help command get + send to avoid warning | 2020-05-19 | -6/+4 | |
| | | | | Only get coroutine when this is gonna be awaited. | |||
* | EH Tests: Created test for `handle_user_input_error` `get_help_command` | 2020-05-19 | -0/+16 | |
| | ||||
* | EH Tests: Added test for `try_get_tag` `ResponseCodeError` ignore | 2020-05-19 | -0/+6 | |
| | ||||
* | EH Tests: Added test for `try_get_tag` `ctx.invoke` calling | 2020-05-19 | -0/+7 | |
| | ||||
* | EH Tests: Added test for `try_get_tag` tag name converting failing | 2020-05-19 | -0/+9 | |
| | ||||
* | EH Tests: Added test for `try_get_tag` successful tag name converting | 2020-05-19 | -0/+9 | |
| | ||||
* | EH Tests: Added test for `try_get_tag` error handling | 2020-05-19 | -2/+11 | |
| | ||||
* | EH Tests: Added test for `try_get_tag` checks fail | 2020-05-19 | -2/+8 | |
| | ||||
* | EH Tests: Added test for `try_get_tag` `invoked_from_error_handler` | 2020-05-19 | -0/+7 | |
| | ||||
* | EH Tests: Added test for `try_get_tag` `get_command` calling | 2020-05-19 | -0/+19 | |
| | ||||
* | EH Tests: Added tests for `get_help_command`in `OtherErrorHandlerTests` | 2020-05-19 | -0/+32 | |
| | ||||
* | EH Tests: Cleanup `try_silence` tests | 2020-05-18 | -39/+27 | |
| | ||||
* | EH Tests: Added test for `try_silence` no match message | 2020-05-18 | -0/+6 | |
| | ||||
* | EH Tests: Created `try_silence` test to test unsilence command calling | 2020-05-18 | -0/+17 | |
| | ||||
* | EH Tests: Created `try_silence` test to test silence command calling | 2020-05-18 | -0/+17 | |
| | ||||
* | EH Tests: Created `try_silence` test to check no permission with except | 2020-05-18 | -0/+8 | |
| | ||||
* | EH Tests: Created `try_silence` test to check no permission response | 2020-05-18 | -2/+9 | |
| | ||||
* | EH Tests: Created `try_silence` test for `get_command` | 2020-05-18 | -0/+9 | |
| | ||||
* | EH Tests: Created first test for `try_silence` | 2020-05-18 | -0/+18 | |
| | ||||
* | EH Tests: Created test for all other errors | 2020-05-18 | -0/+8 | |
| | ||||
* | EH Tests: Added test for 3 errors handling on handler | 2020-05-18 | -1/+17 | |
| | | | | | | | These 3 errors is: - `ConversionError` - `MaxConcurrencyReached` - `ExtensionError` | |||
* | EH Tests: Added test for `CommandInvokeError` handling on handler | 2020-05-18 | -0/+22 | |
| | ||||
* | EH Tests: Added test for `CommandOnCooldown` handling on handler | 2020-05-18 | -0/+8 | |
| | ||||
* | EH Tests: Added test for `CheckFailure` handling on handler | 2020-05-18 | -0/+9 | |
| | ||||
* | EH Tests: Added test for `UserInputError` handling on handler | 2020-05-18 | -0/+9 | |
| | ||||
* | EH Tests: Added another test for `CommandNotFound` error handling | 2020-05-18 | -0/+17 | |
| | | | | | Added test for case when `Context.invoked_from_error_handler` is `True` | |||
* | Test Helpers: Added new attribute to `MockContext` | 2020-05-18 | -0/+2 | |
| | | | | Added `invoked_from_error_handler` attribute that is `False` default. | |||
* | Error Handler: Changed `CommandNotFound` error check for testing | 2020-05-18 | -1/+1 | |
| | | | | Replaced `hasattr` with `getattr` for unit tests | |||
* | EH Tests: Remove class member `cog` + other small changes | 2020-05-18 | -14/+15 | |
| | | | | | - Added `assertIsNone` to `test_error_handler_already_handled`. - Removed `ErrorHandlerTests.cog`, moved it to each test recreation. | |||
* | EH Tests: Created test for `CommandNotFound` error when call isn't by EH | 2020-05-18 | -0/+49 | |
| | ||||
* | EH Tests: Created test cases set + already handled error test | 2020-05-18 | -0/+22 | |
| | | | | | | Created test that make sure when error is already handled in local error handler, this don't await `ctx.send` to make sure that this don't move forward. | |||
* | Merge pull request #858 from python-discord/decorator-factory-mutability-tag | 2020-05-17 | -0/+37 | |
|\ | | | | | Add mutability.md tag | |||
| * | Merge branch 'master' into decorator-factory-mutability-tag | 2020-05-17 | -890/+2365 | |
| |\ | |/ |/| | ||||
* | | Merge pull request #949 from python-discord/help-command-fix-invocation | 2020-05-17 | -41/+33 | |
|\ \ | | | | | | | Use `send_help` to ensure that our help command is correctly invoked | |||
| * | | Use `Command`-object for `send_help` | 2020-05-17 | -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)` | |||
| * | | Use `send_help` to invoke command help | 2020-05-17 | -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. | |||
* | | Merge pull request #519 from mathsman5133/help-refactor | 2020-05-17 | -460/+269 | |
|\ \ | | | | | | | Refactor the !help command. | |||
| * \ | Merge branch 'master' into help-refactor | 2020-05-17 | -1/+1 | |
| |\ \ | |/ / |/| | | ||||
* | | | Merge pull request #946 from vivax3794/master | 2020-05-17 | -1/+1 | |
|\ \ \ | | | | | | | | | add "solved" as a alias for "closed" | |||
| * | | | added "solved" as a alias for "closed" | 2020-05-17 | -1/+1 | |
|/ / / | ||||
| * | | fix redirect_output decorator; remove ninja code | 2020-05-16 | -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. | |||
| * | | Merge branch 'master' into help-refactor | 2020-05-15 | -447/+2127 | |
| |\ \ | |/ / |/| | | ||||
* | | | Remove @Admins ping from the #verification message | 2020-05-14 | -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. | |||
* | | | Remove everyone-ping from mentions alert | 2020-05-14 | -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. | |||
* | | | Merge pull request #899 from ks129/python-news | 2020-05-13 | -55/+321 | |
|\ \ \ | | | | | | | | | Python News implemention | |||
| * \ \ | Merge branch 'master' into python-news | 2020-05-13 | -219/+522 | |
| |\ \ \ | |/ / / |/| | | | ||||
* | | | | Expand guild whitelist | 2020-05-12 | -1/+7 | |
| | | | |