Revert temporarily commented logic in PrepareRelease. Fix client name bug. Add convenience script for running every smoke test.

Author: Sichan Yoo

AWSSDKSwiftCLI/Sources/AWSSDKSwiftCLI/Commands/AWSSDKSwiftCLI/Subcommands/PrepareRelease.swift

@@ -81,16 +81,16 @@ struct PrepareRelease {
         try FileManager.default.changeWorkingDirectory(repoPath)
 
         let previousVersion = try getPreviousVersion()
-//        guard try repoHasChanges(previousVersion) else {
-//            /// If repo has no changes, create an empty release-manifest.json file.
-//            /// Empty manifest file makes GitHubReleasePublisher be no-op.
-//            /// The manifest file is required regardless of whether there should
-//            /// be a release or not.
-//            try createEmptyReleaseManifest()
-//            /// Return without creating new commit or tag in local repos.
-//            /// This makes GitPublisher be no-op.
-//            return
-//        }
+        guard try repoHasChanges(previousVersion) else {
+            /// If repo has no changes, create an empty release-manifest.json file.
+            /// Empty manifest file makes GitHubReleasePublisher be no-op.
+            /// The manifest file is required regardless of whether there should
+            /// be a release or not.
+            try createEmptyReleaseManifest()
+            /// Return without creating new commit or tag in local repos.
+            /// This makes GitPublisher be no-op.
+            return
+        }
         let newVersion = try createNewVersion(previousVersion)
 
         try stageFiles()

codegen/smithy-aws-swift-codegen/src/main/kotlin/software/amazon/smithy/aws/swift/codegen/AWSSmokeTestGenerator.kt

@@ -13,11 +13,11 @@ class AWSSmokeTestGenerator(
     // 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.
     // E.g., "AWSS3:GetObjectTest" or "AWSS3:BucketTests"
-    override val testIdsToIgnore = setOf<String>(
+    override val smokeTestIdsToIgnore = setOf<String>(
         // Add smoke test name to ignore here:
         // E.g., "AWSACM:GetCertificateFailure",
     )
-    override val testTagsToIgnore = setOf<String>(
+    override val smokeTestTagsToIgnore = setOf<String>(
         // Add smoke test tag to ignore here:
         // E.g., "AWSACM:TagToIgnore",
     )
@@ -27,7 +27,7 @@ class AWSSmokeTestGenerator(
     }
 
     override fun getClientName(): String {
-        return ctx.service.getTrait(ServiceTrait::class.java).get().sdkId.toUpperCamelCase() + "Client"
+        return ctx.service.getTrait(ServiceTrait::class.java).get().sdkId.toUpperCamelCase().removeSuffix("Service") + "Client"
     }
 
     override fun renderCustomFilePrivateVariables(writer: SwiftWriter) {

scripts/run-every-smoke-test.sh

@@ -0,0 +1,46 @@
+#!/bin/bash
+
+# This is a convenience script for developers for running every smoke test under SmokeTests/.
+# The script must be run from aws-sdk-swift/, the directory containing SmokeTests/.
+
+# cd into test module dir
+cd SmokeTests/ || { echo "ERROR: Failed to change directory to SmokeTests."; exit 1; }
+
+# Build and discard output for clean log
+echo "INFO: Building SmokeTests module..."
+swift build > /dev/null 2>&1
+
+# Header print helpers
+print_header() {
+  print_spacer
+  local header=$1
+  echo "##### $header #####"
+  print_spacer
+}
+
+print_spacer() {
+  echo ""
+}
+
+# Build and run each and every test runner; save result to results array
+print_header "TEST RUNS"
+results=()
+for runnerName in ./*; do
+    if [ -d "$runnerName" ]; then
+      swift run "${runnerName#./}"
+      if [ $? -eq 0 ]; then
+        # Record success
+        results+=("SUCCESS: ${runnerName#./}")
+      else
+        # record failure
+        results+=("FAILURE: ${runnerName#./}")
+      fi
+      print_spacer
+    fi
+done
+
+# Print result summary
+print_header "TEST RESULT SUMMARY"
+for result in "${results[@]}"; do
+  echo "$result"
+done