[fit] optimize test framework to support executing specific actions at test startup#205
Merged
CodeCasterX merged 1 commit intoModelEngine-Group:3.5.xfrom Jul 17, 2025
Merged
Conversation
Closed
4 tasks
loveTsong
approved these changes
Jul 17, 2025
Yager-42
pushed a commit
to Yager-42/fit-framework
that referenced
this pull request
Aug 16, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 相关问题 / Related Issue
Issue 链接 / Issue Link: Closes #204
📋 变更类型 / Type of Change
📝 变更目的 / Purpose of the Change
This change optimizes the test framework to support database access operations within Bean constructors. Previously, the framework couldn't properly handle scenarios where business logic Beans perform database operations during initialization, leading to test failures or inconsistent behavior. This enhancement adds specific execution phases to ensure proper database connection management and transaction handling during Bean construction in test environments.
📋 主要变更 / Brief Changelog
🧪 验证变更 / Verifying this Change
测试步骤 / Test Steps
测试覆盖 / Test Coverage
📸 截图 / Screenshots
✅ 贡献者检查清单 / Contributor Checklist
请确保你的 Pull Request 符合以下要求 / Please ensure your Pull Request meets the following requirements:
基本要求 / Basic Requirements:
代码质量 / Code Quality:
测试要求 / Testing Requirements:
mvn -B clean package -Dmaven.test.skip=true,elsa README 中的编译检查 / Basic checks passmvn clean install/ Unit tests pass文档和兼容性 / Documentation and Compatibility:
📋 附加信息 / Additional Notes
Breaking Change Notice:
This PR introduces a breaking change to the @SQL annotation in the test framework. The original scripts attribute has been removed and replaced with before and after attributes. The previous scripts attribute only represented pre-execution scripts and didn't consider post-execution scenarios. The new design provides:
This change requires updating existing test code that uses the @SQL annotation's scripts attribute to use the new before attribute instead.
Migration Guide:
Other Enhancements:
This enhancement optimizes Bean constructor database access support through new execution phases. The new execution phase is only activated when Beans with constructor database access are detected. Performance impact is minimal as the optimization only applies to relevant scenarios.
Future improvements may include:
审查者注意事项 / Reviewer Notes:
Please pay special attention to: