Re-enable global locale data support for Ivy

packages/angular_devkit/build_angular/src/utils/i18n-options.ts

@@ -106,14 +106,6 @@ export async function configureI18nBuild<T extends BrowserBuilderSchema | Server
   }
 
   const buildOptions = { ...options };
-
-  if (
-    buildOptions.localize === true ||
-    (Array.isArray(buildOptions.localize) && buildOptions.localize.length > 1)
-  ) {
-    throw new Error('Using the localize option for multiple locales is temporarily disabled.');
-  }
-
   const tsConfig = readTsconfig(buildOptions.tsConfig, context.workspaceRoot);
   const usingIvy = tsConfig.options.enableIvy !== false;
   const metadata = await context.getProjectMetadata(context.target);
@@ -172,8 +164,7 @@ export async function configureI18nBuild<T extends BrowserBuilderSchema | Server
             `Locale data for '${locale}' cannot be found.  No locale data will be included for this locale.`,
           );
         } else {
-          // Temporarily disable pending FW locale data fix
-          // desc.dataPath = localeDataPath;
+          desc.dataPath = localeDataPath;
         }
       }
     }
@@ -182,12 +173,6 @@ export async function configureI18nBuild<T extends BrowserBuilderSchema | Server
     if (usedFormats.size > 0) {
       buildOptions.i18nFormat = [...usedFormats][0];
     }
-
-    // If only one locale is specified set the deprecated option to enable the webpack plugin
-    // transform to register the locale directly in the output bundle.
-    if (i18n.inlineLocales.size === 1) {
-      buildOptions.i18nLocale = [...i18n.inlineLocales][0];
-    }
   }
 
   // If inlining store the output in a temporary location to facilitate post-processing

tests/legacy-cli/e2e/tests/i18n/ivy-localize-dl.ts

@@ -14,27 +14,21 @@ export default async function() {
     config.angularCompilerOptions.disableTypeScriptVersionCheck = true;
   });
 
-  // TODO: re-enable all locales once localeData support is added.
-  const tempLangTranslations = langTranslations.filter(l => l.lang == 'fr');
-
   // Build each locale and verify the output.
-  // NOTE: this should not fail in general, but multi-locale translation is currently disabled.
-  // TODO: remove expectToFail once localeData support is added.
-  await expectToFail(() => ng('build', '--localize', 'true'));
   await ng('build');
-  for (const { lang, outputPath, translation } of tempLangTranslations) {
+  for (const { lang, outputPath, translation } of langTranslations) {
     await expectFileToMatch(`${outputPath}/main-es5.js`, translation.helloPartial);
     await expectFileToMatch(`${outputPath}/main-es2015.js`, translation.helloPartial);
     await expectToFail(() => expectFileToMatch(`${outputPath}/main-es5.js`, '$localize`'));
     await expectToFail(() => expectFileToMatch(`${outputPath}/main-es2015.js`, '$localize`'));
 
     // Verify the locale ID is present
-    await expectFileToMatch(`${outputPath}/main-es5.js`, lang);
-    await expectFileToMatch(`${outputPath}/main-es2015.js`, lang);
+    await expectFileToMatch(`${outputPath}/vendor-es5.js`, lang);
+    await expectFileToMatch(`${outputPath}/vendor-es2015.js`, lang);
 
     // Verify the locale data is registered using the global files
-    // await expectFileToMatch(`${outputPath}/main-es5.js`, '.ng.common.locales');
-    // await expectFileToMatch(`${outputPath}/main-es2015.js`, '.ng.common.locales');
+    await expectFileToMatch(`${outputPath}/vendor-es5.js`, '.ng.common.locales');
+    await expectFileToMatch(`${outputPath}/vendor-es2015.js`, '.ng.common.locales');
 
     // Execute Application E2E tests with dev server
     await ng('e2e', `--configuration=${lang}`, '--port=0');
@@ -49,9 +43,9 @@ export default async function() {
   }
 
   // Verify deprecated locale data registration is not present
-  // await ng('build', '--configuration=fr', '--optimization=false');
-  // await expectToFail(() => expectFileToMatch(`${baseDir}/fr/main-es5.js`, 'registerLocaleData('));
-  // await expectToFail(() => expectFileToMatch(`${baseDir}/fr/main-es2015.js`, 'registerLocaleData('));
+  await ng('build', '--configuration=fr', '--optimization=false');
+  await expectToFail(() => expectFileToMatch(`${baseDir}/fr/main-es5.js`, 'registerLocaleData('));
+  await expectToFail(() => expectFileToMatch(`${baseDir}/fr/main-es2015.js`, 'registerLocaleData('));
 
   // Verify missing translation behaviour.
   await appendToFile('src/app/app.component.html', '<p i18n>Other content</p>');
@@ -60,7 +54,11 @@ export default async function() {
   await expectFileToMatch(`${baseDir}/fr/main-es2015.js`, /Other content/);
   await expectToFail(() => ng('build'));
   try {
-    await execAndWaitForOutputToMatch('ng', ['serve', '--port=0'], /No translation found for/);
+    await execAndWaitForOutputToMatch(
+      'ng',
+      ['serve', '--configuration=fr', '--port=0'],
+      /No translation found for/,
+    );
   } finally {
     killAllProcesses();
   }

tests/legacy-cli/e2e/tests/i18n/ivy-localize-es2015.ts

@@ -15,12 +15,8 @@ export default async function() {
     config.angularCompilerOptions.disableTypeScriptVersionCheck = true;
   });
 
-
-  // TODO: re-enable all locales once localeData support is added.
-  const tempLangTranslations = langTranslations.filter(l => l.lang == 'fr');
-
   await ng('build');
-  for (const { lang, outputPath, translation } of tempLangTranslations) {
+  for (const { lang, outputPath, translation } of langTranslations) {
     await expectFileToMatch(`${outputPath}/main.js`, translation.helloPartial);
     await expectToFail(() => expectFileToMatch(`${outputPath}/main.js`, '$localize`'));
     await expectFileNotToExist(`${outputPath}/main-es5.js`);

tests/legacy-cli/e2e/tests/i18n/ivy-localize-es5.ts

@@ -14,12 +14,9 @@ export default async function() {
     config.angularCompilerOptions.disableTypeScriptVersionCheck = true;
   });
 
-  // TODO: re-enable all locales once localeData support is added.
-  const tempLangTranslations = langTranslations.filter(l => l.lang == 'fr');
-
   // Build each locale and verify the output.
   await ng('build');
-  for (const { lang, outputPath, translation } of tempLangTranslations) {
+  for (const { lang, outputPath, translation } of langTranslations) {
     await expectFileToMatch(`${outputPath}/main.js`, translation.helloPartial);
     await expectToFail(() => expectFileToMatch(`${outputPath}/main.js`, '$localize`'));
     await expectFileNotToExist(`${outputPath}/main-es2015.js`);

tests/legacy-cli/e2e/tests/i18n/ivy-localize-server.ts

@@ -32,9 +32,6 @@ export default async function () {
   const serverbaseDir = 'dist/test-project/server';
   const serverBuildArgs = ['run', 'test-project:server'];
 
-  // TODO: re-enable all locales once localeData support is added.
-  const tempLangTranslations = langTranslations.filter(l => l.lang == 'fr');
-
   // Add server-specific config.
   await updateJsonFile('angular.json', workspaceJson => {
     const appProject = workspaceJson.projects['test-project'];
@@ -106,7 +103,7 @@ export default async function () {
   await ng('build');
   await ng(...serverBuildArgs);
 
-  for (const { lang, translation } of tempLangTranslations) {
+  for (const { lang, translation } of langTranslations) {
     await expectFileToMatch(`${serverbaseDir}/${lang}/main.js`, translation.helloPartial);
     await expectToFail(() => expectFileToMatch(`${serverbaseDir}/${lang}/main.js`, '$localize`'));
 

tests/legacy-cli/e2e/tests/i18n/legacy.ts

@@ -7,12 +7,11 @@ import { updateJsonFile } from '../../utils/project';
 import { expectToFail } from '../../utils/utils';
 import { readNgVersion } from '../../utils/version';
 
-
 // Configurations for each locale.
 export const baseDir = 'dist/test-project';
 export const langTranslations = [
   {
-    lang: 'en-US', outputPath: `${baseDir}/en`,
+    lang: 'en-US', outputPath: `${baseDir}/en-US`,
     translation: {
       helloPartial: 'Hello',
       hello: 'Hello i18n!',
@@ -59,7 +58,6 @@ export const langTranslations = [
 ];
 export const sourceLocale = langTranslations[0].lang;
 export const externalServer = (outputPath: string) => {
-  // Ivy i18n doesn't yet work with `ng serve` so we must use a separate server.
   const app = express();
   app.use(express.static(resolve(outputPath)));
 
@@ -142,9 +140,7 @@ export async function setupI18nConfig(useLocalize = true) {
 
     if (useLocalize) {
       // Enable localization for all locales
-      // TODO: re-enable all locales once localeData support is added.
-      // appArchitect['build'].options.localize = true;
-      appArchitect['build'].options.localize = ['fr'];
+      appArchitect['build'].options.localize = true;
     }
 
     // Add i18n config items (app, build, serve, e2e).