Add for comprehension support to UsingUtils with resource management and guards, and extend test coverage.

[yruslan]

Summary by CodeRabbit

• New Features

  • Resource management utilities now support for-comprehension syntax with guards and filtering, enabling more expressive AutoCloseable resource handling.

• Tests

  • Added comprehensive test coverage for for-comprehension resource management, including multi-resource sequencing, error handling, and exception propagation scenarios.

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

[coderabbitai]

Walkthrough

Adds implicit Scala classes and a filtered resource wrapper to enable for-comprehension syntax for managing AutoCloseable resources. The ResourceWrapper and FilteredResourceWrapper classes provide foreach, map, flatMap, and withFilter methods that delegate to the existing using() mechanism, enabling concise resource handling with for-comprehension syntax.

Changes

Cohort / File(s)	Summary
Implementation cobol-parser/src/main/scala/za/co/absa/cobrix/cobol/utils/UsingUtils.scala	Introduces implicit ResourceWrapper class with foreach, map, flatMap, and withFilter; adds FilteredResourceWrapper for predicate-based filtering with chainable withFilter. Both delegate to existing using() resource management.
Test Coverage cobol-parser/src/test/scala/za/co/absa/cobrix/cobol/utils/UsingUtilsSuite.scala	Comprehensive test suite validating for-comprehension syntax usage with UsingUtils.Implicits, including single/multi-resource cases, error handling, guards, yields, and exception suppression behavior.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 With whiskers twitched and joy so bright,
Resources now flow left and right,
For-comprehensions, clean and neat,
Implicit wraps make syntax sweet,
Closure guards keep cleanup tight! 🌿

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main changes: adding for-comprehension support to UsingUtils with resource management and guards, plus expanded test coverage, which align perfectly with the actual modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/add-for-comprehensions-for-using-semantics

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!

---

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]

JaCoCo code coverage report - 'cobol-parser'

Overall Project	91.64% -0.03%	🍏
Files changed	82.61%	🍏

File	Coverage
UsingUtils.scala	87.32% -11.27%	🍏

[github-actions]

JaCoCo code coverage report - 'spark-cobol'

Overall Project	81.96%	🍏

There is no coverage information present for the Files changed