fix: update test expectations for validateOnly mode to include warnings field

Fixed failing CI test by updating test expectations to match the new response
structure that includes a details.warnings field in validateOnly mode.

Changes:
- Updated test mock to include warnings: [] in applyDiff response
- Updated test expectations to include details: { warnings: [] }

Related to issue #360 fix.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en
This commit is contained in:
czlonkowski
2025-10-24 14:53:44 +02:00
parent a4c7e097e8
commit 95ea6ca0bb
2 changed files with 4 additions and 0 deletions

Binary file not shown.

View File

@@ -188,6 +188,7 @@ describe('handlers-workflow-diff', () => {
operationsApplied: 1, operationsApplied: 1,
message: 'Validation successful', message: 'Validation successful',
errors: [], errors: [],
warnings: []
}); });
const result = await handleUpdatePartialWorkflow(diffRequest, mockRepository); const result = await handleUpdatePartialWorkflow(diffRequest, mockRepository);
@@ -199,6 +200,9 @@ describe('handlers-workflow-diff', () => {
valid: true, valid: true,
operationsToApply: 1, operationsToApply: 1,
}, },
details: {
warnings: []
}
}); });
expect(mockApiClient.updateWorkflow).not.toHaveBeenCalled(); expect(mockApiClient.updateWorkflow).not.toHaveBeenCalled();