@W-18741334: remove hard coded api names for download

package.json

@@ -109,7 +109,7 @@
     "@babel/preset-env": "7.18.6",
     "@babel/preset-react": "7.18.6",
     "@babel/preset-typescript": "^7.18.6",
-    "@commerce-apps/raml-toolkit": "0.6.0",
+    "@commerce-apps/raml-toolkit": "file:../raml-toolkit",
     "@rollup/plugin-babel": "5.3.1",
     "@rollup/plugin-commonjs": "13.0.2",
     "@rollup/plugin-node-resolve": "8.4.0",

scripts/updateApis.ts

@@ -9,7 +9,6 @@ import path from 'path';
 import fs from 'fs-extra';
 import dotenv from 'dotenv';
 import {downloadLatestApis} from './utils';
-import {API_LIST} from './config';
 
 dotenv.config();
 
@@ -26,7 +25,9 @@ const PRODUCTION_API_PATH = path.join(__dirname, '../apis');
 fs.moveSync(PRODUCTION_API_PATH, OLD_APIS_PATH, {overwrite: true});
 fs.ensureDirSync(PRODUCTION_API_PATH);
 
-API_LIST.forEach(name => {
-  // eslint-disable-next-line no-console
-  downloadLatestApis(name, PRODUCTION_API_PATH).catch(console.error);
-});
+// eslint-disable-next-line no-console
+
+downloadLatestApis(
+  'category:Visibility = "External" category:"SDK Type" = "Commerce" category:"SDK Type" = "Isomorphic"',
+  PRODUCTION_API_PATH
+).catch(console.error);

scripts/utils.test.ts

@@ -9,22 +9,16 @@ import {downloadLatestApis} from './utils';
 
 describe('test downloadLatestApis script', () => {
   it('throws error when no results', async () => {
-    await expect(downloadLatestApis('noResults', '/tmp')).rejects.toThrow(
-      "No results in Exchange for 'noResults'"
+    await expect(downloadLatestApis('"noResults"', '/tmp')).rejects.toThrow(
+      'No results in Exchange for \'"noResults"\''
     );
   });
 
-  it('throws error when no exact match', async () => {
-    await expect(downloadLatestApis('noMatch', '/tmp')).rejects.toThrow(
-      "No exact match in Exchange for 'noMatch'"
-    );
-  });
-
-  it('downloads when exact match', async () => {
+  it('downloads the apis with default search query', async () => {
     jest.spyOn(download, 'downloadRestApis').mockResolvedValue('');
 
     await expect(
-      downloadLatestApis('shopper-customers', '/tmp')
+      downloadLatestApis('category:Visibility = "External"', '/tmp')
     ).resolves.toBeUndefined();
   });
 
@@ -33,7 +27,7 @@ describe('test downloadLatestApis script', () => {
       .spyOn(download, 'downloadRestApis')
       .mockRejectedValue(new Error('It failed.'));
     await expect(
-      downloadLatestApis('shopper-customers', '/tmp')
-    ).rejects.toThrow('Failed to download shopper-customers: It failed.');
+      downloadLatestApis('category:Visibility = "External"', '/tmp')
+    ).rejects.toThrow('Failed to download API specs: It failed.');
   });
 });

scripts/utils.ts

@@ -29,31 +29,24 @@ import {download} from '@commerce-apps/raml-toolkit';
  *  We should have some followup to figure out how to cover it.
  *  Ive spent hours trying to mock download
  *
- * @param name - Api name to search for
+ * @param searchQuery - Query to search exchange
  * @param rootPath - Root path to download to
  *
  * @returns a promise that we will complete
  */
 export async function downloadLatestApis(
-  name: string,
-  rootPath: string,
-  isOAS = true
+  searchQuery: string,
+  rootPath: string
 ): Promise<void> {
-  const matchedApis = await download.search(`"${name}"`);
+  const matchedApis = await download.search(searchQuery, undefined, true);
   if (!(matchedApis?.length > 0)) {
-    throw new Error(`No results in Exchange for '${name}'`);
-  }
-  const api = matchedApis.find(
-    (matchedApi: {assetId: string}) => matchedApi?.assetId === name
-  );
-  if (!api) {
-    throw new Error(`No exact match in Exchange for '${name}'`);
+    throw new Error(`No results in Exchange for '${searchQuery}'`);
   }
   try {
-    await download.downloadRestApis([api], rootPath, isOAS);
+    await download.downloadRestApis(matchedApis, rootPath, true);
   } catch (err: unknown) {
     if (err instanceof Error) {
-      err.message = `Failed to download ${name}: ${err.message}`;
+      err.message = `Failed to download API specs: ${err.message}`;
     }
     throw err;
   }