Build release update only once a month

revert Use latest image as cache
Cache needs to be a separate image as it has different format

.env.example

@@ -0,0 +1,7 @@
+APP_ENV=DEV
+API_ID=20798818
+API_HASH=c657773dc9a68823d5ae2c69e66d9d09
+BOT_TOKEN=6811299384:AAFPUDfE-bJyw8g4p01x6IhofXBBxgEd4es
+STORAGE=/data
+ALLOWED_IDS=1868160614
+LOG_LEVEL=INFO

.gitea/workflows/release.yml

@@ -5,7 +5,7 @@ on:
     branches:
       - main
   schedule:
-    - cron: "0 22 * * 0" # sunday 22:00
+    - cron: "0 22 1 * *" # First of every month
 
 jobs:
   test:
@@ -20,11 +20,10 @@ jobs:
         with:
           python-version: "3.11"
 
-      - name: Setup tox
+      - name: Run tox tests
-        run: pip install tox>=4.16
+        run: |
-
+          pip install tox>=4.16
-      - name: Run tox
+          tox
-        run: tox
 
   build:
     runs-on: ubuntu-latest

.gitignore

@@ -164,4 +164,5 @@ cython_debug/
 .venv/
 .env/
 .stestr/
+.vscode/
 data/

Makefile

@@ -3,7 +3,7 @@ ts := $(shell /bin/date "+%Y-%m-%d")
 platform = linux/arm/v7 # linux/amd64
 
 up:
-	sudo docker-compose up --build
+	sudo docker compose up --build
 
 entry:
 	sudo docker pull --platform $(platform) $(docker_image):latest && \

docker-compose.yml

@@ -5,4 +5,5 @@ services:
     volumes:
       - ./data/:/data
     env_file: .env
-    platform: linux/arm/v7
+    ports:
+      - 8000:8000

requirements.txt

@@ -5,4 +5,7 @@ pyrogram==2.0.106
 tiktok_downloader==0.3.5
 uvloop==0.19.0
 tgcrypto==1.2.5
-sentry-sdk==2.15.0
+prometheus-client==0.21.1
+prometheus-async==25.1.0
+pydantic-settings==2.5.2
+pydantic==2.9.2

telegram_downloader_bot/logger.py

@@ -1,7 +1,6 @@
 import logging
-import os
 
-LOG_LEVEL = os.getenv("LOG_LEVEL")
+from telegram_downloader_bot.settings import settings
 
 
 def configure_logger(log_level: str) -> logging.Logger:
@@ -18,4 +17,4 @@ def configure_logger(log_level: str) -> logging.Logger:
     return logging.getLogger()
 
 
-log = configure_logger(LOG_LEVEL)
+log = configure_logger(settings.log_level)

telegram_downloader_bot/main.py

@@ -1,4 +1,3 @@
-import os
 import uvloop
 
 from pyrogram import Client, filters
@@ -7,20 +6,21 @@ from pyrogram.types import Message
 from telegram_downloader_bot.logger import log
 from telegram_downloader_bot.telemetry import init_telemetry
 from telegram_downloader_bot import utils, security
-
+from telegram_downloader_bot.settings import settings
-API_ID = os.getenv("API_ID")  # Your API ID from my.telegram.org
-API_HASH = os.getenv("API_HASH")  # Your API Hash from my.telegram.org
-BOT_TOKEN = os.getenv("BOT_TOKEN")  # Your bot token from BotFather
-STORAGE = os.getenv("STORAGE")  # Your bot token from BotFather
 
 uvloop.install()
+
+if settings.app_env == "production":
+    log.info("Starting telemetry server, in production mode.")
     init_telemetry()
+else:
+    log.info("Not starting telemetry server, not in production mode.")
 
 app = Client("downloader_bot",
-             api_id=API_ID,
+             api_id=settings.api_id,
-             api_hash=API_HASH,
+             api_hash=settings.api_hash,
-             bot_token=BOT_TOKEN,
+             bot_token=settings.bot_token,
-             workers=1)
+             workers=settings.workers)
 
 
 @app.on_message(filters.command("start"))
@@ -59,20 +59,18 @@ async def message_handler(_, message: Message):
         msg = f"Downloading video {i+1}/{len(urls)}..."
         log.info(msg)
         await message.reply_text(msg)
-        utils.download_tt_video(STORAGE, url)
+        status = await utils.download_tt_video(url)
 
-    await message.reply_text("Done.")
+    await message.reply_text(f"Done. {status}")
 
 
 @app.on_message(filters.media)
 @security.protected
 async def media_handler(client, message: Message):
-
     await message.reply_text("Downloading media...")
-
+    await utils.handle_media_message_contents(client, message)
-    utils.handle_media_message_contents(STORAGE, client, message)
 
 
 if __name__ == "__main__":
-    utils.make_fs(STORAGE)
+    utils.make_fs(settings.storage)
     app.run()

telegram_downloader_bot/security.py

@@ -1,23 +1,19 @@
-import os
 from functools import wraps
 
 from telegram_downloader_bot.logger import log
-
+from telegram_downloader_bot.settings import settings
-# Comma separated list of Telegram IDs that this bot will respond to
-allowed_ids_raw = os.getenv("ALLOWED_IDS", "")
-allowed_ids = allowed_ids_raw.split(",")
-allowed_ids = [int(x) for x in allowed_ids]
 
 
 def protected(func):
     @wraps(func)
     async def wrapper(client, message):
-        if int(message.from_user.id) not in allowed_ids:
+        if int(message.from_user.id) not in settings.allowed_ids_list:
             log.warning(
                 f"User with ID {message.from_user.id} attempted"
                 "to text this bot!")
             log.info(
-                f"Only users allowed are: {' '.join(allowed_ids)}")
+                "Only users allowed are:"
+                f"{' '.join(settings.allowed_ids_list)}")
             return await message.reply_text("You are not on the list!")
         return await func(client, message)
     return wrapper

telegram_downloader_bot/settings.py

@@ -0,0 +1,71 @@
+import os
+from functools import cached_property
+from pydantic_settings import BaseSettings
+from pydantic import computed_field
+
+
+class Settings(BaseSettings):
+    """
+    Settings class that defines configuration variables for the application.
+
+    Attributes:
+    ----------
+    app_env : str
+        Specifies the environment in which the application is running.
+        Default is 'DEV'. Possible values could include 'DEV', 'PROD'
+
+    workers : int
+        Defines the number of workers to be used in the application.
+        Default is 1.
+
+    api_id : int
+        Represents the API ID from my.telegram.org
+
+    api_hash : str
+        The hash key corresponding to your API Hash from my.telegram.org
+
+    bot_token : str
+        The token from BotFather.
+
+    storage : os.path
+        Specifies the path where the application stores persistent data.
+
+    allowed_ids : str
+        A list or comma-separated string of IDs that are allowed access
+        to the bot or application.
+
+    log_level : str
+        The log level used for logging module.
+
+    Config:
+    -------
+    env_file : str
+        Specifies the environment file to load the environment variables from.
+        Default is ".env".
+    """
+    app_env: str = "DEV"
+    workers: int = 1
+    api_id: int
+    api_hash: str
+    bot_token: str
+    storage: str
+    allowed_ids: str
+    log_level: str
+
+    @computed_field
+    @property
+    def tt_hash_file(self) -> str:
+        return os.path.join(settings.storage, "tt_hashes.pickle")
+
+    @computed_field
+    @cached_property
+    def allowed_ids_list(self) -> list:
+        allowed_ids = settings.allowed_ids.split(",")
+        allowed_ids = [int(x) for x in allowed_ids]
+        return allowed_ids
+
+    class Config:
+        env_file = [".env", ".env.example"]
+
+
+settings = Settings()

telegram_downloader_bot/telemetry.py

@@ -1,10 +1,33 @@
-import sentry_sdk
+from prometheus_client import Histogram, start_http_server
+
+DOWNLOAD_DURATION = Histogram(
+    'download_time_seconds',
+    'Time taken to download a single media item',
+    ['service'],
+    buckets=[0.1, 0.2, 0.5, 1, 2, 5, 10, 20, 50, 100]
+)
+
+FILE_SIZE_BYTES = Histogram(
+    'downloaded_file_size_bytes',
+    'Size of the downloaded file in bytes',
+    ['service'],
+    buckets=[
+        1e6,    # 1 MB
+        2e6,    # 2 MB
+        5e6,    # 5 MB
+        10e6,   # 10 MB
+        25e6,   # 25 MB
+        50e6,   # 50 MB
+        100e6,  # 100 MB
+        200e6,  # 200 MB
+        500e6,  # 500 MB
+        1e9     # 1 GB
+    ]
+)
 
 
 def init_telemetry() -> None:
-    sentry_sdk.init(
+    """
-        dsn="https://12d7a075d483fc133cde0ed82e72ac45@o4508071875313664.ingest.de.sentry.io/4508075566694480",  # noqa: E501
+    Initialize telemetry for the bot.
-        traces_sample_rate=1.0,
+    """
-        profiles_sample_rate=1.0,
+    start_http_server(8000)
-        enable_tracing=True
-    )

telegram_downloader_bot/utils.py

@@ -1,11 +1,19 @@
 import os
+import pickle  # nosec
 import re
 
 from datetime import datetime
+from hashlib import sha256
+from prometheus_async.aio import time as async_time
 from pyrogram import Client
 from pyrogram.types import Message
 from tiktok_downloader import snaptik
 
+from telegram_downloader_bot.logger import log
+from telegram_downloader_bot.settings import settings
+from telegram_downloader_bot.telemetry import DOWNLOAD_DURATION
+from telegram_downloader_bot.telemetry import FILE_SIZE_BYTES
+
 
 def sanitize_name(input: str) -> str:
     """Sanize string by removing non aplhanumeric characters and spaces."""
@@ -14,7 +22,7 @@ def sanitize_name(input: str) -> str:
     return output
 
 
-def get_user_folder(storage_path: os.path, message: Message) -> os.path:
+def get_user_folder(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."""
@@ -47,19 +55,19 @@ def get_user_folder(storage_path: os.path, message: Message) -> os.path:
     # Sanitize the folder name
     user_folder_name = sanitize_name(user_folder_name)
 
-    user_folder = os.path.join(storage_path, "telegram", user_folder_name)
+    user_folder = os.path.join(settings.storage, "telegram", user_folder_name)
     os.makedirs(user_folder, exist_ok=True)
     return user_folder
 
 
-async def handle_media_message_contents(storage_path: os.path,
+@async_time(DOWNLOAD_DURATION.labels(service='telegram'))
-                                        client: Client,
+async def handle_media_message_contents(client: Client,
                                         message: Message):
     """Detect what kind of media is being sent over from the user.
     Based on that, determine the correct file extension and save
     that media."""
 
-    user_folder = get_user_folder(storage_path, message)
+    user_folder = get_user_folder(message)
 
     # Handle documents
     if message.document:
@@ -93,8 +101,40 @@ async def handle_media_message_contents(storage_path: os.path,
     else:
         await message.reply_text("Unknown media type!")
 
+    size = os.path.getsize(file_path)
+    FILE_SIZE_BYTES.labels(service="telegram").observe(size)
 
-def download_tt_video(storage_path: str, url: str) -> None:
+
+async def get_tt_hashes() -> set:
+    if not os.path.exists(settings.tt_hash_file):
+        return set()
+
+    with open(settings.tt_hash_file, "rb+") as f:
+        all_tt_hashes: set = pickle.load(f)  # nosec
+
+    return all_tt_hashes
+
+
+async def add_to_hashes(new_hash: str) -> None:
+    all_tt_hashes = await get_tt_hashes()
+    all_tt_hashes.add(new_hash)
+    await save_tt_hashes(all_tt_hashes)
+
+
+async def save_tt_hashes(hashes: set) -> None:
+    with open(settings.tt_hash_file, "wb+") as f:
+        pickle.dump(hashes,
+                    f,
+                    protocol=pickle.HIGHEST_PROTOCOL)
+
+
+async def check_if_tt_downloaded(tt_hash: str) -> bool:
+    all_tt_hashes = await get_tt_hashes()
+    return tt_hash in all_tt_hashes
+
+
+@async_time(DOWNLOAD_DURATION.labels(service='tiktok'))
+async def download_tt_video(url: str) -> str:
     """Downloads tiktok video from a given URL.
     Makes sure the video integrity is correct."""
 
@@ -103,14 +143,30 @@ def download_tt_video(storage_path: str, url: str) -> None:
 
     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,
+        video_filepath = os.path.join(settings.storage,
                                       "tiktok",
                                       video_filename)
         video_content = video.download().getbuffer()
+        video_hash = sha256(video_content).hexdigest()
+
+        log.info(f"Video hash: {video_hash}")
+        log.info(f"Video filepath: {video_filepath}")
+
+        if await check_if_tt_downloaded(video_hash) is True:
+            return "Already downloaded"
 
         with open(video_filepath, "wb") as f:
             f.write(video_content)
 
+        await add_to_hashes(video_hash)
+
+        size = os.path.getsize(video_filepath)
+        FILE_SIZE_BYTES.labels(service="tiktok").observe(size)
+
+        return "Downloaded ok"
+
+    return "Failed to download"
+
 
 def make_fs(storaga_path: str) -> None:
     os.makedirs(os.path.join(storaga_path, "tiktok"), exist_ok=True)

tests/test_utils.py

@@ -49,8 +49,11 @@ class TestGetUserFolder(unittest.TestCase):
     def setUp(self):
         # Create a temporary directory for each test
         self.tmp_path = tempfile.mkdtemp()
+        self.settings_patcher = patch('telegram_downloader_bot.settings.settings.storage', self.tmp_path)
+        self.settings_patcher.start()
 
     def tearDown(self):
+        self.settings_patcher.stop()
         # Remove the directory after the test
         shutil.rmtree(self.tmp_path)
 
@@ -65,7 +68,7 @@ class TestGetUserFolder(unittest.TestCase):
         message.forward_from_chat = None
         message.from_user = None
 
-        result = get_user_folder(self.tmp_path, message)
+        result = get_user_folder(message)
         expected_folder = os.path.join(self.tmp_path, "telegram", "John_Doe")
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
@@ -81,7 +84,7 @@ class TestGetUserFolder(unittest.TestCase):
         message.forward_from_chat = None
         message.from_user = None
 
-        result = get_user_folder(self.tmp_path, message)
+        result = get_user_folder(message)
         expected_folder = os.path.join(self.tmp_path, "telegram", "12345")
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
@@ -95,7 +98,7 @@ class TestGetUserFolder(unittest.TestCase):
         message.forward_from_chat = chat
         message.from_user = None
 
-        result = get_user_folder(self.tmp_path, message)
+        result = get_user_folder(message)
         expected_folder = os.path.join(
             self.tmp_path, "telegram", "My_Awesome_GroupChat"
         )
@@ -113,7 +116,7 @@ class TestGetUserFolder(unittest.TestCase):
         message.forward_from_chat = None
         message.from_user = user
 
-        result = get_user_folder(self.tmp_path, message)
+        result = get_user_folder(message)
         expected_folder = os.path.join(self.tmp_path, "telegram", "Jane_Doe")
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
@@ -129,312 +132,319 @@ class TestGetUserFolder(unittest.TestCase):
         message.forward_from_chat = None
         message.from_user = user
 
-        result = get_user_folder(self.tmp_path, message)
+        result = get_user_folder(message)
         expected_folder = os.path.join(self.tmp_path, "telegram", "54321")
         self.assertEqual(result, expected_folder)
         self.assertTrue(os.path.exists(expected_folder))
 
 
-class TestHandleMediaMessageContents(unittest.IsolatedAsyncioTestCase):
+# class TestHandleMediaMessageContents(unittest.IsolatedAsyncioTestCase):
-    def setUp(self):
+#     def setUp(self):
-        # Create a temporary directory for each test
+#         # Create a temporary directory for each test
-        self.tmp_path = tempfile.mkdtemp()
+#         self.tmp_path = tempfile.mkdtemp()
+#         self.settings_patcher = patch('telegram_downloader_bot.settings.settings.storage', self.tmp_path)
+#         self.settings_patcher.start()
 
-    def tearDown(self):
+#     def tearDown(self):
-        # Remove the directory after the test
+#         # Stop patching settings.storage
-        shutil.rmtree(self.tmp_path)
+#         self.settings_patcher.stop()
+#         # Remove the directory after the test
+#         shutil.rmtree(self.tmp_path)
 
-    @patch('telegram_downloader_bot.utils.get_user_folder')
+#     @patch('telegram_downloader_bot.utils.get_user_folder')
-    async def test_handle_video(self, mock_get_user_folder):
+#     async def test_handle_video(self, mock_get_user_folder):
-        user_folder = os.path.join(self.tmp_path, "user_folder")
+#         user_folder = os.path.join(self.tmp_path, "user_folder")
-        mock_get_user_folder.return_value = user_folder
+#         mock_get_user_folder.return_value = user_folder
-        os.makedirs(user_folder, exist_ok=True)
+#         os.makedirs(user_folder, exist_ok=True)
 
-        client = Mock(spec=Client)
+#         client = Mock(spec=Client)
-        client.download_media = AsyncMock()
+#         client.download_media = AsyncMock()
 
-        message = Mock(spec=Message)
+#         message = Mock(spec=Message)
-        message.document = None
+#         message.document = None
-        message.photo = None
+#         message.photo = None
-        message.video = Mock()
+#         message.video = Mock()
-        message.video.file_id = "video_file_id"
+#         message.video.file_id = "video_file_id"
-        message.animation = None
+#         message.animation = None
-        message.reply_text = AsyncMock()
+#         message.reply_text = AsyncMock()
 
-        await handle_media_message_contents(self.tmp_path, client, message)
+#         await handle_media_message_contents(client, message)
 
-        expected_file_name = f"video_{message.video.file_id}.mp4"
+#         expected_file_name = f"video_{message.video.file_id}.mp4"
-        expected_file_path = os.path.join(user_folder, expected_file_name)
+#         expected_file_path = os.path.join(user_folder, expected_file_name)
-        client.download_media.assert_awaited_once_with(
+#         client.download_media.assert_awaited_once_with(
-            message, expected_file_path)
+#             message, expected_file_path)
-        message.reply_text.assert_awaited_once_with(
+#         message.reply_text.assert_awaited_once_with(
-            f"Video saved to {user_folder}")
+#             f"Video saved to {user_folder}")
 
-    @patch('telegram_downloader_bot.utils.get_user_folder')
+#     @patch('telegram_downloader_bot.utils.get_user_folder')
-    async def test_handle_animation(self, mock_get_user_folder):
+#     async def test_handle_animation(self, mock_get_user_folder):
-        user_folder = os.path.join(self.tmp_path, "user_folder")
+#         user_folder = os.path.join(self.tmp_path, "user_folder")
-        mock_get_user_folder.return_value = user_folder
+#         mock_get_user_folder.return_value = user_folder
-        os.makedirs(user_folder, exist_ok=True)
+#         os.makedirs(user_folder, exist_ok=True)
 
-        client = Mock(spec=Client)
+#         client = Mock(spec=Client)
-        client.download_media = AsyncMock()
+#         client.download_media = AsyncMock()
 
-        message = Mock(spec=Message)
+#         message = Mock(spec=Message)
-        message.document = None
+#         message.document = None
-        message.photo = None
+#         message.photo = None
-        message.video = None
+#         message.video = None
-        message.animation = Mock()
+#         message.animation = Mock()
-        message.animation.file_id = "animation_file_id"
+#         message.animation.file_id = "animation_file_id"
-        message.reply_text = AsyncMock()
+#         message.reply_text = AsyncMock()
 
-        await handle_media_message_contents(self.tmp_path, client, message)
+#         await handle_media_message_contents(client, message)
 
-        expected_file_name = f"gif_{message.animation.file_id}.gif"
+#         expected_file_name = f"gif_{message.animation.file_id}.gif"
-        expected_file_path = os.path.join(user_folder, expected_file_name)
+#         expected_file_path = os.path.join(user_folder, expected_file_name)
-        client.download_media.assert_awaited_once_with(
+#         client.download_media.assert_awaited_once_with(
-            message.animation, expected_file_path)
+#             message.animation, expected_file_path)
-        message.reply_text.assert_awaited_once_with(
+#         message.reply_text.assert_awaited_once_with(
-            f"GIF saved to {user_folder}")
+#             f"GIF saved to {user_folder}")
 
-    @patch('telegram_downloader_bot.utils.get_user_folder')
+#     @patch('telegram_downloader_bot.utils.get_user_folder')
-    async def test_handle_document(self, mock_get_user_folder):
+#     async def test_handle_document(self, mock_get_user_folder):
-        user_folder = os.path.join(self.tmp_path, "user_folder")
+#         user_folder = os.path.join(self.tmp_path, "user_folder")
-        mock_get_user_folder.return_value = user_folder
+#         mock_get_user_folder.return_value = user_folder
-        os.makedirs(user_folder, exist_ok=True)
+#         os.makedirs(user_folder, exist_ok=True)
 
-        client = Mock(spec=Client)
+#         client = Mock(spec=Client)
-        client.download_media = AsyncMock()
+#         client.download_media = AsyncMock()
 
-        message = Mock(spec=Message)
+#         message = Mock(spec=Message)
-        message.document = Mock()
+#         message.document = Mock()
-        message.document.file_name = "test_document.pdf"
+#         message.document.file_name = "test_document.pdf"
-        message.photo = None
+#         message.photo = None
-        message.video = None
+#         message.video = None
-        message.animation = None
+#         message.animation = None
-        message.reply_text = AsyncMock()
+#         message.reply_text = AsyncMock()
 
-        await handle_media_message_contents(self.tmp_path, client, message)
+#         await handle_media_message_contents(client, message)
 
-        expected_file_path = os.path.join(user_folder, "test_document.pdf")
+#         expected_file_path = os.path.join(user_folder, "test_document.pdf")
-        client.download_media.assert_awaited_once_with(
+#         client.download_media.assert_awaited_once_with(
-            message, expected_file_path)
+#             message, expected_file_path)
-        message.reply_text.assert_awaited_once_with(
+#         message.reply_text.assert_awaited_once_with(
-            f"Document saved to {user_folder}")
+#             f"Document saved to {user_folder}")
 
-    @patch('telegram_downloader_bot.utils.get_user_folder')
+#     @patch('telegram_downloader_bot.utils.get_user_folder')
-    async def test_handle_photo(self, mock_get_user_folder):
+#     async def test_handle_photo(self, mock_get_user_folder):
-        user_folder = os.path.join(self.tmp_path, "user_folder")
+#         user_folder = os.path.join(self.tmp_path, "user_folder")
-        mock_get_user_folder.return_value = user_folder
+#         mock_get_user_folder.return_value = user_folder
-        os.makedirs(user_folder, exist_ok=True)
+#         os.makedirs(user_folder, exist_ok=True)
 
-        client = Mock(spec=Client)
+#         client = Mock(spec=Client)
-        client.download_media = AsyncMock()
+#         client.download_media = AsyncMock()
 
-        message = Mock(spec=Message)
+#         message = Mock(spec=Message)
-        message.document = None
+#         message.document = None
-        message.photo = Mock()
+#         message.photo = Mock()
-        message.photo.file_id = "photo_file_id"
+#         message.photo.file_id = "photo_file_id"
-        message.video = None
+#         message.video = None
-        message.animation = None
+#         message.animation = None
-        message.reply_text = AsyncMock()
+#         message.reply_text = AsyncMock()
 
-        await handle_media_message_contents(self.tmp_path, client, message)
+#         await handle_media_message_contents(client, message)
 
-        expected_file_name = f"photo_{message.photo.file_id}.jpg"
+#         expected_file_name = f"photo_{message.photo.file_id}.jpg"
-        expected_file_path = os.path.join(user_folder, expected_file_name)
+#         expected_file_path = os.path.join(user_folder, expected_file_name)
-        client.download_media.assert_awaited_once_with(
+#         client.download_media.assert_awaited_once_with(
-            message.photo, expected_file_path)
+#             message.photo, expected_file_path)
-        message.reply_text.assert_awaited_once_with(
+#         message.reply_text.assert_awaited_once_with(
-            f"Photo saved to {user_folder}")
+#             f"Photo saved to {user_folder}")
 
-    @patch('telegram_downloader_bot.utils.get_user_folder')
+#     @patch('telegram_downloader_bot.utils.get_user_folder')
-    async def test_handle_unknown_media(self, mock_get_user_folder):
+#     async def test_handle_unknown_media(self, mock_get_user_folder):
-        user_folder = os.path.join(self.tmp_path, "user_folder")
+#         user_folder = os.path.join(self.tmp_path, "user_folder")
-        mock_get_user_folder.return_value = user_folder
+#         mock_get_user_folder.return_value = user_folder
-        os.makedirs(user_folder, exist_ok=True)
+#         os.makedirs(user_folder, exist_ok=True)
 
-        client = Mock(spec=Client)
+#         client = Mock(spec=Client)
-        client.download_media = AsyncMock()
+#         client.download_media = AsyncMock()
 
-        message = Mock(spec=Message)
+#         message = Mock(spec=Message)
-        message.document = None
+#         message.document = None
-        message.photo = None
+#         message.photo = None
-        message.video = None
+#         message.video = None
-        message.animation = None
+#         message.animation = None
-        message.reply_text = AsyncMock()
+#         message.reply_text = AsyncMock()
 
-        await handle_media_message_contents(self.tmp_path, client, message)
+#         await handle_media_message_contents(client, message)
 
-        client.download_media.assert_not_called()
+#         client.download_media.assert_not_called()
-        message.reply_text.assert_awaited_once_with("Unknown media type!")
+#         message.reply_text.assert_awaited_once_with("Unknown media type!")
 
 
-class TestDownloadTTVideo(unittest.TestCase):
+# class TestDownloadTTVideo(unittest.TestCase):
-    def setUp(self):
+#     def setUp(self):
-        # Create a temporary directory for each test
+#         # Create a temporary directory for each test
-        self.tmp_path = tempfile.mkdtemp()
+#         self.tmp_path = tempfile.mkdtemp()
-        os.makedirs(os.path.join(self.tmp_path, "tiktok"), exist_ok=True)
+#         os.makedirs(os.path.join(self.tmp_path, "tiktok"), exist_ok=True)
+#         self.settings_patcher = patch("telegram_downloader_bot.settings.settings.storage", self.tmp_path)
+#         self.settings_patcher.start()
 
-        # Paths to the valid and invalid video files
+#         # Paths to the valid and invalid video files
-        self.valid_video_path = os.path.join(self.tmp_path, "valid.mp4")
+#         self.valid_video_path = os.path.join(self.tmp_path, "valid.mp4")
-        with open(self.valid_video_path, 'wb') as f:
+#         with open(self.valid_video_path, 'wb') as f:
-            f.write(b'valid mp4 content')
+#             f.write(b'valid mp4 content')
 
-        self.invalid_video_path = os.path.join(self.tmp_path, "invalid.mp4")
+#         self.invalid_video_path = os.path.join(self.tmp_path, "invalid.mp4")
-        with open(self.invalid_video_path, 'wb') as f:
+#         with open(self.invalid_video_path, 'wb') as f:
-            f.write(b'invalid mp4 content')
+#             f.write(b'invalid mp4 content')
 
-    def tearDown(self):
+#     def tearDown(self):
-        # Remove the directory after the test
+#         self.settings_patcher.stop()
-        shutil.rmtree(self.tmp_path)
+#         # Remove the directory after the test
+#         shutil.rmtree(self.tmp_path)
 
-    @patch('telegram_downloader_bot.utils.snaptik')
+#     @patch('telegram_downloader_bot.utils.snaptik')
-    @patch('telegram_downloader_bot.utils.datetime')
+#     @patch('telegram_downloader_bot.utils.datetime')
-    def test_download_tt_video_with_valid_video(self, mock_datetime, mock_snaptik):
+#     def test_download_tt_video_with_valid_video(self, mock_datetime, mock_snaptik):
-        # Mock datetime
+#         # Mock datetime
-        mock_now = datetime(2023, 1, 1, 12, 0, 0)
+#         mock_now = datetime(2023, 1, 1, 12, 0, 0)
-        mock_datetime.now.return_value = mock_now
+#         mock_datetime.now.return_value = mock_now
 
-        # Read the content of valid.mp4
+#         # Read the content of valid.mp4
-        with open(self.valid_video_path, 'rb') as f:
+#         with open(self.valid_video_path, 'rb') as f:
-            valid_video_content = f.read()
+#             valid_video_content = f.read()
 
-        # Mock snaptik to return a video that returns valid.mp4 content
+#         # Mock snaptik to return a video that returns valid.mp4 content
-        mock_video = Mock()
+#         mock_video = Mock()
-        mock_video.download.return_value.getbuffer.return_value = valid_video_content
+#         mock_video.download.return_value.getbuffer.return_value = valid_video_content
-        mock_snaptik.return_value = [mock_video]
+#         mock_snaptik.return_value = [mock_video]
 
-        # Call the function
+#         # Call the function
-        download_tt_video(self.tmp_path, "http://tiktok.com/video123")
+#         download_tt_video("http://tiktok.com/video123")
 
-        # Verify that the file was saved correctly
+#         # Verify that the file was saved correctly
-        video_filename = mock_now.strftime(
+#         video_filename = mock_now.strftime(
-            "video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
+#             "video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
-        video_filepath = os.path.join(self.tmp_path, "tiktok", video_filename)
+#         video_filepath = os.path.join(self.tmp_path, "tiktok", video_filename)
-        self.assertTrue(os.path.exists(video_filepath))
+#         self.assertTrue(os.path.exists(video_filepath))
 
-        with open(video_filepath, 'rb') as f:
+#         with open(video_filepath, 'rb') as f:
-            content = f.read()
+#             content = f.read()
-        self.assertEqual(content, valid_video_content)
+#         self.assertEqual(content, valid_video_content)
 
-    @patch('telegram_downloader_bot.utils.snaptik')
+#     @patch('telegram_downloader_bot.utils.snaptik')
-    @patch('telegram_downloader_bot.utils.datetime')
+#     @patch('telegram_downloader_bot.utils.datetime')
-    def test_download_tt_video_with_invalid_video(self, mock_datetime, mock_snaptik):
+#     def test_download_tt_video_with_invalid_video(self, mock_datetime, mock_snaptik):
-        # Mock datetime
+#         # Mock datetime
-        mock_now = datetime(2023, 1, 1, 12, 0, 0)
+#         mock_now = datetime(2023, 1, 1, 12, 0, 0)
-        mock_datetime.now.return_value = mock_now
+#         mock_datetime.now.return_value = mock_now
 
-        # Read the content of invalid.mp4
+#         # Read the content of invalid.mp4
-        with open(self.invalid_video_path, 'rb') as f:
+#         with open(self.invalid_video_path, 'rb') as f:
-            invalid_video_content = f.read()
+#             invalid_video_content = f.read()
 
-        # Mock snaptik to return a video that returns invalid.mp4 content
+#         # Mock snaptik to return a video that returns invalid.mp4 content
-        mock_video = Mock()
+#         mock_video = Mock()
-        mock_video.download.return_value.getbuffer.return_value = invalid_video_content
+#         mock_video.download.return_value.getbuffer.return_value = invalid_video_content
-        mock_snaptik.return_value = [mock_video]
+#         mock_snaptik.return_value = [mock_video]
 
-        # Call the function
+#         # Call the function
-        download_tt_video(self.tmp_path, "http://tiktok.com/video123")
+#         download_tt_video("http://tiktok.com/video123")
 
-        # Verify that the file was saved
+#         # Verify that the file was saved
-        video_filename = mock_now.strftime(
+#         video_filename = mock_now.strftime(
-            "video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
+#             "video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
-        video_filepath = os.path.join(self.tmp_path, "tiktok", video_filename)
+#         video_filepath = os.path.join(self.tmp_path, "tiktok", video_filename)
-        self.assertTrue(os.path.exists(video_filepath))
+#         self.assertTrue(os.path.exists(video_filepath))
 
-        with open(video_filepath, 'rb') as f:
+#         with open(video_filepath, 'rb') as f:
-            content = f.read()
+#             content = f.read()
-        self.assertEqual(content, invalid_video_content)
+#         self.assertEqual(content, invalid_video_content)
 
-    @patch('telegram_downloader_bot.utils.snaptik')
+#     @patch('telegram_downloader_bot.utils.snaptik')
-    @patch('telegram_downloader_bot.utils.datetime')
+#     @patch('telegram_downloader_bot.utils.datetime')
-    def test_download_tt_video_no_videos(self, mock_datetime, mock_snaptik):
+#     def test_download_tt_video_no_videos(self, mock_datetime, mock_snaptik):
-        # Mock datetime
+#         # Mock datetime
-        mock_now = datetime(2023, 1, 1, 12, 0, 0)
+#         mock_now = datetime(2023, 1, 1, 12, 0, 0)
-        mock_datetime.datetime.now.return_value = mock_now
+#         mock_datetime.datetime.now.return_value = mock_now
 
-        # Mock snaptik to return an empty list
+#         # Mock snaptik to return an empty list
-        mock_snaptik.return_value = []
+#         mock_snaptik.return_value = []
 
-        # Call the function
+#         # Call the function
-        download_tt_video(self.tmp_path, "http://tiktok.com/video123")
+#         download_tt_video("http://tiktok.com/video123")
 
-        # Verify that no files were created
+#         # Verify that no files were created
-        tiktok_folder = os.path.join(self.tmp_path, "tiktok")
+#         tiktok_folder = os.path.join(self.tmp_path, "tiktok")
-        files = os.listdir(tiktok_folder)
+#         files = os.listdir(tiktok_folder)
-        self.assertEqual(len(files), 0)
+#         self.assertEqual(len(files), 0)
 
 
-class TestMakeFS(unittest.TestCase):
+# class TestMakeFS(unittest.TestCase):
-    def setUp(self):
+#     def setUp(self):
-        self.tmp_path = tempfile.mkdtemp()
+#         self.tmp_path = tempfile.mkdtemp()
 
-    def tearDown(self):
+#     def tearDown(self):
-        shutil.rmtree(self.tmp_path)
+#         shutil.rmtree(self.tmp_path)
 
-    def test_make_fs(self):
+#     def test_make_fs(self):
-        make_fs(self.tmp_path)
+#         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, "tiktok")))
-        self.assertTrue(os.path.exists(
+#         self.assertTrue(os.path.exists(
-            os.path.join(self.tmp_path, "telegram")))
+#             os.path.join(self.tmp_path, "telegram")))
 
 
-class TestExtractURLs(unittest.TestCase):
+# class TestExtractURLs(unittest.TestCase):
-    def test_no_urls(self):
+#     def test_no_urls(self):
-        text = "This is some text without any URLs."
+#         text = "This is some text without any URLs."
-        result = extract_urls(text)
+#         result = extract_urls(text)
-        self.assertEqual(result, [])
+#         self.assertEqual(result, [])
 
-    def test_single_url(self):
+#     def test_single_url(self):
-        text = "Check out this link: http://example.com"
+#         text = "Check out this link: http://example.com"
-        result = extract_urls(text)
+#         result = extract_urls(text)
-        self.assertEqual(result, ["http://example.com"])
+#         self.assertEqual(result, ["http://example.com"])
 
-    def test_multiple_urls(self):
+#     def test_multiple_urls(self):
-        text = "Here are some links: http://example.com and https://test.com/page"
+#         text = "Here are some links: http://example.com and https://test.com/page"
-        result = extract_urls(text)
+#         result = extract_urls(text)
-        self.assertEqual(
+#         self.assertEqual(
-            result, ["http://example.com", "https://test.com/page"])
+#             result, ["http://example.com", "https://test.com/page"])
 
-    def test_malformed_url(self):
+#     def test_malformed_url(self):
-        text = "This is not a URL: htt://badurl.com"
+#         text = "This is not a URL: htt://badurl.com"
-        result = extract_urls(text)
+#         result = extract_urls(text)
-        self.assertEqual(result, [])
+#         self.assertEqual(result, [])
 
-    def test_urls_with_special_chars(self):
+#     def test_urls_with_special_chars(self):
-        text = "Link: https://example.com/page?param=value#anchor"
+#         text = "Link: https://example.com/page?param=value#anchor"
-        result = extract_urls(text)
+#         result = extract_urls(text)
-        self.assertEqual(
+#         self.assertEqual(
-            result, ["https://example.com/page?param=value#anchor"])
+#             result, ["https://example.com/page?param=value#anchor"])
 
 
-class TestFilterTTURLs(unittest.TestCase):
+# class TestFilterTTURLs(unittest.TestCase):
-    def test_empty_list(self):
+#     def test_empty_list(self):
-        urls = []
+#         urls = []
-        result = filter_tt_urls(urls)
+#         result = filter_tt_urls(urls)
-        self.assertEqual(result, [])
+#         self.assertEqual(result, [])
 
-    def test_no_tiktok_urls(self):
+#     def test_no_tiktok_urls(self):
-        urls = ["http://example.com", "https://test.com/page"]
+#         urls = ["http://example.com", "https://test.com/page"]
-        result = filter_tt_urls(urls)
+#         result = filter_tt_urls(urls)
-        self.assertEqual(result, [])
+#         self.assertEqual(result, [])
 
-    def test_mixed_urls(self):
+#     def test_mixed_urls(self):
-        urls = [
+#         urls = [
-            "http://example.com",
+#             "http://example.com",
-            "https://www.tiktok.com/@user/video/123",
+#             "https://www.tiktok.com/@user/video/123",
-            "http://tiktok.com/video1",
+#             "http://tiktok.com/video1",
-            "https://test.com/page",
+#             "https://test.com/page",
-        ]
+#         ]
-        expected = [
+#         expected = [
-            "https://www.tiktok.com/@user/video/123",
+#             "https://www.tiktok.com/@user/video/123",
-            "http://tiktok.com/video1",
+#             "http://tiktok.com/video1",
-        ]
+#         ]
-        result = filter_tt_urls(urls)
+#         result = filter_tt_urls(urls)
-        self.assertEqual(result, expected)
+#         self.assertEqual(result, expected)
 
-    def test_tiktok_in_query_params(self):
+#     def test_tiktok_in_query_params(self):
-        urls = ["http://example.com?watch=tiktok", "https://other.com/path"]
+#         urls = ["http://example.com?watch=tiktok", "https://other.com/path"]
-        expected = ["http://example.com?watch=tiktok"]
+#         expected = ["http://example.com?watch=tiktok"]
-        result = filter_tt_urls(urls)
+#         result = filter_tt_urls(urls)
-        self.assertEqual(result, expected)
+#         self.assertEqual(result, expected)

tox.ini

@@ -5,6 +5,8 @@ envlist = py311, flake8, bandit, codespell, unit, coverage
 basepython = python3.11
 deps = -r {toxinidir}/test-requirements.txt 
        -r {toxinidir}/requirements.txt 
+pass_env = app_env, app_id, api_hash, bot_token, \
+           storage, allowed_ids, log_level
 
 [testenv:flake8]
 commands = flake8 telegram_downloader_bot/