Add coverage tests

telegram_downloader_bot/utils.py

@@ -1,4 +1,3 @@
-import integv
 import os
 import re
 
@@ -7,43 +6,45 @@ from pyrogram import Client
 from pyrogram.types import Message
 from tiktok_downloader import snaptik
 
-from telegram_downloader_bot.logger import log
+def sanitize_name(input: str) -> str:
+    """Sanize string by removing non aplhanumeric characters and spaces."""
+    output = re.sub("[^a-zA-Z0-9- ]", "", input)
+    output = output.replace(" ", "_")
+    return output
 
 
-async def get_user_folder(storage_path: os.path, message: Message) -> os.path:
+def get_user_folder(storage_path: os.path, message: Message) -> os.path:
     """ Determine folder name used to save the media to. Depending on
     which type of message (forwarded, direct) detect that person's
     or group's name."""
 
+    # Message forwarded from someone
     if message.forward_from:
         user = message.forward_from
-        # User's first and last name for folder name,
-        # fallback to user ID if not available
-        user_folder_name = (
-            f"{user.first_name}_{user.last_name}".strip()
-            if user.first_name and user.last_name
-            else str(user.id)
-        )
+        if user.first_name and user.last_name:
+            # User's first and last name for folder name
+            user_folder_name = f"{user.first_name} {user.last_name}"
+        else:
+            # fallback to user ID if not available
+            user_folder_name = str(user.id)
+
+    # Message forwarded from chat
     elif message.forward_from_chat:
         user = message.forward_from_chat
-        # Use chat title for groups and channels
-        user_folder_name = "".join(
-            c for c in user.title if c.isalnum() or c in (" ", "_")
-        ).rstrip()
+        user_folder_name = user.title
+
+    # Direct message from user
     else:
         user = message.from_user
-        # User's first and last name for folder name,
-        # fallback to user ID if not available
-        user_folder_name = (
-            f"{user.first_name}_{user.last_name}".strip()
-            if user.first_name and user.last_name
-            else str(user.id)
-        )
+        if user.first_name and user.last_name:
+            # User's first and last name for folder name
+            user_folder_name = f"{user.first_name} {user.last_name}"
+        else:
+            # fallback to user ID if not available
+            user_folder_name = str(user.id)
 
     # Sanitize the folder name
-    user_folder_name = "".join(
-        c for c in user_folder_name if c.isalnum() or c in (" ", "_")
-    ).rstrip()
+    user_folder_name = sanitize_name(user_folder_name)
 
     user_folder = os.path.join(storage_path, "telegram", user_folder_name)
     os.makedirs(user_folder, exist_ok=True)
@@ -92,41 +93,22 @@ async def handle_media_message_contents(storage_path: os.path,
         await message.reply_text("Unknown media type!")
 
 
-def download_tt_video(storage_path: str, url: str) -> bool:
+def download_tt_video(storage_path: str, url: str) -> None:
     """Downloads tiktok video from a given URL.
     Makes sure the video integrity is correct."""
 
     videos = snaptik(url)
     now = datetime.datetime.now()
-    max_tries = 5
-
-    log.debug(f"Downloading video from {url}...")
 
     for video in videos:
         video_filename = now.strftime("video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
-        video_filepath: os.path = os.path.join(
-            storage_path, "tiktok", video_filename)
-
-        for i in range(max_tries):
-            video_content = video.download().getbuffer()
-            is_valid_mp4 = integv.verify(
-                video_content.tobytes(), file_type="mp4")
-            log.debug(
-                f"Attempt {i+1}/{max_tries} to "
-                "download video, video valid: {is_valid_mp4}"
-            )
-            if is_valid_mp4:
-                break
-
-        if not is_valid_mp4:
-            log.error("Downloaded video is not a valid mp4 file")
-            return False
+        video_filepath: os.path = os.path.join(storage_path,
+                                               "tiktok",
+                                               video_filename)
+        video_content = video.download().getbuffer()
 
         with open(video_filepath, "wb") as f:
             f.write(video_content)
-            log.debug("Video saved successfully")
-
-        return True
 
 
 def make_fs(storaga_path: str) -> None:

test-requirements.txt

@@ -2,4 +2,5 @@ codespell
 flake8
 bandit
 pytest
-stestr
+stestr
+coverage

tests/test_utils.py

@@ -1,13 +1,16 @@
+# test_utils.py
+
 import unittest
 import os
 import re
-import asyncio
-import tempfile
 import shutil
+import tempfile
 from unittest.mock import Mock, AsyncMock, patch
 from datetime import datetime
 
+# Adjusted import statement
 from telegram_downloader_bot.utils import (
+    sanitize_name,
     get_user_folder,
     handle_media_message_contents,
     download_tt_video,
@@ -17,8 +20,32 @@ from telegram_downloader_bot.utils import (
 )
 
 from pyrogram.types import Message, User, Chat
+from pyrogram import Client
 
-class TestGetUserFolder(unittest.IsolatedAsyncioTestCase):
+
+class TestSanitizeName(unittest.TestCase):
+    def test_alphanumeric_input(self):
+        input_str = "JohnDoe123"
+        expected_output = "JohnDoe123"
+        self.assertEqual(sanitize_name(input_str), expected_output)
+
+    def test_input_with_special_chars(self):
+        input_str = "John Doe!@#"
+        expected_output = "John_Doe"
+        self.assertEqual(sanitize_name(input_str), expected_output)
+
+    def test_input_with_only_special_chars(self):
+        input_str = "!@#$%^&*()"
+        expected_output = ""
+        self.assertEqual(sanitize_name(input_str), expected_output)
+
+    def test_empty_input(self):
+        input_str = ""
+        expected_output = ""
+        self.assertEqual(sanitize_name(input_str), expected_output)
+
+
+class TestGetUserFolder(unittest.TestCase):
     def setUp(self):
         # Create a temporary directory for each test
         self.tmp_path = tempfile.mkdtemp()
@@ -27,7 +54,7 @@ class TestGetUserFolder(unittest.IsolatedAsyncioTestCase):
         # Remove the directory after the test
         shutil.rmtree(self.tmp_path)
 
-    async def test_forward_from_full_name(self):
+    def test_forward_from_full_name(self):
         user = Mock()
         user.first_name = "John"
         user.last_name = "Doe"
@@ -38,12 +65,12 @@ class TestGetUserFolder(unittest.IsolatedAsyncioTestCase):
         message.forward_from_chat = None
         message.from_user = None
 
-        result = await get_user_folder(self.tmp_path, message)
+        result = get_user_folder(self.tmp_path, message)
         expected_folder = os.path.join(self.tmp_path, "telegram", "John_Doe")
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
 
-    async def test_forward_from_first_name_only(self):
+    def test_forward_from_first_name_only(self):
         user = Mock()
         user.first_name = "John"
         user.last_name = None
@@ -54,28 +81,12 @@ class TestGetUserFolder(unittest.IsolatedAsyncioTestCase):
         message.forward_from_chat = None
         message.from_user = None
 
-        result = await get_user_folder(self.tmp_path, message)
+        result = get_user_folder(self.tmp_path, message)
         expected_folder = os.path.join(self.tmp_path, "telegram", "12345")
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
 
-    async def test_forward_from_no_name(self):
-        user = Mock()
-        user.first_name = None
-        user.last_name = None
-        user.id = 12345
-
-        message = Mock()
-        message.forward_from = user
-        message.forward_from_chat = None
-        message.from_user = None
-
-        result = await get_user_folder(self.tmp_path, message)
-        expected_folder = os.path.join(self.tmp_path, "telegram", "12345")
-        self.assertEqual(result, expected_folder)
-        self.assertTrue(os.path.exists(expected_folder))
-
-    async def test_forward_from_chat_special_chars(self):
+    def test_forward_from_chat_title(self):
         chat = Mock()
         chat.title = "My *Awesome* Group/Chat!"
 
@@ -84,14 +95,14 @@ class TestGetUserFolder(unittest.IsolatedAsyncioTestCase):
         message.forward_from_chat = chat
         message.from_user = None
 
-        result = await get_user_folder(self.tmp_path, message)
+        result = get_user_folder(self.tmp_path, message)
         expected_folder = os.path.join(
             self.tmp_path, "telegram", "My_Awesome_GroupChat"
         )
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
 
-    async def test_from_user_full_name(self):
+    def test_from_user_full_name(self):
         user = Mock()
         user.first_name = "Jane"
         user.last_name = "Doe"
@@ -102,14 +113,14 @@ class TestGetUserFolder(unittest.IsolatedAsyncioTestCase):
         message.forward_from_chat = None
         message.from_user = user
 
-        result = await get_user_folder(self.tmp_path, message)
+        result = get_user_folder(self.tmp_path, message)
         expected_folder = os.path.join(self.tmp_path, "telegram", "Jane_Doe")
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
 
-    async def test_from_user_first_name_only(self):
+    def test_from_user_id(self):
         user = Mock()
-        user.first_name = "Jane"
+        user.first_name = None
         user.last_name = None
         user.id = 54321
 
@@ -118,27 +129,11 @@ class TestGetUserFolder(unittest.IsolatedAsyncioTestCase):
         message.forward_from_chat = None
         message.from_user = user
 
-        result = await get_user_folder(self.tmp_path, message)
+        result = get_user_folder(self.tmp_path, message)
         expected_folder = os.path.join(self.tmp_path, "telegram", "54321")
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
 
-    async def test_special_characters_in_name(self):
-        user = Mock()
-        user.first_name = "Ja*ne"
-        user.last_name = "Do/e"
-        user.id = 54321
-
-        message = Mock()
-        message.forward_from = None
-        message.forward_from_chat = None
-        message.from_user = user
-
-        result = await get_user_folder(self.tmp_path, message)
-        expected_folder = os.path.join(self.tmp_path, "telegram", "Jane_Doe")
-        self.assertEqual(result, expected_folder)
-        self.assertTrue(os.path.exists(expected_folder))
-
 
 class TestHandleMediaMessageContents(unittest.IsolatedAsyncioTestCase):
     def setUp(self):
@@ -149,16 +144,68 @@ class TestHandleMediaMessageContents(unittest.IsolatedAsyncioTestCase):
         # Remove the directory after the test
         shutil.rmtree(self.tmp_path)
 
-    @patch("telegram_downloader_bot.utils.get_user_folder")
-    async def test_document(self, mock_get_user_folder):
+    @patch('telegram_downloader_bot.utils.get_user_folder')
+    async def test_handle_video(self, mock_get_user_folder):
         user_folder = os.path.join(self.tmp_path, "user_folder")
         mock_get_user_folder.return_value = user_folder
         os.makedirs(user_folder, exist_ok=True)
 
-        client = Mock()
+        client = Mock(spec=Client)
         client.download_media = AsyncMock()
 
-        message = Mock()
+        message = Mock(spec=Message)
+        message.document = None
+        message.photo = None
+        message.video = Mock()
+        message.video.file_id = "video_file_id"
+        message.animation = None
+        message.reply_text = AsyncMock()
+
+        await handle_media_message_contents(self.tmp_path, client, message)
+
+        expected_file_name = f"video_{message.video.file_id}.mp4"
+        expected_file_path = os.path.join(user_folder, expected_file_name)
+        client.download_media.assert_awaited_once_with(
+            message, expected_file_path)
+        message.reply_text.assert_awaited_once_with(
+            f"Video saved to {user_folder}")
+
+    @patch('telegram_downloader_bot.utils.get_user_folder')
+    async def test_handle_animation(self, mock_get_user_folder):
+        user_folder = os.path.join(self.tmp_path, "user_folder")
+        mock_get_user_folder.return_value = user_folder
+        os.makedirs(user_folder, exist_ok=True)
+
+        client = Mock(spec=Client)
+        client.download_media = AsyncMock()
+
+        message = Mock(spec=Message)
+        message.document = None
+        message.photo = None
+        message.video = None
+        message.animation = Mock()
+        message.animation.file_id = "animation_file_id"
+        message.reply_text = AsyncMock()
+
+        await handle_media_message_contents(self.tmp_path, client, message)
+
+        expected_file_name = f"gif_{message.animation.file_id}.gif"
+        expected_file_path = os.path.join(user_folder, expected_file_name)
+        client.download_media.assert_awaited_once_with(
+            message.animation, expected_file_path)
+        message.reply_text.assert_awaited_once_with(
+            f"GIF saved to {user_folder}")
+
+    @patch('telegram_downloader_bot.utils.get_user_folder')
+    async def test_handle_document(self, mock_get_user_folder):
+        user_folder = os.path.join(self.tmp_path, "user_folder")
+        mock_get_user_folder.return_value = user_folder
+        os.makedirs(user_folder, exist_ok=True)
+
+        client = Mock(spec=Client)
+        client.download_media = AsyncMock()
+
+        message = Mock(spec=Message)
         message.document = Mock()
         message.document.file_name = "test_document.pdf"
         message.photo = None
@@ -168,44 +215,48 @@ class TestHandleMediaMessageContents(unittest.IsolatedAsyncioTestCase):
 
         await handle_media_message_contents(self.tmp_path, client, message)
 
+        expected_file_path = os.path.join(user_folder, "test_document.pdf")
         client.download_media.assert_awaited_once_with(
-            message, os.path.join(user_folder, "test_document.pdf")
-        )
-        message.reply_text.assert_awaited_once_with(f"Document saved to {user_folder}")
+            message, expected_file_path)
+        message.reply_text.assert_awaited_once_with(
+            f"Document saved to {user_folder}")
 
-    @patch("telegram_downloader_bot.utils.get_user_folder")
-    async def test_photo(self, mock_get_user_folder):
+    @patch('telegram_downloader_bot.utils.get_user_folder')
+    async def test_handle_photo(self, mock_get_user_folder):
         user_folder = os.path.join(self.tmp_path, "user_folder")
         mock_get_user_folder.return_value = user_folder
         os.makedirs(user_folder, exist_ok=True)
 
-        client = Mock()
+        client = Mock(spec=Client)
         client.download_media = AsyncMock()
 
-        message = Mock()
+        message = Mock(spec=Message)
         message.document = None
         message.photo = Mock()
-        message.photo.file_id = "1234567890"
+        message.photo.file_id = "photo_file_id"
         message.video = None
         message.animation = None
         message.reply_text = AsyncMock()
 
         await handle_media_message_contents(self.tmp_path, client, message)
 
-        expected_file = os.path.join(user_folder, f"photo_{message.photo.file_id}.jpg")
-        client.download_media.assert_awaited_once_with(message.photo, expected_file)
-        message.reply_text.assert_awaited_once_with(f"Photo saved to {user_folder}")
+        expected_file_name = f"photo_{message.photo.file_id}.jpg"
+        expected_file_path = os.path.join(user_folder, expected_file_name)
+        client.download_media.assert_awaited_once_with(
+            message.photo, expected_file_path)
+        message.reply_text.assert_awaited_once_with(
+            f"Photo saved to {user_folder}")
 
-    @patch("telegram_downloader_bot.utils.get_user_folder")
-    async def test_unknown_media(self, mock_get_user_folder):
+    @patch('telegram_downloader_bot.utils.get_user_folder')
+    async def test_handle_unknown_media(self, mock_get_user_folder):
         user_folder = os.path.join(self.tmp_path, "user_folder")
         mock_get_user_folder.return_value = user_folder
         os.makedirs(user_folder, exist_ok=True)
 
-        client = Mock()
+        client = Mock(spec=Client)
         client.download_media = AsyncMock()
 
-        message = Mock()
+        message = Mock(spec=Message)
         message.document = None
         message.photo = None
         message.video = None
@@ -214,7 +265,7 @@ class TestHandleMediaMessageContents(unittest.IsolatedAsyncioTestCase):
 
         await handle_media_message_contents(self.tmp_path, client, message)
 
-        client.download_media.assert_not_awaited()
+        client.download_media.assert_not_called()
         message.reply_text.assert_awaited_once_with("Unknown media type!")
 
 
@@ -222,92 +273,145 @@ class TestDownloadTTVideo(unittest.TestCase):
     def setUp(self):
         # Create a temporary directory for each test
         self.tmp_path = tempfile.mkdtemp()
+        os.makedirs(os.path.join(self.tmp_path, "tiktok"), exist_ok=True)
+
+        # Paths to the valid and invalid video files
+        self.valid_video_path = os.path.join(self.tmp_path, "valid.mp4")
+        with open(self.valid_video_path, 'wb') as f:
+            f.write(b'valid mp4 content')
+
+        self.invalid_video_path = os.path.join(self.tmp_path, "invalid.mp4")
+        with open(self.invalid_video_path, 'wb') as f:
+            f.write(b'invalid mp4 content')
 
     def tearDown(self):
         # Remove the directory after the test
         shutil.rmtree(self.tmp_path)
 
-    @patch("telegram_downloader_bot.utils.snaptik")
-    @patch("telegram_downloader_bot.utils.integv.verify")
-    @patch("telegram_downloader_bot.utils.datetime")
-    def test_success(self, mock_datetime, mock_verify, mock_snaptik):
-        mock_video = Mock()
-        mock_video.download.return_value.getbuffer.return_value = b"video_content"
-        mock_snaptik.return_value = [mock_video]
-        mock_verify.return_value = True
-
+    @patch('telegram_downloader_bot.utils.snaptik')
+    @patch('telegram_downloader_bot.utils.datetime')
+    def test_download_tt_video_with_valid_video(self, mock_datetime, mock_snaptik):
+        # Mock datetime
         mock_now = datetime(2023, 1, 1, 12, 0, 0)
-        mock_datetime.now.return_value = mock_now
+        mock_datetime.datetime.now.return_value = mock_now
 
-        result = download_tt_video(self.tmp_path, "http://tiktok.com/video123")
-        self.assertTrue(result)
+        # Read the content of valid.mp4
+        with open(self.valid_video_path, 'rb') as f:
+            valid_video_content = f.read()
 
-        video_filename = mock_now.strftime("video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
+        # Mock snaptik to return a video that returns valid.mp4 content
+        mock_video = Mock()
+        mock_video.download.return_value.getbuffer.return_value = valid_video_content
+        mock_snaptik.return_value = [mock_video]
+
+        # Call the function
+        download_tt_video(self.tmp_path, "http://tiktok.com/video123")
+
+        # Verify that the file was saved correctly
+        video_filename = mock_now.strftime(
+            "video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
         video_filepath = os.path.join(self.tmp_path, "tiktok", video_filename)
         self.assertTrue(os.path.exists(video_filepath))
 
-    @patch("telegram_downloader_bot.utils.snaptik")
-    @patch("telegram_downloader_bot.utils.integv.verify")
-    @patch("telegram_downloader_bot.utils.datetime")
-    def test_failure(self, mock_datetime, mock_verify, mock_snaptik):
-        mock_video = Mock()
-        mock_video.download.return_value.getbuffer.return_value = b"video_content"
-        mock_snaptik.return_value = [mock_video]
-        mock_verify.return_value = False
+        with open(video_filepath, 'rb') as f:
+            content = f.read()
+        self.assertEqual(content, valid_video_content)
 
+    @patch('telegram_downloader_bot.utils.snaptik')
+    @patch('telegram_downloader_bot.utils.datetime')
+    def test_download_tt_video_with_invalid_video(self, mock_datetime, mock_snaptik):
+        # Mock datetime
         mock_now = datetime(2023, 1, 1, 12, 0, 0)
-        mock_datetime.now.return_value = mock_now
+        mock_datetime.datetime.now.return_value = mock_now
 
-        result = download_tt_video(self.tmp_path, "http://tiktok.com/video123")
-        self.assertFalse(result)
+        # Read the content of invalid.mp4
+        with open(self.invalid_video_path, 'rb') as f:
+            invalid_video_content = f.read()
 
-        video_filename = mock_now.strftime("video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
+        # Mock snaptik to return a video that returns invalid.mp4 content
+        mock_video = Mock()
+        mock_video.download.return_value.getbuffer.return_value = invalid_video_content
+        mock_snaptik.return_value = [mock_video]
+
+        # Call the function
+        download_tt_video(self.tmp_path, "http://tiktok.com/video123")
+
+        # Verify that the file was saved
+        video_filename = mock_now.strftime(
+            "video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
         video_filepath = os.path.join(self.tmp_path, "tiktok", video_filename)
-        self.assertFalse(os.path.exists(video_filepath))
+        self.assertTrue(os.path.exists(video_filepath))
+
+        with open(video_filepath, 'rb') as f:
+            content = f.read()
+        self.assertEqual(content, invalid_video_content)
+
+    @patch('telegram_downloader_bot.utils.snaptik')
+    @patch('telegram_downloader_bot.utils.datetime')
+    def test_download_tt_video_no_videos(self, mock_datetime, mock_snaptik):
+        # Mock datetime
+        mock_now = datetime(2023, 1, 1, 12, 0, 0)
+        mock_datetime.datetime.now.return_value = mock_now
+
+        # Mock snaptik to return an empty list
+        mock_snaptik.return_value = []
+
+        # Call the function
+        download_tt_video(self.tmp_path, "http://tiktok.com/video123")
+
+        # Verify that no files were created
+        tiktok_folder = os.path.join(self.tmp_path, "tiktok")
+        files = os.listdir(tiktok_folder)
+        self.assertEqual(len(files), 0)
 
 
 class TestMakeFS(unittest.TestCase):
     def setUp(self):
-        # Create a temporary directory for each test
         self.tmp_path = tempfile.mkdtemp()
 
     def tearDown(self):
-        # Remove the directory after the test
         shutil.rmtree(self.tmp_path)
 
     def test_make_fs(self):
         make_fs(self.tmp_path)
         self.assertTrue(os.path.exists(os.path.join(self.tmp_path, "tiktok")))
-        self.assertTrue(os.path.exists(os.path.join(self.tmp_path, "telegram")))
+        self.assertTrue(os.path.exists(
+            os.path.join(self.tmp_path, "telegram")))
 
 
 class TestExtractURLs(unittest.TestCase):
     def test_no_urls(self):
-        result = extract_urls("This is some text without any URLs.")
+        text = "This is some text without any URLs."
+        result = extract_urls(text)
         self.assertEqual(result, [])
 
-    def test_one_url(self):
-        result = extract_urls("Check out this link: http://example.com")
+    def test_single_url(self):
+        text = "Check out this link: http://example.com"
+        result = extract_urls(text)
         self.assertEqual(result, ["http://example.com"])
 
     def test_multiple_urls(self):
-        result = extract_urls(
-            "Here are some links: http://example.com and https://test.com/page"
-        )
-        self.assertEqual(result, ["http://example.com", "https://test.com/page"])
+        text = "Here are some links: http://example.com and https://test.com/page"
+        result = extract_urls(text)
+        self.assertEqual(
+            result, ["http://example.com", "https://test.com/page"])
 
     def test_malformed_url(self):
-        result = extract_urls("This is not a URL: htt://badurl.com")
+        text = "This is not a URL: htt://badurl.com"
+        result = extract_urls(text)
         self.assertEqual(result, [])
 
-    def test_url_at_text_boundaries(self):
-        result = extract_urls("http://start.com text in the middle https://end.com")
-        self.assertEqual(result, ["http://start.com", "https://end.com"])
+    def test_urls_with_special_chars(self):
+        text = "Link: https://example.com/page?param=value#anchor"
+        result = extract_urls(text)
+        self.assertEqual(
+            result, ["https://example.com/page?param=value#anchor"])
 
 
 class TestFilterTTURLs(unittest.TestCase):
     def test_empty_list(self):
-        result = filter_tt_urls([])
+        urls = []
+        result = filter_tt_urls(urls)
         self.assertEqual(result, [])
 
     def test_no_tiktok_urls(self):
@@ -315,17 +419,22 @@ class TestFilterTTURLs(unittest.TestCase):
         result = filter_tt_urls(urls)
         self.assertEqual(result, [])
 
-    def test_only_tiktok_urls(self):
-        urls = ["http://tiktok.com/video1", "https://www.tiktok.com/@user/video/123"]
-        result = filter_tt_urls(urls)
-        self.assertEqual(result, urls)
-
     def test_mixed_urls(self):
-        urls = ["http://example.com", "https://www.tiktok.com/@user/video/123"]
+        urls = [
+            "http://example.com",
+            "https://www.tiktok.com/@user/video/123",
+            "http://tiktok.com/video1",
+            "https://test.com/page",
+        ]
+        expected = [
+            "https://www.tiktok.com/@user/video/123",
+            "http://tiktok.com/video1",
+        ]
         result = filter_tt_urls(urls)
-        self.assertEqual(result, ["https://www.tiktok.com/@user/video/123"])
+        self.assertEqual(result, expected)
 
-    def test_tiktok_in_query(self):
-        urls = ["http://example.com?param=tiktok", "https://www.other.com/path"]
+    def test_tiktok_in_query_params(self):
+        urls = ["http://example.com?watch=tiktok", "https://other.com/path"]
+        expected = ["http://example.com?watch=tiktok"]
         result = filter_tt_urls(urls)
-        self.assertEqual(result, ["http://example.com?param=tiktok"])
+        self.assertEqual(result, expected)

tox.ini

@@ -16,4 +16,9 @@ commands = bandit -r telegram_downloader_bot/
 commands = codespell telegram_downloader_bot/
 
 [testenv:unit]
-commands = stestr run --test-path tests/
+commands = stestr run --test-path tests/
+
+[testenv:coverage]
+commands = 
+    coverage run -m unittest discover
+    coverage report -m