Refactor configuration logging and cache clearing

In this commit, the logging of the configuration file usage has been moved to the `read_config` function from the `get_config_file` function. This change ensures that the log message is only generated when the configuration file is actually read, providing a more accurate representation of the program's operations.

Additionally, the cache of the `read_config` function is now explicitly cleared in the `test_configure` function in the test suite. This ensures that each test starts with a clean state, preventing potential interference from previous tests.

Author: TechNickAI

aicodebot/config.py

@@ -3,20 +3,19 @@
 import functools, os, yaml
 
 
-@functools.lru_cache
 def get_config_file():
     if "AICODEBOT_CONFIG_FILE" in os.environ:
         config_file = Path(os.getenv("AICODEBOT_CONFIG_FILE"))
     else:
         config_file = Path(Path.home() / ".aicodebot.yaml")
-    logger.debug(f"Using config file {config_file}")
     return config_file
 
 
 @functools.lru_cache
 def read_config():
     """Read the config file and return its contents as a dictionary."""
     config_file = get_config_file()
+    logger.debug(f"Using config file {config_file}")
     if config_file.exists():
         logger.debug(f"Config file {config_file} exists")
         with Path(config_file).open("r") as f:

tests/test_cli.py

@@ -22,6 +22,7 @@ def test_configure(cli_runner, tmp_path, monkeypatch):
     # set aicodebot_config_file to the temp config file
     monkeypatch.setenv("AICODEBOT_CONFIG_FILE", str(temp_config_file))
 
+    read_config.cache_clear()
     assert read_config() is None
 
     # run the setup command, should work with the env var set
@@ -33,13 +34,15 @@ def test_configure(cli_runner, tmp_path, monkeypatch):
     assert Path(temp_config_file).exists()
 
     # load the config file
+    read_config.cache_clear()
     config_data = read_config()
     # check if the config file contains the correct data
     assert config_data["openai_api_key"] == key
     assert config_data["personality"] == DEFAULT_PERSONALITY.name
 
     # remove the config file
     Path.unlink(temp_config_file)
+    read_config.cache_clear()
     assert read_config() is None
 
     # now unset the env var and run the command again with it passed as a flag
@@ -50,6 +53,7 @@ def test_configure(cli_runner, tmp_path, monkeypatch):
     assert result.exit_code == 0, f"output: {result.output}"
 
     # load the config file
+    read_config.cache_clear()
     config_data = read_config()
     # check if the config file contains the correct data
     assert config_data["openai_api_key"] == key