aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | Merge branch 'master' into python-news-statsGravatar ks1292020-05-14-2/+1
| |\ \ \ \ \ \ \ \ \
| * | | | | | | | | | Python News: Implement statsGravatar ks1292020-05-14-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add stat increaser to PEP and maillist posting.
| | | * | | | | | | | Eval Stats: Replaced `elif` with `else` on icon checkGravatar ks1292020-05-22-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
| | | * | | | | | | | Stats: Fix docstringsGravatar ks1292020-05-19-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joseph Banks <[email protected]>
| | | * | | | | | | | Stats: Fixed stat namesGravatar ks1292020-05-19-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joseph Banks <[email protected]>
| | | * | | | | | | | Stats: Added stats for eval channel using (Help/Bot commands/Topical)Gravatar ks1292020-05-16-0/+7
| | | | | | | | | | |
| | | * | | | | | | | Stats: Added stats for eval role uses (Helpers/Developers)Gravatar ks1292020-05-16-0/+5
| | | | | | | | | | |
| | | * | | | | | | | Stats: Added stats for eval successes + failsGravatar ks1292020-05-16-0/+6
| | | | | | | | | | |
| | | * | | | | | | | Stats: Added codeblock correction statsGravatar ks1292020-05-16-0/+2
| | | | | | | | | | |
| | | * | | | | | | | Stats: Create guild boost stat collectionGravatar ks1292020-05-16-1/+16
| | |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Collect Guild boost amount + level and post it to StatsD every hour in task. Added starting to cog `__init__.py` and stopping to `cog_unload`.
| | | | | | * / / / Better docstring for RedisCache.containsGravatar Leon Sandøy2020-05-23-1/+5
| | | | | | |/ / /
| | | | | | * | | Better docstring for RedisCacheGravatar Leon Sandøy2020-05-23-1/+31
| | | | | | | | |
| | | | | | * | | Complete asyncified test suite for RedisCacheGravatar Leon Sandøy2020-05-23-94/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit just alters existing code to work with the new interface, and with async. All tests are passing successfully.
| | | | | | * | | Finish asyncifying RedisCache methodsGravatar Leon Sandøy2020-05-23-16/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - All methods will now do a validation check - Complete interface spec added to class: - .update - .clear - .pop - .to_dict - .length - .contains - .delete - .get - .set
| | | | | | * | | Finish .set and .get, and add tests.Gravatar Leon Sandøy2020-05-23-25/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The .set and .get will accept ints, floats, and strings. These will be converted into "typestrings", which is basically just a simple format that's been invented for this object. For example, an int looks like `b"i|2423"`. Note how it is still stored as a bytestring (like everything in Redis), but because of this prefix we are able to coerce it into the type we want on the way out of the db.
| | | | | | * | | Use autospecced mocks in MockBot for the stats and aiohttp sessionGravatar MarkKoz2020-05-22-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will help catch anything that tries to get/set an attribute/method which doesn't exist. It'll also catch missing/too many parameters being passed to methods.
| | | | | | * | | Properly mock the redis pool in MockBotGravatar MarkKoz2020-05-22-9/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because some of the redis pool/connection methods return futures rather than being coroutines, the redis pool had to be mocked using the CustomMockMixin so it could take advantage of `additional_spec_asyncs` to use AsyncMocks for these future-returning methods.
| | | | | | * | | Fix unawaited coro warning when instantiating Bot for MockBot's specGravatar MarkKoz2020-05-22-10/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fix is to mock the loop and pass it to the Bot. It will then set it as `self.loop` rather than trying to get an event loop from asyncio. The `create_task` patch has been moved to this loop mock rather than being done in MockBot to ensure that it applies to anything calling it when instantiating the Bot.
| | | | | | * | | Improve aiohttp context manager mocking in snekbox testsGravatar MarkKoz2020-05-22-3/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm not sure how it even managed to work before. It was calling the `post` coroutine (without specifying a URL) and then changing `__aenter__`. Now, a separate mock is created for the context manager and the `post` simply returns that mocked context manager.
| | | | | | * | | Fix assertion for `create_task` in duck pond testsGravatar MarkKoz2020-05-22-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The assertion wasn't using the assertion method. Furthermore, it was testing a non-existent function `create_loop` rather than `create_task`.
| | | | | | * | | Set up async testbedGravatar Leon Sandøy2020-05-22-198/+135
| | | | | | | | |
| | | | | | * | | Opens a Redis connection in the Bot class.Gravatar Leon Sandøy2020-05-22-3/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This global connection is the one we will be using in RedisCache to power all our commands. This also ensures that connection is closed when the bot starts its shutdown process.
| | | | | | * | | Replace redis-py with aioredis.Gravatar Leon Sandøy2020-05-22-60/+104
| | | | | | | | |
| | | | | | * | | Add the REDIS_PASSWORD environment variableGravatar Leon Sandøy2020-05-22-137/+115
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In production, we will need this password to make a connection to Redis.
| | | | | | * | | Add the REDIS_PASSWORD environment variableGravatar Leon Sandøy2020-05-17-6/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In production, we will need this password to make a connection to Redis.
| | | | | | * | | Changes discord-py to discord.py in PipfileGravatar Leon Sandøy2020-05-17-7/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `discord-py` package is no longer the official release, and so making this change silences some warnings about deprecation.
| | | | | | * | | Merge branch 'master' into redis_persistenceGravatar Leon Sandøy2020-05-17-460/+269
| | | | | | |\ \ \
| | | | | | * | | | Fix linting errors introduced by flake8 3.8Gravatar Leon Sandøy2020-05-17-34/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Turns out that bumping the flake8 version up to 3.8 introduces a long list of new linting errors. Since this PR is the one that bumps the version, I suppose we will also fix all the linting errors in this branch.
| | | | | | * | | | Merge branch 'master' into redis_persistenceGravatar Leon Sandøy2020-05-17-1/+1
| | | | | | |\ \ \ \
| | | | | | * | | | | namespace "general" -> "global"Gravatar Leon Sandøy2020-05-17-1/+1
| | | | | | | | | | |
| | | | | | * | | | | Make redis host and port configurable.Gravatar Leon Sandøy2020-05-17-1/+11
| | | | | | | | | | |
| | | | | | * | | | | Test suite for the redis dict.Gravatar Leon Sandøy2020-05-17-0/+189
| | | | | | | | | | |
| | | | | | * | | | | Implement .pop, .popitem and .setdefault.Gravatar Leon Sandøy2020-05-17-2/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Turns out the MutableMapping class doesn't give us servicable implementations of these, so we need to implement them ourselves. Also, let's not have keys returned as bytestrings.
| | | | | | * | | | | Add fakeredis to the PipfileGravatar Leon Sandøy2020-05-17-3/+27
| | | | | | | | | | |
| | | | | | * | | | | Implement .get, equality, and membership checkGravatar Leon Sandøy2020-05-17-2/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is supposed to be provided by our MutableMapping mixin, but unit tests are demonstrating that these don't really work as intended.
| | | | | | * | | | | Implements .clear with hash deletion.Gravatar Leon Sandøy2020-05-17-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This would've been implemented by MutableMapping, but that implementation is O(n) instead of O(1) since it just iterates the entire hash and does HDEL. Feels wasteful.
| | | | | | * | | | | copy should dictify the .items(), not just keys.Gravatar Leon Sandøy2020-05-16-1/+1
| | | | | | | | | | |
| | | | | | * | | | | Add basic dict methods for RedisDict.Gravatar Leon Sandøy2020-05-16-35/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The rest of the features should be provided by the MutableMapping abc we're interfacing. Specifically, MutableMapping provides these: .pop, .popitem, .clear, .update, .setdefault, __contains__, .keys, .items, .values, .get, __eq__, and __ne__.
| | | | | | * | | | | Refactor - no more mixins!Gravatar Leon Sandøy2020-05-16-59/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was brought to my attention that we may need several caches per Cog for some of our Cogs. This means that the original approach of having this be a mixin is a little bit problematic. Instead, RedisDict will be instantiated directly inside the class you want it in. By leveraging __set_name__, we can create a namespace containing both the class name and the variable name without the user having to provide anything. For example, if you create an attribute MyClass.cache = RedisDict(), this will be using the redis namespace 'MyClass.cache.' before anything you store in it. With this approach, it is also possible to instantiate a RedisDict with a custom namespace by simply passing it into the constructor. - RedisDict("firedog") will create items with the 'firedog.your_item' prefix. - If there are multiple RedisDicts using the same namespace, an underscore will be appended to the namespace, such that the second RedisDict("firedog") will actually create items in the 'firedog_.your_item' namespace. This is also possible to use outside of classes, so long as you provide a custom namespace when you instantiate it. Custom namespaces will always take precedence over automatic 'Class.attribute_name' ones.
| | | | | | * | | | | Boilerplate for the RedisCacheMixinGravatar Leon Sandøy2020-05-16-0/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're using __init_subclass__ to initialize our RedisDict with the subclass name as a namespace. This will be prefixed to all data that we store, so that there won't be collisions between different subclasses.
| | | | | | * | | | | Adding redis-py to the PipfileGravatar Leon Sandøy2020-05-16-65/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the module we will be using to interface with Redis.
| | | | | | * | | | | Adding redis to docker-compose file.Gravatar Leon Sandøy2020-05-16-0/+4
| | | |_|_|/ / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is almost hilariously easy since we can just use the official image for it.
| | | | | | | | * | AntiMalware tests - Switched from monkeypatch to unittest.patchGravatar Jannes Jonkers2020-05-25-2/+2
| | | | | | | | | |
| | | | | | | | * | AntiMalware Refactor - Moved embed descriptions into constants, added tests ↵Gravatar Jannes Jonkers2020-05-11-36/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | for embed descriptions
| | | | | | | | * | AntiMalware Tests - Removed exact log content checksGravatar Jannes Jonkers2020-05-11-19/+2
| | | | | | | | | |
| | | | | | | | * | AntiMalware Tests - added a missing case for no extensions in ↵Gravatar Jannes Jonkers2020-05-11-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_get_disallowed_extensions
| | | | | | | | * | AntiMalware Tests - extracted the method for determining disallowed ↵Gravatar Jannes Jonkers2020-05-11-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | extensions and added a test for it.
| | | | | | | | * | AntiMalware Tests - extracted the method for determining disallowed ↵Gravatar Jannes Jonkers2020-05-11-6/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | extensions and added a test for it.
| | | | | | | | * | AntiMalware Tests - implemented minor feedbackGravatar Jannes Jonkers2020-05-11-12/+12
| | | | | | | | | |
| | | | | | | | * | Update tests/bot/cogs/test_antimalware.pyGravatar MrGrote2020-05-08-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-28 16:33:05 +0000