refactor: Update import statements to include 'src' prefix for module paths
Build and Push Docker Image / build-and-push (push) Has been cancelled

This commit is contained in:
William Valentin
2025-07-29 16:52:46 -07:00
parent 5243352867
commit 21dd1fc9c8
22 changed files with 68 additions and 68 deletions
+25 -25
View File
@@ -20,9 +20,9 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
assert constants.LOG_LEVEL == "INFO"
assert src.constants.LOG_LEVEL == "INFO"
def test_custom_log_level(self):
"""Test custom LOG_LEVEL from environment."""
@@ -31,9 +31,9 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
assert constants.LOG_LEVEL == "DEBUG"
assert src.constants.LOG_LEVEL == "DEBUG"
def test_default_log_path(self):
"""Test default LOG_PATH when not set in environment."""
@@ -42,9 +42,9 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
assert constants.LOG_PATH == "/tmp/logs/thechart"
assert src.constants.LOG_PATH == "/tmp/logs/thechart"
def test_custom_log_path(self):
"""Test custom LOG_PATH from environment."""
@@ -53,9 +53,9 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
assert constants.LOG_PATH == "/custom/log/path"
assert src.constants.LOG_PATH == "/custom/log/path"
def test_default_log_clear(self):
"""Test default LOG_CLEAR when not set in environment."""
@@ -64,9 +64,9 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
assert constants.LOG_CLEAR == "False"
assert src.constants.LOG_CLEAR == "False"
def test_custom_log_clear_true(self):
"""Test LOG_CLEAR when set to true in environment."""
@@ -75,9 +75,9 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
assert constants.LOG_CLEAR == "True"
assert src.constants.LOG_CLEAR == "True"
def test_custom_log_clear_false(self):
"""Test LOG_CLEAR when set to false in environment."""
@@ -86,9 +86,9 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
assert constants.LOG_CLEAR == "False"
assert src.constants.LOG_CLEAR == "False"
def test_log_level_case_insensitive(self):
"""Test that LOG_LEVEL is converted to uppercase."""
@@ -97,9 +97,9 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
assert constants.LOG_LEVEL == "WARNING"
assert src.constants.LOG_LEVEL == "WARNING"
def test_dotenv_override(self):
"""Test that dotenv override parameter is set to True."""
@@ -109,22 +109,22 @@ class TestConstants:
if 'constants' in sys.modules:
importlib.reload(sys.modules['constants'])
else:
import constants
import src.constants
mock_load_dotenv.assert_called_once_with(override=True)
def test_all_constants_are_strings(self):
"""Test that all constants are string type."""
import constants
import src.constants
assert isinstance(constants.LOG_LEVEL, str)
assert isinstance(constants.LOG_PATH, str)
assert isinstance(constants.LOG_CLEAR, str)
assert isinstance(src.constants.LOG_LEVEL, str)
assert isinstance(src.constants.LOG_PATH, str)
assert isinstance(src.constants.LOG_CLEAR, str)
def test_constants_not_empty(self):
"""Test that constants are not empty strings."""
import constants
import src.constants
assert constants.LOG_LEVEL != ""
assert constants.LOG_PATH != ""
assert constants.LOG_CLEAR != ""
assert src.constants.LOG_LEVEL != ""
assert src.constants.LOG_PATH != ""
assert src.constants.LOG_CLEAR != ""
+1 -1
View File
@@ -12,7 +12,7 @@ import matplotlib.pyplot as plt
import sys
sys.path.insert(0, os.path.join(os.path.dirname(__file__), '..', 'src'))
from graph_manager import GraphManager
from src.graph_manager import GraphManager
class TestGraphManager:
+19 -19
View File
@@ -24,7 +24,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
mock_mkdir.assert_called_once()
@@ -38,7 +38,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
mock_mkdir.assert_not_called()
@@ -53,7 +53,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
mock_print.assert_called()
@@ -70,7 +70,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
mock_init_logger.assert_called_once_with('init', testing_mode=False)
@@ -87,7 +87,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
mock_init_logger.assert_called_once_with('init', testing_mode=True)
@@ -98,7 +98,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
expected_files = (
f"{temp_log_dir}/thechart.log",
@@ -106,7 +106,7 @@ class TestInit:
f"{temp_log_dir}/thechart.error.log",
)
assert init.log_files == expected_files
assert src.init.log_files == expected_files
def test_testing_mode_detection(self, temp_log_dir):
"""Test that testing mode is detected correctly."""
@@ -117,14 +117,14 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
assert init.testing_mode is True
assert src.init.testing_mode is True
# Test with non-DEBUG level
with patch('init.LOG_LEVEL', 'INFO'):
importlib.reload(sys.modules['init'])
assert init.testing_mode is False
assert src.init.testing_mode is False
def test_log_clear_true(self, temp_log_dir):
"""Test log file clearing when LOG_CLEAR is True."""
@@ -147,7 +147,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
# Check that files were truncated
for log_file in log_files:
@@ -176,7 +176,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
# Check that files were not truncated
for log_file in log_files:
@@ -203,7 +203,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
def test_log_clear_permission_error(self, temp_log_dir):
"""Test handling of permission errors during log clearing."""
@@ -226,7 +226,7 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
def test_module_exports(self, temp_log_dir):
"""Test that module exports expected objects."""
@@ -235,12 +235,12 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
# Check that expected objects are available
assert hasattr(init, 'logger')
assert hasattr(init, 'log_files')
assert hasattr(init, 'testing_mode')
assert hasattr(src.init, 'logger')
assert hasattr(src.init, 'log_files')
assert hasattr(src.init, 'testing_mode')
def test_log_path_printing(self, temp_log_dir):
"""Test that LOG_PATH is printed when directory is created."""
@@ -253,6 +253,6 @@ class TestInit:
if 'init' in sys.modules:
importlib.reload(sys.modules['init'])
else:
import init
import src.init
mock_print.assert_called_with(temp_log_dir + '/new_dir')
+1 -1
View File
@@ -10,7 +10,7 @@ from unittest.mock import patch, Mock
import sys
sys.path.insert(0, os.path.join(os.path.dirname(__file__), '..', 'src'))
from logger import init_logger
from src.logger import init_logger
class TestLogger:
+1 -1
View File
@@ -10,7 +10,7 @@ import pandas as pd
import sys
sys.path.insert(0, os.path.join(os.path.dirname(__file__), '..', 'src'))
from main import MedTrackerApp
from src.main import MedTrackerApp
class TestMedTrackerApp: