aboutsummaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeLines
* Merge branch 'master' into fuzzy_zero_divGravatar Shirayuki Nekomata2020-02-29-0/+380
|\
| * Merge remote-tracking branch 'origin/master' into eval-enhancementsGravatar MarkKoz2020-02-28-471/+1844
| |\
| * | Return only the new code in continue_eval and check for truthiness insteadGravatar Matteo2020-02-28-4/+4
| | |
| * | Use unicode code point instead of literal for the snekbox re-eval emojiGravatar Matteo2020-02-27-3/+3
| | | | | | | | | | | | Unicode literals aren't really safe compared to code points
| * | Split the eval command procedure into two functions.Gravatar Matteo2020-02-27-69/+81
| | | | | | | | | | | | Two functions were created: send_eval and continue_eval, in order to facilitate testing. The corresponding tests are also changed in this commit.
| * | Merge branch 'master' into eval-enhancementsGravatar Matteo Bertucci2020-02-18-1/+8
| |\ \
| * | | Use kwargs to set mock attributesGravatar Matteo2020-02-18-25/+11
| | | |
| * | | Assert return value of Snekbox.post_evalGravatar Matteo2020-02-18-1/+3
| | | |
| * | | Use a space instead of an empty string in test_get_status_emoji Gravatar Matteo Bertucci2020-02-18-1/+1
| | | | | | | | | | | | | | | | | | | | Because of the stripping, it should still be considered as empty Co-Authored-By: Mark <[email protected]>
| * | | Delete additional informations from subtest Gravatar Matteo Bertucci2020-02-18-1/+1
| | | | | | | | | | | | | | | | | | | | Reduce visual clutter Co-Authored-By: Mark <[email protected]>
| * | | Split assertions onto separate lines Gravatar Matteo Bertucci2020-02-18-4/+8
| | | | | | | | | | | | | | | | | | | | Reads better as separate lines Co-Authored-By: Mark <[email protected]>
| * | | Split assertion onto separate linesGravatar Matteo Bertucci2020-02-18-1/+2
| | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| * | | Write tests for bot/cogs/test_snekbox.pyGravatar Akarys422020-02-09-0/+363
| | | |
| * | | Create an AsyncContextManagerMock mock for testing asynchronous context managersGravatar Akarys422020-02-09-0/+12
| | | | | | | | | | | | | | | | It can be used to test aiohttp request functions, since they are async context managers
* | | | Merge branch 'master' into fuzzy_zero_divGravatar Numerlor2020-02-28-7/+7
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | # Conflicts: # bot/cogs/error_handler.py
| * | | Merge branch 'master' into feat/backend/b496/rename-constantsGravatar Leon Sandøy2020-02-28-15/+0
| |\ \ \
| * | | | Fix mismatched constant names in syncer testsGravatar S. Co12020-02-25-2/+2
| | | | |
| * | | | Merge branch 'master' into feat/backend/b496/rename-constantsGravatar S. Co12020-02-25-205/+1197
| |\ \ \ \
| * | | | | Tests: fix setting bot-commands ID in information testsGravatar MarkKoz2020-02-23-4/+4
| | | | | |
| * | | | | Constants: rename roles to match their names in the guildGravatar MarkKoz2020-02-23-1/+1
| | | | | |
* | | | | | Merge branch 'master' into fuzzy_zero_divGravatar Sebastiaan Zeeff2020-02-27-15/+0
|\ \ \ \ \ \ | | |_|/ / / | |/| | | |
| * | | | | Merge branch 'master' into channel_constants_checkGravatar Numerlor2020-02-25-205/+1197
| |\ \ \ \ \ | | | |/ / / | | |/| | |
| * | | | | Remove unused `chunks` function and its tests.Gravatar Numerlor2020-02-24-15/+0
| | |/ / / | |/| | | | | | | | | | | | | The function was only used in the since removed `Events` cog.
* | | | | Adjust tests for new converter behavior.Gravatar Numerlor2020-02-26-1/+1
| |/ / / |/| | |
* | | | Merge remote-tracking branch 'origin/master' into ↵Gravatar MarkKoz2020-02-23-246/+642
|\| | | | | | | | | | | | | | | bug/backend/b704/ready-missing-cache
| * | | Merge branch 'master' into information-refactorGravatar S. Co12020-02-23-239/+635
| |\ \ \
| | * \ \ Merge branch 'master' into unittest-antispam-rulesGravatar Sebastiaan Zeeff2020-02-21-62/+2
| | |\ \ \
| | | * | | Remove tests for custom bot logGravatar Joseph Banks2020-02-21-62/+2
| | | | |/ | | | |/|
| | * | | Add unit test for newlines antispam ruleGravatar kwzrd2020-02-04-0/+105
| | | | |
| | * | | Add unit test for duplicates antispam ruleGravatar kwzrd2020-02-04-0/+66
| | | | |
| | * | | Merge branch 'master' into unittest-antispam-rulesGravatar Joseph2020-02-02-1/+8
| | |\| |
| | * | | Make RuleTest use ABCMetaGravatar kwzrd2020-02-02-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | This will prevent child classes to be instantiated unless they implement all abstract methods, leading to a more descriptive error message.
| | * | | Adjust existing tests to inherit from RuleTest ABCGravatar kwzrd2020-02-02-294/+157
| | | | |
| | * | | Implement RuleTest ABCGravatar kwzrd2020-02-02-0/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will serve as an ABC for tests for individual rules. The base class provides runners for allowed and disallowed cases, and the children classes then only provide the cases and implementations of helper methods specific to each rule.
| | * | | Adjust multi-line docstrings to prevailing styleGravatar kwzrd2020-01-31-3/+6
| | | | |
| | * | | Refactor msg helper function name to make_msgGravatar kwzrd2020-01-26-25/+25
| | | | | | | | | | | | | | | | | | | | The name msg is less descriptive and creates a needless name conflict in local gen exp.
| | * | | Fix incorrect config key in attachments antispam ruleGravatar kwzrd2020-01-26-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The rule was incorrectly printing out the maximum amount of allowed attachments instead of the configured interval. This commit also adjusts the rule's unit test case.
| | * | | Add unit test case for role mentions antispam ruleGravatar kwzrd2020-01-26-0/+66
| | | | |
| | * | | Add unit test case for discord emojis antispam ruleGravatar kwzrd2020-01-26-0/+68
| | | | |
| | * | | Add unit test case for chars antispam ruleGravatar kwzrd2020-01-26-0/+75
| | | | |
| | * | | Add unit test case for burst shared antispam ruleGravatar kwzrd2020-01-26-0/+65
| | | | |
| | * | | Add unit test case for burst antispam ruleGravatar kwzrd2020-01-26-0/+69
| | | |/ | | |/|
| * | | Update the tests accordingly to reflect the new changesGravatar Deniz2020-02-13-3/+3
| | | |
| * | | Fix some of the testing for information.py; I think this should be it. ↵Gravatar Deniz2020-02-06-2/+2
| | | | | | | | | | | | | | | | (hopefully).
| * | | Update tests to reflect status changesGravatar Deniz2020-02-06-4/+4
| | |/ | |/|
* | | Sync: add confirmation timeout and max diff to configGravatar MarkKoz2020-02-19-2/+2
| | |
* | | Tests: fix unawaited error for MockAPIClientGravatar MarkKoz2020-02-12-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This error is due to the use of an actual instance of APIClient as the spec for the mock. recreate() is called in __init__ which in turn creates a task for the _create_session coroutine. The approach to the solution is to use the type for the spec rather than and instance, thus avoiding any call of __init__. However, without an instance, instance attributes will not be included in the spec. Therefore, they are defined as class attributes on the actual APIClient class definition and given default values. Alternatively, a subclass of APIClient could have been made in the tests.helpers module to define those class attributes. However, it seems easier to maintain if the attributes are in the original class definition.
* | | Sync tests: use async_test decoratorGravatar MarkKoz2020-02-12-94/+135
| | |
* | | Sync tests: test on_member_joinGravatar MarkKoz2020-02-12-0/+52
| | | | | | | | | | | | | | | | | | Should PUT user's data or POST it if the user doesn't exist. ResponseCodeError should be re-raised if status code isn't a 404. A helper method was added to reduce code redundancy between the 2 tests.
* | | Sync tests: add a third role with a lower ID to on_member_update testGravatar MarkKoz2020-02-12-1/+2
| | | | | | | | | | | | This better ensures that roles are being sorted when patching.