[LOGGING-185] mvn verify should also be run as part of the github actions workflow - ASF JIRA

Voters

Watch issue

Watchers

Link

Clone

Update Comment Author

Replace String in Comment

Update Comment Visibility

Delete Comments

XML

Word

Printable

JSON

Details

Type: Improvement
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: None
Fix Version/s: 1.3.0
Labels:
None

Description

mvn verify should be run as part of the build pipeline.

The integration tests are currently broken and I believe they have been since this PR was merged because it introduced static imports to org.junit.Assert which is not on the classpath when running mvn verify.

There is also some incorrect config in the failsafe plugin.

By reverting the change to both ParentFirstTestCase.java and SecurityForbiddenTestCase.java in that PR and fixing the pom I was able to successfully run the tests.