fix(changelog): changelog hook was not called on dry run

Author: noirbizarre

commitizen/commands/changelog.py

@@ -132,6 +132,11 @@ def write_changelog(
 
             if changelog_hook:
                 changelog_out = changelog_hook(changelog_out, partial_changelog)
+
+            if self.dry_run:
+                out.write(changelog_out)
+                raise DryRunExit()
+
             changelog_file.write(changelog_out)
 
     def export_template(self):
@@ -216,10 +221,6 @@ def __call__(self):
         )
         changelog_out = changelog_out.lstrip("\n")
 
-        if self.dry_run:
-            out.write(changelog_out)
-            raise DryRunExit()
-
         lines = []
         if self.incremental and os.path.isfile(self.file_name):
             with open(self.file_name, encoding=self.encoding) as changelog_file:

tests/commands/test_changelog_command.py

@@ -12,6 +12,7 @@
 from commitizen import __file__ as commitizen_init
 from commitizen import cli, git
 from commitizen.commands.changelog import Changelog
+from commitizen.config.base_config import BaseConfig
 from commitizen.cz.base import BaseCommitizen
 from commitizen.exceptions import (
     DryRunExit,
@@ -275,20 +276,24 @@ def test_changelog_incremental_keep_a_changelog_sample(
 
 
 @pytest.mark.usefixtures("tmp_commitizen_project")
-def test_changelog_hook(mocker: MockFixture, config):
+@pytest.mark.parametrize("dry_run", [True, False])
+def test_changelog_hook(mocker: MockFixture, config: BaseConfig, dry_run: bool):
     changelog_hook_mock = mocker.Mock()
     changelog_hook_mock.return_value = "cool changelog hook"
 
     create_file_and_commit("feat: new file")
     create_file_and_commit("refactor: is in changelog")
     create_file_and_commit("Merge into master")
 
-    config.settings["change_type_order"] = ["Refactor", "Feat"]
+    config.settings["change_type_order"] = ["Refactor", "Feat"]  # type: ignore[typeddict-unknown-key]
     changelog = Changelog(
-        config, {"unreleased_version": None, "incremental": True, "dry_run": False}
+        config, {"unreleased_version": None, "incremental": True, "dry_run": dry_run}
     )
     mocker.patch.object(changelog.cz, "changelog_hook", changelog_hook_mock)
-    changelog()
+    try:
+        changelog()
+    except DryRunExit:
+        pass
     full_changelog = (
         "## Unreleased\n\n### Refactor\n\n- is in changelog\n\n### Feat\n\n- new file\n"
     )