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

.dockerignore

@@ -0,0 +1,6 @@
+# Ignore everything
+**
+
+# Except for
+!telegram_downloader_bot/
+!requirements.txt

.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,37 +20,36 @@ jobs:
         with:
           python-version: "3.11"
 
-      - name: Setup tox
-        run: pip install tox>=4.16
-
-      - name: Run tox
-        run: tox
+      - name: Run tox tests
+        run: |
+          pip install tox>=4.16
+          tox
 
   build:
     runs-on: ubuntu-latest
     needs: test
     steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
       - name: Get date for image label
         id: date
         run: echo "::set-output name=date::$(date +'%Y-%m-%d')"
 
-      - name: Checkout code
-        uses: https://github.com/actions/checkout@v4
-
       - name: Set up Docker Buildx
-        uses: https://github.com/docker/setup-buildx-action@v3
+        uses: docker/setup-buildx-action@v3
         with:
           driver: docker-container
 
       - name: Login to Docker Registry
-        uses: https://github.com/docker/login-action@v3
+        uses: docker/login-action@v3
         with:
           registry: git.orebolt.cz
           username: ${{ secrets.REGISTRY_USERNAME }}
           password: ${{ secrets.REGISTRY_TOKEN }}
 
       - name: Build and push image
-        uses: https://github.com/docker/build-push-action@v5
+        uses: docker/build-push-action@v5
         with:
           context: .
           push: true
@@ -58,3 +57,17 @@ jobs:
           platforms: linux/amd64,linux/arm/v7
           cache-to: "mode=max,image-manifest=true,oci-mediatypes=true,type=registry,ref=${{ vars.DOCKER_IMAGE }}:cache"
           cache-from: "mode=max,image-manifest=true,oci-mediatypes=true,type=registry,ref=${{ vars.DOCKER_IMAGE }}:cache"
+          labels: |
+            org.opencontainers.image.created=${{ steps.date.outputs.date }}
+            org.opencontainers.image.authors=Roman Krček
+            org.opencontainers.image.source=${{ env.GITHUB_REPOSITORY }}
+            org.opencontainers.image.revision=${{ env.GITHUB_SHA }}
+            org.opencontainers.image.vendor=Orebolt.cz
+            org.opencontainers.image.ref.name=${{ env.GITHUB_REF }}
+            org.opencontainers.image.title=Telegram Downloader
+
+      - name: Run Trivy vulnerability scanner
+        uses: aquasecurity/trivy-action@0.24.0
+        with:
+          image-ref: '${{ vars.DOCKER_IMAGE }}:latest'
+          format: 'table'

.gitignore

@@ -162,5 +162,7 @@ cython_debug/
 #.idea/
 
 .venv/
-.env
+.env/
+.stestr/
+.vscode/
 data/

Dockerfile

@@ -8,7 +8,7 @@ FROM base AS builder
 
 COPY requirements.txt ./
 RUN --mount=type=cache,target=/tmp/pip_cache \
-    python3 -m pip install --upgrade pip  && \
+    python3 -m pip install --upgrade pip setuptools  && \
     pip install \
     -r requirements.txt \
     --extra-index-url https://www.piwheels.org/simple \

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

@@ -1,6 +1,11 @@
-integv==1.3.0
+# Pin aiohttp version until builds start passing 
+# https://www.piwheels.org/project/aiohttp/
+aiohttp==3.10.9 
 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,5 +1,7 @@
 import logging
 
+from telegram_downloader_bot.settings import settings
+
 
 def configure_logger(log_level: str) -> logging.Logger:
     log_format = (
@@ -13,3 +15,6 @@ def configure_logger(log_level: str) -> logging.Logger:
     )
 
     return logging.getLogger()
+
+
+log = configure_logger(settings.log_level)

telegram_downloader_bot/main.py

@@ -1,183 +1,76 @@
-import datetime
-import integv
-import os
-import re
-import sentry_sdk
 import uvloop
 
 from pyrogram import Client, filters
 from pyrogram.types import Message
-from tiktok_downloader import snaptik
 
-from telegram_downloader_bot import logger
-
-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")  # Storage directory for downloads
-MY_MSG_ID = int(os.getenv("MY_MSG_ID"))  # Your message ID for authorization
-LOG_LEVEL = os.getenv("LOG_LEVEL")  # Log level
-
-log = logger.configure_logger(LOG_LEVEL)
+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
 
 uvloop.install()
 
-sentry_sdk.init(
-    dsn="https://12d7a075d483fc133cde0ed82e72ac45@o4508071875313664.ingest.de.sentry.io/4508075566694480",
-    traces_sample_rate=1.0,
-    profiles_sample_rate=1.0,
-    enable_tracing=True
-)
+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_hash=API_HASH, bot_token=BOT_TOKEN)
-
-
-def download_tt_video(url: str) -> bool:
-    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, "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
-
-        with open(video_filepath, "wb") as f:
-            f.write(video_content)
-            log.debug("Video saved successfully")
-
-        return True
+app = Client("downloader_bot",
+             api_id=settings.api_id,
+             api_hash=settings.api_hash,
+             bot_token=settings.bot_token,
+             workers=settings.workers)
 
 
 @app.on_message(filters.command("start"))
-async def start_handler(client, message: Message):
+@security.protected
+async def start_handler(_, message: Message):
     await message.reply_text(
-        "This bot downloads TikTok videos " "to my personal server"
+        "This bot downloads TikTok videos to my personal server"
     )
 
 
 @app.on_message(filters.command("help"))
-async def help_handler(client, message: Message):
+@security.protected
+async def help_handler(_, message: Message):
     await message.reply_text("I won't help you!")
 
 
 @app.on_message(filters.text)
-async def message_handler(client, message: Message):
-    if message.chat.id != MY_MSG_ID:
-        return await message.reply_text("Nope, not talking to you!")
+@security.protected
+async def message_handler(_, message: Message):
+
+    urls = utils.extract_urls(message.text)
 
-    urls = re.findall(r"\bhttps?://[^\s]+", message.text)
     if not urls:
         return await message.reply_text(
-            "No links found in the message. " "Nothing to download!"
+            "No links found in the message. Nothing to download!"
+        )
+
+    tt_urls = utils.filter_tt_urls(urls)
+
+    if not tt_urls:
+        return await message.reply_text(
+            "No TikTok URLs found! Nothing to download!"
         )
 
-    success_count = 0
     for i, url in enumerate(urls):
         msg = f"Downloading video {i+1}/{len(urls)}..."
         log.info(msg)
         await message.reply_text(msg)
-        outcome = download_tt_video(url)
-        success_count += 1 if outcome else 0
+        status = await utils.download_tt_video(url)
 
-    await message.reply_text(f"{success_count}/{len(urls)} "
-                             "video(s) downloaded")
+    await message.reply_text(f"Done. {status}")
 
 
 @app.on_message(filters.media)
+@security.protected
 async def media_handler(client, message: Message):
-    if message.chat.id != MY_MSG_ID:
-        return await message.reply_text("Nope, not talking to you!")
-
-    # Determine folder name based on whether the message was forwarded
-    # and who it was forwarded from
-    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)
-        )
-    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()
-    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)
-        )
-
-    user_folder_name = "".join(
-        c for c in user_folder_name if c.isalnum() or c in (" ", "_")
-    ).rstrip()  # Sanitize the folder name
-
-    user_folder = os.path.join(STORAGE, "telegram", user_folder_name)
-    os.makedirs(user_folder, exist_ok=True)
-
-    # Reply to user that the download is starting
     await message.reply_text("Downloading media...")
-
-    # Handle documents
-    if message.document:
-        file_name = message.document.file_name
-        file_path = os.path.join(user_folder, file_name)
-        await client.download_media(message, file_path)
-        await message.reply_text(f"Document saved to {user_folder}")
-
-    # Handle single or multiple photos
-    elif message.photo:
-        file_name = f"photo_{message.photo.file_id}.jpg"
-        file_path = os.path.join(user_folder, file_name)
-        await client.download_media(message.photo, file_path)
-        await message.reply_text(f"Photo saved to {user_folder}")
-
-    # Handle videos
-    elif message.video:
-        file_name = f"video_{message.video.file_id}.mp4"
-        file_path = os.path.join(user_folder, file_name)
-        await client.download_media(message, file_path)
-        await message.reply_text(f"Video saved to {user_folder}")
-
-    # Handle GIFs
-    elif message.animation:
-        file_name = f"gif_{message.animation.file_id}.gif"
-        file_path = os.path.join(user_folder, file_name)
-        await client.download_media(message.animation, file_path)
-        await message.reply_text(f"GIF saved to {user_folder}")
-
-    # Handle unknown data types
-    else:
-        await message.reply_text("Unknown media type!")
+    await utils.handle_media_message_contents(client, message)
 
 
 if __name__ == "__main__":
-    os.makedirs(os.path.join(STORAGE, "tiktok"), exist_ok=True)
-    os.makedirs(os.path.join(STORAGE, "telegram"), exist_ok=True)
+    utils.make_fs(settings.storage)
     app.run()

telegram_downloader_bot/security.py

@@ -0,0 +1,19 @@
+from functools import wraps
+
+from telegram_downloader_bot.logger import log
+from telegram_downloader_bot.settings import settings
+
+
+def protected(func):
+    @wraps(func)
+    async def wrapper(client, message):
+        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(
+                "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

@@ -0,0 +1,33 @@
+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:
+    """
+    Initialize telemetry for the bot.
+    """
+    start_http_server(8000)

telegram_downloader_bot/utils.py

@@ -0,0 +1,181 @@
+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."""
+    output = re.sub("[^a-zA-Z0-9- ]", "", input)
+    output = output.replace(" ", "_")
+    return output
+
+
+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."""
+
+    # Message forwarded from someone
+    if message.forward_from:
+        user = message.forward_from
+        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
+        user_folder_name = user.title
+
+    # Direct message from user
+    else:
+        user = message.from_user
+        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 = sanitize_name(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_time(DOWNLOAD_DURATION.labels(service='telegram'))
+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(message)
+
+    # Handle documents
+    if message.document:
+        file_name = message.document.file_name
+        file_path = os.path.join(user_folder, file_name)
+        await client.download_media(message, file_path)
+        await message.reply_text(f"Document saved to {user_folder}")
+
+    # Handle single or multiple photos
+    elif message.photo:
+        file_name = f"photo_{message.photo.file_id}.jpg"
+        file_path = os.path.join(user_folder, file_name)
+        await client.download_media(message.photo, file_path)
+        await message.reply_text(f"Photo saved to {user_folder}")
+
+    # Handle videos
+    elif message.video:
+        file_name = f"video_{message.video.file_id}.mp4"
+        file_path = os.path.join(user_folder, file_name)
+        await client.download_media(message, file_path)
+        await message.reply_text(f"Video saved to {user_folder}")
+
+    # Handle GIFs
+    elif message.animation:
+        file_name = f"gif_{message.animation.file_id}.gif"
+        file_path = os.path.join(user_folder, file_name)
+        await client.download_media(message.animation, file_path)
+        await message.reply_text(f"GIF saved to {user_folder}")
+
+    # Handle unknown data types
+    else:
+        await message.reply_text("Unknown media type!")
+
+    size = os.path.getsize(file_path)
+    FILE_SIZE_BYTES.labels(service="telegram").observe(size)
+
+
+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."""
+
+    videos = snaptik(url)
+    now = datetime.now()
+
+    for video in videos:
+        video_filename = now.strftime("video-tiktok-%Y-%m-%d_%H-%M-%S.mp4")
+        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)
+    os.makedirs(os.path.join(storaga_path, "telegram"), exist_ok=True)
+
+
+def extract_urls(text: str) -> list:
+    return re.findall(r"\bhttps?://[^\s]+", text)
+
+
+def filter_tt_urls(urls: list) -> list:
+    return [x for x in urls if "tiktok" in x]

test-requirements.txt

@@ -1,4 +1,5 @@
-codespell
-flake8
-bandit
-pytest
+codespell==2.3.0
+flake8==7.1.1
+bandit==1.7.10
+stestr==4.1.0
+coverage==7.6.2

tests/integration_tests.py

@@ -0,0 +1 @@
+# To be implemented

tests/test_utils.py

@@ -0,0 +1,450 @@
+# test_utils.py
+
+import unittest
+import os
+import re
+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,
+    make_fs,
+    extract_urls,
+    filter_tt_urls,
+)
+
+from pyrogram.types import Message, User, Chat
+from pyrogram import Client
+
+
+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()
+        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)
+
+    def test_forward_from_full_name(self):
+        user = Mock()
+        user.first_name = "John"
+        user.last_name = "Doe"
+        user.id = 12345
+
+        message = Mock()
+        message.forward_from = user
+        message.forward_from_chat = None
+        message.from_user = None
+
+        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))
+
+    def test_forward_from_first_name_only(self):
+        user = Mock()
+        user.first_name = "John"
+        user.last_name = None
+        user.id = 12345
+
+        message = Mock()
+        message.forward_from = user
+        message.forward_from_chat = None
+        message.from_user = None
+
+        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))
+
+    def test_forward_from_chat_title(self):
+        chat = Mock()
+        chat.title = "My *Awesome* Group/Chat!"
+
+        message = Mock()
+        message.forward_from = None
+        message.forward_from_chat = chat
+        message.from_user = None
+
+        result = get_user_folder(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))
+
+    def test_from_user_full_name(self):
+        user = Mock()
+        user.first_name = "Jane"
+        user.last_name = "Doe"
+        user.id = 54321
+
+        message = Mock()
+        message.forward_from = None
+        message.forward_from_chat = None
+        message.from_user = user
+
+        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))
+
+    def test_from_user_id(self):
+        user = Mock()
+        user.first_name = None
+        user.last_name = None
+        user.id = 54321
+
+        message = Mock()
+        message.forward_from = None
+        message.forward_from_chat = None
+        message.from_user = user
+
+        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):
+#     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):
+#         # Stop patching settings.storage
+#         self.settings_patcher.stop()
+#         # Remove the directory after the test
+#         shutil.rmtree(self.tmp_path)
+
+#     @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(spec=Client)
+#         client.download_media = AsyncMock()
+
+#         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(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(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
+#         message.video = None
+#         message.animation = None
+#         message.reply_text = AsyncMock()
+
+#         await handle_media_message_contents(client, message)
+
+#         expected_file_path = os.path.join(user_folder, "test_document.pdf")
+#         client.download_media.assert_awaited_once_with(
+#             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_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(spec=Client)
+#         client.download_media = AsyncMock()
+
+#         message = Mock(spec=Message)
+#         message.document = None
+#         message.photo = Mock()
+#         message.photo.file_id = "photo_file_id"
+#         message.video = None
+#         message.animation = None
+#         message.reply_text = AsyncMock()
+
+#         await handle_media_message_contents(client, message)
+
+#         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_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(spec=Client)
+#         client.download_media = AsyncMock()
+
+#         message = Mock(spec=Message)
+#         message.document = None
+#         message.photo = None
+#         message.video = None
+#         message.animation = None
+#         message.reply_text = AsyncMock()
+
+#         await handle_media_message_contents(client, message)
+
+#         client.download_media.assert_not_called()
+#         message.reply_text.assert_awaited_once_with("Unknown media type!")
+
+
+# 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)
+#         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
+#         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):
+#         self.settings_patcher.stop()
+#         # Remove the directory after the test
+#         shutil.rmtree(self.tmp_path)
+
+#     @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
+
+#         # Read the content of valid.mp4
+#         with open(self.valid_video_path, 'rb') as f:
+#             valid_video_content = f.read()
+
+#         # 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("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))
+
+#         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
+
+#         # Read the content of invalid.mp4
+#         with open(self.invalid_video_path, 'rb') as f:
+#             invalid_video_content = f.read()
+
+#         # 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("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.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("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):
+#         self.tmp_path = tempfile.mkdtemp()
+
+#     def tearDown(self):
+#         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")))
+
+
+# class TestExtractURLs(unittest.TestCase):
+#     def test_no_urls(self):
+#         text = "This is some text without any URLs."
+#         result = extract_urls(text)
+#         self.assertEqual(result, [])
+
+#     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):
+#         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):
+#         text = "This is not a URL: htt://badurl.com"
+#         result = extract_urls(text)
+#         self.assertEqual(result, [])
+
+#     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):
+#         urls = []
+#         result = filter_tt_urls(urls)
+#         self.assertEqual(result, [])
+
+#     def test_no_tiktok_urls(self):
+#         urls = ["http://example.com", "https://test.com/page"]
+#         result = filter_tt_urls(urls)
+#         self.assertEqual(result, [])
+
+#     def test_mixed_urls(self):
+#         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, expected)
+
+#     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, expected)

tox.ini

@@ -1,9 +1,12 @@
 [tox]
-envlist = py311, flake8, bandit
+envlist = py311, flake8, bandit, codespell, unit, coverage
 
 [testenv]
 basepython = python3.11
-deps = -r {toxinidir}/test-requirements.txt
+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/
@@ -12,4 +15,12 @@ commands = flake8 telegram_downloader_bot/
 commands = bandit -r telegram_downloader_bot/
 
 [testenv:codespell]
-commands = codespell telegram_downloader_bot/
+commands = codespell telegram_downloader_bot/
+
+[testenv:unit]
+commands = stestr run --test-path tests/
+
+[testenv:coverage]
+commands = 
+    coverage run -m unittest discover
+    coverage report -m