Skip to content

Commit a881bbd

Browse files
committed
fix(nested-clients): introduce submodules for nested-clients
1 parent a502644 commit a881bbd

File tree

77 files changed

+598
-319
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

77 files changed

+598
-319
lines changed

.eslintrc.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,5 +29,8 @@ module.exports = {
2929
"simple-import-sort/imports": "error",
3030
"sort-export-all/sort-export-all": "error",
3131
},
32-
ignorePatterns: ["packages/nested-clients/**/protocols/**.ts"],
32+
ignorePatterns: [
33+
"packages/nested-clients/src/submodules/**/protocols/*.ts",
34+
"packages/nested-clients/src/submodules/**/models/*.ts",
35+
],
3336
};

packages/core/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"build:include:deps": "lerna run --scope $npm_package_name --include-dependencies build",
1010
"build:types": "tsc -p tsconfig.types.json",
1111
"build:types:downlevel": "downlevel-dts dist-types dist-types/ts3.4",
12-
"lint": "node ./scripts/lint.js",
12+
"lint": "node ../../scripts/validation/submodules-linter.js --pkg core",
1313
"clean": "rimraf ./dist-* && rimraf *.tsbuildinfo",
1414
"extract:docs": "api-extractor run --local",
1515
"test": "yarn g:vitest run",

packages/core/scripts/lint.js

Lines changed: 0 additions & 118 deletions
This file was deleted.

packages/credential-provider-ini/src/resolveAssumeRoleCredentials.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { isAssumeRoleProfile, resolveAssumeRoleCredentials } from "./resolveAssu
66
import { resolveCredentialSource } from "./resolveCredentialSource";
77
import { resolveProfileData } from "./resolveProfileData";
88

9-
vi.mock("@aws-sdk/nested-clients", () => {
9+
vi.mock("@aws-sdk/nested-clients/sts", () => {
1010
return {
1111
getDefaultRoleAssumer: vi.fn().mockReturnValue(async () => ({})),
1212
};

packages/credential-provider-ini/src/resolveAssumeRoleCredentials.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ export const resolveAssumeRoleCredentials = async (
111111
const { source_profile, region } = profileData;
112112

113113
if (!options.roleAssumer) {
114-
const { getDefaultRoleAssumer } = await import("@aws-sdk/nested-clients");
114+
const { getDefaultRoleAssumer } = await import("@aws-sdk/nested-clients/sts");
115115
options.roleAssumer = getDefaultRoleAssumer(
116116
{
117117
...options.clientConfig,

packages/credential-provider-web-identity/src/fromWebToken.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { getDefaultRoleAssumerWithWebIdentity } from "@aws-sdk/nested-clients";
1+
import { getDefaultRoleAssumerWithWebIdentity } from "@aws-sdk/nested-clients/sts";
22
import { afterEach, describe, expect, test as it, vi } from "vitest";
33

44
import { fromWebToken } from "./fromWebToken";
@@ -16,7 +16,7 @@ const MOCK_CREDS = {
1616
sessionToken: "sessionToken",
1717
};
1818

19-
vi.mock("@aws-sdk/nested-clients", () => ({
19+
vi.mock("@aws-sdk/nested-clients/sts", () => ({
2020
getDefaultRoleAssumerWithWebIdentity: vi.fn().mockReturnValue(() => {}),
2121
}));
2222

packages/credential-provider-web-identity/src/fromWebToken.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ export const fromWebToken =
163163
let { roleAssumerWithWebIdentity } = init;
164164

165165
if (!roleAssumerWithWebIdentity) {
166-
const { getDefaultRoleAssumerWithWebIdentity } = await import("@aws-sdk/nested-clients");
166+
const { getDefaultRoleAssumerWithWebIdentity } = await import("@aws-sdk/nested-clients/sts");
167167
roleAssumerWithWebIdentity = getDefaultRoleAssumerWithWebIdentity(
168168
{
169169
...init.clientConfig,

packages/credential-providers/src/fromIni.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
import { fromIni as coreProvider } from "@aws-sdk/credential-provider-ini";
2-
import { getDefaultRoleAssumer, getDefaultRoleAssumerWithWebIdentity } from "@aws-sdk/nested-clients";
2+
import { getDefaultRoleAssumer, getDefaultRoleAssumerWithWebIdentity } from "@aws-sdk/nested-clients/sts";
33
import { beforeEach, describe, expect, test as it, vi } from "vitest";
44

55
import { fromIni } from "./fromIni";
66

77
const mockRoleAssumer = vi.fn().mockResolvedValue("ROLE_ASSUMER");
88
const mockRoleAssumerWithWebIdentity = vi.fn().mockResolvedValue("ROLE_ASSUMER_WITH_WEB_IDENTITY");
99

10-
vi.mock("@aws-sdk/nested-clients", () => ({
10+
vi.mock("@aws-sdk/nested-clients/sts", () => ({
1111
getDefaultRoleAssumer: vi.fn().mockImplementation(() => mockRoleAssumer),
1212
getDefaultRoleAssumerWithWebIdentity: vi.fn().mockImplementation(() => mockRoleAssumerWithWebIdentity),
1313
}));

packages/credential-providers/src/fromNodeProviderChain.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
import { defaultProvider } from "@aws-sdk/credential-provider-node";
2-
import { getDefaultRoleAssumer, getDefaultRoleAssumerWithWebIdentity } from "@aws-sdk/nested-clients";
2+
import { getDefaultRoleAssumer, getDefaultRoleAssumerWithWebIdentity } from "@aws-sdk/nested-clients/sts";
33
import { beforeEach, describe, expect, test as it, vi } from "vitest";
44

55
import { fromNodeProviderChain } from "./fromNodeProviderChain";
66

77
const mockRoleAssumer = vi.fn().mockResolvedValue("ROLE_ASSUMER");
88
const mockRoleAssumerWithWebIdentity = vi.fn().mockResolvedValue("ROLE_ASSUMER_WITH_WEB_IDENTITY");
99

10-
vi.mock("@aws-sdk/nested-clients", () => ({
10+
vi.mock("@aws-sdk/nested-clients/sts", () => ({
1111
getDefaultRoleAssumer: vi.fn().mockImplementation(() => mockRoleAssumer),
1212
getDefaultRoleAssumerWithWebIdentity: vi.fn().mockImplementation(() => mockRoleAssumerWithWebIdentity),
1313
}));

packages/credential-providers/src/fromTemporaryCredentials.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
1-
import { AssumeRoleCommand, STSClient } from "@aws-sdk/nested-clients";
1+
import { AssumeRoleCommand, STSClient } from "@aws-sdk/nested-clients/sts";
22
import { beforeEach, describe, expect, test as it, vi } from "vitest";
33

44
import { fromTemporaryCredentials } from "./fromTemporaryCredentials";
55

66
const mockSend = vi.fn();
77
const mockUsePlugin = vi.fn();
8-
vi.mock("@aws-sdk/nested-clients", () => ({
8+
vi.mock("@aws-sdk/nested-clients/sts", () => ({
99
STSClient: vi.fn().mockImplementation((config) => ({
1010
config,
1111
send: vi.fn().mockImplementation(async function (command) {

0 commit comments

Comments
 (0)