test: change browserslist file name to .browserslistrc

Author: alan-agius4

packages/angular_devkit/build_angular/src/utils/build-browser-features_spec.ts

@@ -31,7 +31,7 @@ describe('BuildBrowserFeatures', () => {
   describe('isDifferentialLoadingNeeded', () => {
     it('should be true for for IE 9-11 and ES2015', () => {
       host.writeMultipleFiles({
-        'browserslist': 'IE 9-11',
+        '.browserslistrc': 'IE 9-11',
       });
 
       const buildBrowserFeatures = new BuildBrowserFeatures(
@@ -43,7 +43,7 @@ describe('BuildBrowserFeatures', () => {
 
     it('should be false for Chrome and ES2015', () => {
       host.writeMultipleFiles({
-        'browserslist': 'last 1 chrome version',
+        '.browserslistrc': 'last 1 chrome version',
       });
 
       const buildBrowserFeatures = new BuildBrowserFeatures(
@@ -55,7 +55,7 @@ describe('BuildBrowserFeatures', () => {
 
     it('detects no need for differential loading for target is ES5', () => {
       host.writeMultipleFiles({
-        'browserslist': 'last 1 chrome version',
+        '.browserslistrc': 'last 1 chrome version',
       });
 
       const buildBrowserFeatures = new BuildBrowserFeatures(
@@ -67,7 +67,7 @@ describe('BuildBrowserFeatures', () => {
 
     it('should be false for Safari 10.1 when target is ES2015', () => {
       host.writeMultipleFiles({
-        'browserslist': 'Safari 10.1',
+        '.browserslistrc': 'Safari 10.1',
       });
 
       const buildBrowserFeatures = new BuildBrowserFeatures(
@@ -81,7 +81,7 @@ describe('BuildBrowserFeatures', () => {
   describe('isFeatureSupported', () => {
     it('should be true for es6-module and Safari 10.1', () => {
       host.writeMultipleFiles({
-        'browserslist': 'Safari 10.1',
+        '.browserslistrc': 'Safari 10.1',
       });
 
       const buildBrowserFeatures = new BuildBrowserFeatures(
@@ -93,7 +93,7 @@ describe('BuildBrowserFeatures', () => {
 
     it('should be false for es6-module and IE9', () => {
       host.writeMultipleFiles({
-        'browserslist': 'IE 9',
+        '.browserslistrc': 'IE 9',
       });
 
       const buildBrowserFeatures = new BuildBrowserFeatures(
@@ -105,7 +105,7 @@ describe('BuildBrowserFeatures', () => {
 
     it('should be true for es6-module and last 1 chrome version', () => {
       host.writeMultipleFiles({
-        'browserslist': 'last 1 chrome version',
+        '.browserslistrc': 'last 1 chrome version',
       });
 
       const buildBrowserFeatures = new BuildBrowserFeatures(
@@ -117,7 +117,7 @@ describe('BuildBrowserFeatures', () => {
 
     it('should be true for es6-module and Edge 18', () => {
       host.writeMultipleFiles({
-        'browserslist': 'Edge 18',
+        '.browserslistrc': 'Edge 18',
       });
 
       const buildBrowserFeatures = new BuildBrowserFeatures(

packages/angular_devkit/build_angular/test/browser/differential_loading_spec_large.ts

@@ -18,7 +18,7 @@ describe('Browser Builder with differential loading', () => {
     await host.initialize().toPromise();
     // to trigger differential loading we need an non ever green browser
     host.writeMultipleFiles({
-      browserslist: 'IE 10',
+      '.browserslistrc': 'IE 10',
     });
 
     architect = (await createArchitect(host.root())).architect;
@@ -193,7 +193,7 @@ describe('Browser Builder with differential loading', () => {
 
   it('adds `type="module"` when differential loading is needed', async () => {
     host.writeMultipleFiles({
-      browserslist: `
+      '.browserslistrc': `
         last 1 chrome version
         IE 9
       `,

packages/angular_devkit/build_angular/test/browser/styles_spec_large.ts

@@ -285,7 +285,7 @@ describe('Browser Builder styles', () => {
         /* normal-comment */
         /*! important-comment */
         div { flex: 1 }`,
-      browserslist: 'IE 10',
+      '.browserslistrc': 'IE 10',
     });
 
     const overrides = { extractCss: true, optimization: false };
@@ -319,7 +319,7 @@ describe('Browser Builder styles', () => {
           grid-template-columns: 100px;
         }
       `,
-      browserslist: 'IE 10',
+      '.browserslistrc': 'IE 10',
     });
 
     const overrides = { extractCss: true, optimization: true, styles: ['src/styles.scss'] };

packages/angular_devkit/build_angular/test/dev-server/index_spec_large.ts

@@ -18,7 +18,7 @@ describe('Dev Server Builder index', () => {
 
   it(`adds 'type="module"' when differential loading is needed`, async () => {
     host.writeMultipleFiles({
-      browserslist: `
+      '.browserslistrc': `
         last 1 chrome version
         IE 10
       `,
@@ -41,7 +41,7 @@ describe('Dev Server Builder index', () => {
 
   it(`does not add 'type="module"' to custom scripts when differential loading is needed`, async () => {
     host.writeMultipleFiles({
-      browserslist: `
+      '.browserslistrc': `
         last 1 chrome version
         IE 10
       `,
@@ -85,7 +85,7 @@ describe('Dev Server Builder index', () => {
 
   it(`doesn't 'type="module"' when differential loading is not needed`, async () => {
     host.writeMultipleFiles({
-      browserslist: `
+      '.browserslistrc': `
         last 1 chrome version
       `,
     });

packages/angular_devkit/build_angular/test/server/base_spec_large.ts

@@ -57,7 +57,7 @@ describe('Server Builder', () => {
     // the below is needed because of different code paths
     // for polyfills if differential loading is needed
     host.writeMultipleFiles({
-      'browserslist': 'IE 10',
+      '.browserslistrc': 'IE 10',
     });
 
     const run = await architect.scheduleTarget(target);

packages/schematics/angular/application/files/.browserslistrc.template

@@ -0,0 +1,12 @@
+# This file is used by the build system to adjust CSS and JS output to support the specified browsers below.
+# For additional information regarding the format and rule options, please see:
+# https://github.com/browserslist/browserslist#queries
+
+# You can see what browsers were selected by your queries by running:
+#   npx browserslist
+
+> 0.5%
+last 2 versions
+Firefox ESR
+not dead
+not IE 9-11 # For IE 9-11 support, remove 'not'.

packages/schematics/angular/migrations/update-10/rename-browserslist-config.ts

@@ -8,26 +8,22 @@
 import { Path, join } from '@angular-devkit/core';
 import { DirEntry, Rule } from '@angular-devkit/schematics';
 
-function visit(directory: DirEntry): Path[] {
-  const files: Path[] = [];
-
+function* visit(directory: DirEntry): IterableIterator<string> {
   for (const path of directory.subfiles) {
     if (path !== 'browserslist') {
       continue;
     }
 
-    files.push(join(directory.path, path));
+    yield join(directory.path, path);
   }
 
   for (const path of directory.subdirs) {
     if (path === 'node_modules') {
       continue;
     }
 
-    files.push(...visit(directory.dir(path)));
+    yield* visit(directory.dir(path));
   }
-
-  return files;
 }
 
 export default function (): Rule {

tests/angular_devkit/build_angular/hello-world-app-ve/browserslist renamed to tests/angular_devkit/build_angular/hello-world-app-ve/.browserslistrc

@@ -9,7 +9,7 @@ export default async function() {
   await setupI18nConfig();
 
   // Ensure a ES2015 build is used.
-  await writeFile('browserslist', 'Chrome 65');
+  await writeFile('.browserslistrc', 'Chrome 65');
   await updateJsonFile('tsconfig.json', config => {
     config.compilerOptions.target = 'es2015';
     config.angularCompilerOptions.disableTypeScriptVersionCheck = true;

tests/angular_devkit/build_angular/hello-world-app/browserslist renamed to tests/angular_devkit/build_angular/hello-world-app/.browserslistrc

@@ -22,7 +22,7 @@ export default async function() {
   await npm('install', `${localizeVersion}`);
 
   // Ensure a ES2015 build is used.
-  await writeFile('browserslist', 'Chrome 65');
+  await writeFile('.browserslistrc', 'Chrome 65');
   await updateJsonFile('tsconfig.json', config => {
     config.compilerOptions.target = 'es2015';
     config.angularCompilerOptions.disableTypeScriptVersionCheck = true;

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

@@ -16,7 +16,7 @@ export default async function () {
   }
 
   await replaceInFile(
-    'browserslist',
+    '.browserslistrc',
     'not IE 9-11',
     'Safari 9-10.1\nIE 9-11',
   );

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

@@ -3,7 +3,7 @@ import { ng } from '../../utils/process';
 
 export default async function() {
   // Ensure an ES2015 build is used in test
-  await writeFile('browserslist', 'Chrome 65');
+  await writeFile('.browserslistrc', 'Chrome 65');
 
   await ng('generate', 'service', 'user');
 

tests/legacy-cli/e2e/tests/misc/browsers.ts

@@ -4,7 +4,7 @@ import { expectToFail } from '../../utils/utils';
 
 export default async function() {
   // Ensure an ES2015 build is used in test
-  await writeFile('browserslist', 'Chrome 65');
+  await writeFile('.browserslistrc', 'Chrome 65');
 
   // Update the application to use a forward reference
   await replaceInFile(

tests/legacy-cli/e2e/tests/misc/es2015-nometa.ts

@@ -14,7 +14,7 @@ export default async function () {
       appArchitect.build.options.es5BrowserSupport = false;
   });
 
-  await writeFile('browserslist', 'last 2 Chrome versions');
+  await writeFile('.browserslistrc', 'last 2 Chrome versions');
   await ng('build');
   await expectFileNotToExist('dist/test-project/polyfills-es5.js');
   await expectFileToMatch('dist/test-project/index.html', oneLineTrim`
@@ -40,7 +40,7 @@ export default async function () {
     const appArchitect = workspaceJson.projects['test-project'].architect;
     appArchitect.build.options.es5BrowserSupport = undefined;
   });
-  await writeFile('browserslist', 'IE 10');
+  await writeFile('.browserslistrc', 'IE 10');
   await ng('build');
   await expectFileToMatch('dist/test-project/polyfills-es5.js', 'core-js');
   await expectFileToMatch('dist/test-project/index.html', oneLineTrim`