Skip to content

Commit b509c8e

Browse files
committed
test(codegen): add accId test for RoleAssumerWebIdentity
1 parent af5cd87 commit b509c8e

File tree

3 files changed

+15
-4
lines changed

3 files changed

+15
-4
lines changed

clients/client-sts/src/defaultStsRoleAssumers.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ export const getDefaultRoleAssumer = (
9090
}
9191
let accountId;
9292
try {
93-
accountId = AssumedRoleUser.Arn.split(":")[4];
93+
accountId = AssumedRoleUser!.Arn!.split(":")[4];
9494
} catch (error) {
9595
accountId = undefined;
9696
}
@@ -147,7 +147,7 @@ export const getDefaultRoleAssumerWithWebIdentity = (
147147
}
148148
let accountId;
149149
try {
150-
accountId = AssumedRoleUser.Arn.split(":")[4];
150+
accountId = AssumedRoleUser!.Arn!.split(":")[4];
151151
} catch (error) {
152152
accountId = undefined;
153153
}

codegen/smithy-aws-typescript-codegen/src/main/resources/software/amazon/smithy/aws/typescript/codegen/sts-client-defaultRoleAssumers.spec.ts

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -218,6 +218,17 @@ describe("getDefaultRoleAssumerWithWebIdentity", () => {
218218
});
219219
});
220220

221+
it("should return accountId in the credentials", async () => {
222+
const roleAssumerWithWebIdentity = getDefaultRoleAssumerWithWebIdentity();
223+
const params: AssumeRoleWithWebIdentityCommandInput = {
224+
RoleArn: "arn:aws:foo",
225+
RoleSessionName: "session",
226+
WebIdentityToken: "token",
227+
};
228+
const assumedRole = await roleAssumerWithWebIdentity(params);
229+
expect(assumedRole.accountId).toEqual("123456789012");
230+
});
231+
221232
it("should use the STS client middleware", async () => {
222233
const customMiddlewareFunction = jest.fn();
223234
const roleAssumerWithWebIdentity = getDefaultRoleAssumerWithWebIdentity({}, [

codegen/smithy-aws-typescript-codegen/src/main/resources/software/amazon/smithy/aws/typescript/codegen/sts-client-defaultStsRoleAssumers.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ export const getDefaultRoleAssumer = (
8787
}
8888
let accountId;
8989
try {
90-
accountId = AssumedRoleUser.Arn.split(":")[4];
90+
accountId = AssumedRoleUser!.Arn!.split(":")[4];
9191
} catch (error) {
9292
accountId = undefined;
9393
}
@@ -144,7 +144,7 @@ export const getDefaultRoleAssumerWithWebIdentity = (
144144
}
145145
let accountId;
146146
try {
147-
accountId = AssumedRoleUser.Arn.split(":")[4];
147+
accountId = AssumedRoleUser!.Arn!.split(":")[4];
148148
} catch (error) {
149149
accountId = undefined;
150150
}

0 commit comments

Comments
 (0)