v3.0.3 (#134)

* feat: dynamically retrieve package version from metadata with fallback to hardcoded string

* refactor: remove redundant global declarations and update development dependencies

* version 3.0.3

* test: implement comprehensive test suite and Docker-based cross-version CI pipeline

* chore: add --verbose flag to twine upload command in deploy target

Author: dpnkrpl

.dockerignore

@@ -0,0 +1,10 @@
+.git
+venv
+.venv
+__pycache__
+*.pyc
+.pytest_cache
+*.egg-info
+build
+dist
+.mypy_cache

.gitignore

@@ -133,8 +133,7 @@ deploy.sh
 
 pylint.txt
 .gitpod.yml
-tests/
-tests/
+
 
 *.mp3
 cache.sqlite

Makefile

@@ -33,7 +33,7 @@ dist: clean
 	${PYTHON} setup.py sdist bdist_wheel
 
 deploy: dist
-	twine upload --repository-url https://upload.pypi.org/legacy/ dist/*
+	twine upload --repository-url https://upload.pypi.org/legacy/ dist/* --verbose
 
 test-deploy: dist
 	@echo "Sending to testpypi server......."

docker/Dockerfile.test

@@ -0,0 +1,26 @@
+# Multi-Python CI image: build with --build-arg PYTHON_VERSION=3.11
+ARG PYTHON_VERSION=3.11
+FROM python:${PYTHON_VERSION}-slim-bookworm
+
+ENV PIP_DISABLE_PIP_VERSION_CHECK=1 \
+    PYTHONDONTWRITEBYTECODE=1 \
+    PYTHONUNBUFFERED=1
+
+WORKDIR /app
+
+# Some dependencies may need a compiler when wheels are missing
+RUN apt-get update \
+    && apt-get install -y --no-install-recommends gcc make \
+    && rm -rf /var/lib/apt/lists/*
+
+COPY requirements.txt requirements-dev.txt setup.py README.md Makefile configure.py features.conf ./
+COPY radioactive ./radioactive
+
+RUN pip install --upgrade pip setuptools wheel \
+    && pip install -r requirements.txt -r requirements-dev.txt \
+    && pip install -e .
+
+COPY pytest.ini ./
+COPY tests ./tests
+
+CMD ["pytest", "-v", "tests/"]

pytest.ini

@@ -0,0 +1,4 @@
+[pytest]
+testpaths = tests
+python_files = test_*.py
+addopts = -ra

radioactive/__main__.py

@@ -479,8 +479,6 @@ def main():
 
 
 def signal_handler(sig, frame):
-    global ffplay
-    global player
     log.debug("You pressed Ctrl+C!")
     log.debug("Stopping the radio")
     if ffplay and ffplay.is_playing:

radioactive/app.py

@@ -1,15 +1,24 @@
 """
-    Version of the current program, (in development mode
-    it needs to be updated in every release)
-    and to check if an updated version available for the app or not
+Version of the current program, (in development mode
+it needs to be updated in every release)
+and to check if an updated version available for the app or not
 """
 
 import json
+import sys
+
+if sys.version_info >= (3, 8):
+    from importlib import metadata
+else:
+    import importlib_metadata as metadata
 
 
 class App:
     def __init__(self):
-        self.__VERSION__ = "3.0.2"  # change this on every update #
+        try:
+            self.__VERSION__ = metadata.version("radio-active")
+        except metadata.PackageNotFoundError:
+            self.__VERSION__ = "3.0.3"  # change this on every update #
         self.pypi_api = "https://pypi.org/pypi/radio-active/json"
         self.remote_version = ""
 

radioactive/handler.py

@@ -1,5 +1,5 @@
 """
-    This handler solely depends on pyradios module to communicate with our remote API
+This handler solely depends on pyradios module to communicate with our remote API
 """
 
 import datetime

radioactive/last_station.py

@@ -1,8 +1,7 @@
-""" This module saves the current playing station information to a hidden file,
-and loads the data when no arguments are provide """
+"""This module saves the current playing station information to a hidden file,
+and loads the data when no arguments are provide"""
 
 import json
-import os.path
 
 from zenlog import log
 

radioactive/ui.py

@@ -111,7 +111,6 @@ def handle_history_table(history) -> None:
 def handle_show_station_info() -> None:
     """Show important information regarding the current station."""
     # pylint: disable=global-statement
-    global global_current_station_info
     custom_info = {}
     try:
         custom_info["name"] = global_current_station_info.get("name")
@@ -150,5 +149,4 @@ def set_global_station_info(info: dict) -> None:
 
 def get_global_station_info() -> dict:
     """Helper to get global station info."""
-    global global_current_station_info
     return global_current_station_info