Skip to content

refactor(test): move toFieldError tests to validation-types.test.ts#443

Merged
nullvariant merged 1 commit into
mainfrom
refactor/move-to-field-error-tests-to-validation-types
Apr 6, 2026
Merged

refactor(test): move toFieldError tests to validation-types.test.ts#443
nullvariant merged 1 commit into
mainfrom
refactor/move-to-field-error-tests-to-validation-types

Conversation

@nullvariant
Copy link
Copy Markdown
Owner

@nullvariant nullvariant commented Apr 6, 2026

Summary

  • Move toFieldError tests from errors.test.ts to new validation-types.test.ts
  • Remove toFieldError import from errors.test.ts
  • Register new test file in runTests.ts

Aligns test file with source module for 1:1 correspondence (validation-types.ts → validation-types.test.ts).

Test plan

  • npx tsc --noEmit passes
  • npx eslint . --max-warnings=0 passes
  • npm run test:coverage passes with 100% statement coverage
  • All existing tests still pass (no behavioral change)

🤖 Generated with Claude Code

@nullvariant @claude
refactor(test): move toFieldError tests to validation-types.test.ts
8a24cee 
Aligns test file with source module for 1:1 correspondence.
toFieldError is defined in validation-types.ts, so its tests
belong in validation-types.test.ts rather than errors.test.ts.
Signed-off-by: Null;Variant <null@nullvariant.com>

🖥️ IDE: [VS Code](https://code.visualstudio.com/)
🔌 Extension: [Claude Code](https://claude.ai/download)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Model-Raw: claude-opus-4-6
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 6, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 8a24cee.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@nullvariant-mimi
Copy link
Copy Markdown
Contributor

nullvariant-mimi Bot commented Apr 6, 2026

🐰 Mimi's Validation Report ✅

All checks are looking good! Great job! 🎉

⏳ Some checks are still running. I will keep watching!

バリデーターを通してくださいね

This report was carefully prepared by nullvariant-mimi[bot]

@nullvariant-slow
Copy link
Copy Markdown
Contributor

nullvariant-slow Bot commented Apr 6, 2026

🦥 Slow's Code Review 😩

...yawn... Do I really have to review this?

⚠️ TOO LONG... I can barely keep my eyes open reading these:

File Lines

| extensions/git-id-switcher/src/test/errors.test.ts | 467 |

Split it up... reading long files is exhausting.

働きたくないでござる

This review was reluctantly filed by nullvariant-slow[bot]

@nullvariant-ciel
Copy link
Copy Markdown
Contributor

nullvariant-ciel Bot commented Apr 6, 2026 •
edited
Loading

🕊️ Ciel's Mediation 🌤️

*~~ floating down from the clouds ~~ The zoo seems a bit noisy today...*

2 zoo members have reviewed this PR.

Zoo Member Status
🦥 Slow Commented
🐰 Mimi Commented

⚖️ The zoo has mixed opinions. Some are concerned, some are fine with it. Please review each comment carefully and make the final call.

まあまあ、ほどほどに。

This mediation was peacefully delivered by nullvariant-ciel[bot]

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 6, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

nullvariant-justice[bot]
nullvariant-justice Bot approved these changes Apr 6, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@nullvariant-justice nullvariant-justice Bot left a comment

Choose a reason for hiding this comment

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

⚖️ Justice grants passage. CI checks passed — this code meets the garden's standards.

@nullvariant nullvariant merged commit 4d76dc6 into main Apr 6, 2026
33 of 35 checks passed
@nullvariant nullvariant deleted the refactor/move-to-field-error-tests-to-validation-types branch April 6, 2026 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nullvariant-justice nullvariant-justice[bot] nullvariant-justice[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@nullvariant