Aias00 opened a new issue, #6593:
URL: https://github.com/apache/shenyu/issues/6593

   ## Description
   Both LRU/TinyLFU eviction tests put the entire logic + all `assertEquals` 
inside a try whose catch is `// just ignore`. Any `AssertionError`/NPE/logic 
bug is silently discarded -> test is green no matter what. Comment admits 
"results … not stable, just ignore".
   
   ## Location
   ```
   
shenyu-common/src/test/java/org/apache/shenyu/common/cache/MemorySafeLRUMapTest.java:40-65
   
shenyu-common/src/test/java/org/apache/shenyu/common/cache/MemorySafeWindowTinyLFUMapTest.java:73-76
   ```
   
   ## Impact
   Core cache eviction correctness is effectively unverified.
   
   ## Suggested fix
   Remove the try/catch; if memory-stability genuinely matters, isolate that 
one assertion behind `@Disabled` or `Assumptions.assumeTrue`, not a blanket 
swallow.
   
   ## Related existing issue(s)
   None
   
   _Identified during the 2026-08-02 audit; full list in 
[`docs/issue-candidates-2026-08-02.md`](docs/issue-candidates-2026-08-02.md)._


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to