test(bump): add tests for bumping only version files

Author: marcusalstrom

tests/commands/test_bump_command.py

@@ -535,6 +535,24 @@ def test_bump_major_version_zero_when_major_is_not_zero(mocker, tmp_commitizen_p
     assert expected_error_message in str(excinfo.value)
 
 
+def test_bump_not_add_unstaged(mocker: MockFixture, tmp_commitizen_project):
+    unstaged_file = "ui.py"
+    create_file_and_commit("feat: new file", unstaged_file)
+    tmp_file = tmp_commitizen_project.join(unstaged_file)
+    tmp_file.write("updated content")
+    # print("Before")
+    # cmd_res = cmd.run('git status')
+    # print(cmd_res.out)
+
+    testargs = ["cz", "bump", "--yes"]
+    mocker.patch.object(sys, "argv", testargs)
+    cli.main()
+    # print("After")
+    cmd_res = cmd.run("git diff --name-only")
+    # print(cmd_res.out)
+    assert unstaged_file in cmd_res.out
+
+
 def test_bump_files_only(mocker: MockFixture, tmp_commitizen_project):
     tmp_version_file = tmp_commitizen_project.join("__version__.py")
     tmp_version_file.write("0.1.0")

tests/providers/test_cargo_provider.py

@@ -58,3 +58,6 @@ def test_cargo_provider(
 
     provider.set_version("42.1")
     assert file.read_text() == dedent(expected)
+
+    files = provider.get_files()
+    assert filename in files

tests/providers/test_composer_provider.py

@@ -45,3 +45,6 @@ def test_composer_provider(
 
     provider.set_version("42.1")
     assert file.read_text() == dedent(expected)
+
+    files = provider.get_files()
+    assert filename in files

tests/providers/test_npm_provider.py

@@ -96,3 +96,10 @@ def test_npm_provider(
         assert pkg_lock.read_text() == dedent(pkg_lock_expected)
     if pkg_shrinkwrap_content:
         assert pkg_shrinkwrap.read_text() == dedent(pkg_shrinkwrap_expected)
+
+    files = provider.get_files()
+    assert NpmProvider.package_filename in files
+    if pkg_lock_content:
+        assert NpmProvider.lock_filename in files
+    if pkg_shrinkwrap_content:
+        assert NpmProvider.shrinkwrap_filename in files

tests/providers/test_pep621_provider.py

@@ -41,3 +41,6 @@ def test_cargo_provider(
 
     provider.set_version("42.1")
     assert file.read_text() == dedent(expected)
+
+    files = provider.get_files()
+    assert filename in files

tests/providers/test_poetry_provider.py

@@ -41,3 +41,6 @@ def test_cargo_provider(
 
     provider.set_version("42.1")
     assert file.read_text() == dedent(expected)
+
+    files = provider.get_files()
+    assert filename in files

tests/providers/test_scm_provider.py

@@ -60,6 +60,10 @@ def test_scm_provider(
     # Should not fail on set_version()
     provider.set_version("43.1")
 
+    # Shouldn not have any files
+    files = provider.get_files()
+    assert files == []
+
 
 @pytest.mark.usefixtures("tmp_git_project")
 def test_scm_provider_default_without_commits_and_tags(config: BaseConfig):