Merge ~sayun/qa-regression-testing:master into qa-regression-testing:master

Proposed by Chris
Status: Merged
Merged at revision: 3ddeb20d19ec93f14c76f7abc4a9c18c8344e5a0
Proposed branch: ~sayun/qa-regression-testing:master
Merge into: qa-regression-testing:master
Diff against target: 18 lines (+11/-0)
1 file modified
notes_testing/maven-shared-utils/README (+11/-0)
Reviewer Review Type Date Requested Status
Steve Beattie Approve
Review via email: mp+463639@code.launchpad.net

Commit message

Added a readme on how to run maven-shared-utils tests.

To post a comment you must log in.
Revision history for this message
Steve Beattie (sbeattie) wrote :

 review approve

LGTM, merged as is. One thing you might have added was making sure
someone testing does `apt source maven-shared-utils` to get the version
of the testsuite included in the source (versus say pulling the upstream
git repo); one reason to do this is that for some packages where we
manually or otherwise run the included test suite from upstream,
additional steps sometimes need to be taken to build e.g. test binaries,
but that's not the case here.

Thanks!

--
Steve Beattie
<email address hidden>

review: Approve
Revision history for this message
Alex Murray (alexmurray) wrote :

Next step would be to write a test-maved-shared-utils.py script which does this automatically :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/notes_testing/maven-shared-utils/README b/notes_testing/maven-shared-utils/README
2new file mode 100644
3index 0000000..086f277
4--- /dev/null
5+++ b/notes_testing/maven-shared-utils/README
6@@ -0,0 +1,11 @@
7+To run maven tests:
8+
9+1. Install maven and OpenJDK:
10+
11+ apt install maven default-jdk
12+
13+
14+2. In the project folder, run maven verify:
15+ (The rat.skip flag is necessary to skip license checking, which results in errors.)
16+
17+ mvn -Drat.skip=true verify
18\ No newline at end of file

Subscribers

People subscribed via source and target branches