| Commit message (Collapse) | Author | Age | Lines | ||
|---|---|---|---|---|---|
| ... | |||||
| * | | | Sync tests: test diff for all 3 role changes simultaneously |  MarkKoz | 2020-02-12 | -21/+17 | |
| | | | | |||||
| * | | | Sync tests: test diff for deleted roles | MarkKoz | 2020-02-12 | -17/+10 | |
| | | | | |||||
| * | | | Sync tests: test diff for new roles | MarkKoz | 2020-02-12 | -20/+15 | |
| | | | | |||||
| * | | | Sync tests: create a role in setUp to use as a constant | MarkKoz | 2020-02-12 | -6/+5 | |
| | | | | |||||
| * | | | Sync tests: test diff for updated roles | MarkKoz | 2020-02-12 | -27/+16 | |
| | | | | |||||
| * | | | Sync tests: test empty diff for identical roles | MarkKoz | 2020-02-12 | -8/+11 | |
| | | | | |||||
| * | | | Sync tests: fix creation of MockRoles | MarkKoz | 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 | MarkKoz | 2020-02-12 | -1/+21 | |
| | | | | |||||
| * | | | Sync tests: rename the role syncer test case | MarkKoz | 2020-02-12 | -1/+1 | |
| | | | | |||||
| * | | | Tests: add a MockAPIClient | MarkKoz | 2020-02-12 | -0/+18 | |
| |/ / | |||||
| * | | Merge the note with the additional resources section |  Matteo Bertucci | 2020-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 readme | Matteo Bertucci | 2020-01-30 | -0/+6 | |
| |/ | |||||
| * | Merge branch 'master' into unittest-mentions | Matteo Bertucci | 2019-12-13 | -18/+894 | |
| |\ | |||||
| | * | Fixed a typo ( due to poor copy pasta and eyeballing skills ) |  Shirayuki Nekomata | 2019-12-13 | -1/+1 | |
| | | | |||||
| | * | Added tests for `until_expiration` | Shirayuki Nekomata | 2019-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` |  scragly | 2019-12-12 | -25/+10 | |
| | |\ | |||||
| | | * | Fix test failures for setup log messages | MarkKoz | 2019-12-08 | -24/+7 | |
| | | | | |||||
| | | * | Change all Bot imports to use the subclass | MarkKoz | 2019-12-07 | -1/+3 | |
| | | | | |||||
| | * | | Removed `setUp()` from `TimeTests` since it is not being used for anything. | Shirayuki Nekomata | 2019-12-05 | -3/+0 | |
| | | | | |||||
| | * | | Moved all individual test cases into iterables and test with `self.subTest` ↵ | Shirayuki Nekomata | 2019-12-05 | -14/+18 | |
| | | | | | | | | | | | | | context manager. | ||||
| | * | | Added `self.subTest` for tests with multiple test cases & simplified single ↵ | Shirayuki Nekomata | 2019-12-05 | -19/+11 | |
| | | | | | | | | | | | | | test case tests. | ||||
| | * | | Introduced test for `test_format_infraction`, refactored ↵ | Shirayuki Nekomata | 2019-12-05 | -5/+7 | |
| | | | | | | | | | | | | | `test_parse_rfc1123`, fixed typo. | ||||
| | * | | Splitting test cases for `format_infraction_with_duration` into proper, ↵ | Shirayuki Nekomata | 2019-12-05 | -7/+27 | |
| | | | | | | | | | | | | | independent tests. | ||||
| | * | | Changed `assert` to `self.assertIs` for `test_wait_until` | Shirayuki Nekomata | 2019-12-05 | -1/+1 | |
| | | | | |||||
| | * | | Added missing docstring for `test_humanize_delta_raises_for_invalid_max_units` | Shirayuki Nekomata | 2019-12-05 | -0/+1 | |
| | | | | |||||
| | * | | Splitting test cases for `humanize_delta` into proper, independent tests. | Shirayuki Nekomata | 2019-12-05 | -7/+21 | |
| | | | | |||||
| | * | | Added `unittest` for `bot.utils.time` | Shirayuki Nekomata | 2019-12-04 | -0/+87 | |
| | |/ | |||||
| | * | Apply review comments to duckpond's unit tests |  Sebastiaan Zeeff | 2019-11-27 | -72/+128 | |
| | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/pull/621 I've changed to unit tests according to the comments made on the issue. Most changes are straightforward enough, but, for context, see the PR linked above. | ||||
| | * | Apply suggestions from code review | Sebastiaan Zeeff | 2019-11-16 | -4/+3 | |
| | | | | | | | Co-Authored-By: Mark <[email protected]> | ||||
| | * | Add unit tests with full coverage for `bot.cogs.duck_pond` | Sebastiaan Zeeff | 2019-11-15 | -159/+490 | |
| | | | | | | | | | | | This commit adds unit tests that provide a full branch coverage of the `bot.cogs.duck_pond` file. | ||||
| | * | Add MockAsyncWebhook to mock `discord.Webhook` objects | Sebastiaan Zeeff | 2019-11-15 | -0/+21 | |
| | | | | | | | | | | | | | | | I have added a mock type to mock `discord.Webhook` instances. Note that the current type is specifically meant to mock webhooks that use an AsyncAdaptor and therefore has AsyncMock/coroutine mocks for the "maybe-coroutine" methods specified in the `discord.py` docs. | ||||
| | * | Adjust MockReaction for new AsyncIteratorMock protocol | Sebastiaan Zeeff | 2019-11-15 | -3/+1 | |
| | | | | | | | | | | | The new AsyncIteratorMock no longer needs an additional method to be used with a Mock object. | ||||
| | * | Add MockUser to mock `discord.User` objects | Sebastiaan Zeeff | 2019-11-15 | -0/+19 | |
| | | | | | | | | | | | | | | | | | I have added a special mock that follows the specifications of a `discord.User` instance. This is useful, since `Users` have less attributes available than `discord.Members`. Since this difference in availability of information can be important, we should not use a `MockMember` to mock a `discord.user`. | ||||
| | * | Add `return_value` support and assertions to AsyncIteratorMock | Sebastiaan Zeeff | 2019-11-15 | -4/+54 | |
| | | | | | | | | | | | | | | | | | | | | | | | The AsyncIteratorMock included in Python 3.8 will work similarly to the mocks of callabes. This means that it allows you to set the items it will yield using the `return_value` attribute. It will also have support for the common Mock-specific assertions. This commit introduces some backports of those features in a slightly simplified way to make the transition to Python 3.8 easier in the future. | ||||
| | * | Add MockAttachment type and attachments default for MockMessage | Sebastiaan Zeeff | 2019-11-14 | -1/+16 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | As stated from the start, our intention is to add custom mock types as we need them for testing. While writing tests for DuckPond, I noticed that we did not have a mock type for Attachments, so I added one with this commit. In addition, I think it's a very sensible for MockMessage to have an empty list as a default value for the `attachements` attribute. This is equal to what `discord.Message` returns for a message without attachments and makes sure that if you don't explicitely add an attachment to a message, `MockMessage.attachments` tests as falsey. | ||||
| | * | Add bot=False default value to MockMember | Sebastiaan Zeeff | 2019-11-14 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default, a mocked value is considered `truthy` in Python, like all non-empty/non-zero/non-None values in Python. This means that if an attribute is not explicitly set on a mock, it will evaluate at as truthy in a boolean context, since the mock will provide a truthy mocked value by default. This is not the best default value for the `bot` attribute of our MockMember type, since members are rarely bots. It makes much more intuitive sense to me to consider a member to not be a bot, unless we explicitly set `bot=True`. This commit sets that sensible default value that can be overwritten by passing `bot=False` to the constructor or setting the `object.bot` attribute to `False` after the creation of the mock. | ||||
| | * | Merging in master |  Leon Sandøy | 2019-11-13 | -144/+637 | |
| | |\ | |||||
| | * | | Add tests for on_raw_reaction_add. | Leon Sandøy | 2019-11-12 | -14/+70 | |
| | | | | | | | | | | | | | | | | | | | | | | Basically I suck at this and I can't get this return_value thing to work. I'll have Ves look at it to resolve it. As of right now, multiple tests are failing. | ||||
| | * | | Implement a mixed duck test. |  Leon Sandøy | 2019-11-11 | -10/+20 | |
| | | | | | | | | | | | | | | | | Also gets started setting up for the final tests, which will require more mockwork. | ||||
| | * | | Adding ducky count tests and a new AsyncIteratorMock | Leon Sandøy | 2019-11-11 | -15/+93 | |
| | | | | |||||
| | * | | Test is_staff and has_green_checkmark. | Leon Sandøy | 2019-11-03 | -17/+35 | |
| | | | | |||||
| | * | | Resolving merge conflicts from master | Leon Sandøy | 2019-11-03 | -179/+256 | |
| | |\ \ | |||||
| | * | | | Figure out which tests we need. | Leon Sandøy | 2019-10-31 | -0/+81 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds empty tests for all the tests I'd like to add to this pull request. It also adds a few more duckies to the emoji constant list, and adds a single line of clarification to the testing readme. | ||||
| * | | | | Specify assertion to be a tuple comparison |  kwzrd | 2019-11-14 | -1/+1 | |
| | | | | | |||||
| * | | | | Make complex test cases namedtuples, recognize between various authors, pass ↵ | kwzrd | 2019-11-14 | -12/+43 | |
| | | | | | | | | | | | | | | | | | config to subTest | ||||
| * | | | | Hold recent_messages in a list to respect type hint, set config in setUp | kwzrd | 2019-11-14 | -5/+8 | |
| | | | | | |||||
| * | | | | Adjust attachments rule unit test to correcty build the arguments for the ↵ | kwzrd | 2019-11-14 | -7/+11 | |
| | | | | | | | | | | | | | | | | | tested rule | ||||
| * | | | | Use async_test helper to simplify coro testing | kwzrd | 2019-11-13 | -9/+14 | |
| | | | | | |||||
| * | | | | Adjust attachments rule to use MockMessage, restructure test cases | kwzrd | 2019-11-13 | -20/+23 | |
| | | | | | |||||
| * | | | | Adjust links rule to use proper MockMessage | kwzrd | 2019-11-13 | -14/+10 | |
| | | | | | |||||
 |
index : bot | |
| A mirrored copy of the python-discord/bot repository. |
| aboutsummaryrefslogtreecommitdiffstats |