chore: add onlyFilesWithFlowAnnotation tests

gajus · Nov 27, 2016 · d311f22 · d311f22
1 parent 52b71f3
commit d311f22
Show file tree

Hide file tree

Showing 9 changed files with 90 additions and 8 deletions.
diff --git a/tests/rules/assertions/booleanStyle.js b/tests/rules/assertions/booleanStyle.js
@@ -29,6 +29,15 @@ export default {
     {
       code: 'type X = bool',
       options: ['bool']
+    },
+    {
+      code: 'type X = bool',
+      options: ['boolean'],
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
diff --git a/tests/rules/assertions/noDupeKeys.js b/tests/rules/assertions/noDupeKeys.js
@@ -12,6 +12,14 @@ export default {
   valid: [
     {
       code: 'type FooType = { a: number, b: string, c: number }'
+    },
+    {
+      code: 'type FooType = { a: number, b: string, a: number }',
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
diff --git a/tests/rules/assertions/noWeakTypes.js b/tests/rules/assertions/noWeakTypes.js
@@ -246,6 +246,14 @@ export default {
     {
       code: 'type X = Function',
       options: [{Function: false}]
+    },
+    {
+      code: 'function foo(thing): Function {}',
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
diff --git a/tests/rules/assertions/objectTypeDelimiter.js b/tests/rules/assertions/objectTypeDelimiter.js
@@ -142,6 +142,15 @@ export default {
     {
       code: 'declare class Foo { (): Foo, }',
       options: ['comma']
+    },
+    {
+      code: 'type Foo = { a: Foo, b: Bar }',
+      options: ['semicolon'],
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
diff --git a/tests/rules/assertions/requireParameterType.js b/tests/rules/assertions/requireParameterType.js
@@ -190,6 +190,14 @@ export default {
           excludeParameterMatch: '^_'
         }
       ]
+    },
+    {
+      code: '(foo) => {}',
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
diff --git a/tests/rules/assertions/semi.js b/tests/rules/assertions/semi.js
@@ -50,6 +50,14 @@ export default {
     {
       code: 'type FooType = {}',
       options: ['never']
+    },
+    {
+      code: 'type FooType = {}',
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
diff --git a/tests/rules/assertions/sortKeys.js b/tests/rules/assertions/sortKeys.js
@@ -79,6 +79,14 @@ export default {
     {
       code: 'type FooType = { 1:number, 2: number, 10: number }',
       options: ['asc', {natural: true}]
+    },
+    {
+      code: 'type FooType = { b: number, a: number }',
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
diff --git a/tests/rules/assertions/typeIdMatch.js b/tests/rules/assertions/typeIdMatch.js
@@ -29,6 +29,14 @@ export default {
       options: [
         '^foo$'
       ]
+    },
+    {
+      code: 'type foo = {};',
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
diff --git a/tests/rules/assertions/unionIntersectionSpacing.js b/tests/rules/assertions/unionIntersectionSpacing.js
@@ -74,10 +74,10 @@ const UNION = {
     }
   ],
   valid: [
-        {code: 'type X = string | number;'},
-        {code: 'type X = string | number | boolean;'},
-        {code: 'type X = (string) | number;'},
-        {code: 'type X = ((string)) | (number | foo);'},
+    {code: 'type X = string | number;'},
+    {code: 'type X = string | number | boolean;'},
+    {code: 'type X = (string) | number;'},
+    {code: 'type X = ((string)) | (number | foo);'},
     {
       code: 'type X = string|number',
       options: ['never']
@@ -93,6 +93,14 @@ const UNION = {
         '| number',
         '}'
       ].join('\n')
+    },
+    {
+      code: 'type X = string| number;',
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };
@@ -173,10 +181,10 @@ const INTERSECTION = {
     }
   ],
   valid: [
-        {code: 'type X = string & number;'},
-        {code: 'type X = string & number & boolean;'},
-        {code: 'type X = (string) & number;'},
-        {code: 'type X = ((string)) & (number & foo);'},
+    {code: 'type X = string & number;'},
+    {code: 'type X = string & number & boolean;'},
+    {code: 'type X = (string) & number;'},
+    {code: 'type X = ((string)) & (number & foo);'},
     {
       code: 'type X = string&number',
       options: ['never']
@@ -192,6 +200,14 @@ const INTERSECTION = {
         '& number',
         '}'
       ].join('\n')
+    },
+    {
+      code: 'type X = string& number;',
+      settings: {
+        flowtype: {
+          onlyFilesWithFlowAnnotation: true
+        }
+      }
     }
   ]
 };