Skip to content

#376: JsonProviderTest fails if run after other Junits in a test suite #380

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.

Sign up for GitHub

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

Merged
lukasj merged 5 commits into from
Jul 15, 2022
Merged

#376: JsonProviderTest fails if run after other Junits in a test suite #380

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b3f7629
#376: JsonProviderTest fails if run after other Junits in a test suite
lukasj May 19, 2022
0a59c34
simplify the provider lookup, add logging
lukasj May 19, 2022
afac3ef
cpright
lukasj May 19, 2022
ffbd680
fix exception message
lukasj May 19, 2022
94f5522
fix javadoc
lukasj May 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix exception message 
Signed-off-by: Lukas Jungmann <lukas.jungmann@oracle.com>
  • Loading branch information
@lukasj
lukasj committed May 19, 2022
commit ffbd680c15105a2c2cf1de24b81bb0b756ee86d6
4 changes: 2 additions & 2 deletions api/src/main/java/jakarta/json/spi/JsonProvider.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -158,10 +158,10 @@ private static JsonProvider newInstance(String className) {
return clazz.getConstructor().newInstance();
} catch (ClassNotFoundException x) {
throw new JsonException(
"Provider " + DEFAULT_PROVIDER + " not found", x);
"Provider " + className + " not found", x);
} catch (Exception x) {
throw new JsonException(
"Provider " + DEFAULT_PROVIDER + " could not be instantiated: " + x,
"Provider " + className + " could not be instantiated: " + x,
x);
}
}
Expand Down