aboutsummaryrefslogtreecommitdiffstats
path: root/tests (follow)
Commit message (Collapse)AuthorAgeLines
...
| * | | | Sync tests: rename user sync test caseGravatar MarkKoz2020-02-12-2/+2
| | | | |
| * | | | Sync tests: test syncs with multiple rolesGravatar MarkKoz2020-02-12-15/+37
| | | | |
| * | | | Sync tests: test API requests for role syncingGravatar MarkKoz2020-02-12-1/+34
| | | | |
| * | | | Sync tests: create separate role test cases for diff and sync testsGravatar MarkKoz2020-02-12-2/+10
| | | | |
| * | | | Sync tests: remove guild_roles lists and assign roles to variablesGravatar MarkKoz2020-02-12-14/+8
| | | | | | | | | | | | | | | | | | | | | | | | | Makes the creation of the expected diff clearer since the variable has a name compared to accessing some index of a list.
| * | | | Sync tests: remove diff test for updated and new roles togetherGravatar MarkKoz2020-02-12-19/+0
| | | | | | | | | | | | | | | | | | | | | | | | | Redundant since test_diff_for_new_updated_and_deleted_roles tests all 3 types together.
| * | | | Sync tests: test diff for all 3 role changes simultaneouslyGravatar MarkKoz2020-02-12-21/+17
| | | | |
| * | | | Sync tests: test diff for deleted rolesGravatar MarkKoz2020-02-12-17/+10
| | | | |
| * | | | Sync tests: test diff for new rolesGravatar MarkKoz2020-02-12-20/+15
| | | | |
| * | | | Sync tests: create a role in setUp to use as a constantGravatar MarkKoz2020-02-12-6/+5
| | | | |
| * | | | Sync tests: test diff for updated rolesGravatar MarkKoz2020-02-12-27/+16
| | | | |
| * | | | Sync tests: test empty diff for identical rolesGravatar MarkKoz2020-02-12-8/+11
| | | | |
| * | | | Sync tests: fix creation of MockRolesGravatar MarkKoz2020-02-12-3/+4
| | | | | | | | | | | | | | | | | | | | Role was being accessed like a class when it is actually a dict.
| * | | | Sync tests: add fixture to create a guild with rolesGravatar MarkKoz2020-02-12-1/+21
| | | | |
| * | | | Sync tests: rename the role syncer test caseGravatar MarkKoz2020-02-12-1/+1
| | | | |
| * | | | Tests: add a MockAPIClientGravatar MarkKoz2020-02-12-0/+18
| | |_|/ | |/| |
* | | | Fix test_time test and ensure coverageGravatar Sebastiaan Zeeff2020-02-23-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | One of the test_time methods did not actually assert the exception message it was trying to detect as the assertion statement was contained within the context manager handling the exception. I've moved it out of the context so it actually runs. I've also added a few `praga: no cover` comments for parts that were artifically lowering coverage of the test suite.
* | | | Use unittest's AsyncMock instead of our AsyncMockGravatar Sebastiaan Zeeff2020-02-23-213/+103
| | | | | | | | | | | | | | | | | | | | | | | | Python 3.8 introduced an `unittest.mock.AsyncMock` class that can be used to mock coroutines and other types of asynchronous operations like async iterators and async context managers. As we were using our custom, but limited, AsyncMock, I have replaced our mock with unittest's AsyncMock. Since Python 3.8 also introduces a different way of automatically detecting which attributes should be mocked with an AsyncMock, I've changed our CustomMockMixin to use this new method as well. Together with a couple other small changes, this means that our Custom Mocks now use a lazy method of detecting coroutine attributes, which significantly speeds up the test suite.
* | | | Remove lingering pytest test_time.py fileGravatar Sebastiaan Zeeff2020-02-23-62/+0
| | | | | | | | | | | | | | | | I forgot to remove one pytest test file during the migration from pytest to unittest. Since we have sinced added a unittest version of the same file, I've now removed the lingering pytest file.
* | | | Use IsolatedAsyncioTestCase instead of async_testGravatar Sebastiaan Zeeff2020-02-23-70/+13
| | | | | | | | | | | | | | | | Since we upgraded to Python 3.8, we can now use the new IsolatedAsyncioTestCase test class to use coroutine-based test methods instead of our own, custom async_test decorator. I have changed the base class for all of our test classes that use coroutine-based test methods and removed the now obsolete decorator from our helpers.
* | | | Use mixin-composition not inheritance for LoggingTestCaseGravatar Sebastiaan Zeeff2020-02-23-16/+14
| |/ / |/| | | | | | | | | | | | | | We used inheritence to add additional logging assertion methods to unittest's TestCase class. However, with the introduction of the new IsolatedAsyncioTestCase this extension strategy means we'd have to create multiple child classes to be able to use the extended functionality in all of the TestCase variants. Since that leads to undesirable code reuse and an inheritance relationship is not at all needed, I've switched to a mixin-composition based approach that allows the user to extend the functionality of any TestCase variant with a mixin where needed.
* | | 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
| |/ |/|
* | Merge the note with the additional resources sectionGravatar Matteo Bertucci2020-01-30-1/+2
| | | | | | Move the link to Ned Batchelder’s talk and link the note to the section
* | Add additional resources to the test readmeGravatar Matteo Bertucci2020-01-30-0/+6
|/
* Merge branch 'master' into unittest-mentionsGravatar Matteo Bertucci2019-12-13-18/+894
|\
| * Fixed a typo ( due to poor copy pasta and eyeballing skills )Gravatar Shirayuki Nekomata2019-12-13-1/+1
| |
| * Added tests for `until_expiration`Gravatar Shirayuki Nekomata2019-12-13-0/+45
| | | | | | | | | | | | | | Similar to `format_infraction_with_duration` ( if not outright copying it ), added 3 tests for `until_expiration`: - None `expiry`. - Custom `max_units`. - Normal use cases.
| * Merge branch 'master' into Write-unit-tests-for-`bot/utils/time.py`Gravatar scragly2019-12-12-25/+10
| |\
| | * Fix test failures for setup log messagesGravatar MarkKoz2019-12-08-24/+7
| | |
| | * Change all Bot imports to use the subclassGravatar MarkKoz2019-12-07-1/+3
| | |
| * | Removed `setUp()` from `TimeTests` since it is not being used for anything.Gravatar Shirayuki Nekomata2019-12-05-3/+0
| | |