Skip to content

Fixed an incorrect test method #2972

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

oah1021
Copy link
Contributor

@oah1021 oah1021 commented Oct 13, 2023

I'm testing XmlConfigBuilderTest#shouldSuccessfullyLoadXMLConfigFile() , throws the error
1697193985613_C901D413-94FB-4fa1-AB03-DA56217CB481

It's actually it throwing an exception here
1697193985613_C901D413-94FB-4fa1-AB03-DA56217CB481

When I switch to JAVASSIST, this is correct
But I don't know why

@coveralls
Copy link

Coverage Status

coverage: 86.61% (-0.009%) from 86.619% when pulling 1cea8f7 on oah1021:About_the_CGLIB_question into 6e84c85 on mybatis:master.

@wuwen5
Copy link
Contributor

wuwen5 commented Oct 13, 2023

Currently, all ci tests have been successful and there are no incorrect testing methods, which should be a local issue for you.

@oah1021
Copy link
Contributor Author

oah1021 commented Oct 13, 2023

目前所有ci测试均已成功,不存在测试方法不正确的情况,这应该是您本地的问题。
Thanks, but I don't have a good solution to deal with this problem
I will close this pr

@oah1021 oah1021 closed this Oct 13, 2023
@oah1021 oah1021 deleted the About_the_CGLIB_question branch October 13, 2023 11:12
@harawata
Copy link
Member

Just FYI, tests involving CGLIB should be ignored on JDK 17 (see #2499 ).
We use @Tag to filter tests on the CI environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants