Fix install of mjpeg-consumer

appium · May 7, 2019 · 2936115 · 2936115
1 parent e655ce1
commit 2936115
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.travis.yml b/.travis.yml
@@ -20,7 +20,8 @@ install:
     npm i opencv4nodejs > /dev/null 2>&1
     kill `cat ping.pid`
   - npm install
-  - npm install --no-save mjpeg-consumer
+  # on node 12 mjpeg-consumer cannot be installed, but we don't want to error
+  - npm install --no-save mjpeg-consumer && echo 'done'
 script:
   - npm run test && npm run e2e-test
 after_success:

diff --git a/test/mjpeg-e2e-specs.js b/test/mjpeg-e2e-specs.js
@@ -13,6 +13,10 @@ const MJPEG_SERVER_PORT = 8589;
 const MJPEG_SERVER_URL = `http://localhost:${MJPEG_SERVER_PORT}`;
 
 describe('MJpeg Stream (e2e)', function () {
+  // TODO: remove when buffertools can handle v12
+  if (process.version.startsWith('v12')) {
+    return this.skip();
+  }
 
   let mJpegServer, stream;