feat: add test history functionality and update README

Author: AlexTkDev

README.md

@@ -94,7 +94,8 @@ macOS_application_speedtest_for_python/
 ├── .github/workflows/         # GitHub Actions configuration
 │   └── pylint.yml
 ├── LICENSE                    # MIT License
-└── README.md                  # This documentation
+├──  README.md                 # This documentation
+└── test_history.py            # Test history file 
 ```
 
 #### How It Works

main.spec

@@ -4,6 +4,7 @@ a = Analysis(
     ['speedtest_app/alexs_speedtest.py'],
     pathex=[],
     binaries=[],
+    datas=[('test_history.json', '.')],
     hiddenimports=[
         'speedtest_cli',
         'speedtest_app.network_adapter_information',

speedtest_app/test_history.py

@@ -6,41 +6,54 @@
 from matplotlib import pyplot as plt
 from matplotlib.backends.backend_tkagg import FigureCanvasTkAgg, NavigationToolbar2Tk
 
-# Получаем логгер
 logger = logging.getLogger("SpeedTest")
 
-# Path to the file for storing history
-HISTORY_FILE = "test_history.json"
 
+def get_history_file_path():
+    """Returns the path to the history file in Downloads directory."""
+    downloads_dir = os.path.join(os.path.expanduser("~"), "Downloads")
+    return os.path.join(downloads_dir, "speedtest_history.json")
+
+
+def save_test_results(download_speed, upload_speed, ping, file_path=None):
+    """Saves the test results to the Downloads directory."""
+    if file_path is None:
+        file_path = get_history_file_path()
 
-def save_test_results(download_speed, upload_speed, ping, file_path="test_history.json"):
-    """Saves the test results to a specified JSON file with timestamp."""
     data = {
         "timestamp": datetime.now().strftime("%Y-%m-%d %H:%M:%S"),
         "download_speed": download_speed,
         "upload_speed": upload_speed,
         "ping": ping
     }
 
-    # If the file exists, load the current data
-    if os.path.exists(file_path):
-        try:
-            with open(file_path, "r", encoding="utf-8") as file:
-                history = json.load(file)
-        except json.JSONDecodeError:
-            logger.error(f"Error decoding JSON from {file_path}. Creating new history.")
+    try:
+        # Ensure directory exists (although Downloads should always exist)
+        downloads_dir = os.path.dirname(file_path)
+        if not os.path.exists(downloads_dir):
+            logger.warning(f"Downloads directory not found at {downloads_dir}")
+            messagebox.showerror("Error", "Downloads directory not found")
+            return
+
+        # Load existing history or create new
+        if os.path.exists(file_path):
+            try:
+                with open(file_path, "r", encoding="utf-8") as file:
+                    history = json.load(file)
+            except json.JSONDecodeError:
+                logger.error(f"Error decoding JSON from {file_path}. Creating new history.")
+                history = []
+        else:
             history = []
-    else:
-        history = []
 
-    # Add new data
-    history.append(data)
+        # Add new data
+        history.append(data)
 
-    # Save the updated file
-    try:
+        # Save the updated file
         with open(file_path, "w", encoding="utf-8") as file:
             json.dump(history, file, indent=4)
         logger.info(f"Test results saved to {file_path}")
+
     except Exception as e:
         logger.error(f"Error saving test results: {e}", exc_info=True)
         messagebox.showerror("Error", f"Could not save test results: {e}")
@@ -246,4 +259,4 @@ def save_plot():
             messagebox.showerror("Save Error", f"Could not save plot: {e}")
 
     save_button = Button(button_frame, text="Save Plot", command=save_plot)
-    save_button.pack(side="left", padx=5)
+    save_button.pack(side="left", padx=5)