Skip to content

Commit a59e7f4

Browse files
committed
Remove inputs for deprecated outputs
1 parent 953de5a commit a59e7f4

2 files changed

Lines changed: 5 additions & 37 deletions

File tree

packages/cdkConstructs/src/constructs/SsmParametersConstruct.ts

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -33,16 +33,6 @@ export interface SsmParameterDefinition {
3333
* Value stored in the SSM parameter.
3434
*/
3535
readonly value: string
36-
/**
37-
* Optional export suffix for the output containing the parameter name.
38-
* @default nameSuffix value
39-
*/
40-
readonly outputExportSuffix?: string
41-
/**
42-
* Optional output description.
43-
* @default description value
44-
*/
45-
readonly outputDescription?: string
4636
}
4737

4838
/**
@@ -53,10 +43,6 @@ export interface SsmParameterDefinition {
5343
* @property parameters List of SSM parameters to create.
5444
* @property readPolicyDescription Description for the managed policy that grants
5545
* read access. Defaults to "Allows reading SSM parameters".
56-
* @property readPolicyOutputDescription Description for the output exporting the
57-
* managed policy ARN. Defaults to "Access to the parameters used by the integration".
58-
* @property readPolicyExportSuffix Export suffix for the output exporting the
59-
* managed policy ARN.
6046
*/
6147
export interface SsmParametersConstructProps {
6248
/**
@@ -72,15 +58,6 @@ export interface SsmParametersConstructProps {
7258
* @default "Allows reading SSM parameters"
7359
*/
7460
readonly readPolicyDescription?: string
75-
/**
76-
* Description for the output exporting the managed policy ARN.
77-
* @default "Access to the parameters used by the integration"
78-
*/
79-
readonly readPolicyOutputDescription?: string
80-
/**
81-
* Export suffix for the output exporting the managed policy ARN.
82-
*/
83-
readonly readPolicyExportSuffix: string
8461
}
8562

8663
/**

packages/cdkConstructs/tests/constructs/ssmParametersConstruct.test.ts

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -24,30 +24,22 @@ describe("SsmParametersConstruct", () => {
2424
id: "MockParam1",
2525
nameSuffix: "MockParam1",
2626
description: "Description for mock parameter 1",
27-
value: "mock-value-1",
28-
outputExportSuffix: "MockParam1Parameter",
29-
outputDescription: "Name of the SSM parameter holding MockParam1"
27+
value: "mock-value-1"
3028
},
3129
{
3230
id: "MockParam2",
3331
nameSuffix: "MockParam2",
3432
description: "Description for mock parameter 2",
35-
value: "mock-value-2",
36-
outputExportSuffix: "MockParam2Parameter",
37-
outputDescription: "Name of the SSM parameter holding MockParam2"
33+
value: "mock-value-2"
3834
},
3935
{
4036
id: "MockParam3",
4137
nameSuffix: "MockParam3",
4238
description: "Description for mock parameter 3",
43-
value: "mock-value-3",
44-
outputExportSuffix: "MockParam3Parameter",
45-
outputDescription: "Name of the SSM parameter holding MockParam3"
39+
value: "mock-value-3"
4640
}
4741
],
48-
readPolicyDescription: "Mock policy description",
49-
readPolicyOutputDescription: "Mock read policy output description",
50-
readPolicyExportSuffix: "MockGetParametersPolicy"
42+
readPolicyDescription: "Mock policy description"
5143
})
5244
// Sonarcloud complains that the construct is not used, so we add an assertion to sidestep that.
5345
assert(params, "SsmParametersConstruct should be created successfully")
@@ -114,8 +106,7 @@ describe("SsmParametersConstruct uses defaults when optional fields are omitted"
114106
description: "Mock SSM parameter description",
115107
value: "mock-value-1"
116108
}
117-
],
118-
readPolicyExportSuffix: "MockGetParametersPolicy"
109+
]
119110
})
120111
// Get sonar to shup up about the construct not being used
121112
assert(params, "SsmParametersConstruct should be created successfully")

0 commit comments

Comments
 (0)