fix(@angular-devkit/build-optimizer): limit ignore test to safe transformers

clydin · hansl · commit 9eca83404a20 · 2018-06-05T17:55:15.000-07:00
diff --git a/packages/angular_devkit/build_optimizer/src/build-optimizer/build-optimizer.ts b/packages/angular_devkit/build_optimizer/src/build-optimizer/build-optimizer.ts
@@ -112,17 +112,18 @@ export function buildOptimizer(options: BuildOptimizerOptions): TransformJavascr
   }
 
   // tests are not needed for fast path
+  // usage will be expanded once transformers are verified safe
   const ignoreTest = !options.emitSourceMap && !typeCheck;
 
-  if (ignoreTest || testPrefixClasses(content)) {
+  if (testPrefixClasses(content)) {
     getTransforms.unshift(getPrefixClassesTransformer);
   }
 
   if (ignoreTest || testImportTslib(content)) {
     getTransforms.unshift(getImportTslibTransformer);
   }
 
-  if (ignoreTest || testWrapEnums(content)) {
+  if (testWrapEnums(content)) {
     getTransforms.unshift(getWrapEnumsTransformer);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -112,17 +112,18 @@ export function buildOptimizer(options: BuildOptimizerOptions): TransformJavascr`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`// tests are not needed for fast path`
	`115`	`+ // usage will be expanded once transformers are verified safe`
`115`	`116`	`const ignoreTest = !options.emitSourceMap && !typeCheck;`
`116`	`117`
`117`		`- if (ignoreTest \|\| testPrefixClasses(content)) {`
	`118`	`+ if (testPrefixClasses(content)) {`
`118`	`119`	`getTransforms.unshift(getPrefixClassesTransformer);`
`119`	`120`	`}`
`120`	`121`
`121`	`122`	`if (ignoreTest \|\| testImportTslib(content)) {`
`122`	`123`	`getTransforms.unshift(getImportTslibTransformer);`
`123`	`124`	`}`
`124`	`125`
`125`		`- if (ignoreTest \|\| testWrapEnums(content)) {`
	`126`	`+ if (testWrapEnums(content)) {`
`126`	`127`	`getTransforms.unshift(getWrapEnumsTransformer);`
`127`	`128`	`}`
`128`	`129`