ros/n8n-mcp
1
0
Fork 0
mirror of https://github.com/czlonkowski/n8n-mcp.git synced 2026-01-30 06:22:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: update test to match new Google Sheets validation logic

Browse Source 
Updated test expectation to match the new validation that accepts
EITHER range OR columns for Google Sheets append operation. This
fixes the CI test failure.

Test was expecting old message: 'Range is required for append operation'
Now expects: 'Range or columns mapping is required for append operation'

Related to #304 - Google Sheets v4+ resourceMapper validation

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

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
czlonkowski
2025-10-11 20:14:09 +02:00
parent fc8fb66900
commit c580a3dde4
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
tests/unit/services/node-specific-validators.test.ts
View File

@@ -347,14 +347,14 @@ describe('NodeSpecificValidators', () => {
};
});
it('should require range for append', () => {
it('should require range or columns for append', () => {
NodeSpecificValidators.validateGoogleSheets(context);
expect(context.errors).toContainEqual({
type: 'missing_required',
property: 'range',
message: 'Range is required for append operation',
fix: 'Specify range like "Sheet1!A:B" or "Sheet1!A1:B10"'
message: 'Range or columns mapping is required for append operation',
fix: 'Specify range like "Sheet1!A:B" OR use columns with mappingMode'
});
});