From e5a0d05b334b3f6e3a9f3a40b98d47a936dc74f8 Mon Sep 17 00:00:00 2001
From: Ralph Khreish <35776126+Crunchyman-ralph@users.noreply.github.com>
Date: Mon, 6 Oct 2025 17:58:31 +0200
Subject: [PATCH] chore: apply requested changes

---
 tests/unit/dependency-manager.test.js | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/tests/unit/dependency-manager.test.js b/tests/unit/dependency-manager.test.js
index 7743cd0b..a584ce26 100644
--- a/tests/unit/dependency-manager.test.js
+++ b/tests/unit/dependency-manager.test.js
@@ -996,7 +996,10 @@ describe('Dependency Manager Module', () => {
 				() => ({ default: jest.fn() })
 			);
 			// Set up test data that matches the issue report
-			mockReadJSON.mockReturnValue(crossLevelDependencyTasks);
+			// Clone fixture data before each test to prevent mutation issues
+			mockReadJSON.mockImplementation(() =>
+				structuredClone(crossLevelDependencyTasks)
+			);
 
 			// Configure mockTaskExists to properly validate cross-level dependencies
 			mockTaskExists.mockImplementation((tasks, taskId) => {
@@ -1091,7 +1094,13 @@ describe('Dependency Manager Module', () => {
 
 			// Verify the dependency was actually added to task 11
 			expect(task11.dependencies).toContain('2.1');
-			expect(mockLog).toHaveBeenCalled();
+			// Verify a success log was emitted mentioning both task 11 and subtask 2.1
+			const successCall = mockLog.mock.calls.find(
+				([level]) => level === 'success'
+			);
+			expect(successCall).toBeDefined();
+			expect(successCall[1]).toContain('11');
+			expect(successCall[1]).toContain('2.1');
 		});
 
 		test('should properly validate cross-level dependencies exist', async () => {