Skip to content

Commit 38920b3

Browse files
authored
Update new-clients tests to run subset of clients (#281)
1 parent 70a313b commit 38920b3

24 files changed

+44
-8316
lines changed
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export const CLIENTS_TO_TEST = ["ACM", "AccessAnalyzer", "Discovery"];
Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
import { CLIENT_PACKAGE_NAMES_MAP } from "../../src/transforms/v2-to-v3/config";
22

3-
export const getClientNamesSortedByPackageName = () =>
4-
Object.keys(CLIENT_PACKAGE_NAMES_MAP).sort((a, b) =>
5-
CLIENT_PACKAGE_NAMES_MAP[a].localeCompare(CLIENT_PACKAGE_NAMES_MAP[b])
6-
);
3+
export const getClientNamesSortedByPackageName = (clientsToTest: string[]) =>
4+
Object.keys(
5+
clientsToTest.reduce((acc, clientName) => {
6+
acc[clientName] = CLIENT_PACKAGE_NAMES_MAP[clientName];
7+
return acc;
8+
}, {})
9+
).sort((a, b) => CLIENT_PACKAGE_NAMES_MAP[a].localeCompare(CLIENT_PACKAGE_NAMES_MAP[b]));
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
1+
import { CLIENT_NAMES } from "../../src/transforms/v2-to-v3/config";
12
import { getV2ClientsNewExpressionCode } from "./getV2ClientsNewExpressionCode";
23

34
export const getGlobalImportInput = (codegenComment: string) => {
45
let globalImportInputContent = `${codegenComment}\n`;
56

67
globalImportInputContent += `import AWS from "aws-sdk";\n\n`;
7-
globalImportInputContent += getV2ClientsNewExpressionCode(`AWS.`);
8+
globalImportInputContent += getV2ClientsNewExpressionCode(CLIENT_NAMES, `AWS.`);
89

910
return globalImportInputContent;
1011
};
Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,16 @@
1+
import { CLIENT_NAMES } from "../../src/transforms/v2-to-v3/config";
12
import { getClientNamesSortedByPackageName } from "./getClientNamesSortedByPackageName";
23
import { getV3ClientsNewExpressionCode } from "./getV3ClientsNewExpressionCode";
34
import { getV3PackageImportsCode } from "./getV3PackageImportsCode";
45

56
export const getGlobalImportOutput = (codegenComment: string) => {
67
let globalImportOutputContent = `${codegenComment}\n`;
78

8-
globalImportOutputContent += getV3PackageImportsCode(getClientNamesSortedByPackageName());
9+
globalImportOutputContent += getV3PackageImportsCode(
10+
getClientNamesSortedByPackageName(CLIENT_NAMES)
11+
);
912
globalImportOutputContent += `\n`;
10-
globalImportOutputContent += getV3ClientsNewExpressionCode();
13+
globalImportOutputContent += getV3ClientsNewExpressionCode(CLIENT_NAMES);
1114

1215
return globalImportOutputContent;
1316
};
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
1+
import { CLIENTS_TO_TEST } from "./config";
12
import { getV2ClientsNewExpressionCode } from "./getV2ClientsNewExpressionCode";
23

34
export const getGlobalRequireInput = (codegenComment: string) => {
45
let globalRequireInputContent = `${codegenComment}\n`;
56

67
globalRequireInputContent += `const AWS = require("aws-sdk");\n\n`;
7-
globalRequireInputContent += getV2ClientsNewExpressionCode(`AWS.`);
8+
globalRequireInputContent += getV2ClientsNewExpressionCode(CLIENTS_TO_TEST, `AWS.`);
89

910
return globalRequireInputContent;
1011
};

scripts/generateNewClientTests/getGlobalRequireOutput.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
import { CLIENT_NAMES_MAP, CLIENT_PACKAGE_NAMES_MAP } from "../../src/transforms/v2-to-v3/config";
2+
import { CLIENTS_TO_TEST } from "./config";
23
import { getClientNamesSortedByPackageName } from "./getClientNamesSortedByPackageName";
34
import { getV3ClientsNewExpressionCode } from "./getV3ClientsNewExpressionCode";
45

56
export const getGlobalRequireOutput = (codegenComment: string) => {
67
let globalRequireOutputContent = `${codegenComment}\n`;
78

8-
const sortedClientNames = getClientNamesSortedByPackageName();
9+
const sortedClientNames = getClientNamesSortedByPackageName(CLIENTS_TO_TEST);
910
globalRequireOutputContent += `const `;
1011
for (const v2ClientName of sortedClientNames) {
1112
const v3ClientName = CLIENT_NAMES_MAP[v2ClientName];
@@ -19,7 +20,7 @@ export const getGlobalRequireOutput = (codegenComment: string) => {
1920
` `;
2021
}
2122
globalRequireOutputContent = globalRequireOutputContent.replace(/,\n {6}$/, ";\n\n");
22-
globalRequireOutputContent += getV3ClientsNewExpressionCode();
23+
globalRequireOutputContent += getV3ClientsNewExpressionCode(CLIENTS_TO_TEST);
2324

2425
return globalRequireOutputContent;
2526
};
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
1-
import { CLIENT_NAMES } from "../../src/transforms/v2-to-v3/config";
1+
import { CLIENTS_TO_TEST } from "./config";
22
import { getV2ClientsNewExpressionCode } from "./getV2ClientsNewExpressionCode";
33

44
export const getServiceImportDeepInput = (codegenComment: string) => {
55
let serviceImportInputContent = `${codegenComment}\n`;
66

7-
for (const clientName of CLIENT_NAMES) {
7+
for (const clientName of CLIENTS_TO_TEST) {
88
serviceImportInputContent += `import ${clientName} from "aws-sdk/clients/${clientName.toLowerCase()}";\n`;
99
}
1010
serviceImportInputContent += `\n`;
11-
serviceImportInputContent += getV2ClientsNewExpressionCode();
11+
serviceImportInputContent += getV2ClientsNewExpressionCode(CLIENTS_TO_TEST);
1212

1313
return serviceImportInputContent;
1414
};
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
1-
import { CLIENT_NAMES } from "../../src/transforms/v2-to-v3/config";
1+
import { CLIENTS_TO_TEST } from "./config";
22
import { getV2ClientsNewExpressionCode } from "./getV2ClientsNewExpressionCode";
33

44
export const getServiceImportInput = (codegenComment: string) => {
55
let serviceImportInputContent = `${codegenComment}\n`;
66

7-
for (const clientName of CLIENT_NAMES) {
7+
for (const clientName of CLIENTS_TO_TEST) {
88
serviceImportInputContent += `import { ${clientName} } from "aws-sdk";\n`;
99
}
1010
serviceImportInputContent += `\n`;
11-
serviceImportInputContent += getV2ClientsNewExpressionCode();
11+
serviceImportInputContent += getV2ClientsNewExpressionCode(CLIENTS_TO_TEST);
1212

1313
return serviceImportInputContent;
1414
};
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
1-
import { CLIENT_NAMES } from "../../src/transforms/v2-to-v3/config";
1+
import { CLIENTS_TO_TEST } from "./config";
22
import { getV3ClientsNewExpressionCode } from "./getV3ClientsNewExpressionCode";
33
import { getV3PackageImportsCode } from "./getV3PackageImportsCode";
44

55
export const getServiceImportOutput = (codegenComment: string) => {
66
let serviceImportOutputContent = `${codegenComment}\n`;
77

8-
serviceImportOutputContent += getV3PackageImportsCode(CLIENT_NAMES);
8+
serviceImportOutputContent += getV3PackageImportsCode(CLIENTS_TO_TEST);
99
serviceImportOutputContent += `\n`;
10-
serviceImportOutputContent += getV3ClientsNewExpressionCode();
10+
serviceImportOutputContent += getV3ClientsNewExpressionCode(CLIENTS_TO_TEST);
1111

1212
return serviceImportOutputContent;
1313
};
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
1-
import { CLIENT_NAMES } from "../../src/transforms/v2-to-v3/config";
1+
import { CLIENTS_TO_TEST } from "./config";
22
import { getV2ClientsNewExpressionCode } from "./getV2ClientsNewExpressionCode";
33

44
export const getServiceRequireDeepInput = (codegenComment: string) => {
55
let serviceRequireInputContent = `${codegenComment}\n`;
66

7-
for (const clientName of CLIENT_NAMES) {
7+
for (const clientName of CLIENTS_TO_TEST) {
88
serviceRequireInputContent += `const ${clientName} = require("aws-sdk/clients/${clientName.toLowerCase()}");\n`;
99
}
1010
serviceRequireInputContent += `\n`;
11-
serviceRequireInputContent += getV2ClientsNewExpressionCode();
11+
serviceRequireInputContent += getV2ClientsNewExpressionCode(CLIENTS_TO_TEST);
1212

1313
return serviceRequireInputContent;
1414
};

0 commit comments

Comments
 (0)