take into account splitted modules in compat data result diff

Author: zloirock

scripts/build-compat-data.mjs

@@ -1,5 +1,5 @@
 /* https://github.com/import-js/eslint-plugin-import/issues/2181 */
-import { data, modules } from 'core-js-compat/src/data.mjs';
+import { dataWithIgnored as data, ignored, modules } from 'core-js-compat/src/data.mjs';
 import external from 'core-js-compat/src/external.mjs';
 import mappings from 'core-js-compat/src/mapping.mjs';
 import helpers from 'core-js-compat/helpers.js';
@@ -63,10 +63,18 @@ function write(filename, content) {
   return fs.writeJson(`packages/core-js-compat/${ filename }.json`, content, { spaces: '  ' });
 }
 
+const dataWithoutIgnored = { ...data };
+
+for (const ignore of ignored) delete dataWithoutIgnored[ignore];
+
 await Promise.all([
-  write('data', data),
+  write('data', dataWithoutIgnored),
   write('modules', modules),
   write('external', external),
+  // version for compat data tests
+  fs.writeFile('tests/compat/compat-data.js', `;(typeof global != 'undefined' ? global : Function('return this')()).data = ${
+    JSON.stringify(data, null, '  ')
+  }`),
 ]);
 
 console.log(chalk.green('compat data rebuilt'));

tests/compat/common-runner.js

@@ -1,5 +1,6 @@
 var GLOBAL = typeof global != 'undefined' ? global : Function('return this')();
 var results = GLOBAL.results = Object.create(null);
+var data = GLOBAL.data;
 var tests = GLOBAL.tests;
 
 for (var testName in tests) {
@@ -13,7 +14,7 @@ for (var testName in tests) {
   }
 }
 
-GLOBAL.showResults = function (data, engine, logger) {
+GLOBAL.showResults = function (engine, logger) {
   var difference = false;
 
   function logResults(showDifference) {

tests/compat/deno-runner.mjs

@@ -1,10 +1,8 @@
 /* global Deno -- it's Deno */
 import './tests.js';
+import './compat-data.js';
 import './common-runner.js';
 
 if (Deno.args.includes('--mode=JSON')) {
   console.log(JSON.stringify(globalThis.results, null, '  '));
-} else {
-  const data = JSON.parse(await Deno.readTextFile('packages/core-js-compat/data.json'));
-  globalThis.showResults(data, 'deno', console.log);
-}
+} else globalThis.showResults('deno', console.log);

tests/compat/node-runner.js

@@ -1,8 +1,8 @@
 /* eslint-disable no-console -- output */
 require('./tests');
+require('./compat-data');
 require('./common-runner');
-var data = require('../../packages/core-js-compat/data');
 
 if (process.argv.indexOf('--mode=JSON') !== -1) {
   console.log(JSON.stringify(global.results, null, '  '));
-} else global.showResults(data, 'node', console.log);
+} else global.showResults('node', console.log);

tests/compat/rhino-prepare.mjs

@@ -5,5 +5,3 @@ const rhino = await fetch(
 );
 
 await fs.writeFile('tests/compat/rhino.jar', Buffer.from(await rhino.arrayBuffer()));
-
-await fs.writeFile('tests/compat/compat-data.js', `module.exports = ${ await fs.readFile('packages/core-js-compat/data.json') }`);

tests/compat/rhino-runner.js

@@ -1,7 +1,6 @@
 require('./tests');
+require('./compat-data');
 require('./common-runner');
-// eslint-disable-next-line import/no-unresolved -- can be created after linting
-var data = require('./compat-data');
 
 /* eslint-disable no-restricted-globals -- output */
-global.showResults(data, 'rhino', print);
+global.showResults('rhino', print);