Handle empty declaration list in Convert to ES6 Module (microsoft#21843)

Andy · web-flow · commit 425a4182a3eb · 2018-02-09T14:46:57.000-08:00
* Handle empty declaration list in Convert to ES6 Module

* Fix test
diff --git a/src/services/refactors/convertToEs6Module.ts b/src/services/refactors/convertToEs6Module.ts
@@ -33,8 +33,8 @@ namespace ts.refactor {
                 return isExportsOrModuleExportsOrAlias(sourceFile, node as PropertyAccessExpression)
                     || isExportsOrModuleExportsOrAlias(sourceFile, (node as PropertyAccessExpression).expression);
             case SyntaxKind.VariableDeclarationList:
-                const decl = (node as VariableDeclarationList).declarations[0];
-                return isExportsOrModuleExportsOrAlias(sourceFile, decl.initializer);
+                const decl = firstOrUndefined((node as VariableDeclarationList).declarations);
+                return !!decl && isExportsOrModuleExportsOrAlias(sourceFile, decl.initializer);
             case SyntaxKind.VariableDeclaration:
                 return isExportsOrModuleExportsOrAlias(sourceFile, (node as VariableDeclaration).initializer);
             default:
diff --git a/tests/cases/fourslash/refactorConvertToEs6Module_triggers_declarationList.ts b/tests/cases/fourslash/refactorConvertToEs6Module_triggers_declarationList.ts
@@ -0,0 +1,9 @@
+/// <reference path='fourslash.ts' />
+
+// @allowJs: true
+
+// @Filename: /a.js
+////c[|o|]nst;
+////require("x");
+
+goTo.eachRange(() => verify.not.refactorAvailable("Convert to ES6 module"));
