Fix for uninitialized variables

jj-ivx · jj-ivx · commit 425ed9413b0a · 2019-10-11T16:19:46.000+02:00
diff --git a/lib/utils.js b/lib/utils.js
@@ -47,7 +47,7 @@ const getImport = (context, callback) => ({
   },
   VariableDeclaration: node => {
     node.declarations.forEach(({ init }) => {
-      if (isImportExpression(init)) {
+      if (init && isImportExpression(init)) {
         callback(resolveImport(context, node, init.arguments[0]));
       }
     });
diff --git a/tests/rules/no-cross-imports.js b/tests/rules/no-cross-imports.js
@@ -11,6 +11,7 @@ ruleTester.run('no-cross-imports', rule, {
     "import 'module';",
     "import '../some/relative/path';",
     'someFunction();',
+    'let a;',
     {
       options: [{ allow: '@test/workspace' }],
       filename: '/some/path.js',

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ const getImport = (context, callback) => ({`
`47`	`47`	`},`
`48`	`48`	`VariableDeclaration: node => {`
`49`	`49`	`node.declarations.forEach(({ init }) => {`
`50`		`- if (isImportExpression(init)) {`
	`50`	`+ if (init && isImportExpression(init)) {`
`51`	`51`	`callback(resolveImport(context, node, init.arguments[0]));`
`52`	`52`	`}`
`53`	`53`	`});`
Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,7 @@ ruleTester.run('no-cross-imports', rule, {`
`11`	`11`	`"import 'module';",`
`12`	`12`	`"import '../some/relative/path';",`
`13`	`13`	`'someFunction();',`
	`14`	`+ 'let a;',`
`14`	`15`	`{`
`15`	`16`	`options: [{ allow: '@test/workspace' }],`
`16`	`17`	`filename: '/some/path.js',`