fix checkstyle

[EtienneLt]

PR Summary

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c7ccf9fe-4d18-46d8-b49d-3da612537816

📥 Commits

Reviewing files that changed from the base of the PR and between 932ad27 and 6a33f65.

📒 Files selected for processing (1)

• src/test/java/org/gridsuite/actions/server/ContingencyListControllerTest.java

---

📝 Walkthrough

Walkthrough

The PR refactors a test utility method to improve test robustness by ensuring queue cleanup always occurs, even when assertions fail. The assertQueuesEmptyThenClear method now wraps the queue-emptiness checks in a try block with output clearing moved to a finally block.

Changes

Test Queue Assertion Cleanup

Layer / File(s)	Summary
Assert-and-clear utility method robustness src/test/java/org/gridsuite/actions/server/ContingencyListControllerTest.java	The assertQueuesEmptyThenClear method refactored to use try/finally structure, guaranteeing output.clear() is called after queue assertions regardless of whether the assertions pass or fail.

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (1 warning, 2 inconclusive)

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.
Title check	❓ Inconclusive	The title 'fix checkstyle' is vague and generic, using a non-descriptive term that doesn't convey meaningful information about what checkstyle issue was fixed.	Replace the generic title with a more specific description of the checkstyle fix, such as 'Ensure output cleanup in assertQueuesEmptyThenClear even on assertion failure'.
Description check	❓ Inconclusive	The PR description is empty with only a template header, providing no meaningful information about the changes or their rationale.	Fill in the PR description with details about what checkstyle issue was being addressed and why the changes were necessary.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

---

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.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[GhilesA]

One of the greatest achievement in humankind