test(#4853): add comprehensive tests for Cache class in CacheTest.java

[volodya-lombrozo]

This PR enhances the CacheTest class by adding comprehensive tests to improve coverage and robustness.

Fixes #4853

Summary by CodeRabbit

• Tests
  • Enhanced test coverage for the caching mechanism, including verification of recompilation behavior when source content changes, handling of incomplete cache entries, and SHA-256 hash file generation.

[coderabbitai]

📝 Walkthrough

Walkthrough

Three new test methods are added to CacheTest.java to verify cache behavior: recompilation on source changes, handling missing cache entries, and SHA-256 hash generation. A long-form TODO block is removed from class Javadoc. Supporting imports for I/O, cryptography, and Base64 encoding are added.

Changes

Cohort / File(s)	Summary
Cache Testing Enhancements eo-maven-plugin/src/test/java/org/eolang/maven/CacheTest.java	Adds three new test methods: compilesAgainWhenChanged (verifies recompilation on source modification), compilesIfHashExistsButCacheMissing (verifies recompilation when cache entry is missing), and writesCorrectShaHash (verifies SHA-256 hash file content). Introduces imports for I/O, charset, cryptography utilities, and removes TODO documentation block.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• #4847: Introduces the Cache implementation that these new tests directly exercise for hash generation, caching behavior, and recompilation scenarios.
• #4845: Also targets cache/transpilation behavior with related test coverage in FpDefaultTest and cache configuration changes.

Suggested labels

core

Suggested reviewers

• yegor256

Poem

🐰 A rabbit hops through tests with glee,
Hashes checked so carefully,
When sources change, recompile we must,
Cache behavior? In tests we trust! 🔐

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ 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%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding comprehensive tests for the Cache class in CacheTest.java, with reference to issue #4853.
Linked Issues check	✅ Passed	The PR successfully adds three new test methods to CacheTest.java and removes the puzzle TODO block from Javadoc, directly addressing issue #4853's requirement to fix the puzzle and remove its text from source code.
Out of Scope Changes check	✅ Passed	All changes are within scope: new test methods and imports support comprehensive Cache testing, and removal of the puzzle TODO block directly satisfies the issue requirement.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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]

🚀 Performance Analysis

All benchmarks are within the acceptable range. No critical degradation detected (threshold is 100%). Please refer to the detailed report for more information.

Click to see the detailed report

Test	Base Score	PR Score	Change	% Change	Unit	Mode
benchmarks.XmirBench.xmirToEO	181.052	202.287	21.236	11.73%	ms/op	Average Time

⚠️ Performance loss: benchmarks.XmirBench.xmirToEO is slower by 21.236 ms/op (11.73%)

[Copilot]

Pull request overview

This PR enhances test coverage for the Cache class by adding three comprehensive test methods and removing a resolved TODO comment. The changes address issue #4853, which identified inadequate testing in CacheTest.java.

Changes:

• Removed TODO comment requesting more tests for the Cache class
• Added test for cache invalidation when source file is modified
• Added test for cache miss scenario when cached file is deleted but hash remains
• Added test verifying correct SHA-256 hash generation and storage

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[volodya-lombrozo]

@yegor256 could you have a look, please?

[0crat]

@volodya-lombrozo Thanks for your contribution! You've earned +16 points for your code submission. This reward follows our team policy which awards 16 points for reviews provided, plus additional points based on your hits-of-code and review feedback. Please keep them coming! Your running score is +316; don't forget to check your Zerocracy account too.