| Commit message (Collapse) | Author | Age | Lines | ||
|---|---|---|---|---|---|
| ... | |||||
| | * | | | | Sync tests: test _send_prompt adds reactions | 2020-02-12 | -0/+8 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test _send_prompt sends a new message if one isn't given | 2020-02-12 | -0/+11 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test _send_prompt returns None if channel fetch fails | 2020-02-12 | -0/+12 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test _send_prompt fetches channel on a cache miss | 2020-02-12 | -0/+11 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test that _send_prompt gets channel from cache | 2020-02-12 | -0/+11 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test that _send_prompt edits message contents | 2020-02-12 | -0/+8 | ||
| | | | | | | |||||
| | * | | | | Sync tests: create a Syncer subclass for testing | 2020-02-12 | -0/+9 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test instantiation fails without abstract methods | 2020-02-12 | -0/+17 | ||
| | | | | | | |||||
| | * | | | | Sync tests: fix wait_until_ready in duck pond tests | 2020-02-12 | -2/+2 | ||
| | | | | | | |||||
| | * | | | | Sync tests: fix docstring for UserSyncerSyncTests | 2020-02-12 | -1/+1 | ||
| | | | | | | |||||
| | * | | | | Sync tests: create and use a fake_role fixture | 2020-02-12 | -33/+31 | ||
| | | | | | | |||||
| | * | | | | Sync tests: add tests for API requests for syncing users | 2020-02-12 | -1/+40 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test diff is empty when DB has a user not in the guild | 2020-02-12 | -9/+9 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test diff for all 3 changes simultaneously | 2020-02-12 | -8/+12 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test diff for users which leave the guild | 2020-02-12 | -7/+9 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test diff for new users | 2020-02-12 | -8/+10 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test diff for updated users | 2020-02-12 | -9/+11 | ||
| | | | | | | |||||
| | * | | | | Sync tests: fix get_guild modifying the original member dicts | 2020-02-12 | -0/+1 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test empty diff for identical users | 2020-02-12 | -8/+8 | ||
| | | | | | | |||||
| | * | | | | Sync tests: work around @everyone role being added by MockMember | 2020-02-12 | -2/+3 | ||
| | | | | | | |||||
| | * | | | | Sync tests: fix mismatched attributes when creating a mock user | 2020-02-12 | -0/+3 | ||
| | | | | | | |||||
| | * | | | | Sync tests: fix fake_user fixture | 2020-02-12 | -7/+9 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test empty diff for no users | 2020-02-12 | -6/+9 | ||
| | | | | | | |||||
| | * | | | | Sync tests: add fixture to get a guild with members | 2020-02-12 | -1/+19 | ||
| | | | | | | |||||
| | * | | | | Sync tests: rename user sync test case | 2020-02-12 | -2/+2 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test syncs with multiple roles | 2020-02-12 | -15/+37 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test API requests for role syncing | 2020-02-12 | -1/+34 | ||
| | | | | | | |||||
| | * | | | | Sync tests: create separate role test cases for diff and sync tests | 2020-02-12 | -2/+10 | ||
| | | | | | | |||||
| | * | | | | Sync tests: remove guild_roles lists and assign roles to variables | 2020-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 together | 2020-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 simultaneously | 2020-02-12 | -21/+17 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test diff for deleted roles | 2020-02-12 | -17/+10 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test diff for new roles | 2020-02-12 | -20/+15 | ||
| | | | | | | |||||
| | * | | | | Sync tests: create a role in setUp to use as a constant | 2020-02-12 | -6/+5 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test diff for updated roles | 2020-02-12 | -27/+16 | ||
| | | | | | | |||||
| | * | | | | Sync tests: test empty diff for identical roles | 2020-02-12 | -8/+11 | ||
| | | | | | | |||||
| | * | | | | Sync tests: fix creation of MockRoles | 2020-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 roles | 2020-02-12 | -1/+21 | ||
| | | | | | | |||||
| | * | | | | Sync tests: rename the role syncer test case | 2020-02-12 | -1/+1 | ||
| | | | | | | |||||
| | * | | | | Tests: add a MockAPIClient | 2020-02-12 | -0/+18 | ||
| | | |_|/ | |/| | | |||||
| * | | | | Fix test_time test and ensure coverage | 2020-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 AsyncMock | 2020-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 file | 2020-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_test | 2020-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 LoggingTestCase | 2020-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-refactor | 2020-02-23 | -239/+635 | ||
| |\ \ \ | |||||
| | * \ \ | Merge branch 'master' into unittest-antispam-rules | 2020-02-21 | -62/+2 | ||
| | |\ \ \ | |||||
| | | * | | | Remove tests for custom bot log | 2020-02-21 | -62/+2 | ||
| | | |/ / | |||||
| | * | | | Add unit test for newlines antispam rule | 2020-02-04 | -0/+105 | ||
| | | | | | |||||
| | * | | | Add unit test for duplicates antispam rule | 2020-02-04 | -0/+66 | ||
| | | | | | |||||