fix: Removed debug code

unlight · unlight · commit e7afd372a91c · 2019-06-14T21:46:42.000+04:00
diff --git a/package.json b/package.json
@@ -2,11 +2,16 @@
   "name": "typescript-transform-unspec",
   "version": "0.0.0-dev",
   "license": "MIT",
-  "description": "Typescript transform plugin",
+  "description": "Typescript transform plugin removes spec definition from source file",
   "main": "index.js",
   "typings": "index.d.ts",
   "author": "2019",
-  "keywords": [],
+  "keywords": [
+    "typescript-transformer",
+    "typescript-plugin",
+    "typescript-transform-plugin",
+    "typescript-compiler"
+  ],
   "engines": {
     "node": ">=8"
   },
diff --git a/src/example.ts b/src/example.ts
@@ -6,10 +6,8 @@ it('hello world test', () => {
     expect(hello()).toBe('hello world');
 });
 
-jest.mock('foo');
-
 describe('hello world test', () => {
     beforeAll(() => {
-        jest.mock('foo');
+        jest.mock('inspector');
     });
 });
diff --git a/src/index.spec.ts b/src/index.spec.ts
@@ -1,15 +1,13 @@
 /* eslint-disable @typescript-eslint/tslint/config */
-import * as lib from './index';
+import transformPlugin from './index';
 import * as ts from 'typescript';
 
-let transformPlugin: typeof lib.typescriptTransformUnspec = lib as any;
-
 function transform(sourceText: string) {
     const program = ts.createProgram({
         rootNames: [],
         options: {},
     });
-    const transformer = transformPlugin(program, {});
+    const transformer = transformPlugin(program);
     const sourceFile = ts.createSourceFile('filename.tsx', sourceText, ts.ScriptTarget.ES5, true, ts.ScriptKind.TSX);
     let result: string | undefined;
     const writeCallback = (fileName: string, data: string) => {
@@ -20,7 +18,7 @@ function transform(sourceText: string) {
 }
 
 it('smoke test', () => {
-    expect(lib).toBeTruthy();
+    expect(transformPlugin).toBeTruthy();
     expect(transform(';')).toBe(';');
 });
 
diff --git a/src/index.ts b/src/index.ts
@@ -1,14 +1,15 @@
 import * as ts from 'typescript';
 
-export function typescriptTransformUnspec(program: ts.Program, options?): ts.TransformerFactory<ts.SourceFile> {
+export default function typescriptTransformUnspec(program: ts.Program): ts.TransformerFactory<ts.SourceFile> { // tslint:disable-line:no-default-export
     return (context) => (file) => visitNodeAndChildren(file, program, context);
 }
 
 function visitNodeAndChildren(node: ts.Node, program: ts.Program, context: ts.TransformationContext): ts.SourceFile {
-    return ts.visitEachChild(visitor(node), (childNode) => visitNodeAndChildren(childNode, program, context), context) as ts.SourceFile;
+    const visitor = (childNode) => visitNodeAndChildren(childNode, program, context);
+    return ts.visitEachChild<ts.SourceFile>(<ts.SourceFile>filterNode(node), visitor, context);
 }
 
-function visitor(node: ts.Node): ts.Node | undefined {
+function filterNode<T extends ts.Node>(node: T): T | undefined {
     if (isSpecExpressionStatement(node)) {
         return undefined;
     }
@@ -44,18 +45,16 @@ function isSpecExpressionStatement(node: ts.Node) {
     if (ts.isExpressionStatement(node) && ts.isCallExpression(node.expression)) {
         const callExpr = node.expression.expression;
         if (ts.isIdentifier(callExpr) && isTestExpressionText(callExpr.escapedText)) {
-            console.log("callExpr", callExpr.flags);
             return true;
         }
         if (ts.isPropertyAccessExpression(callExpr)) {
             const propertyAccessExpr = callExpr.expression;
             if (ts.isIdentifier(propertyAccessExpr) && isTestExpressionText(propertyAccessExpr.escapedText)) {
-                console.log("propertyAccessExpr", propertyAccessExpr.flags);
                 return true;
             }
         }
-        return false;
     }
+    return false;
 }
 
 module.exports = typescriptTransformUnspec;

Original file line number	Diff line number	Diff line change
`@@ -1,14 +1,15 @@`
`1`	`1`	`import * as ts from 'typescript';`
`2`	`2`
`3`		`-export function typescriptTransformUnspec(program: ts.Program, options?): ts.TransformerFactory<ts.SourceFile> {`
	`3`	`+export default function typescriptTransformUnspec(program: ts.Program): ts.TransformerFactory<ts.SourceFile> { // tslint:disable-line:no-default-export`
`4`	`4`	`return (context) => (file) => visitNodeAndChildren(file, program, context);`
`5`	`5`	`}`
`6`	`6`
`7`	`7`	`function visitNodeAndChildren(node: ts.Node, program: ts.Program, context: ts.TransformationContext): ts.SourceFile {`
`8`		`- return ts.visitEachChild(visitor(node), (childNode) => visitNodeAndChildren(childNode, program, context), context) as ts.SourceFile;`
	`8`	`+ const visitor = (childNode) => visitNodeAndChildren(childNode, program, context);`
	`9`	`+ return ts.visitEachChild<ts.SourceFile>(<ts.SourceFile>filterNode(node), visitor, context);`
`9`	`10`	`}`
`10`	`11`
`11`		`-function visitor(node: ts.Node): ts.Node \| undefined {`
	`12`	`+function filterNode<T extends ts.Node>(node: T): T \| undefined {`
`12`	`13`	`if (isSpecExpressionStatement(node)) {`
`13`	`14`	`return undefined;`
`14`	`15`	`}`
`@@ -44,18 +45,16 @@ function isSpecExpressionStatement(node: ts.Node) {`
`44`	`45`	`if (ts.isExpressionStatement(node) && ts.isCallExpression(node.expression)) {`
`45`	`46`	`const callExpr = node.expression.expression;`
`46`	`47`	`if (ts.isIdentifier(callExpr) && isTestExpressionText(callExpr.escapedText)) {`
`47`		`- console.log("callExpr", callExpr.flags);`
`48`	`48`	`return true;`
`49`	`49`	`}`
`50`	`50`	`if (ts.isPropertyAccessExpression(callExpr)) {`
`51`	`51`	`const propertyAccessExpr = callExpr.expression;`
`52`	`52`	`if (ts.isIdentifier(propertyAccessExpr) && isTestExpressionText(propertyAccessExpr.escapedText)) {`
`53`		`- console.log("propertyAccessExpr", propertyAccessExpr.flags);`
`54`	`53`	`return true;`
`55`	`54`	`}`
`56`	`55`	`}`
`57`		`- return false;`
`58`	`56`	`}`
	`57`	`+ return false;`
`59`	`58`	`}`
`60`	`59`
`61`	`60`	`module.exports = typescriptTransformUnspec;`