test executable in docker

test/docker.spec.js

@@ -5,14 +5,18 @@ import sinon from 'sinon'
 
 import { Config, log } from '../'
 
-// !Config.isDocker && test('Docker test skipped', function(t) {
-//   t.pass('not in docker, skip docker tests')
-// })
+/**
+ * need keep this !Config.isDocker because ava need at least one test() inside.
+ *   × No tests found in test\docker.spec.js
+ */
+!Config.isDocker && test('Docker test skipped', function(t) {
+  t.pass('not in docker. this test is to prevent AVA `× No tests found in test\docker.spec.js` error.')
+})
 
 Config.isDocker && test('Docker smoking test', function(t) {
   // const n = execSync('ps a | grep Xvfb | grep -v grep | wc -l').toString().replace(/\n/, '', 'g')
   // t.is(parseInt(n), 1, 'should has Xvfb started')
   t.notThrows(_ => {
-    fs.accessSync(Config.BINARY_CHROMIUM, fs.F_OK)
+    fs.accessSync(Config.BINARY_CHROMIUM, fs.X_OK)
   }, 'should exist xvfb-chrome exectable')
 })