aboutsummaryrefslogtreecommitdiffstats
path: root/tests/helpers.py (unfollow)
Commit message (Collapse)AuthorLines
2020-03-16(Mod Log): Fixed case when `on_guild_channel_update` old or new value is ↵Gravatar ks123-1/+1
empty and with this message formatting go wrong.
2020-03-16Fixed tag search via contents, any keywords.Gravatar Shirayuki Nekomata-1/+1
Fixed `!tag search any` raises `AttributeError`. Changed default value of `keywords` from `None` to `'any'`. This will make it search for keyword `'any'` when there is no keyword.
2020-03-15not awaiting _get_tags_via_content() method as it is non-asyncGravatar RohanJnr-2/+2
2020-03-15convert _get_tags_via_content() method to non-asyncGravatar RohanJnr-1/+1
2020-03-14Tags: add restrictions 1 & 9 from YouTube ToS to ytdl tagGravatar MarkKoz-2/+6
2020-03-14Tags: use constant for command prefix in embed footerGravatar MarkKoz-7/+9
* Add a constant for the footer text * Import constants module rather than its classes
2020-03-14Remove line that calls get_tags() methodGravatar RohanJnr-2/+0
The tags have now been shifted from the database to being static files and hence the get_tags() method has undergone changes. It now dosen't fetch from the database but looks at the local files and we need not call it more than once.
2020-03-12convert get_tags() method to staticmethodGravatar RohanJnr-1/+2
2020-03-12remove repetitive file searchGravatar RohanJnr-3/+2
2020-03-12Update ytdl tag to the new YouTube ToSGravatar Matteo Bertucci-5/+4
2020-03-09Disambiguate codepoint value.Gravatar scragly-1/+2
The usage of 127462 as a unicode start point isn't super clear for other devs coming across the code in future, so assigning it to a nicely named variable with an accompanying inline comment should help make things clearer.
2020-03-09Check lower bound for vote options.Gravatar scragly-0/+2
If the vote command receives less than 2 options, it's not being used for it's intended usage and is considered a user input error.
2020-03-09Implement vote command.Gravatar scragly-0/+18
The vote command takes a given list of options and generates a simple message and corresponding reactions for each so members can quickly take a vote on a subject during in-server discussions and meetings.
2020-03-09Refactored to use paginator like normal `!tag`Gravatar Shirayuki Nekomata-12/+20
- Split `_get_tags_via_content` - introduce `_send_matching_tags` - `_send_matching_tags` will send and paginate like `!tag` - Simplified `is_plural` even more.
2020-03-08Refactor token detection to check all potential substrings in messageGravatar S. Co1-3/+10
2020-03-08Fix filtered extension string out of scope for log messageGravatar S. Co1-2/+2
* Fix typo in file extensions list comprehension
2020-03-08Fixed `is_plural` counting 1 less space.Gravatar Shirayuki Nekomata-1/+1
Co-Authored-By: Mark <[email protected]>
2020-03-08Refactored dense codes, removed obvious type hint.Gravatar Shirayuki Nekomata-15/+25
- Show the process of sanitizing the List[str] `keywords_processed`. - Show the process of finding tag for `matching_tags` ( was `founds` ). - Refactored the logic to find boolean `is_plural`. - Minor wording changes for docstring.
2020-03-08Fixed searching for `,` returing all tags. Made it more descriptive when ↵Gravatar Shirayuki Nekomata-3/+7
multiple tags are found. - Added a truthy check for each `query` since `','.split()` returns a list of two empty strings. - Changed from `Did you mean ...` to `Here are the tags containing the given keyword(s):` to be much more descriptive about the results - they are `tag` and not `term` to be searched.
2020-03-08Refactored tag searching via keywords in contentsGravatar Shirayuki Nekomata-8/+6
- Refactored `if` block - change to only send result when there is any result. - Added better type hinting for `check` argument of `_get_tags_via_content` - changed from `callable` to `Callable[[Iterable], bool]`. Thanks to @markkoz 's reviews Co-Authored-By: Mark <[email protected]>
2020-03-08Implemented `search` as a subcommand for `tag` that will search in contents ↵Gravatar Shirayuki Nekomata-1/+50
instead of names - `!tag search` will search for multiple keywords, separated by comma, and return tags that has ALL of these keywords. ` !tag search any` is the same as `!tag search` but it return tags that has ANY of the keyword instead.
2020-03-07ModLog: fix posting null attachments for deleted message logsGravatar MarkKoz-1/+1
If attachments are not given to `upload_log`, an empty list is used. By default, `zip_longest` uses `None` ass the fill value, so each message was getting paired with a `None` (AKA null) attachment. The filed in the DB is non-nullable so an empty list must be used instead. Fixes #792
2020-03-05Remove excludes from pre-commitGravatar MarkKoz-1/+0
It was excluding files that are already ignored by git. Pre-commit respects git ignore, even with --all-files, so these ignores were redundant.
2020-03-05CI: don't show diff after pre-commit hooksGravatar MarkKoz-1/+1
It's noisy, messy output. It's not of much benefit anyway as users can run git diff locally if they really need to see a diff. They have to do work locally anyway since CI won't commit the fixes pre-commit makes.
2020-03-05Raise BadArgument if no string match.Gravatar scragly-0/+3
2020-03-05Re-use embed, use command converter, raise BadArgument.Gravatar scragly-37/+22
2020-03-05(Information Cog, !roles command test): Removed 's' at end of "Total 1 ↵Gravatar ks123-1/+1
role(s)" due changes in command.
2020-03-05(Information Cog, !roles command): Added 's' to end of 'role' only if there ↵Gravatar ks123-1/+1
is more then 1 role.
2020-03-05(Information Cog, !roles command test): Applied empty parameter change.Gravatar ks123-1/+1
2020-03-05(Information Cog, !roles command): Added empty parameter to pagination (False)Gravatar ks123-1/+1
2020-03-04Update tag files for new linting hooksGravatar S. Co1-59/+59
2020-03-04Convert "get_tags()" and "_get_tag()" to sync functionsGravatar RohanJnr-10/+7
"get_tags()" and "_get_tag()" functions need not be async as we are no longer doing any API call but instead reading from local files.
2020-03-04Use "pathlib" instead of "os" module and context managerGravatar RohanJnr-11/+9
The pathlib module simplifies opening and reading files, hence the os module and the context manager are no longer used.
2020-03-04Use raw strings for docstrings with forward slashesGravatar Sebastiaan Zeeff-3/+3
A few docstrings in `bot.cogs.extensions` have forward slashed in them to escape Markdown rendering when our help feature uses these docstring in a Discord message. However, the use of forward slashes with an invalid escape sequence in docstrings now raises a DeprecationWarning in Python: /home/sebastiaan/pydis/repositories/bot/bot/cogs/extensions.py:72: DeprecationWarning: invalid escape sequence \* PEP 257 (Docstring Conventions, https://www.python.org/dev/peps/pep-0257/) states that raw strings should be used for docstrings that use forward slashes, so I've added the `r`-prefix to the docstrings that use forward slashes.
2020-03-03Update code for pep8-naming complianceGravatar S. Co1-6/+6
2020-03-03Remove CaseInsensitiveDictGravatar S. Co1-94/+0
This was added by the now-removed Snake cog & is not used elsewhere on bot.
2020-03-03Add pep8-naming & relockGravatar S. Co1-14/+28
2020-03-03CI: mock the pipenv binaryGravatar MarkKoz-0/+10
The mock gets used by the flake8 pre-commit hook, which invokes flake8 via `pipenv run flake8`. It's normally useful to use pipenv here cause it ensures flake8 is invoked within the context of the venv. However, in CI, there is no venv - dependencies are installed directly to the system site-packages. `pipenv run` does not work in such case because it tries to create a new venv if one doesn't exist (it doesn't consider the system interpreter to be a venv). This workaround (okay, it's a hack) creates an executable shell script which replaces the original pipenv binary. The shell script simply ignores the first argument (i.e. ignores `run` in `pipenv run`) and executes the rest of the arguments as a command. It essentially makes `pipenv run flake8` equivalent to just having ran `flake8`. When pre-commit executes pipenv, the aforementioned script is what will run.
2020-03-03CI: add a restore key for the pre-commit cacheGravatar MarkKoz-1/+3
A cache for an outdated pre-commit environment may still be useful. It may be the case that only some hooks need to be updated rather than all.
2020-03-03Use pre-commit in pipenv lint scriptGravatar MarkKoz-4/+12
2020-03-03Prevent exception if a watched user sends a DM to the botGravatar S. Co1-2/+9
The previous embed assumed that the messages would be sent on the server, where the channel would have a name and the message would have a jump URL. For a DM, neither of these are present and an exception will be raised when attempting to construct the embed for the webhook to send.
2020-03-03Fix typo in commentGravatar Mark-1/+1
Co-Authored-By: Leon Sandøy <[email protected]>
2020-03-02Add more pre-commit hooksGravatar MarkKoz-10/+27
* Remove trailing whitespaces * Specify error code for a noqa in the free command
2020-03-02Adding helpers to the Filtering whitelistGravatar Leon Sandøy-0/+1
Resolves an issue mentioned in https://github.com/python-discord/bot/issues/767, giving Helpers access to post invites and other things caught by the Filtering cog.
2020-03-01Add grabify (IP logger) domains to banned domainsGravatar Joseph-0/+24
2020-03-01Modify log test regex to be non-os-specificGravatar S. Co1-1/+1
Previous regex utilized a `/`, which doesn't work for comparing against Windows paths, which use `\`
2020-03-01Move syncer confirmation reaction check out of finally clauseGravatar S. Co1-11/+11
Returning directly out of a `finally` clause can cause any exceptions raised in the clause to be discarded, so we can remove the finally clause entirely and shift the control statements into the body of the function
2020-03-01Bump Dependencies & RelockGravatar S. Co1-78/+76
* Remove explicit urllib3 pinning, CVE that caused its pinning has been resolved by 1.25+. This is a child dependency of requests.
2020-03-01Add logging to antimalware cog & expand user feedbackGravatar S. Co1-17/+24
* Add generic handling for multi-file uploads * Log user, id, and blocked extensions * Provide the full list of attachment filenames as a logging extra * Provide feedback on all blacklisted file types uploaded
2020-03-01Optimize DockerfileGravatar Spencer Young-2/+5
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-31 11:02:29 +0000