aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* 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.
* Update to Python 3.8 and discord.py 1.3.2Gravatar Sebastiaan Zeeff2020-02-23-79/+37
| | | | | | I've changed the Python version in our Pipfile to Python 3.8. The main advantage of Python 3.8 is that it comes with significant upgrades to the unittest module which allow us to test asyncio-based code more easily. While our current test suite runs in P3.8 "out of the box", it currently still relies on many of the workarounds we had to use to test asynchronous code in Python 3.7. A future commit will replace these workarounds with the new tools available in Python 3.8. This commit also updates our discord.py version to 1.3.2. Versions of discord.py <= 1.3.1 contain a bug that causes errors in the new unittest tools that come with Python 3.8. For more specific details, see https://github.com/Rapptz/discord.py/pull/2570.
* Merge pull request #749 from python-discord/reminder-enhancementsGravatar Leon Sandøy2020-02-23-15/+47
|\ | | | | Reminder Embed Enhancements & Whitelist Expansion
| * Merge branch 'master' into reminder-enhancementsGravatar S. Co12020-02-23-607/+951
| |\ | |/ |/|
* | Add Sentdex server to whitelistGravatar Leon Sandøy2020-02-23-0/+1
| |
* | Merge pull request #773 from python-discord/reminder_missing_usersGravatar Joseph2020-02-23-17/+37
|\ \ | | | | | | Check reminder user and channel before send and schedule.
| * \ Merge branch 'master' into reminder_missing_usersGravatar Joseph2020-02-23-120/+101
| |\ \ | |/ / |/| |
* | | Merge pull request #776 from python-discord/checkout-msg-logGravatar S. Co12020-02-23-14/+13
|\ \ \ | | | | | | | | Change verification post log level to info, tidy code.
| * \ \ Merge branch 'master' into checkout-msg-logGravatar S. Co12020-02-23-2/+6
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #756 from python-discord/bug/backend/s329/docker-compose-ttyGravatar scragly2020-02-24-0/+2
|\ \ \ \ | | | | | | | | | | Fix missing Django logs when using Docker Compose
| * \ \ \ Merge branch 'master' into bug/backend/s329/docker-compose-ttyGravatar scragly2020-02-24-571/+888
| |\ \ \ \
| * | | | | Fix missing Django logs when using Docker ComposeGravatar MarkKoz2020-02-16-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed by allocating a pseudo-tty to the web and bot services in Docker Compose.
* | | | | | Merge pull request #774 from python-discord/cooldown-log-levelGravatar S. Co12020-02-23-2/+4
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Reduce log level of tag cooldown notice.
| * | | | | Reduce log level of tag cooldown notice.Gravatar scragly2020-02-24-2/+4
|/ / / / /
| | * / / Change verification post log level to info, tidy code.Gravatar scragly2020-02-24-14/+13
| |/ / / |/| | |
* | | | Merge pull request #741 from Denayder/information-refactorGravatar S. Co12020-02-23-73/+50
|\ \ \ \ | | | | | | | | | | Refactor information.py with lots of changes that makes the overall code cleaner and faster
| * \ \ \ Merge branch 'master' into information-refactorGravatar S. Co12020-02-23-562/+978
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #771 from python-discord/paginator-react-404Gravatar S. Co12020-02-23-18/+22
|\ \ \ \ \ | | | | | | | | | | | | Fix paginator erroring on clearing reactions from non-existing messages.
| * \ \ \ \ Merge branch 'master' into paginator-react-404Gravatar S. Co12020-02-23-14/+11
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #759 from ↵Gravatar S. Co12020-02-23-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/optimize/pagination/emoji_collection_structure Pagination migrations - Emoji Data Structure Modified
| * \ \ \ \ \ Merge branch 'master' into optimize/pagination/emoji_collection_structureGravatar S. Co12020-02-23-478/+814
| |\ \ \ \ \ \ | |/ / / / / / |/| | | | | |
* | | | | | | Merge pull request #772 from python-discord/dm_failure_fixGravatar S. Co12020-02-23-12/+9
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Don't log exception traceback on Forbidden for welcomes.
| * | | | | | | Don't log exception traceback on Forbidden for welcomes.Gravatar scragly2020-02-23-12/+9
|/ / / / / / /
| * | | | | | Pagination migrations - Emoji Data Structure ModifiedGravatar F4zi2020-02-17-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changed the pagination emoji collection from list to tuple This change was suggested since this collection is constant
| | * | | | | Define `_count` in `__init__`.Gravatar scragly2020-02-23-0/+1
| | | | | | |
| | * | | | | Suppress NotFound on react clear, tidy imports.Gravatar scragly2020-02-23-18/+21
| | | | | | |
| | | * | | | Remove the space that makes the test failGravatar Deniz2020-02-21-1/+1
| | | | | | |
| | | * | | | Use the code provided by sco1 to fix the checks failing.Gravatar Deniz2020-02-21-5/+10
| | | | | | |
| | | * | | | Add thousand separators to Members count, closes #744Gravatar Deniz2020-02-13-4/+4
| | | | | | |
| | | * | | | Update the tests accordingly to reflect the new changesGravatar Deniz2020-02-13-3/+3
| | | | | | |
| | | * | | | Use collections.Counter properly. Use the ChannelType enum instead of the ↵Gravatar Deniz2020-02-12-12/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | __class__ attribute, and re-add the None check for !user roles.
| | | * | | | Use the enum itself instead of its string valueGravatar Deniz2020-02-08-6/+6
| | | | | | |
| | | * | | | Fix some of the testing for information.py; I think this should be it. ↵Gravatar Deniz2020-02-06-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | (hopefully).
| | | * | | | Change if statement to elif; if the first if statement returns true, the ↵Gravatar Deniz2020-02-06-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | second cannot be true making it unneccesary to check
| | | * | | | Fix up imports a bit; there's no need to import all of discord and typing ↵Gravatar Deniz2020-02-06-10/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | for just 1 or 2 uses (e.g. Union, and Message).
| | | * | | | Instead of sending a message everytime a role can't be converted, append it ↵Gravatar Deniz2020-02-06-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | to a list, and then send them it at once (less spammy)
| | | * | | | Update tests to reflect status changesGravatar Deniz2020-02-06-4/+4
| | | | | | |
| | | * | | | Refactor how channels and statuses are counted; using Counter() - way cleaner.Gravatar Deniz2020-02-06-35/+17
| | | | | | |
| | | * | | | Remove some a lot of unneccesary newlines that arguably make it harder to readGravatar Deniz2020-02-06-9/+2
| | | | | | |
| | | * | | | Roles cannot return None because everyone has the Developer role by default, ↵Gravatar Deniz2020-02-06-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | and non-verified users cannot use this command.
| | | * | | | No longer check if every role is @everyone; just skip the first element in ↵Gravatar Deniz2020-02-06-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | the list
| | | | | * | Remove call to delete reminder, as ensure method already does it.Gravatar scragly2020-02-23-1/+0
| | | | | | |
| | | | | * | Check reminder user and channel before send and schedule.Gravatar scragly2020-02-23-17/+38
| |_|_|_|/ / |/| | | | |
* | | | | | Don't set project log level so it uses root level.Gravatar scragly2020-02-23-1/+1
| |/ / / / |/| | | |
* | | | | Change defcon log levels from warning to infoGravatar Joseph Banks2020-02-21-2/+2
| | | | |
* | | | | Enable logging warnings to SentryGravatar Joseph Banks2020-02-21-3/+12
| | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-11-02 20:37:30 +0000