Removed preloader from AoT Webpack config (#225)

* Removed preloader from aot webpack config

* Update config-webpack-build-aot.spec.js

config/webpack/build-aot.webpack.config.js

@@ -19,6 +19,14 @@ function getWebpackConfig(skyPagesConfig) {
   let commonConfig = common.getWebpackConfig(skyPagesConfig);
   commonConfig.entry = null;
 
+  // Since the preloader is executed against the file system during an AoT build,
+  // we need to remove it from the webpack config, otherwise it will get executed twice.
+  commonConfig.module.rules = commonConfig.module.rules
+    .filter((rule) => {
+      const isPreloader = /\/sky-processor\//.test(rule.loader);
+      return (!isPreloader);
+    });
+
   return webpackMerge(commonConfig, {
     entry: {
       polyfills: [skyPagesConfigUtil.spaPathTempSrc('polyfills.ts')],

test/config-webpack-build-aot.spec.js

@@ -18,7 +18,7 @@ describe('config webpack build-aot', () => {
   });
 
   afterEach(() => {
-    mock.stop(ngtoolsWebpackPath);
+    mock.stopAll();
   });
 
   it('should expose a getWebpackConfig method', () => {
@@ -193,4 +193,40 @@ describe('config webpack build-aot', () => {
     });
   });
 
+  it('should remove the sky-processor loader from the rules array', () => {
+    const f = './common.webpack.config';
+    mock(f, {
+      getWebpackConfig: () => ({
+        module: {
+          rules: [
+            {
+              loader: '/sky-processor/'
+            }
+          ]
+        }
+      })
+    });
+
+    const lib = require('../config/webpack/build-aot.webpack.config');
+
+    const skyPagesConfig = {
+      runtime: runtimeUtils.getDefaultRuntime(),
+      skyux: {}
+    };
+
+    const config = lib.getWebpackConfig(skyPagesConfig);
+
+    let found = false;
+
+    config.module.rules.forEach((rule) => {
+      if (found) {
+        return;
+      }
+
+      found = /\/sky-processor\//.test(rule.loader);
+    });
+
+    expect(found).toEqual(false);
+  });
+
 });