Skip to content
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

JUnit5 assertThrows SerializationProxyTest #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nhojpatrick
Copy link

Convert all legacy junit v4 exception checking to junit v5 assertThrows.

Test not run locally due to this error in master.

% mvn clean install
[INFO] Scanning for projects...
[ERROR] [ERROR] The projects in the reactor contain a cyclic reference: Edge between 'Vertex{label='org.apache.commons:commons-proxy2-build-tools:2.1-SNAPSHOT'}' and 'Vertex{label='org.apache.commons:commons-proxy2-parent:2.1-SNAPSHOT'}' introduces to cycle in the graph org.apache.commons:commons-proxy2-parent:2.1-SNAPSHOT --> org.apache.commons:commons-proxy2-build-tools:2.1-SNAPSHOT --> org.apache.commons:commons-proxy2-parent:2.1-SNAPSHOT @ 
[ERROR] The projects in the reactor contain a cyclic reference: Edge between 'Vertex{label='org.apache.commons:commons-proxy2-build-tools:2.1-SNAPSHOT'}' and 'Vertex{label='org.apache.commons:commons-proxy2-parent:2.1-SNAPSHOT'}' introduces to cycle in the graph org.apache.commons:commons-proxy2-parent:2.1-SNAPSHOT --> org.apache.commons:commons-proxy2-build-tools:2.1-SNAPSHOT --> org.apache.commons:commons-proxy2-parent:2.1-SNAPSHOT -> [Help 1]

@nhojpatrick nhojpatrick changed the title Junit5 assertThrows SerializationProxyTest JUnit5 assertThrows SerializationProxyTest Oct 19, 2022
@nhojpatrick nhojpatrick force-pushed the junit5-assertThrows-test-SerializationProxyTest branch from d7c5d7a to 7a494d2 Compare November 21, 2022 22:21
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.

1 participant