fix: units tests

react-native-community · Aug 30, 2024 · 6f0db32 · 6f0db32
1 parent ebe3911
commit 6f0db32
Showing 1 changed file with 26 additions and 26 deletions.
diff --git a/packages/cli-config/src/__tests__/index-test.ts b/packages/cli-config/src/__tests__/index-test.ts
@@ -59,18 +59,18 @@ beforeEach(async () => {
 
 afterEach(() => cleanup(DIR));
 
-test('should have a valid structure by default', () => {
+test('should have a valid structure by default', async () => {
  DIR = getTempDirectory('config_test_structure');
  writeFiles(DIR, {
  'react-native.config.js': `module.exports = {
  reactNativePath: "."
  }`,
  });
- const config = loadConfig({projectRoot: DIR});
+ const config = await loadConfig({projectRoot: DIR});
  expect(removeString(config, DIR)).toMatchSnapshot();
 });
 
-test('should return dependencies from package.json', () => {
+test('should return dependencies from package.json', async () => {
  DIR = getTempDirectory('config_test_deps');
  writeFiles(DIR, {
  ...REACT_NATIVE_MOCK,
@@ -83,11 +83,11 @@ test('should return dependencies from package.json', () => {
  }
  }`,
  });
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(removeString(dependencies, DIR)).toMatchSnapshot();
 });
 
-test('should read a config of a dependency and use it to load other settings', () => {
+test('should read a config of a dependency and use it to load other settings', async () => {
  DIR = getTempDirectory('config_test_settings');
  writeFiles(DIR, {
  ...REACT_NATIVE_MOCK,
@@ -122,13 +122,13 @@ test('should read a config of a dependency and use it to load other settings', (
  }
  }`,
  });
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(
  removeString(dependencies['react-native-test'], DIR),
  ).toMatchSnapshot();
 });
 
-test('command specified in root config should overwrite command in "react-native-foo" and "react-native-bar" packages', () => {
+test('command specified in root config should overwrite command in "react-native-foo" and "react-native-bar" packages', async () => {
  DIR = getTempDirectory('config_test_packages');
  writeFiles(DIR, {
  'node_modules/react-native-foo/package.json': '{}',
@@ -173,15 +173,15 @@ test('command specified in root config should overwrite command in "react-native
  ],
  };`,
  });
- const {commands} = loadConfig({projectRoot: DIR});
+ const {commands} = await loadConfig({projectRoot: DIR});
  const commandsNames = commands.map(({name}) => name);
  const commandIndex = commandsNames.indexOf('foo-command');
 
  expect(commands[commandIndex].options).not.toBeNull();
  expect(commands[commandIndex]).toMatchSnapshot();
 });
 
-test('should merge project configuration with default values', () => {
+test('should merge project configuration with default values', async () => {
  DIR = getTempDirectory('config_test_merge');
  writeFiles(DIR, {
  ...REACT_NATIVE_MOCK,
@@ -206,13 +206,13 @@ test('should merge project configuration with default values', () => {
  }
  }`,
  });
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(removeString(dependencies['react-native-test'], DIR)).toMatchSnapshot(
  'snapshoting `react-native-test` config',
  );
 });
 
-test('should load commands from "react-native-foo" and "react-native-bar" packages', () => {
+test('should load commands from "react-native-foo" and "react-native-bar" packages', async () => {
  DIR = getTempDirectory('config_test_packages');
  writeFiles(DIR, {
  'react-native.config.js': 'module.exports = { reactNativePath: "." }',
@@ -241,11 +241,11 @@ test('should load commands from "react-native-foo" and "react-native-bar" packag
  }
  }`,
  });
- const {commands} = loadConfig({projectRoot: DIR});
+ const {commands} = await loadConfig({projectRoot: DIR});
  expect(commands).toMatchSnapshot();
 });
 
-test('should not skip packages that have invalid configuration (to avoid breaking users)', () => {
+test('should not skip packages that have invalid configuration (to avoid breaking users)', async () => {
  process.env.FORCE_COLOR = '0'; // To disable chalk
  DIR = getTempDirectory('config_test_skip');
  writeFiles(DIR, {
@@ -261,14 +261,14 @@ test('should not skip packages that have invalid configuration (to avoid breakin
  }
  }`,
  });
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(removeString(dependencies, DIR)).toMatchSnapshot(
  'dependencies config',
  );
  expect(spy.mock.calls[0][0]).toMatchSnapshot('logged warning');
 });
 
-test('does not use restricted "react-native" key to resolve config from package.json', () => {
+test('does not use restricted "react-native" key to resolve config from package.json', async () => {
  DIR = getTempDirectory('config_test_restricted');
  writeFiles(DIR, {
  'node_modules/react-native-netinfo/package.json': `{
@@ -281,12 +281,12 @@ test('does not use restricted "react-native" key to resolve config from package.
  }
  }`,
  });
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(dependencies).toHaveProperty('react-native-netinfo');
  expect(spy).not.toHaveBeenCalled();
 });
 
-test('supports dependencies from user configuration with custom root and properties', () => {
+test('supports dependencies from user configuration with custom root and properties', async () => {
  DIR = getTempDirectory('config_test_custom_root');
  const escapePathSeparator = (value: string) =>
  path.sep === '\\' ? value.replace(/(\/|\\)/g, '\\\\') : value;
@@ -327,7 +327,7 @@ module.exports = {
  }`,
  });
 
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(removeString(dependencies['local-lib'], DIR)).toMatchInlineSnapshot(`
  Object {
  "name": "local-lib",
@@ -345,7 +345,7 @@ module.exports = {
  `);
 });
 
-test('should apply build types from dependency config', () => {
+test('should apply build types from dependency config', async () => {
  DIR = getTempDirectory('config_test_apply_dependency_config');
  writeFiles(DIR, {
  ...REACT_NATIVE_MOCK,
@@ -367,13 +367,13 @@ test('should apply build types from dependency config', () => {
  }
  }`,
  });
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(
  removeString(dependencies['react-native-test'], DIR),
  ).toMatchSnapshot();
 });
 
-test('supports dependencies from user configuration with custom build type', () => {
+test('supports dependencies from user configuration with custom build type', async () => {
  DIR = getTempDirectory('config_test_apply_custom_build_config');
  writeFiles(DIR, {
  ...REACT_NATIVE_MOCK,
@@ -400,13 +400,13 @@ test('supports dependencies from user configuration with custom build type', ()
  }`,
  });
 
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(
  removeString(dependencies['react-native-test'], DIR),
  ).toMatchSnapshot();
 });
 
-test('supports disabling dependency for ios platform', () => {
+test('supports disabling dependency for ios platform', async () => {
  DIR = getTempDirectory('config_test_disable_dependency_platform');
  writeFiles(DIR, {
  ...REACT_NATIVE_MOCK,
@@ -429,13 +429,13 @@ test('supports disabling dependency for ios platform', () => {
  }`,
  });
 
- const {dependencies} = loadConfig({projectRoot: DIR});
+ const {dependencies} = await loadConfig({projectRoot: DIR});
  expect(
  removeString(dependencies['react-native-test'], DIR),
  ).toMatchSnapshot();
 });
 
-test('should convert project sourceDir relative path to absolute', () => {
+test('should convert project sourceDir relative path to absolute', async () => {
  DIR = getTempDirectory('config_test_absolute_project_source_dir');
  const iosProjectDir = './ios2';
  const androidProjectDir = './android2';
@@ -494,7 +494,7 @@ test('should convert project sourceDir relative path to absolute', () => {
  `,
  });
 
- const config = loadConfig({projectRoot: DIR});
+ const config = await loadConfig({projectRoot: DIR});
 
  expect(config.project.ios?.sourceDir).toBe(path.join(DIR, iosProjectDir));
  expect(config.project.android?.sourceDir).toBe(