kn: enable many @IgnoreNative tests

.brazil.json

@@ -2,15 +2,15 @@
   "dependencies": {
     "org.jetbrains.kotlin:kotlin-stdlib:2.*": "KotlinStdlib-2.x",
     "org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.*": "KotlinxCoroutinesCoreJvm-1.x",
-
     "com.squareup.okhttp3:okhttp-coroutines:5.*": "OkHttp3Coroutines-5.x",
     "com.squareup.okhttp3:okhttp:5.*": "OkHttp3-5.x",
     "com.squareup.okio:okio-jvm:3.*": "OkioJvm-3.x",
     "io.opentelemetry:opentelemetry-api:1.*": "Maven-io-opentelemetry_opentelemetry-api-1.x",
     "org.slf4j:slf4j-api:2.*": "Maven-org-slf4j_slf4j-api-2.x",
     "aws.sdk.kotlin.crt:aws-crt-kotlin:0.9.*": "AwsCrtKotlin-0.9.x",
     "aws.sdk.kotlin.crt:aws-crt-kotlin:0.8.*": "AwsCrtKotlin-0.8.x",
-    "com.squareup.okhttp3:okhttp:4.*": "OkHttp3-4.x"
+    "com.squareup.okhttp3:okhttp:4.*": "OkHttp3-4.x",
+    "org.jetbrains.kotlinx:kotlinx-datetime-jvm:0.*": "KotlinxDatetimeJvm-0.x"
   },
   "packageHandlingRules": {
     "versioning": {

.changes/4820850c-8916-47f5-a7e1-8880e6a00d22.json

@@ -0,0 +1,5 @@
+{
+    "id": "4820850c-8916-47f5-a7e1-8880e6a00d22",
+    "type": "bugfix",
+    "description": "Fix errors in equality checks for `CaseInsensitiveMap` which affect `Headers` and `ValuesMap` implementations"
+}

.changes/c0040355-ffdc-4813-80e9-baf859ef02b9.json

@@ -0,0 +1,5 @@
+{
+    "id": "c0040355-ffdc-4813-80e9-baf859ef02b9",
+    "type": "bugfix",
+    "description": "fix: correct hash code calculation for case-insensitive map entries"
+}

.github/workflows/artifact-size-metrics.yml

@@ -31,6 +31,8 @@ jobs:
         with:
           role-to-assume: ${{ secrets.CI_AWS_ROLE_ARN }}
           aws-region: us-west-2
+      - name: Configure Gradle
+        uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
       - name: Generate Artifact Size Metrics
         run: ./gradlew artifactSizeMetrics
       - name: Save Artifact Size Metrics
@@ -54,6 +56,8 @@ jobs:
         with:
           role-to-assume: ${{ secrets.CI_AWS_ROLE_ARN }}
           aws-region: us-west-2
+      - name: Configure Gradle
+        uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
       - name: Generate Artifact Size Metrics
         run: ./gradlew -Paws.kotlin.native=false artifactSizeMetrics
       - name: Analyze Artifact Size Metrics

.github/workflows/continuous-integration.yml

@@ -89,6 +89,11 @@ jobs:
         with:
           path: 'smithy-kotlin'
 
+      - name: Configure Gradle
+        uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
+        with:
+          working-directory: 'smithy-kotlin'
+
       - name: Setup build
         uses: ./smithy-kotlin/.github/actions/setup-build
 
@@ -152,12 +157,10 @@ jobs:
       - name: Setup build
         uses: ./smithy-kotlin/.github/actions/setup-build
 
-      - name: Configure JDK
-        uses: actions/setup-java@v3
+      - name: Configure Gradle
+        uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
         with:
-          distribution: 'corretto'
-          java-version: 17
-          cache: 'gradle'
+          working-directory: 'smithy-kotlin'
 
       - name: Test
         working-directory: ./smithy-kotlin
@@ -186,6 +189,11 @@ jobs:
       - name: Setup build
         uses: ./smithy-kotlin/.github/actions/setup-build
 
+      - name: Configure Gradle
+        uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
+        with:
+          working-directory: 'smithy-kotlin'
+
       - name: Test
         working-directory: ./smithy-kotlin
         shell: bash
@@ -211,6 +219,16 @@ jobs:
         path: 'aws-sdk-kotlin'
         repository: 'awslabs/aws-sdk-kotlin'
 
+    - name: Configure Gradle - smithy-kotlin
+      uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
+      with:
+        working-directory: ./smithy-kotlin
+
+    - name: Configure Gradle - aws-sdk-kotlin
+      uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
+      with:
+        working-directory: ./aws-sdk-kotlin
+
     - name: Build and Test aws-sdk-kotlin downstream
       working-directory: ./smithy-kotlin
       run: |

.github/workflows/kat-transform.yml

@@ -38,6 +38,11 @@ jobs:
           role-to-assume: ${{ secrets.CI_AWS_ROLE_ARN }}
           aws-region: us-west-2
 
+      - name: Configure Gradle
+        uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
+        with:
+          working-directory: ./smithy-kotlin
+
       - name: Setup kat
         uses: awslabs/aws-kotlin-repo-tools/.github/actions/setup-kat@main
 

.github/workflows/lint.yml

@@ -21,6 +21,8 @@ jobs:
     steps:
     - name: Checkout sources
       uses: actions/checkout@v2
+    - name: Configure Gradle
+      uses: awslabs/aws-kotlin-repo-tools/.github/actions/configure-gradle@main
     - name: Lint ${{ env.PACKAGE_NAME }}
       run: |
         ./gradlew ktlint

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.12.1-bin.zip
+distributionUrl=https://services.gradle.org/distributions/gradle-8.12.1-bin.zip
 networkTimeout=10000
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

runtime/auth/aws-credentials/common/test/aws/smithy/kotlin/runtime/auth/awscredentials/CachedCredentialsProviderTest.kt

@@ -5,7 +5,6 @@
 
 package aws.smithy.kotlin.runtime.auth.awscredentials
 
-import aws.smithy.kotlin.runtime.IgnoreNative
 import aws.smithy.kotlin.runtime.collections.Attributes
 import aws.smithy.kotlin.runtime.time.Instant
 import aws.smithy.kotlin.runtime.time.ManualClock
@@ -37,7 +36,6 @@ class CachedCredentialsProviderTest {
         }
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testLoadFirstCall() = runTest {
         // explicit expiration
@@ -52,7 +50,6 @@ class CachedCredentialsProviderTest {
         assertEquals(1, source.callCount)
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testDefaultExpiration() = runTest {
         // expiration should come from the cached provider
@@ -65,7 +62,6 @@ class CachedCredentialsProviderTest {
         assertEquals(1, source.callCount)
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
     @Test
     fun testReloadExpiredCredentials() = runTest {
         val source = TestCredentialsProvider(expiration = testExpiration)
@@ -81,7 +77,6 @@ class CachedCredentialsProviderTest {
         assertEquals(2, source.callCount)
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
     @Test
     fun testRefreshBufferWindow() = runTest {
         val source = TestCredentialsProvider(expiration = testExpiration)
@@ -103,7 +98,6 @@ class CachedCredentialsProviderTest {
         assertEquals(2, source.callCount)
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
     @Test
     fun testLoadFailed() = runTest {
         val source = object : CredentialsProvider {
@@ -126,7 +120,6 @@ class CachedCredentialsProviderTest {
         provider.resolve()
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
     @Test
     fun testItThrowsOnGetCredentialsAfterClose() = runTest {
         val source = TestCredentialsProvider(expiration = testExpiration)
@@ -144,7 +137,6 @@ class CachedCredentialsProviderTest {
         assertEquals(1, source.callCount)
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
     @Test
     fun testCachedConvenienceFunction() = runTest {
         val source = TestCredentialsProvider(expiration = testExpiration)

runtime/auth/aws-signing-common/common/test/aws/smithy/kotlin/runtime/auth/awssigning/PresignerTest.kt

@@ -4,7 +4,6 @@
  */
 package aws.smithy.kotlin.runtime.auth.awssigning
 
-import aws.smithy.kotlin.runtime.IgnoreNative
 import aws.smithy.kotlin.runtime.auth.awscredentials.Credentials
 import aws.smithy.kotlin.runtime.auth.awscredentials.CredentialsProvider
 import aws.smithy.kotlin.runtime.client.endpoints.Endpoint
@@ -25,14 +24,12 @@ import kotlin.test.assertTrue
 class PresignerTest {
     // Verify that custom endpoint URL schemes aren't changed.
     // See https://github.com/awslabs/aws-sdk-kotlin/issues/938
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testSignedUrlAllowsHttp() = testSigningUrl("http://localhost:8080/path/to/resource?foo=bar")
 
     // Verify that custom endpoint URL schemes aren't changed.
     // See https://github.com/awslabs/aws-sdk-kotlin/issues/938
     @Test
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     fun testSignedUrlAllowsHttps() = testSigningUrl("https://localhost:8088/path/to/resource?bar=foo")
 
     private fun testSigningUrl(url: String) = runTest {

runtime/auth/aws-signing-default/common/test/aws/smithy/kotlin/runtime/auth/awssigning/DefaultCanonicalizerTest.kt

@@ -4,7 +4,6 @@
  */
 package aws.smithy.kotlin.runtime.auth.awssigning
 
-import aws.smithy.kotlin.runtime.IgnoreNative
 import aws.smithy.kotlin.runtime.auth.awscredentials.Credentials
 import aws.smithy.kotlin.runtime.auth.awssigning.tests.DEFAULT_TEST_CREDENTIALS
 import aws.smithy.kotlin.runtime.http.Headers
@@ -23,7 +22,6 @@ import kotlin.test.assertEquals
 
 class DefaultCanonicalizerTest {
     // Test adapted from https://docs.aws.amazon.com/general/latest/gr/sigv4-create-canonical-request.html
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testCanonicalize() = runTest {
         val request = HttpRequest {
@@ -84,7 +82,6 @@ class DefaultCanonicalizerTest {
     }
 
     // Targeted test for proper URI path escaping. See https://github.com/smithy-lang/smithy-kotlin/issues/657
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testEscapablePath() {
         val uri = Url.Builder()
@@ -101,7 +98,6 @@ class DefaultCanonicalizerTest {
         assertEquals("/2013-04-01/healthcheck/foo%253Cbar%253Ebaz%253C%252Fbar%253E", uri.canonicalPath(config))
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testCanonicalPath() {
         val config = AwsSigningConfig {
@@ -117,7 +113,6 @@ class DefaultCanonicalizerTest {
         assertEquals("/foo/%40bar/baz%253Cqux%253Aquux", uri.canonicalPath(config))
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testCanonicalQueryParams() {
         Url.Builder().apply {
@@ -132,7 +127,6 @@ class DefaultCanonicalizerTest {
         }
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testUnsignedHeaders() = runTest {
         val request = HttpRequest {
@@ -166,7 +160,6 @@ class DefaultCanonicalizerTest {
         assertEquals(expectedSignedHeaders, actual.signedHeaders)
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testCustomPort() = runTest {
         val request = HttpRequest {

runtime/auth/aws-signing-default/common/test/aws/smithy/kotlin/runtime/auth/awssigning/DefaultRequestMutatorTest.kt

@@ -4,7 +4,6 @@
  */
 package aws.smithy.kotlin.runtime.auth.awssigning
 
-import aws.smithy.kotlin.runtime.IgnoreNative
 import aws.smithy.kotlin.runtime.auth.awscredentials.Credentials
 import aws.smithy.kotlin.runtime.http.Headers
 import aws.smithy.kotlin.runtime.http.HttpBody
@@ -16,7 +15,6 @@ import kotlin.test.Test
 import kotlin.test.assertEquals
 
 class DefaultRequestMutatorTest {
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
     @Test
     fun testAppendAuthHeader() {
         val canonical = CanonicalRequest(baseRequest.toBuilder(), "", "action;host;x-amz-date", "")

runtime/auth/aws-signing-default/common/test/aws/smithy/kotlin/runtime/auth/awssigning/DefaultSignatureCalculatorTest.kt

@@ -4,7 +4,6 @@
  */
 package aws.smithy.kotlin.runtime.auth.awssigning
 
-import aws.smithy.kotlin.runtime.IgnoreNative
 import aws.smithy.kotlin.runtime.auth.awscredentials.Credentials
 import aws.smithy.kotlin.runtime.auth.awssigning.tests.DEFAULT_TEST_CREDENTIALS
 import aws.smithy.kotlin.runtime.hashing.sha256
@@ -17,7 +16,6 @@ import kotlin.test.assertEquals
 
 class DefaultSignatureCalculatorTest {
     // Test adapted from https://docs.aws.amazon.com/general/latest/gr/sigv4-calculate-signature.html
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testCalculate() {
         val signingKey = "c4afb1cc5771d871763a393e44b703571b55cc28424d1a5e86da6ed3c154a4b9".decodeHexBytes()
@@ -34,7 +32,6 @@ class DefaultSignatureCalculatorTest {
     }
 
     // Test adapted from https://docs.aws.amazon.com/general/latest/gr/sigv4-calculate-signature.html
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testSigningKey() = runTest {
         val config = AwsSigningConfig {
@@ -50,7 +47,6 @@ class DefaultSignatureCalculatorTest {
     }
 
     // Test adapted from https://docs.aws.amazon.com/general/latest/gr/sigv4-create-string-to-sign.html
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testStringToSign() {
         val canonicalRequest = """
@@ -84,7 +80,6 @@ class DefaultSignatureCalculatorTest {
 
     private data class ChunkStringToSignTest(val signatureType: AwsSignatureType, val expectedNonSignatureHeaderHash: String)
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     fun testChunkStringToSign() {
         // Test event stream signing

runtime/auth/http-auth-aws/common/test/aws/smithy/kotlin/runtime/http/auth/AwsHttpSignerTestBase.kt

@@ -108,7 +108,6 @@ public abstract class AwsHttpSignerTestBase(
         return operation.context[HttpOperationContext.HttpCallList].last().request
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     public fun testSignRequest(): TestResult = runTest {
         val op = buildOperation()
@@ -122,7 +121,6 @@ public abstract class AwsHttpSignerTestBase(
         assertEquals(expectedSig, signed.headers["Authorization"])
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
     @Test
     public fun testUnsignedRequest(): TestResult = runTest {
         val op = buildOperation(unsigned = true)
@@ -136,7 +134,7 @@ public abstract class AwsHttpSignerTestBase(
         assertEquals(expectedSig, signed.headers["Authorization"])
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
+    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation.
     @Test
     public fun testSignReplayableStreamingRequest(): TestResult = runTest {
         val op = buildOperation(streaming = true)
@@ -150,7 +148,7 @@ public abstract class AwsHttpSignerTestBase(
         assertEquals(expectedSig, signed.headers["Authorization"])
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
+    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation.
     @Test
     public fun testSignAwsChunkedStreamNonReplayable(): TestResult = runTest {
         val op = buildOperation(streaming = true, replayable = false, requestBody = "a".repeat(AWS_CHUNKED_THRESHOLD + 1))
@@ -164,7 +162,7 @@ public abstract class AwsHttpSignerTestBase(
         assertEquals(expectedSig, signed.headers["Authorization"])
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
+    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation.
     @Test
     public fun testSignAwsChunkedStreamReplayable(): TestResult = runTest {
         val op = buildOperation(streaming = true, replayable = true, requestBody = "a".repeat(AWS_CHUNKED_THRESHOLD + 1))
@@ -178,7 +176,7 @@ public abstract class AwsHttpSignerTestBase(
         assertEquals(expectedSig, signed.headers["Authorization"])
     }
 
-    @IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
+    @IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation.
     @Test
     public fun testSignOneShotStream(): TestResult = runTest {
         val op = buildOperation(streaming = true, replayable = false)