Skip to content

Add unit tests for utils, network, and pre-push hook#17

Merged
dadachi merged 1 commit intomainfrom
add_unit_tests_and_pre_push_hook
Mar 27, 2026
Merged

Add unit tests for utils, network, and pre-push hook#17
dadachi merged 1 commit intomainfrom
add_unit_tests_and_pre_push_hook

Conversation

@dadachi
Copy link
Copy Markdown
Contributor

@dadachi dadachi commented Mar 27, 2026

Summary

  • Add 34 new unit tests across 4 test files covering previously untested code
    • RequestHelperTest (6 tests): header generation with and without auth tokens
    • DateTimeFormatterUtilityTest (4 tests): locale-safe formatter assertions
    • DateUtilityTest (10 tests): ZonedDateTime/String extensions and timezone conversion
    • UtilityTest (14 tests): email validation, alphanumeric checks, scanUri
  • Add scripts/pre-push hook running spotlessCheck, lintDebug, and tests before push
  • Skip pre-push checks on branch deletions (zero-hash detection)
  • Document pre-push hook install instructions in CLAUDE.md

Test plan

  • All 34 new tests pass locally via ./gradlew testDebugUnitTest
  • ./gradlew spotlessCheck passes
  • CI passes on PR

🤖 Generated with Claude Code

- Add 34 new unit tests across 4 test files covering previously untested code
- RequestHelperTest (6 tests): header generation with and without auth tokens
- DateTimeFormatterUtilityTest (4 tests): locale-safe formatter assertions
- DateUtilityTest (10 tests): ZonedDateTime/String extensions and timezone conversion
- UtilityTest (14 tests): email validation, alphanumeric checks, scanUri
- Add scripts/pre-push hook running spotlessCheck, lintDebug, and tests
- Skip pre-push checks on branch deletions (zero-hash detection)
- Document pre-push hook in CLAUDE.md

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dadachi dadachi merged commit 846dcd3 into main Mar 27, 2026
1 check passed
@dadachi dadachi deleted the add_unit_tests_and_pre_push_hook branch March 27, 2026 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant