[GitHub] [metron] mmiklavc commented on a change in pull request #1522: METRON-2271 Reorganize Travis Builds

[GitBox <gi...@apache.org>]

mmiklavc commented on a change in pull request #1522: METRON-2271 Reorganize Travis Builds
URL: https://github.com/apache/metron/pull/1522#discussion_r330802353
 
 

 ##########
 File path: .travis.yml
 ##########
 @@ -38,14 +31,36 @@ before_install:
   - npm config set cache $HOME/.npm-cache --global
   - npm config set prefix $HOME/.npm-prefix --global
 
-install:
-  - time mvn install -T 2C -q -DskipTests=true -Dmaven.javadoc.skip=true -B -V
-
-script:
-  - time $SCRIPT
-
 matrix:
   fast_finish: true
+  include:
+    - name: Unit Tests
+      script:
+        - time mvn install -T 2C -q -DskipTests=true -Dmaven.javadoc.skip=true -B -V -pl '!:metron-config,!:metron-alerts'
+        - time mvn surefire:test@unit-tests -T 2C
+
+    - name: Integration Tests
+      script:
+        - time mvn install -T 2C -q -DskipTests=true -Dmaven.javadoc.skip=true -B -V -pl '!:metron-config,!:metron-alerts'
+        - time mvn surefire:test@integration-tests
+
+    - name: Alerts UI Tests
+      script:
+        - time mvn install -T 2C -q -DskipTests=true -Dmaven.javadoc.skip=true -B -V -pl ':metron-alerts'
 
 Review comment:
   I don't think I realized you could use the colons for the `-pl` list as well (I always use it with `-rf`). I like that. Way cleaner than typing the full module path.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services