Fixes and tests for config directives

wcjohnson · Jul 6, 2017 · 69913bd · 69913bd
1 parent 5db0616
commit 69913bd
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 2 deletions.
diff --git a/src/config.lsc b/src/config.lsc
@@ -122,8 +122,8 @@ eatDirective(code, directives) ->
   else:
     dqMatches = code.match(/^"((?:[^"\\]|\\.)*)"\s*;?\s*([^]*)$/)
     if dqMatches:
-      directives.push(matches[1])
-      matches[2]
+      directives.push(dqMatches[1])
+      dqMatches[2]
     else:
       code
 

diff --git a/test/fixtures/configuration-directives/multiple/actual.js b/test/fixtures/configuration-directives/multiple/actual.js
@@ -0,0 +1,15 @@
+// comment 1
+'use @oigroup/lightscript'
+/*
+* comment 2
+*/
+"use @oigroup/lightscript with noEnforcedSubscriptIndentation";
+// comment 3
+'use strict'
+/*
+comment 4
+*/
+'use @oigroup/lightscript with safeCall'
+
+a?()
+.b
diff --git a/test/fixtures/configuration-directives/multiple/expected.js b/test/fixtures/configuration-directives/multiple/expected.js
@@ -0,0 +1,14 @@
+// comment 1
+'use @oigroup/lightscript';
+/*
+* comment 2
+*/
+"use @oigroup/lightscript with noEnforcedSubscriptIndentation";
+// comment 3
+'use strict';
+/*
+comment 4
+*/
+'use @oigroup/lightscript with safeCall';
+
+(typeof a === 'function' ? a() : null).b;