fix(health): update health status method to handle persistent instances correctly#14635
Merged
Merged
Conversation
|
Thanks for your this PR. 🙏 感谢您提交的PR。 🙏 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
|
KomachiSion
added a commit
that referenced
this pull request
Mar 19, 2026
* Add core coverage (#14608) * test(core-utils): remove unused method and add comprehensive unit tests. * test(web): add unit tests for NacosCoreWebConfiguration and NacosWebServerListener。 * test(core): add unit tests for parts of remote package. * add core unit test coverage (#14618) * test(core): add unit tests for plugin forms and prompt request extractor. * test(core): add unit tests for remote param check and connection ejector. * fix(naming): 添加分页参数验证 (#14616) - 在实例列表查询接口中增加 pageForm.validate() 验证调用 - 确保分页参数在处理前进行有效性检查 * test(auth): add and improve unit tests for auth module. (#14619) * [ISSUE #14622] fix: exclude json-schema-validator 3.0.0 from agentscope-core to resolve NoClassDefFoundError (#14625) agentscope-core:1.0.7 directly depends on json-schema-validator:3.0.0 (Jackson 3), which overrides the 1.5.7 version required by mcp-json-jackson2:0.14.1 (Jackson 2). This causes NoClassDefFoundError for SpecVersion$VersionFlag when using the 'Auto import from MCP Server' feature, since SpecVersion.VersionFlag was renamed to SpecificationVersion in json-schema-validator 2.0.0+. Excluding json-schema-validator from agentscope-core allows mcp's transitive dependency (1.5.7) to take effect, resolving the conflict. * test(consistency): add unit tests for serializers, processors and utils. (#14628) * fix(health): update health status method to handle persistent instances correctly (#14635) * Refactor database plugins and update CI report paths (#14640) * refactor(plugin): Refactor default database plugin, split database plugins and move depend jdbc to plugins dir. * fix(ci): update JaCoCo report paths for datasource plugins in CI workflow * fix(pom): update maven-antrun-plugin version and correct task tags * chore(db): remove derby and mysql schemas, add AI resource mappers. --------- Co-authored-by: Lichenkang <2188718831@qq.com> Co-authored-by: Sunrisea <1336317033@qq.com>
KomachiSion
added a commit
that referenced
this pull request
Mar 19, 2026
* Add core coverage (#14608) * test(core-utils): remove unused method and add comprehensive unit tests. * test(web): add unit tests for NacosCoreWebConfiguration and NacosWebServerListener。 * test(core): add unit tests for parts of remote package. * add core unit test coverage (#14618) * test(core): add unit tests for plugin forms and prompt request extractor. * test(core): add unit tests for remote param check and connection ejector. * fix(naming): 添加分页参数验证 (#14616) - 在实例列表查询接口中增加 pageForm.validate() 验证调用 - 确保分页参数在处理前进行有效性检查 * test(auth): add and improve unit tests for auth module. (#14619) * [ISSUE #14622] fix: exclude json-schema-validator 3.0.0 from agentscope-core to resolve NoClassDefFoundError (#14625) agentscope-core:1.0.7 directly depends on json-schema-validator:3.0.0 (Jackson 3), which overrides the 1.5.7 version required by mcp-json-jackson2:0.14.1 (Jackson 2). This causes NoClassDefFoundError for SpecVersion$VersionFlag when using the 'Auto import from MCP Server' feature, since SpecVersion.VersionFlag was renamed to SpecificationVersion in json-schema-validator 2.0.0+. Excluding json-schema-validator from agentscope-core allows mcp's transitive dependency (1.5.7) to take effect, resolving the conflict. * test(consistency): add unit tests for serializers, processors and utils. (#14628) * fix(health): update health status method to handle persistent instances correctly (#14635) * Refactor database plugins and update CI report paths (#14640) * refactor(plugin): Refactor default database plugin, split database plugins and move depend jdbc to plugins dir. * fix(ci): update JaCoCo report paths for datasource plugins in CI workflow * fix(pom): update maven-antrun-plugin version and correct task tags * chore(db): remove derby and mysql schemas, add AI resource mappers. --------- Co-authored-by: Lichenkang <2188718831@qq.com> Co-authored-by: Sunrisea <1336317033@qq.com>
KomachiSion
added a commit
that referenced
this pull request
Mar 26, 2026
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
3 participants
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.
Fix #14600
Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean package apache-rat:check spotbugs:check -DskipTeststo make sure basic checks pass. Runmvn clean install -DskipITsto make sure unit-test pass. Runmvn clean test-compile failsafe:integration-testto make sure integration-test pass.