Merge pull request #125 from moranje/unicode-checks

Fix unicode checks for fallbacks with test

moo.js

@@ -219,8 +219,8 @@
 
         if (unicodeFlag === null) {
           unicodeFlag = obj.unicode
-        } else if (unicodeFlag !== obj.unicode) {
-          throw new Error("If one rule is /u then all must be")
+        } else if (unicodeFlag !== obj.unicode && options.fallback === false) {
+          throw new Error('If one rule is /u then all must be')
         }
       }
 

test/test.js

@@ -1217,13 +1217,21 @@ describe('include', () => {
   })
 })
 
+describe('unicode flag', () => {
+  test('allows all rules to be /u', () => {
+    expect(() => compile({a: /foo/u, b: /bar/u, c: 'quxx'})).not.toThrow()
+    expect(() => compile({a: /foo/u, b: /bar/, c: 'quxx'})).toThrow(
+      'If one rule is /u then all must be'
+    )
+    expect(() => compile({a: /foo/, b: /bar/u, c: 'quxx'})).toThrow(
+      'If one rule is /u then all must be'
+    )
+  })
 
-describe("unicode flag", () => {
-
-  test("allows all rules to be /u", () => {
-    expect(() => compile({ a: /foo/u, b: /bar/u, c: "quxx" })).not.toThrow()
-    expect(() => compile({ a: /foo/u, b: /bar/, c: "quxx" })).toThrow("If one rule is /u then all must be")
-    expect(() => compile({ a: /foo/, b: /bar/u, c: "quxx" })).toThrow("If one rule is /u then all must be")
+  test('unicode rules work with fallback token', () => {
+    expect(() =>
+      compile({a: moo.fallback, b: /bar/u, c: /quxx/u})
+    ).not.toThrow()
   })
 
   test("supports unicode", () => {