Skip to content

Create a GitHub action to run maven tests#247

Merged
jrha merged 3 commits intomasterfrom
github-action-maven
Jun 23, 2021
Merged

Create a GitHub action to run maven tests#247
jrha merged 3 commits intomasterfrom
github-action-maven

Conversation

@jrha
Copy link
Member

@jrha jrha commented Jun 21, 2021

No description provided.

@jrha jrha force-pushed the github-action-maven branch 6 times, most recently from 648c36f to 5096f5c Compare June 21, 2021 13:03
@jrha jrha force-pushed the github-action-maven branch 11 times, most recently from 239b940 to d37e4ea Compare June 21, 2021 14:05
@wpoely86
Copy link
Member

@jrha I think you're hitting the same issue as I did with the tests for the template. You must first download all of the maven dependencies and then run the tests. For some reason, downloads at a later stage hang or fail.
I fixed it by running:

mvn_run "dependency:resolve $MVN_ARGS" && mvn_test

@jrha
Copy link
Member Author

jrha commented Jun 22, 2021

I think you may be right, but I was trying to build panc in a clean environment, oddly it is falling over at:

Error: Failed to execute goal on project panc-rpm: Could not resolve dependencies for project org.quattor.pan:panc-rpm:rpm:10.8-SNAPSHOT: Could not find artifact org.quattor.pan:panc:zip:10.8-SNAPSHOT in clojars.org (https://repo.clojars.org/) -> [Help 1]

Which it shouldn't be looking for, because that is what it should be building.
Nor should it be looking in the clojars repo, which makes me think it tried everything else first.

@jrha
Copy link
Member Author

jrha commented Jun 22, 2021

Aha! It fails the same way on my desktop now.

@jrha jrha force-pushed the github-action-maven branch from d37e4ea to 2906a26 Compare June 23, 2021 08:38
@jrha jrha force-pushed the github-action-maven branch 2 times, most recently from b2c1472 to 1999dd4 Compare June 23, 2021 09:14
@jrha jrha force-pushed the github-action-maven branch from 1999dd4 to 07ec273 Compare June 23, 2021 09:19
@jrha
Copy link
Member Author

jrha commented Jun 23, 2021

Ok, it's a circular dependency in the build process, the secret is to run install as well as test.

@jrha jrha merged commit 86e9e65 into master Jun 23, 2021
@jrha jrha deleted the github-action-maven branch June 23, 2021 13:48
@jrha jrha added this to the 10.8 milestone May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants