Issue 6890 - run npm tests in parallel (PENDING)

.gitlab-ci.yml

 # This file is part of Adblock Plus <https://adblockplus.org/>,
 # Copyright (C) 2006-present eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
 #
 # You should have received a copy of the GNU General Public License
 # along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
 
 stages:
   - prepare
   - test
 
-prepare-dependencies:
-  stage: prepare
-  script:
+.dependencies:
+  script: &dependencies
     - mkdir -p .git/info
     - pip install --user Jinja2 cryptography
     - python ensure_dependencies.py
-    - npm install
-  cache:
-    key: cache_$CI_COMMIT_SHA
-    paths:
-      - ./
-    policy: push
-
+    - "[ -d node_modules ] || npm install"
 
 .test_template: &test_template
   stage: test
-  before_script:
-    - python ensure_dependencies.py

[Sebastian Noack, 2018/09/03 19:37:35]

FWIW, the lint job below doesn't require ensure_dependencies.py. But not sure
how much boilerplate a special case would cause?

[tlucas, 2018/09/03 21:35:53]

We'd need to copy this whole template into "lint" - IMHO not worth it :)

-    - npm install

[Sebastian Noack, 2018/09/03 19:37:35]

Perhaps we can check if node_modules already exists to avoid running "npm
install" redundantly (in case the prepare stage took already care of it). Note
that (unlike ensure_dependencies.py) "npm install" will always check for updates
(which can take a while) even if package.json didn't change.

[tlucas, 2018/09/03 21:35:53]

I was thinking about "what if node_modules was created, but npm install was
aborted for some reason?" - but then things would fail anyway. Done.

+  before_script: *dependencies
   cache:
     key: cache_$CI_COMMIT_SHA
     paths:
       - ./
     policy: pull
 
 
+prepare-dependencies:
+  stage: prepare
+  script: *dependencies
+  cache:
+    key: cache_$CI_COMMIT_SHA
+    paths:
+      - ./
+    policy: push
+
 tests:gecko:
   <<: *test_template
   script:
-    - npm test -- -g gecko
+    - npm run test-only -- -g gecko
 
 tests:chrome:
   <<: *test_template
   script:
-    - xvfb-run npm test -- -g chrome
+    - xvfb-run npm run test-only -- -g chrome
 
 lint:
   <<: *test_template
   script:
     - npm run lint