fix(@angular-devkit/build-angular): temporarily disable localize for multiple locales

Related to angular#15974

Author: filipesilva

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

@@ -105,6 +105,11 @@ 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);

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

@@ -54,6 +54,8 @@ export default async function() {
 
     // Enable localization for all locales
     appArchitect['build'].options.localize = true;
+    // Always error on missing translations.
+    appArchitect['build'].options.i18nMissingTranslation = 'error';
 
     // Add locale definitions to the project
     // tslint:disable-next-line: no-any
@@ -103,8 +105,13 @@ export default async function() {
   }
 
   // Build each locale and verify the output.
-  await ng('build', '--i18n-missing-translation', 'error');
+  // NOTE: this should not fail in general, but multi-locale translation is currently disabled.
+  // TODO: remove expectToFail when functionality is re-enabled.
+  await expectToFail(() => ng('build'));
   for (const { lang, translation } of langTranslations) {
+    // NOTE: remove this extra single build once the above expectToFail is removed.
+    await ng('build', '-c', lang);
+
     await expectFileToMatch(`${baseDir}/${lang}/main-es5.js`, translation);
     await expectFileToMatch(`${baseDir}/${lang}/main-es2015.js`, translation);
     await expectToFail(() => expectFileToMatch(`${baseDir}/${lang}/main-es5.js`, '$localize`'));
@@ -155,5 +162,5 @@ export default async function() {
   await ng('build', '--i18n-missing-translation', 'ignore');
   await expectFileToMatch(`${baseDir}/fr/main-es5.js`, /Other content/);
   await expectFileToMatch(`${baseDir}/fr/main-es2015.js`, /Other content/);
-  await expectToFail(() => ng('build', '--i18n-missing-translation', 'error'));
+  await expectToFail(() => ng('build'));
 }

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

@@ -55,6 +55,8 @@ export default async function() {
 
     // Enable localization for all locales
     appArchitect['build'].options.localize = true;
+    // Always error on missing translations.
+    appArchitect['build'].options.i18nMissingTranslation = 'error';
 
     // Add locale definitions to the project
     // tslint:disable-next-line: no-any
@@ -102,9 +104,15 @@ export default async function() {
   }
 
   // Build each locale and verify the output.
-  await ng('build', '--i18n-missing-translation', 'error');
+  // NOTE: this should not fail in general, but multi-locale translation is currently disabled.
+  // TODO: remove expectToFail when functionality is re-enabled.
+  await expectToFail(() => ng('build'));
   for (const { lang, translation } of langTranslations) {
+    // NOTE: remove this extra single build once the above expectToFail is removed.
+    await ng('build', '-c', lang);
+
     await expectFileToMatch(`${baseDir}/${lang}/main.js`, translation);
+    await expectFileToMatch(`${baseDir}/${lang}/main.js`, 'registerLocaleData');
     await expectToFail(() => expectFileToMatch(`${baseDir}/${lang}/main.js`, '$localize`'));
     await expectFileNotToExist(`${baseDir}/${lang}/main-es2015.js`);
     await expectFileToMatch(`${baseDir}/${lang}/main.js`, lang);
@@ -142,9 +150,13 @@ export default async function() {
     }
   }
 
+  // Verify locale data registration (currently only for single locale builds)
+  await ng('build', '--optimization', 'false', '-c', 'fr', '--i18n-missing-translation', 'error');
+  await expectFileToMatch(`${baseDir}/fr/main.js`, 'registerLocaleData');
+
   // Verify missing translation behaviour.
   await appendToFile('src/app/app.component.html', '<p i18n>Other content</p>');
   await ng('build', '--i18n-missing-translation', 'ignore');
   await expectFileToMatch(`${baseDir}/fr/main.js`, /Other content/);
-  await expectToFail(() => ng('build', '--i18n-missing-translation', 'error'));
+  await expectToFail(() => ng('build'));
 }