refactor(test): cleanup test framework and improve code formatting

[marevol]

Summary

• Refactored the test framework by updating UnitFessTestCase.tearDown() to properly accept TestInfo parameter
• Cleaned up redundant JUnit 5 lifecycle annotations across 190+ test classes
• Improved code readability in LLM client classes with better formatting

Changes Made

Test Framework Updates

• Updated UnitFessTestCase.tearDown() method signature to accept TestInfo parameter, aligning with JUnit 5 best practices
• Modified all test classes extending UnitFessTestCase to pass TestInfo to the tearDown() method
• Removed redundant @BeforeEach annotations from test classes where the parent class already handles the setup lifecycle
• Removed redundant @AfterEach annotations from test classes where the parent class already handles the teardown lifecycle

Code Formatting Improvements

• Reformatted GeminiLlmClient.java with improved builder pattern line breaks for better readability
• Reformatted OpenAiLlmClient.java with improved builder pattern line breaks for better readability
• Applied consistent formatting style to OkHttpClient and Request builder chains

Testing

• The changes are primarily cleanup and formatting improvements
• All existing tests should continue to pass as the behavior remains unchanged

Additional Notes

• This PR reduces code duplication by removing ~564 lines of redundant annotation code
• No functional changes to the test behavior or LLM client functionality

[Copilot]

Pull request overview

This PR refactors the test framework to align with JUnit 5 best practices by updating the UnitFessTestCase.tearDown() method signature to accept a TestInfo parameter. It removes redundant @BeforeEach and @AfterEach annotations from test classes that extend UnitFessTestCase, as the parent class already handles the test lifecycle. Additionally, it improves code readability in LLM client classes by reformatting builder pattern method chains.

Changes:

• Updated UnitFessTestCase.tearDown() to accept TestInfo parameter
• Removed redundant JUnit 5 lifecycle annotations (@BeforeEach, @AfterEach) from 190+ test classes
• Reformatted builder patterns in OpenAiLlmClient.java and GeminiLlmClient.java for better readability

Reviewed changes

Copilot reviewed 192 out of 192 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/test/java/org/codelibs/fess/unit/UnitFessTestCase.java	Updated tearDown() method signature to accept TestInfo parameter
src/test/java/org/codelibs/fess/validation/*.java	Removed redundant @BeforeEach annotations from validation test classes
src/test/java/org/codelibs/fess/util/*.java	Removed redundant lifecycle annotations and updated tearDown() calls
src/test/java/org/codelibs/fess/llm/**/*.java	Removed redundant lifecycle annotations and updated tearDown() calls
src/main/java/org/codelibs/fess/llm/openai/OpenAiLlmClient.java	Reformatted builder pattern chains for improved readability
src/main/java/org/codelibs/fess/llm/gemini/GeminiLlmClient.java	Reformatted builder pattern chains for improved readability

---

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