optimizely / agent / 18234484985

86%

master: 87%

Build Type

Pull #444

github

Committed by Mat001

Commit Message

Add unit tests to improve code coverage

Coverage improvements for recent PR review fixes:

1. pkg/syncer/pubsub_test.go:
   - Test database type conversion (int vs float64 from YAML/JSON)
   - Covers the type switch added in getPubSubRedis/getPubSubRedisStreams
   - Tests valid types (int, float64) and invalid types (string, nil)

2. plugins/odpcache/services/redis_cache_test.go:
   - Test RedisCache.UnmarshalJSON method
   - Verifies password field priority: auth_token > redis_secret > password
   - Tests empty password handling and invalid JSON

3. plugins/userprofileservice/services/redis_ups_test.go:
   - Test RedisUserProfileService.UnmarshalJSON method
   - Verifies password field priority: auth_token > redis_secret > password
   - Tests empty password handling and invalid JSON

These tests cover the previously uncovered code paths from the flexible
Redis password configuration implementation.

Pull Request Pull Request #444: [FSSDK-] Add redis-streams

Run Details

424 of 511 new or added lines in 6 files covered. (82.97%)

3 existing lines in 1 file now uncovered.

3285 of 3832 relevant lines covered (85.73%)

2889.07 hits per line