Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Calvin Combs <66279577+comcalvi@users.noreply.github.com>
  • Loading branch information
konokenj and comcalvi committed Mar 6, 2023
1 parent 3f79c30 commit c198aeb
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,7 @@ describe('physicalResourceId patterns', () => {
});

// physicalResourceId pattern #7
test('physicalResourceId is specified in onUpdate with empty onCreate then success', () => {
test('onCreate and onUpdate both have physicalResourceId when physicalResourceId is specified in onUpdate, even when onCreate is unspecified', () => {
// GIVEN
const stack = new cdk.Stack();

Expand Down Expand Up @@ -460,7 +460,7 @@ describe('physicalResourceId patterns', () => {
});

// physicalResourceId pattern #8
test('physicalResourceId is not specified onCreate with empty onUpdate then fail', () => {
test('Omitting physicalResourceId in onCreate when onUpdate is undefined throws an error', () => {
// GIVEN
const stack = new cdk.Stack();

Expand All @@ -478,7 +478,7 @@ describe('physicalResourceId patterns', () => {
},
policy: AwsCustomResourcePolicy.fromSdkCalls({ resources: AwsCustomResourcePolicy.ANY_RESOURCE }),
});
}).toThrow(/`physicalResourceId` must be specified for onUpdate call when onCreate is omitted./);
}).toThrow(/'physicalResourceId' must be specified for 'onUpdate' call when 'onCreate' is omitted./);
});
});

Expand Down

0 comments on commit c198aeb

Please sign in to comment.