Skip to content

Commit ba7bfa4

Browse files
committed
Re-enable more tests
1 parent 7918a4d commit ba7bfa4

File tree

4 files changed

+0
-20
lines changed

4 files changed

+0
-20
lines changed

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

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,6 @@ public abstract class AwsHttpSignerTestBase(
108108
return operation.context[HttpOperationContext.HttpCallList].last().request
109109
}
110110

111-
@IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
112111
@Test
113112
public fun testSignRequest(): TestResult = runTest {
114113
val op = buildOperation()
@@ -122,7 +121,6 @@ public abstract class AwsHttpSignerTestBase(
122121
assertEquals(expectedSig, signed.headers["Authorization"])
123122
}
124123

125-
@IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
126124
@Test
127125
public fun testUnsignedRequest(): TestResult = runTest {
128126
val op = buildOperation(unsigned = true)
@@ -136,7 +134,6 @@ public abstract class AwsHttpSignerTestBase(
136134
assertEquals(expectedSig, signed.headers["Authorization"])
137135
}
138136

139-
@IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
140137
@Test
141138
public fun testSignReplayableStreamingRequest(): TestResult = runTest {
142139
val op = buildOperation(streaming = true)
@@ -150,7 +147,6 @@ public abstract class AwsHttpSignerTestBase(
150147
assertEquals(expectedSig, signed.headers["Authorization"])
151148
}
152149

153-
@IgnoreNative // FIXME Re-enable after Kotlin/Native Implementation
154150
@Test
155151
public fun testSignAwsChunkedStreamNonReplayable(): TestResult = runTest {
156152
val op = buildOperation(streaming = true, replayable = false, requestBody = "a".repeat(AWS_CHUNKED_THRESHOLD + 1))
@@ -164,7 +160,6 @@ public abstract class AwsHttpSignerTestBase(
164160
assertEquals(expectedSig, signed.headers["Authorization"])
165161
}
166162

167-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
168163
@Test
169164
public fun testSignAwsChunkedStreamReplayable(): TestResult = runTest {
170165
val op = buildOperation(streaming = true, replayable = true, requestBody = "a".repeat(AWS_CHUNKED_THRESHOLD + 1))
@@ -178,7 +173,6 @@ public abstract class AwsHttpSignerTestBase(
178173
assertEquals(expectedSig, signed.headers["Authorization"])
179174
}
180175

181-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
182176
@Test
183177
public fun testSignOneShotStream(): TestResult = runTest {
184178
val op = buildOperation(streaming = true, replayable = false)

runtime/protocol/aws-event-stream/common/test/aws/smithy/kotlin/runtime/awsprotocol/eventstream/FrameEncoderTest.kt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import kotlin.test.assertContentEquals
1717
import kotlin.test.assertEquals
1818

1919
class FrameEncoderTest {
20-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
2120
@Test
2221
fun testEncode() = runTest {
2322
val expected = listOf(
@@ -44,7 +43,6 @@ class FrameEncoderTest {
4443
assertContentEquals(expected[2], actual[2].readByteArray())
4544
}
4645

47-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
4846
@Test
4947
fun testAsEventStreamHttpBody() = runTest {
5048
val messages = flowOf(

runtime/protocol/aws-event-stream/common/test/aws/smithy/kotlin/runtime/awsprotocol/eventstream/HeaderValueTest.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ import kotlin.test.assertFails
1616

1717
class HeaderValueTest {
1818

19-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
2019
@Test
2120
fun testExpectAs() {
2221
assertEquals(true, HeaderValue.Bool(true).expectBool())

runtime/protocol/aws-event-stream/common/test/aws/smithy/kotlin/runtime/awsprotocol/eventstream/MessageTest.kt

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ fun validMessageNoHeaders(): ByteArray = byteArrayFrom(
5353

5454
class MessageTest {
5555

56-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
5756
@Test
5857
fun testMessageNoHeaders() {
5958
// Test message taken from the CRT:
@@ -70,7 +69,6 @@ class MessageTest {
7069
assertEquals(expectedPayload, actual.payload.decodeToString())
7170
}
7271

73-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
7472
@Test
7573
fun testMessageOneHeader() {
7674
// Test message taken from the CRT:
@@ -92,7 +90,6 @@ class MessageTest {
9290
assertEquals(expectedHeaders, actual.headers)
9391
}
9492

95-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
9693
@Test
9794
fun testRoundTripAllHeadersPayload() {
9895
val encoded = validMessageWithAllHeaders()
@@ -122,7 +119,6 @@ class MessageTest {
122119
assertContentEquals(message.payload, result.payload)
123120
}
124121

125-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
126122
@Test
127123
fun testInvalidHeaderStringValueLength() {
128124
// header length = -1
@@ -144,7 +140,6 @@ class MessageTest {
144140
}.message.shouldContain("Invalid HeaderValue; type=STRING, len=65535")
145141
}
146142

147-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
148143
@Test
149144
fun testInvalidHeaderStringLengthCutoff() {
150145
val encoded = byteArrayFrom(
@@ -161,7 +156,6 @@ class MessageTest {
161156
}
162157
}
163158

164-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
165159
@Test
166160
fun testInvalidHeaderValueType() {
167161
val encoded = byteArrayFrom(
@@ -182,7 +176,6 @@ class MessageTest {
182176
}.message.shouldContain("Unknown HeaderType: 96")
183177
}
184178

185-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
186179
@Test
187180
fun testInvalidHeaderNameLength() {
188181
val encoded = byteArrayFrom(
@@ -203,7 +196,6 @@ class MessageTest {
203196
}.message.shouldContain("Invalid header name length")
204197
}
205198

206-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
207199
@Test
208200
fun testInvalidHeadersLength() {
209201
val encoded = byteArrayFrom(
@@ -220,7 +212,6 @@ class MessageTest {
220212
}.message.shouldContain("Not enough bytes to read header name; needed: 3; remaining: 1")
221213
}
222214

223-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
224215
@Test
225216
fun testInvalidPreludeChecksum() {
226217
val encoded = byteArrayFrom(
@@ -241,7 +232,6 @@ class MessageTest {
241232
}.message.shouldContain("Prelude checksum mismatch; expected=0xdeadbeef; calculated=0x8bb495fb")
242233
}
243234

244-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
245235
@Test
246236
fun testInvalidMessageChecksum() {
247237
val encoded = byteArrayFrom(
@@ -262,7 +252,6 @@ class MessageTest {
262252
}.message.shouldContain("Message checksum mismatch; expected=0xdeadbeef; calculated=0x01a05860")
263253
}
264254

265-
@IgnoreNative // FIXME Re-enable after Kotlin/Native implementation
266255
@Test
267256
fun testInvalidHeaderNameLengthTooLong() {
268257
val encoded = byteArrayFrom(

0 commit comments

Comments
 (0)