Skip to content

Conversation

@mfsiega
Copy link
Contributor

@mfsiega mfsiega commented Nov 23, 2025

Summary

Fix a broken test expectation.

#21811 changed the behaviour but didn't update the test correctly (and somehow passed CI to get merged). Subsequent CI on master has been failing since then.

Related Linear tickets, Github issues, and Community forum posts

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@codecov
Copy link

codecov bot commented Nov 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team node/improvement New feature or request labels Nov 23, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@mfsiega mfsiega closed this Nov 23, 2025
@mfsiega mfsiega deleted the form-node-date-format-test-fix branch November 23, 2025 13:18
@mfsiega mfsiega restored the form-node-date-format-test-fix branch November 23, 2025 13:18
@mfsiega mfsiega deleted the form-node-date-format-test-fix branch November 23, 2025 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

n8n team Authored by the n8n team node/improvement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants