#68: adds more tests for ...

Author: kaelzhang

test/fixtures/cases.js

@@ -8,6 +8,9 @@ function readPatterns (file) {
   return fs.readFileSync(file).toString()
 }
 
+const IS_WINDOWS = process.platform === 'win32'
+const SHOULD_TEST_WINDOWS = IS_WINDOWS || !!process.env.IGNORE_TEST_WIN32
+
 const cases = [
   /////////////////////////////////////////////////////////////////////
   // [
@@ -988,9 +991,30 @@ const cases = [
   ]
 ]
 
-const IS_WINDOWS = process.platform === 'win32'
-if (!IS_WINDOWS && !process.env.IGNORE_TEST_WIN32) {
+if (!SHOULD_TEST_WINDOWS) {
   cases.push(
+    [
+      '#68: ignore test for files named ...',
+      [
+        '/...',
+        '/.....'
+      ],
+      {
+        '...': 1,
+        '....': 0,
+        '.....': 1,
+        '......': 0
+      }
+    ],
+    [
+      '#68: file named ...',
+      [],
+      {
+        '...': 0,
+        '....': 0,
+        '.....': 0
+      }
+    ],
     [
       '#130: consequent escaped backslashes with whitespaces',
       [
@@ -1153,7 +1177,6 @@ exports.checkEnv = key => hasOnly
   : true
 
 exports.IS_WINDOWS = IS_WINDOWS
-exports.SHOULD_TEST_WINDOWS = process.env.IGNORE_TEST_WIN32
-  || IS_WINDOWS
+exports.SHOULD_TEST_WINDOWS = SHOULD_TEST_WINDOWS
 
 exports.debug = debug

test/others.test.js

@@ -178,9 +178,13 @@ _test('isPathValid', t => {
 
 const IGNORE_TEST_CASES = [
   [
+    // Description
     'test: no rule',
+    // patterns
     null,
+    // path
     'foo',
+    // ignored, unignored
     [false, false]
   ],
   [
@@ -212,14 +216,17 @@ const IGNORE_TEST_CASES = [
     ['*.js', '!a/a.js'],
     'a/a.js',
     [false, true]
-  ],
-  [
+  ]
+]
+
+if (!SHOULD_TEST_WINDOWS) {
+  IGNORE_TEST_CASES.push([
     `test: file which named '...'`,
-    'foo',
+    null,
     '...',
     [false, false]
-  ]
-]
+  ])
+}
 
 IGNORE_TEST_CASES.forEach(([d, patterns, path, [ignored, unignored]]) => {
   _test(d, t => {