From 1fbafc83669ff3137bbc1763d1b7669e161f72b4 Mon Sep 17 00:00:00 2001
From: Russell Cohen <rcoh@amazon.com>
Date: Thu, 27 Jul 2023 12:00:18 -0400
Subject: [PATCH] Fix default features test (#2881)

## Motivation and Context
<!--- Why is this change required? What problem does it solve? -->
<!--- If it fixes an open issue, please link to the issue here -->

## Description
The default features test is actually broken and failed during the
release dry run

## Testing
- ~Added the default features test to CI~ got a linker error


----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._
---
 aws/sdk/integration-tests/dynamodb/Cargo.toml               | 2 +-
 .../no-default-features/tests/client-construction.rs        | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/aws/sdk/integration-tests/dynamodb/Cargo.toml b/aws/sdk/integration-tests/dynamodb/Cargo.toml
index 3cd3a98b4..61663dbbe 100644
--- a/aws/sdk/integration-tests/dynamodb/Cargo.toml
+++ b/aws/sdk/integration-tests/dynamodb/Cargo.toml
@@ -16,7 +16,7 @@ aws-config = { path = "../../build/aws-sdk/sdk/aws-config" }
 aws-credential-types = { path = "../../build/aws-sdk/sdk/aws-credential-types", features = ["test-util"] }
 aws-http = { path = "../../build/aws-sdk/sdk/aws-http" }
 aws-sdk-dynamodb = { path = "../../build/aws-sdk/sdk/dynamodb" }
-aws-smithy-async = { path = "../../build/aws-sdk/sdk/aws-smithy-async" }
+aws-smithy-async = { path = "../../build/aws-sdk/sdk/aws-smithy-async", features = ["test-util"] }
 aws-smithy-client = { path = "../../build/aws-sdk/sdk/aws-smithy-client", features = ["test-util", "rustls"] }
 aws-smithy-http = { path = "../../build/aws-sdk/sdk/aws-smithy-http" }
 aws-smithy-protocol-test = { path = "../../build/aws-sdk/sdk/aws-smithy-protocol-test" }
diff --git a/aws/sdk/integration-tests/no-default-features/tests/client-construction.rs b/aws/sdk/integration-tests/no-default-features/tests/client-construction.rs
index ada59bc6f..26d84c96f 100644
--- a/aws/sdk/integration-tests/no-default-features/tests/client-construction.rs
+++ b/aws/sdk/integration-tests/no-default-features/tests/client-construction.rs
@@ -9,10 +9,10 @@ use aws_smithy_async::rt::sleep::AsyncSleep;
 use std::time::Duration;
 
 // This will fail due to lack of a connector when constructing the SDK Config
+// If this test doesn't panic, you may have accidentally unified features, resulting in
+// the connector being enabled transitively
 #[tokio::test]
-#[should_panic(
-    expected = "No HTTP connector was available. Enable the `rustls` crate feature or set a connector to fix this."
-)]
+#[should_panic(expected = "Enable the `rustls` crate feature or set a connector to fix this.")]
 async fn test_clients_from_sdk_config() {
     aws_config::load_from_env().await;
 }
-- 
GitLab