Fix flow options

Also added tests to ensure we can parse JSX.
facebook · Dec 3, 2018 · 2fd2429 · 2fd2429
1 parent 1cbe93d
commit 2fd2429
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 21 deletions.
diff --git a/parser/flow.js b/parser/flow.js
@@ -19,7 +19,7 @@ const defaultOptions = {
   esproposal_export_star_as: true,
   esproposal_optional_chaining: true,
   esproposal_nullish_coalescing: true,
-  token: true,
+  tokens: true,
   types: true,
 };
 

diff --git a/src/__tests__/Worker-test.js b/src/__tests__/Worker-test.js
@@ -92,31 +92,39 @@ describe('Worker API', () => {
       });
     });
 
-    it('uses babylon if configured as such', done => {
-      const transformPath = getTransformForParser('babylon');
-      const sourcePath = getSourceFile();
-      const emitter = worker([transformPath]);
-
-      emitter.send({files: [sourcePath]});
-      emitter.once('message', (data) => {
-        expect(data.status).toBe('ok');
-        expect(getFileContent(sourcePath)).toBe('changed');
-        done();
+    ['flow', 'babylon'].forEach(parser => {
+      it(`uses ${parser} if configured as such`, done => {
+        const transformPath = getTransformForParser(parser);
+        const sourcePath = getSourceFile();
+        const emitter = worker([transformPath]);
+
+        emitter.send({files: [sourcePath]});
+        emitter.once('message', (data) => {
+          expect(data.status).toBe('ok');
+          expect(getFileContent(sourcePath)).toBe('changed');
+          done();
+        });
       });
     });
 
-    it('uses flow if configured as such', done => {
-      const transformPath = getTransformForParser('flow');
-      const sourcePath = getSourceFile();
-      const emitter = worker([transformPath]);
-
-      emitter.send({files: [sourcePath]});
-      emitter.once('message', (data) => {
-        expect(data.status).toBe('ok');
-        expect(getFileContent(sourcePath)).toBe('changed');
-        done();
+    ['babylon', 'flow', 'tsx'].forEach(parser => {
+      it(`can parse JSX with ${parser}`, done => {
+        const transformPath = getTransformForParser(parser);
+        const sourcePath = createTempFileWith(
+          'var component = <div>{foobar}</div>;'
+        );
+        const emitter = worker([transformPath]);
+
+        emitter.send({files: [sourcePath]});
+        emitter.once('message', (data) => {
+          expect(data.status).toBe('ok');
+          expect(getFileContent(sourcePath)).toBe('changed');
+          done();
+        });
       });
     });
+
   });
 
+
 });