Skip to content

Commit

Permalink
fix: Temporarily disable smoke test codegen for S3Tables (#1840)
Browse files Browse the repository at this point in the history
Co-authored-by: Sichan Yoo <[email protected]>
  • Loading branch information
sichanyoo and Sichan Yoo authored Dec 5, 2024
1 parent 1af7b09 commit 76a27cc
Showing 1 changed file with 5 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ import software.amazon.smithy.swift.codegen.utils.toUpperCamelCase
class AWSSmokeTestGenerator(
private val ctx: ProtocolGenerator.GenerationContext
) : SmokeTestGenerator(ctx) {
// Filter out tests by name or tag at codegen time.
// Each element must have the prefix "<service-name>:" before the test name or tag name.
// Filter out tests by name, test tag, or service name at codegen time.
// Each test name or tag must have the prefix "<service-name>:" before the test name or tag name.
// E.g., "AWSS3:GetObjectTest" or "AWSS3:BucketTests"
override val smokeTestIdsToIgnore = setOf<String>(
// Add smoke test name to ignore here:
Expand All @@ -21,6 +21,9 @@ class AWSSmokeTestGenerator(
// Add smoke test tag to ignore here:
// E.g., "AWSACM:TagToIgnore",
)
override val servicesToIgnore = setOf<String>(
"AWSS3Tables" // TEMPORARY DISABLE UNTIL THEY FIX DUPLICATE TEST IDS (GetTable_AccessDeniedException)
)

override fun getServiceName(): String {
return "AWS" + ctx.service.getTrait(ServiceTrait::class.java).get().sdkId.toUpperCamelCase()
Expand Down

0 comments on commit 76a27cc

Please sign in to comment.