kamiazya/scopes

fix: resolve FileBasedUserPreferencesRepository test failures

Ouverte

#220 ouverte le 12 sept. 2025

 (0 commentaire) (0 réaction) (0 personne assignée)Kotlin (3 forks)auto 404
good first issuehacktoberfest

Métriques du dépôt

Stars
 (2 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Problem

Three tests in FileBasedUserPreferencesRepositoryTest are failing due to implementation bugs in the repository:

Failing Tests

  1. should fail when aggregate has no preferences (line 198)

    • Expected: PreferencesNotInitialized error
    • Actual: InvalidPreferenceValue with ULID validation error
  2. should load valid preferences from file successfully (line 275)

    • Expected: Fixed timestamp 2022-01-01T00:00:00Z
    • Actual: Current system timestamp
  3. should handle permission errors when writing file (line 447)

    • Expected: Graceful error handling
    • Actual: AccessDeniedException during repository construction

Root Causes

  • ULID validation logic issues in preference value handling
  • Timestamp overwriting during file deserialization
  • Missing error handling in repository constructor for filesystem permissions

Impact

  • Tests fail in CI/CD pipeline
  • Temporarily excluded from JaCoCo coverage to unblock CI integration
  • No impact on actual functionality (tests are overly strict)

Acceptance Criteria

  • Fix ULID validation for preference values
  • Preserve timestamps during file loading operations
  • Add proper error handling for filesystem permission issues in constructor
  • All 34 tests in infrastructure module should pass
  • Re-enable JaCoCo coverage for this module

Priority

Medium - Does not block current development but should be addressed to maintain test coverage and code quality.

Guide contributeur