aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2020-05-28 19:52:41 -0700
committerGravatar MarkKoz <[email protected]>2020-05-28 19:52:41 -0700
commit67472080fef5c38b21d74daa2178c3f35081b58f (patch)
tree6ff1ed806117a849a3e3574c2aa6debdacfd626f
parentAdd more thorough and realistic inputs for token ID and timestamp tests (diff)
Remove is_maybe_token tests
The function was removed due to redundancy. Therefore, its tests are obsolete.
-rw-r--r--tests/bot/cogs/test_token_remover.py33
1 files changed, 0 insertions, 33 deletions
diff --git a/tests/bot/cogs/test_token_remover.py b/tests/bot/cogs/test_token_remover.py
index ffe76865a..5dd12636c 100644
--- a/tests/bot/cogs/test_token_remover.py
+++ b/tests/bot/cogs/test_token_remover.py
@@ -213,39 +213,6 @@ class TokenRemoverTests(unittest.IsolatedAsyncioTestCase):
results = [match[0] for match in results]
self.assertCountEqual((token_1, token_2), results)
- @autospec(TokenRemover, "is_valid_user_id", "is_valid_timestamp")
- def test_is_maybe_token_missing_part_returns_false(self, valid_user, valid_time):
- """False should be returned for tokens which do not have all 3 parts."""
- return_value = TokenRemover.is_maybe_token("x.y")
-
- self.assertFalse(return_value)
- valid_user.assert_not_called()
- valid_time.assert_not_called()
-
- @autospec(TokenRemover, "is_valid_user_id", "is_valid_timestamp")
- def test_is_maybe_token(self, valid_user, valid_time):
- """Should return True if the user ID and timestamp are valid or return False otherwise."""
- subtests = (
- (False, True, False),
- (True, False, False),
- (True, True, True),
- )
-
- for user_return, time_return, expected in subtests:
- valid_user.reset_mock()
- valid_time.reset_mock()
-
- with self.subTest(user_return=user_return, time_return=time_return, expected=expected):
- valid_user.return_value = user_return
- valid_time.return_value = time_return
-
- actual = TokenRemover.is_maybe_token("x.y.z")
- self.assertIs(actual, expected)
-
- valid_user.assert_called_once_with("x")
- if user_return:
- valid_time.assert_called_once_with("y")
-
async def test_delete_message(self):
"""The message should be deleted, and a message should be sent to the same channel."""
await TokenRemover.delete_message(self.msg)