add commit-confirm options to vyos_config (#229)

* added vyos_config confirm options

* fix: update commit comment to match main

* fix: documentation example for automatic confirm

* fix: add tests for commit confirm and confirm_timeout options

* fix: add config_confirm changelog fragment

* fix: lint missing diff parameter for edit_config

* fix: add commit confirm integration test

---------

Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>

1 files changed, 38 insertions, 0 deletions

diff --git a/tests/unit/modules/network/vyos/test_vyos_config.py b/tests/unit/modules/network/vyos/test_vyos_config.py
index 4f1cac6a..601971cd 100644
--- a/tests/unit/modules/network/vyos/test_vyos_config.py
+++ b/tests/unit/modules/network/vyos/test_vyos_config.py
@@ -140,3 +140,41 @@ class TestVyosConfigModule(TestVyosModule):
             return_value=self.cliconf_obj.get_diff(candidate, None, diff_match="none"),
         )
         self.execute_module(changed=True, commands=lines, sort=False)
+
+    def test_vyos_config_confirm_automatic(self):
+        src = load_fixture("vyos_config_src.cfg")
+        confirm_timeout = 7
+        set_module_args(dict(src=src, confirm="automatic", confirm_timeout=confirm_timeout))
+        candidate = "\n".join(self.module.format_commands(src.splitlines()))
+        commands = [
+            "set system host-name foo",
+            "delete interfaces ethernet eth0 address",
+        ]
+        self.conn.get_diff = MagicMock(
+            return_value=self.cliconf_obj.get_diff(candidate, self.running_config),
+        )
+
+        self.execute_module(changed=True, commands=commands)
+
+        self.assertEqual(self.load_config.call_args[1]["confirm"], confirm_timeout)
+        self.run_commands.assert_called_once()
+        self.assertEqual(
+            ["configure", "confirm", "exit"],
+            self.run_commands.call_args[0][1],
+        )
+
+    def test_vyos_config_confirm_manual(self):
+        lines = [
+            "set system host-name foo",
+        ]
+        confirm_timeout = 12
+        set_module_args(dict(lines=lines, confirm="manual", confirm_timeout=confirm_timeout))
+        candidate = "\n".join(lines)
+        self.conn.get_diff = MagicMock(
+            return_value=self.cliconf_obj.get_diff(candidate, self.running_config),
+        )
+
+        self.execute_module(changed=True, commands=lines)
+
+        self.assertEqual(self.load_config.call_args[1]["confirm"], confirm_timeout)
+        self.run_commands.assert_not_called()