Include changes in the results when schedule.present state is run wit…

…h test=True.
jynolen · Dec 21, 2023 · 9f5a1a7 · 9f5a1a7
1 parent 05faca6
commit 9f5a1a7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/changelog/65652.fixed.md b/changelog/65652.fixed.md
@@ -0,0 +1 @@
+Include changes in the results when schedule.present state is run with test=True.
diff --git a/salt/states/schedule.py b/salt/states/schedule.py
@@ -264,6 +264,7 @@ def present(name, **kwargs):
             kwargs["test"] = True
             result = __salt__["schedule.add"](name, **kwargs)
             ret["comment"].append(result["comment"])
+            ret["changes"] = {name: "added"}
         else:
             result = __salt__["schedule.add"](name, **kwargs)
             if not result["result"]:

diff --git a/tests/pytests/unit/states/test_schedule.py b/tests/pytests/unit/states/test_schedule.py
@@ -220,7 +220,7 @@ def test_present():
         ret = {
             "name": "job1",
             "result": True,
-            "changes": {},
+            "changes": {"job1": "added"},
             "comment": "Job: test-schedule would be added to schedule.",
         }
         with patch.dict(schedule.__opts__, {"test": True}):
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Include changes in the results when schedule.present state is run with test=True.