Skip to content

Commit 02a45b5

Browse files
authored
Fix broken smoke tests with customizations (#3195)
1 parent 3b98ba4 commit 02a45b5

File tree

5 files changed

+34
-27
lines changed

5 files changed

+34
-27
lines changed

build_tools/customizations.rb

Lines changed: 30 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -61,11 +61,9 @@ def dynamodb_example_deep_transform(subsegment, keys)
6161
subsegment
6262
end
6363
end
64-
6564
end
6665

6766
api('CloudFront') do |api|
68-
6967
api['shapes'].each do |_, shape|
7068
if shape['members'] && shape['members']['MaxItems'] && shape['members']['MaxItems']['shape'] == 'string'
7169
shape['members']['MaxItems']['shape'] = 'integer'
@@ -76,7 +74,6 @@ def dynamodb_example_deep_transform(subsegment, keys)
7674
symbolized = name.sub(/\d{4}_\d{2}_\d{2}$/, '')
7775
api['operations'][symbolized] = api['operations'].delete(name)
7876
end
79-
8077
end
8178

8279
api('EC2') do |api|
@@ -118,10 +115,6 @@ def dynamodb_example_deep_transform(subsegment, keys)
118115
api['shapes']['Timestamp']['type'] = 'timestamp'
119116
end
120117

121-
smoke('MTurk') do |smoke|
122-
smoke['testCases'] = []
123-
end
124-
125118
# Cross Region Copying
126119
%w[RDS Neptune DocDB].each do |service|
127120
operations = %w[
@@ -229,10 +222,6 @@ def dynamodb_example_deep_transform(subsegment, keys)
229222
end
230223
end
231224

232-
smoke('SMS') do |smoke|
233-
smoke['testCases'] = []
234-
end
235-
236225
api('SQS') do |api|
237226
api['metadata']['errorPrefix'] = 'AWS.SimpleQueueService.'
238227
api['shapes']['StringList']['flattened'] = true
@@ -247,5 +236,35 @@ def dynamodb_example_deep_transform(subsegment, keys)
247236
api['operations'][operation]['auth'] = ['smithy.api#noAuth']
248237
end
249238
end
239+
240+
smoke('DSQL') do |smoke|
241+
test = smoke['testCases'].find do |test_case|
242+
test_case['id'] == 'GetClusterNotFound'
243+
end
244+
test['input']['identifier'] = 'a' * 26
245+
end
246+
247+
smoke('ElasticTranscoder') do |smoke|
248+
# discontinued service
249+
smoke['testCases'] = []
250+
end
251+
252+
smoke('NetworkFlowMonitor') do |smoke|
253+
test = smoke['testCases'].find do |test_case|
254+
test_case['id'] == 'GetMonitorSuccess'
255+
end
256+
test['expectation'] = { 'failure' => {} }
257+
end
258+
259+
smoke('ObservabilityAdmin') do |smoke|
260+
test = smoke['testCases'].find do |test_case|
261+
test_case['id'] == 'GetTelemetryEvaluationStatusForOrganization'
262+
end
263+
test['expectation'] = { 'failure' => { 'errorId' => 'ValidationException' } }
264+
test = smoke['testCases'].find do |test_case|
265+
test_case['id'] == 'GetTelemetryEvaluationStatus'
266+
end
267+
test['expectation'] = { 'success' => {} }
268+
end
250269
end
251270
end

gems/aws-sdk-dsql/features/smoke.feature

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ Feature: Smoke tests for DSQL
1515
"""
1616
When I call the operation 'get_cluster' with params:
1717
"""
18-
{"identifier":"someIdentifier"}
18+
{"identifier":"aaaaaaaaaaaaaaaaaaaaaaaaaa"}
1919
"""
2020
Then I expect a 'Aws::DSQL::Errors::ResourceNotFoundException' was raised
2121

gems/aws-sdk-elastictranscoder/features/smoke.feature

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,3 @@
66
# WARNING ABOUT GENERATED CODE
77

88
Feature: Smoke tests for ElasticTranscoder
9-
10-
@elastictranscoder @smoke
11-
Scenario: ListPresetsSuccess
12-
Given I create a 'Aws::ElasticTranscoder' client with config:
13-
"""
14-
{"region":"us-west-2"}
15-
"""
16-
When I call the operation 'list_presets' with params:
17-
"""
18-
{}
19-
"""
20-
Then I expect an error was not raised

gems/aws-sdk-networkflowmonitor/features/smoke.feature

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,4 @@ Feature: Smoke tests for NetworkFlowMonitor
1717
"""
1818
{"monitor_name":"SmokeTestMonitor"}
1919
"""
20-
Then I expect an error was not raised
20+
Then I expect an error was raised

gems/aws-sdk-observabilityadmin/features/smoke.feature

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ Feature: Smoke tests for ObservabilityAdmin
1717
"""
1818
{}
1919
"""
20-
Then I expect a 'Aws::ObservabilityAdmin::Errors::AccessDeniedException' was raised
20+
Then I expect a 'Aws::ObservabilityAdmin::Errors::ValidationException' was raised
2121

2222
@observabilityadmin @smoke
2323
Scenario: GetTelemetryEvaluationStatus
@@ -29,4 +29,4 @@ Feature: Smoke tests for ObservabilityAdmin
2929
"""
3030
{}
3131
"""
32-
Then I expect a 'Aws::ObservabilityAdmin::Errors::AccessDeniedException' was raised
32+
Then I expect an error was not raised

0 commit comments

Comments
 (0)