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

Support for failsafe? #86

Closed
orpiske opened this issue Oct 7, 2020 · 12 comments
Closed

Support for failsafe? #86

orpiske opened this issue Oct 7, 2020 · 12 comments
Labels
wontfix This will not be worked on

Comments

@orpiske
Copy link

orpiske commented Oct 7, 2020

I noticed that our integration tests do not run when using mvnd.

Not knowing anything Maven internals and the challenges of running these in parallel, I am not sure if:

  • it's a bug,
  • working as designed
  • a problem on our own build configuration.

Could you, please, share some insights? Should the integration test execution, managed by the failsafe plugin, be supported?

@ppalaga
Copy link
Contributor

ppalaga commented Oct 7, 2020

Why do you think they do not run? There is #21 but tests not running at all would be new to me.

@orpiske
Copy link
Author

orpiske commented Oct 7, 2020

Looking at our build output, there's a few strange things:

  1. I have several reports of 0 tests run:
    [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

  2. The log message says the tests were skipped

[INFO] --- maven-failsafe-plugin:2.22.2:integration-test (integration-test) @ itests-syslog ---
[INFO] Tests are skipped.
[INFO] 
[INFO] --- maven-failsafe-plugin:2.22.2:verify (integration-test) @ itests-syslog ---
[INFO] Tests are skipped.

@ppalaga
Copy link
Contributor

ppalaga commented Oct 18, 2020

An integration test reproducing the issue would be awesome.

@orpiske
Copy link
Author

orpiske commented Oct 18, 2020

An integration test reproducing the issue would be awesome.

I'll prepare a reproducer this week.

@famod
Copy link
Contributor

famod commented Oct 20, 2020

@orpiske

1. [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

Is this from surefire? Are you using Junit5 parameterized tests?
If yes this could be caused by: https://issues.apache.org/jira/browse/SUREFIRE-1741

@orpiske
Copy link
Author

orpiske commented Oct 21, 2020

@famod this is for failsafe. We are not using parameterized tests.

I am going to provide the reproducer soon. Today I have some time to focus on this item.

@orpiske
Copy link
Author

orpiske commented Oct 21, 2020

Steps to reproduce:

git clone https://github.com/orpiske/camel-kafka-connector.git --branch mvnd-issue-86
cd camel-kafka-connector
mvnd -U -DskipIntegrationTests=false clean verify

Note: it's ok if the integration tests fail when running w/ mvnd. In fact, this is what I want to investigate (whether we can make them work in this scenario or not).

Edit: removed the unnecessary "install¨ target because, as explained below, it repeats several parts of the build cycle.

@khmarbaise
Copy link
Member

Hi,

Steps to reproduce:

git clone https://github.com/orpiske/camel-kafka-connector.git --branch mvnd-issue-86
cd camel-kafka-connector
mvnd -U -DskipIntegrationTests=false clean install verify

Is there a reason why you use install verify ? This will repeat several parts for the build life cycle...

I would suggest to use only clean verify...

@orpiske
Copy link
Author

orpiske commented Oct 29, 2020

Hi,

Steps to reproduce:

git clone https://github.com/orpiske/camel-kafka-connector.git --branch mvnd-issue-86
cd camel-kafka-connector
mvnd -U -DskipIntegrationTests=false clean install verify

Is there a reason why you use install verify ? This will repeat several parts for the build life cycle...

Thanks for the explanation about repeating the cycles! I wasn't aware about it. Using the install verify is just an old habit ...

I would suggest to use only clean verify...

So, I tried with just "clean verify" but it didn't seem to have affected the behavior. The tests still did not execute. I am wondering if we have something on our pom that would cause it to skip under these circumstances.

@ppalaga
Copy link
Contributor

ppalaga commented Oct 29, 2020

Now I realized you pass -DskipIntegrationTests=false. Indeed it is not honored in 0.0.10. The root cause is described in #157 (comment) . #157 is fixed in the master. Could you plz try with a recent build from the CI - e.g. this one https://github.com/mvndaemon/mvnd/actions/runs/334727512

@orpiske
Copy link
Author

orpiske commented Oct 29, 2020

Now I realized you pass -DskipIntegrationTests=false. Indeed it is not honored in 0.10.0. The root cause is described in #157 (comment) . #157 is fixed in the master. Could you plz try with a recent build from the CI - e.g. this one https://github.com/mvndaemon/mvnd/actions/runs/334727512

That was right on point. Using the master does make our tests run! Thanks Peter, I'll make sure to upgrade once the new version is out.

@ppalaga
Copy link
Contributor

ppalaga commented Oct 29, 2020

OK, let's close this as a duplicate of #157.

@ppalaga ppalaga closed this as completed Oct 29, 2020
@ppalaga ppalaga added this to the No fix/wont't fix milestone Oct 29, 2020
@gnodet gnodet added the wontfix This will not be worked on label Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

5 participants