Fixed a bug in the glob tests.

Was not properly looking in the correct relative directories.

Author: coordt

tests/test_config/test_utils.py

@@ -10,7 +10,7 @@
 from bumpversion.config.utils import get_all_file_configs, resolve_glob_files
 from bumpversion.config.models import FileChange
 from bumpversion.config import DEFAULTS
-from tests.conftest import inside_dir
+from tests.conftest import inside_dir, get_config_data
 
 
 def write_config(tmp_path: Path, overrides: dict) -> Path:
@@ -114,6 +114,16 @@ def test_all_attributes_are_copied(self, tmp_path: Path):
             assert resolved_file.ignore_missing_file is True
             assert resolved_file.regex is True
 
+    def test_finds_all_files(self, fixtures_path: Path):
+        """Test that all files are found."""
+        overrides = {
+            "current_version": "1.2.3",
+            "files": [{"glob": "glob/**/*.txt", "ignore_missing_file": True, "ignore_missing_version": True}],
+        }
+        with inside_dir(fixtures_path):
+            conf, version_config, current_version = get_config_data(overrides)
+            assert len(conf.files_to_modify) == 3
+
     @pytest.mark.parametrize(
         ["glob_exclude", "expected_number"],
         [

tests/test_files.py

@@ -117,15 +117,15 @@ def test_dedents_properly_when_file_does_not_contain_pattern(self, fixtures_path
             caplog.set_level(logging.INFO)
             logger = get_indented_logger(__name__)
             logger.reset()
-            globs = f"{fixtures_path}/glob/**/*.txt"
             overrides = {
                 "current_version": "1.2.3",
-                "files": [{"glob": str(globs), "ignore_missing_file": True, "ignore_missing_version": True}],
+                "files": [{"glob": "glob/**/*.txt", "ignore_missing_file": True, "ignore_missing_version": True}],
             }
-            conf, version_config, current_version = get_config_data(overrides)
-            configured_file1 = files.ConfiguredFile(conf.files_to_modify[0], version_config)
-            configured_file2 = files.ConfiguredFile(conf.files_to_modify[1], version_config)
-            configured_file3 = files.ConfiguredFile(conf.files_to_modify[2], version_config)
+            with inside_dir(fixtures_path):
+                conf, version_config, current_version = get_config_data(overrides)
+                configured_file1 = files.ConfiguredFile(conf.files_to_modify[0], version_config)
+                configured_file2 = files.ConfiguredFile(conf.files_to_modify[1], version_config)
+                configured_file3 = files.ConfiguredFile(conf.files_to_modify[2], version_config)
             new_version = current_version.bump("patch")
             ctx = get_context(conf)
 
@@ -141,12 +141,15 @@ def test_dedents_properly_when_file_does_not_contain_pattern(self, fixtures_path
             assert logs[2] == (
                 f"\nFile {configured_file1.file_change.filename}: replace `{{current_version}}` with `{{new_version}}`"
             )
-            assert logs[3] == (
+            assert logs[3] == "  File not found, but ignoring"
+            assert logs[4] == (
                 f"\nFile {configured_file2.file_change.filename}: replace `{{current_version}}` with `{{new_version}}`"
             )
-            assert logs[4] == (
+            assert logs[5] == "  File not found, but ignoring"
+            assert logs[6] == (
                 f"\nFile {configured_file3.file_change.filename}: replace `{{current_version}}` with `{{new_version}}`"
             )
+            assert logs[7] == "  File not found, but ignoring"
 
 
 def test_single_file_processed_twice(tmp_path: Path):