FEATURE 14.1: autoreaction on invalid nick
- если фулл_нейм НЕ содержит букв английских или русских то считаем его невалидным, сообщение удаляем, юзеру пишем о правиле 6
This commit is contained in:
+20
-3
@@ -18,7 +18,7 @@ from src import helpers
|
||||
helpers.is_url_reachable = lambda a: True
|
||||
from src.commands import bot
|
||||
from src.constants import BDMTSS_ID, LUTZ_ID, ADMIN_ID, HELLO_EXAMPLES
|
||||
from src.filters import is_white_id
|
||||
from src.filters import is_white_id, is_invalid_name
|
||||
|
||||
# отключаем логи бота, в тестах они не нужны
|
||||
logger.setLevel(logging.ERROR)
|
||||
@@ -99,6 +99,23 @@ class TestOther(TestCase):
|
||||
m = MagicMock(from_user=MagicMock(id=text))
|
||||
self.assertEqual(is_white_id(m), expected)
|
||||
|
||||
def test_is_invalid_name(self):
|
||||
params = (
|
||||
(True, None),
|
||||
(True, ''),
|
||||
(True, ' '),
|
||||
(True, '.'),
|
||||
(True, '#!%%@$#@'),
|
||||
(True, '⛔️ '),
|
||||
(True, '212434343'),
|
||||
(False, 'a212434343'),
|
||||
(False, '212434343я'),
|
||||
)
|
||||
for expected, text in params:
|
||||
with self.subTest(f"check non-is_invalid_name({text})"):
|
||||
m = MagicMock(from_user=MagicMock(full_name=text))
|
||||
self.assertEqual(is_invalid_name(m), expected)
|
||||
|
||||
|
||||
class TestBot(IsolatedAsyncioTestCase):
|
||||
|
||||
@@ -476,8 +493,8 @@ class TestBot(IsolatedAsyncioTestCase):
|
||||
self.assertEqual(expected, RESULTS[0][0]['text'])
|
||||
RESULTS.clear()
|
||||
|
||||
async def test_empty_name_user(self):
|
||||
await self.bot.process_new_updates([get_update('новый текст', first_name='')])
|
||||
async def test_invalid_name_user(self):
|
||||
await self.bot.process_new_updates([get_update('новый текст', first_name=None)])
|
||||
self.assertEqual(RESULTS[0], ({'chat_id': 11, 'message_id': 1}, 'deleteMessage'))
|
||||
self.assertEqual(RESULTS[1][1], 'sendMessage')
|
||||
self.assertEqual(RESULTS[1][0]['text'], '@None\nНапоминаем правило: Твой ник должен быть читаемым и понятным')
|
||||
|
||||
+16
-1
@@ -2,7 +2,7 @@ from unittest import TestCase, main
|
||||
from unittest.mock import MagicMock
|
||||
from src.constants import SPAM, BAN_WORDS, ADMIN_ID, RULES_TEXT
|
||||
from src.helpers import is_spam, is_mixed, is_ban_words_in_caption, is_in_not_allowed, is_nongrata, is_admin, \
|
||||
fetch_rule, cleaned_text, remove_spaces
|
||||
fetch_rule, cleaned_text, remove_spaces, has_no_letters
|
||||
|
||||
CASES = (
|
||||
"Оформим резидeнтствo ОАЭ без предoплаты за 5000 дирхам. Пoдробнoсти в лc",
|
||||
@@ -115,6 +115,21 @@ class TestOthers(TestCase):
|
||||
with self.subTest(f"remove_spaces {text}"):
|
||||
self.assertEqual(expected, remove_spaces(text))
|
||||
|
||||
def test_has_no_letters(self):
|
||||
params = (
|
||||
(False, 'f1 b2 print text '),
|
||||
(True, ''),
|
||||
(True, ' '),
|
||||
(True, '12 @# &*'),
|
||||
(False, '12 @# &* f'),
|
||||
(False, '12 @# &* Z'),
|
||||
(False, '12 @# &* я'),
|
||||
(False, '12 @# &* Я'),
|
||||
)
|
||||
for expected, text in params:
|
||||
with self.subTest(f"has_no_letters {text}"):
|
||||
self.assertEqual(expected, has_no_letters(text))
|
||||
|
||||
|
||||
if __name__ == '__main__':
|
||||
main()
|
||||
|
||||
Reference in New Issue
Block a user