| Commit message (Collapse) | Author | Age | Lines | 
| | 
| 
| 
|  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
|  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
| 
| 
| 
|  | 
Duplicates existing silence and unsilence cache tests for voice
 channels.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
|  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
| 
| 
| 
|  | 
Reduces the number of members created for each test to the bare minimum
required.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| |\   | 
 | 
| | | 
| | 
| |  | 
The `tests/README.md` file still referenced our old custom `AsyncMock` that has been removed in favour of the standard library one that has been introduced in 3.8. This commit fixes this by updating the section.
 | 
| | |\   | 
 | 
| | |\ \   | 
 | 
| | | | | 
| | | | 
| | | | 
| | | |  | 
Tests for this were added additionally
 | 
| | | | | 
| | | | 
| | | | 
| | | | 
| | | |  | 
The tests were not adjusted after the converter was corrected to accept
digits
 | 
| | | | |  | 
 | 
| | |\ \ \   | 
 | 
| | |\ \ \ \   | 
 | 
| | |\ \ \ \ \   | 
 | 
| | |\ \ \ \ \ \  
| | | | | | | | 
| | | | | | | | 
| | | | | | | | 
| | | | | | | | 
| | | | | | | |  | 
# Conflicts:
#	LICENSE-THIRD-PARTY
#	bot/exts/info/doc.py
 | 
| | |\ \ \ \ \ \ \   | 
 | 
| | |\ \ \ \ \ \ \ \  
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
# Conflicts:
#	bot/cogs/doc/cog.py
#	bot/converters.py
#	tests/bot/test_converters.py
 | 
| | | | | | | | | | |  | 
 | 
| |\ \ \ \ \ \ \ \ \ \  
| | |_|_|_|_|_|_|_|/  
| |/| | | | | | | |    | 
 | 
| | | | | | | | | | |  | 
 | 
| | | | | | | | | | |  | 
 | 
| | | |_|_|_|_|_|_|/  
| |/| | | | | | |   
| | | | | | | | |   
| | | | | | | | |    | 
Accounts for no redirects on extensions that are not `.py`.
 | 
| | |\ \ \ \ \ \ \ \   | 
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
The DM sent to infracted users now instructs them to DM modmail if they want to discuss non-ban infractions, so the tests now check if that instruction is present. Note that there already exists a superfluous test for note infractions, for which no DM is sent by design.
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
Previously, the user was instructed to email the appeals email for infraction types that don't remove one from the server. They are now instructed to DM ModMail except for Ban-type infractions. Also removed the URL string literal from the hyperlink to that URL.
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
Changes the mock used for creating an erroneous function in the silence
tests cog to use the side effect property instead of an extra function.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
Removes redundant functionality from the `test_unsilence_helper_fail`
test as it is covered by another test. Keeps the functionality that
isn't being tested elsewhere.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
Combine two tests that are responsible for checking the silence helper
uses the correct channel and message.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| |\ \ \ \ \ \ \ \ \ \  
| | |/ / / / / / / /  
| |/| | | | | | | |    | 
 | 
| | |/ / / / / / / /   | 
 | 
| | | | | | | | | |  | 
 | 
| | | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | |  | 
Adds a missing test assertion, and seperates the voice and text
components of a test.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | |  | 
Adds a missing test assertion, and seperates the voice and text
components of a test.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | |  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | |  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | | | | | | | |  | 
 | 
| |\| | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | |  | 
# Conflicts:
#	bot/constants.py
 | 
| | |\ \ \ \ \ \ \ \  
| | |_|_|_|_|_|_|/  
| |/| | | | | | |    | 
Adds Member Checks Before Changing Voice
 | 
| | | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | | 
| | | | | | | | |  | 
Updates the voice ban action so the infraction pardoning is always run,
and so all operations are handled in the _scheduler. Updates tests.
 | 
| | | |\ \ \ \ \ \ \   | 
 | 
| | | | | | | | | | |  | 
 | 
| | | | |_|_|_|_|/ /  
| | |/| | | | | |   
| | | | | | | | |   
| | | | | | | | |   
| | | | | | | | |   
| | | | | | | | |   
| | | | | | | | |    | 
Adds a check that checks if the user object is an instance of guild
member, before performing guild operations. Adds tests.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | |\ \ \ \ \ \ \ \   | 
 | 
| | | |\ \ \ \ \ \ \ \  
| | | | |/ / / / / /  
| | | |/| | | | | |    | 
 | 
| | | |\ \ \ \ \ \ \ \  
| | | | |_|_|_|_|_|/  
| | | |/| | | | | |    | 
 | 
| | | |\ \ \ \ \ \ \ \  
| | | | |_|_|_|_|/ /  
| | | |/| | | | | |    | 
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
An arbitrary cutoff score of 80 is chosen because it works. A bug in
the test for the same command is also fixed.
 | 
| | | |_|_|/ / / / /  
| |/| | | | | | |    | 
 |