!Hotfix: Redis getValue null 반환에 맞춰 테스트를 정리한다

[SeongHo5356]

🚀 Why - 해결하려는 문제가 무엇인가요?

• Redis 조회 결과가 null인 경우 테스트가 실패하는 문제가 있었습니다.
• 실제 동작은 (키 없음 → null 반환) 인데, 테스트에서는 다른 값을 기대하고 있어 동작/기대값 불일치를 해결할 필요가 있었습니다.

✅ What - 무엇이 변경됐나요?

• RedisKeyValueMapTest에서 키 미존재(null 반환) 케이스 기대값을 실제 동작에 맞게 수정했습니다.
• 테스트 메서드명과 의도를 정리해, 어떤 케이스를 검증하는지 명확히 했습니다.

🛠️ How - 어떻게 해결했나요?

• getValue()의 “값 없음” 케이스를 null로 검증하도록 테스트를 수정했습니다.
• 테스트 이름을 ReturnNull 형태로 변경해 테스트 의도를 명확화했습니다.

Summary by CodeRabbit

• 버그 수정
  • Redis에서 누락된 값이 빈 문자열 대신 null로 반환되도록 동작이 변경되었습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)

Validation error: Unrecognized key(s) in object: 'tools'

⚙️ Configuration instructions

• Please see the configuration documentation for more information.
• You can also validate your configuration using the online YAML validator.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

📝 Walkthrough

Walkthrough

Redis 캐시 조회 시 누락된 값의 표현을 변경하는 테스트 코드 업데이트입니다. getValue_ReturnEmptyString 테스트 메서드를 getValue_ReturnNull로 이름 변경하고, 어설션을 isEmpty()에서 isNull()로 수정하여 null 반환 동작을 검증합니다.

Changes

코호트 / 파일	변경 요약
Redis 어댑터 테스트 src/test/java/starlight/adapter/member/auth/redis/RedisKeyValueMapTest.java	테스트 메서드명을 getValue_ReturnEmptyString에서 getValue_ReturnNull로 변경하고, 어설션을 isEmpty()에서 isNull()로 수정하여 null 반환 동작을 반영

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경 사항의 핵심을 명확하게 반영합니다. Redis getValue의 null 반환에 맞춰 테스트를 수정했다는 주요 의도가 잘 드러나 있습니다.

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7536202 and 8cd049a.

📒 Files selected for processing (1)

• src/test/java/starlight/adapter/member/auth/redis/RedisKeyValueMapTest.java

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: test
• GitHub Check: build-and-push

🔇 Additional comments (1)

src/test/java/starlight/adapter/member/auth/redis/RedisKeyValueMapTest.java (1)

84-94: Redis의 null 반환 동작을 정확하게 반영한 테스트입니다 👍

메서드명을 getValue_ReturnNull로 변경하고 assertion을 isNull()로 수정한 것이 정확합니다. RedisKeyValueMap.getValue() 구현이 문서화된 대로 키 미존재 시 null을 반환하고 있으며, 호출부(JwtFilter, AuthServiceImpl)에서 모두 null 체크를 통해 안전하게 처리하고 있습니다.

테스트 케이스 구조도 일관성 있게 정리되어 있습니다. getValue_Success → getValue_ReturnNull → getValue_Fail 순서로 성공/null/예외 상황을 모두 커버하고 있네요.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Test Results

249 tests   249 ✅  10s ⏱️
 46 suites    0 💤
 46 files      0 ❌

Results for commit 8cd049a.