aboutsummaryrefslogtreecommitdiffstats
path: root/tests/helpers.py (unfollow)
Commit message (Collapse)AuthorLines
2020-05-23Better docstring for RedisCache.containsGravatar Leon Sandøy-1/+5
2020-05-23Better docstring for RedisCacheGravatar Leon Sandøy-1/+31
2020-05-23Complete asyncified test suite for RedisCacheGravatar Leon Sandøy-94/+112
This commit just alters existing code to work with the new interface, and with async. All tests are passing successfully.
2020-05-23Finish asyncifying RedisCache methodsGravatar Leon Sandøy-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
2020-05-23Finish .set and .get, and add tests.Gravatar Leon Sandøy-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.
2020-05-22Use autospecced mocks in MockBot for the stats and aiohttp sessionGravatar MarkKoz-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.
2020-05-22Properly mock the redis pool in MockBotGravatar MarkKoz-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.
2020-05-22Fix unawaited coro warning when instantiating Bot for MockBot's specGravatar MarkKoz-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.
2020-05-22Improve aiohttp context manager mocking in snekbox testsGravatar MarkKoz-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.
2020-05-22Fix assertion for `create_task` in duck pond testsGravatar MarkKoz-1/+1
The assertion wasn't using the assertion method. Furthermore, it was testing a non-existent function `create_loop` rather than `create_task`.
2020-05-22Set up async testbedGravatar Leon Sandøy-198/+135
2020-05-22Eval Stats: Replaced `elif` with `else` on icon checkGravatar ks129-1/+1
Co-authored-by: Mark <[email protected]>
2020-05-22Opens a Redis connection in the Bot class.Gravatar Leon Sandøy-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.
2020-05-22Replace redis-py with aioredis.Gravatar Leon Sandøy-60/+104
2020-05-22Add the REDIS_PASSWORD environment variableGravatar Leon Sandøy-137/+115
In production, we will need this password to make a connection to Redis.
2020-05-19Stats: Fix docstringsGravatar ks129-2/+2
Co-authored-by: Joseph Banks <[email protected]>
2020-05-19Stats: Fixed stat namesGravatar ks129-7/+7
Co-authored-by: Joseph Banks <[email protected]>
2020-05-18Add Steam gift card scam to domain blacklistGravatar S. Co1-0/+2
2020-05-19[bug] Adjustment to changes in #941, return message sent by webhook so ↵Gravatar Joseph Banks-1/+1
publish can take place
2020-05-17Add the REDIS_PASSWORD environment variableGravatar Leon Sandøy-6/+17
In production, we will need this password to make a connection to Redis.
2020-05-17Changes discord-py to discord.py in PipfileGravatar Leon Sandøy-7/+15
The `discord-py` package is no longer the official release, and so making this change silences some warnings about deprecation.
2020-05-17Fix incomplete variable renamingGravatar decorator-factory-1/+1
2020-05-17Rename `string` to `greeting`Gravatar decorator-factory-10/+10
2020-05-17Use `Command`-object for `send_help`Gravatar Sebastiaan Zeeff-21/+21
As @mathsman5133 pointed out, it's better to use the `Command`-instance we typically already have in the current context than to rely on parsing the qualified name again. The invocation is now done as: `await ctx.send_help(ctx.command)`
2020-05-17Add a note on user-defined classesGravatar decorator-factory-1/+1
2020-05-17Change standalone programs to interactive sessionsGravatar decorator-factory-8/+13
2020-05-17Apply language improvements proposed from kwzrdGravatar decorator-factory-3/+3
Co-authored-by: kwzrd <[email protected]>
2020-05-17Use `send_help` to invoke command helpGravatar Sebastiaan Zeeff-40/+32
After the refactoring of the help command, we need to use the built-in method of calling the help command: `Context.send_help`. As an argument, the qualified name (a string containing the full command path, including parents) of the command can be passed. Examples: - await ctx.send_help("reminders edit") This would send a help embed with information on `!reminders edit` to the Context. - await ctx.send_help(ctx.command.qualified_name) This would extract the qualified name of the command, which is the full command path, and send a help embed to Context. - await ctx.send_help() This will send the main "root" help embed to the Context.
2020-05-17Fix linting errors introduced by flake8 3.8Gravatar Leon Sandøy-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.
2020-05-17namespace "general" -> "global"Gravatar Leon Sandøy-1/+1
2020-05-17Make redis host and port configurable.Gravatar Leon Sandøy-1/+11