Enable Webpack E2Es (#12735)

* test: enable webpack e2e tests

* refactor: remove redundant catch block

* fix(@ngtools/webpack): when stats is not available call callback with an error

Fixes #12686

Author: alan-agius4

packages/ngtools/webpack/src/virtual_file_system_decorator.ts

@@ -26,12 +26,13 @@ export class VirtualFileSystemDecorator implements InputFileSystem {
   }
 
   stat(path: string, callback: (err: Error, stats: Stats) => void): void {
-    try {
+    const result = this._webpackCompilerHost.stat(path);
+    if (result) {
       // tslint:disable-next-line:no-any
-      callback(null as any, this._webpackCompilerHost.stat(path) as any);
-    } catch (e) {
+      callback(null as any, result);
+    } else {
       // tslint:disable-next-line:no-any
-      callback(e, undefined as any);
+      callback(new FileDoesNotExistException(path), undefined as any);
     }
   }
 

tests/legacy-cli/e2e_runner.ts

@@ -124,10 +124,6 @@ const allTests = glob.sync(path.join(e2eRoot, testGlob), { nodir: true, ignore:
   .filter(name => !name.includes('tests/commands/new/'))
   // NEEDS devkit change
   .filter(name => !name.endsWith('/existing-directory.ts'))
-  // ngtools/webpack is now in devkit and needs to be tested there
-  .filter(name => !name.endsWith('/packages/webpack/server-ng5.ts'))
-  .filter(name => !name.endsWith('/packages/webpack/test-ng5.ts'))
-  .filter(name => !name.endsWith('/packages/webpack/weird-ng5.ts'))
   // Disabled on rc.0 due to needed sync with devkit for changes.
   .filter(name => !name.endsWith('/service-worker.ts'))
   .sort();