test: add issue for broken app-shell test

Author: filipesilva

packages/angular_devkit/build_angular/src/app-shell/index.ts

@@ -15,6 +15,7 @@ import { JsonObject, experimental, join, normalize, resolve, schema } from '@ang
 import { NodeJsSyncHost } from '@angular-devkit/core/node';
 import * as fs from 'fs';
 import * as path from 'path';
+import { readTsconfig } from '../angular-cli-files/utilities/read-tsconfig';
 import { augmentAppWithServiceWorker } from '../angular-cli-files/utilities/service-worker';
 import { BrowserBuilderOutput } from '../browser';
 import { Schema as BrowserBuilderSchema } from '../browser/schema';
@@ -31,40 +32,47 @@ async function _renderUniversal(
   const browserIndexOutputPath = path.join(browserResult.outputPath || '', 'index.html');
   const indexHtml = fs.readFileSync(browserIndexOutputPath, 'utf8');
   const serverBundlePath = await _getServerModuleBundlePath(options, context, serverResult);
-
   const root = context.workspaceRoot;
 
+  // Get browser target options.
+  const browserTarget = targetFromTargetString(options.browserTarget);
+  const rawBrowserOptions = await context.getTargetOptions(browserTarget);
+  const browserBuilderName = await context.getBuilderNameForTarget(browserTarget);
+  const browserOptions = await context.validateOptions<JsonObject & BrowserBuilderSchema>(
+    rawBrowserOptions,
+    browserBuilderName,
+  );
+
+  // Determine if browser app was compiled using Ivy.
+  const { options: compilerOptions } = readTsconfig(browserOptions.tsConfig, context.workspaceRoot);
+  const ivy = compilerOptions.enableIvy;
+
   // Initialize zone.js
   const zonePackage = require.resolve('zone.js', { paths: [root] });
   await import(zonePackage);
 
   // Load platform server module renderer
   const platformServerPackage = require.resolve('@angular/platform-server', { paths: [root] });
-  const renderModuleFactory = await import(platformServerPackage)
+  const renderOpts = {
+    document: indexHtml,
+    url: options.route,
+  };
+
+  // Render app using Ivy or VE
+  const renderCall = await import(platformServerPackage)
     // tslint:disable-next-line:no-implicit-dependencies
-    .then((m: typeof import('@angular/platform-server')) => m.renderModuleFactory);
+    .then((m: typeof import('@angular/platform-server')) => ivy
+      ? m.renderModule(require(serverBundlePath).AppServerModule, renderOpts)
+      : m.renderModuleFactory(require(serverBundlePath).AppServerModuleNgFactory, renderOpts));
 
-  const AppServerModuleNgFactory = require(serverBundlePath).AppServerModuleNgFactory;
+  // Render to HTML and overwrite the client index file.
+  const html = await renderCall;
   const outputIndexPath = options.outputIndexPath
     ? path.join(root, options.outputIndexPath)
     : browserIndexOutputPath;
 
-  // Render to HTML and overwrite the client index file.
-  const html = await renderModuleFactory(AppServerModuleNgFactory, {
-    document: indexHtml,
-    url: options.route,
-  });
-
   fs.writeFileSync(outputIndexPath, html);
 
-  const browserTarget = targetFromTargetString(options.browserTarget);
-  const rawBrowserOptions = await context.getTargetOptions(browserTarget);
-  const browserBuilderName = await context.getBuilderNameForTarget(browserTarget);
-  const browserOptions = await context.validateOptions<JsonObject & BrowserBuilderSchema>(
-    rawBrowserOptions,
-    browserBuilderName,
-  );
-
   if (browserOptions.serviceWorker) {
     const host = new NodeJsSyncHost();
     // Create workspace.

packages/angular_devkit/build_angular/test/app-shell/app-shell_spec_large.ts

@@ -13,6 +13,7 @@ import { createArchitect, host, veEnabled } from '../utils';
 
 
 // DISABLED_FOR_IVY   These should pass but are currently not supported
+// See https://github.com/angular/angular-cli/issues/15383 for details.
 (veEnabled ? describe : xdescribe)('AppShell Builder', () => {
   const target = { project: 'app', target: 'app-shell' };
   let architect: Architect;

tests/legacy-cli/e2e_runner.ts

@@ -105,6 +105,7 @@ if (!argv.ve) {
     // We don't have a platform-server usage story yet for Ivy.
     // It's contingent on lazy loading and factory shim considerations that are still being
     // discussed.
+    // Broken currently https://github.com/angular/angular-cli/issues/15383
     .filter(name => !name.endsWith('tests/build/platform-server.ts'))
     .filter(name => !name.endsWith('tests/build/build-app-shell.ts'))
     .filter(name => !name.endsWith('tests/build/build-app-shell-with-schematic.ts'));