-
Notifications
You must be signed in to change notification settings - Fork 6
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
Use maven-failsafe-plugin to handle integration tests #305
Use maven-failsafe-plugin to handle integration tests #305
Conversation
<version>3.1.2</version> | ||
<executions> | ||
<execution> | ||
<phase>process-resources</phase> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The pre-integration-test
phase could be used for this, although it would be redundant if the model haven't been changed.
<configuration> | ||
<driver>com.mysql.cj.jdbc.Driver</driver> | ||
<username>${gemma.db.build.user}</username> | ||
<password>${gemma.db.build.password}</password> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be better to introduce gemma.testdb.user
and gemma.testdb.password
for this instead of reusing credentials.
<goal>execute</goal> | ||
</goals> | ||
<configuration> | ||
<url>${bootstrap.db.url}</url> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here, a gemma.testdb.bootstrap-url
might be a better choice.
<goal>execute</goal> | ||
</goals> | ||
<configuration> | ||
<url>${gemma.testdb.build.url}</url> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace this by gemma.testdb.url
, simply.
<configuration> | ||
<argLine>-Dlog4j1.compatibility=true</argLine> | ||
<includes> | ||
<include>**/*Test.java</include> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add **/*IntegrationTest.java
here as well?
@@ -71,7 +71,8 @@ | |||
/** | |||
* Retrieve all {@link AnalysisResultSet} matching a set of criteria. | |||
* | |||
* @param datasets filter result sets that belong to any of the provided dataset identifiers, or null to ignore | |||
* @param datasets filter result sets that belong to any of the provided dataset identifiers, or null to | |||
* ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is unrelated...
9d0668b
to
03efc49
Compare
Only include tests marked by the UnitTest category for maven-sure-fire-plugin. Include all test ending with Test.java and exclude those marked by the UnitTest category for integration tests. Use integration-test and verify goals for maven-jasmin-plugin as it require a fake browser.
03efc49
to
740f871
Compare
See #293 for more details