Skip to content

Commit 71a0045

Browse files
authored
feat!: Remove unknown case from allCases in int & string enums (#742)
1 parent 16eb445 commit 71a0045

File tree

8 files changed

+10
-20
lines changed

8 files changed

+10
-20
lines changed

Package.swift

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,7 @@ let package = Package(
6363
.target(
6464
name: "SmithyRetries",
6565
dependencies: [
66+
"SmithyRetriesAPI",
6667
.product(name: "AwsCommonRuntimeKit", package: "aws-crt-swift"),
6768
]
6869
),

Sources/WeatherSDK/models/SimpleYesNo.swift

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ extension WeatherClientTypes {
1212
public static var allCases: [SimpleYesNo] {
1313
return [
1414
.no,
15-
.yes,
16-
.sdkUnknown("")
15+
.yes
1716
]
1817
}
1918

Sources/WeatherSDK/models/TypedYesNo.swift

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ extension WeatherClientTypes {
1212
public static var allCases: [TypedYesNo] {
1313
return [
1414
.no,
15-
.yes,
16-
.sdkUnknown("")
15+
.yes
1716
]
1817
}
1918

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,6 @@ class EnumGenerator(
193193
}
194194

195195
fun generateAllCasesBlock() {
196-
allCasesBuilder.add(".sdkUnknown(\"\")")
197196
writer.openBlock("public static var allCases: [\$enum.name:L] {", "}") {
198197
writer.openBlock("return [", "]") {
199198
writer.write(allCasesBuilder.joinToString(",\n"))

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,6 @@ class IntEnumGenerator(
9191
}
9292

9393
fun generateAllCasesBlock() {
94-
allCasesBuilder.add(".sdkUnknown(0)")
9594
writer.openBlock("public static var allCases: [\$enum.name:L] {", "}") {
9695
writer.openBlock("return [", "]") {
9796
writer.write(allCasesBuilder.joinToString(",\n"))

smithy-swift-codegen/src/test/kotlin/EnumGeneratorTests.kt

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,7 @@ public enum MyEnum: Swift.Equatable, Swift.RawRepresentable, Swift.CaseIterable,
4848
public static var allCases: [MyEnum] {
4949
return [
5050
.bar,
51-
.fooBazXap,
52-
.sdkUnknown("")
51+
.fooBazXap
5352
]
5453
}
5554
@@ -108,8 +107,7 @@ public enum MyEnum: Swift.Equatable, Swift.RawRepresentable, Swift.CaseIterable,
108107
public static var allCases: [MyEnum] {
109108
return [
110109
.t2Micro,
111-
.t2Nano,
112-
.sdkUnknown("")
110+
.t2Nano
113111
]
114112
}
115113
@@ -155,8 +153,7 @@ extension ExampleClientTypes {
155153
.club,
156154
.diamond,
157155
.heart,
158-
.spade,
159-
.sdkUnknown("")
156+
.spade
160157
]
161158
}
162159

smithy-swift-codegen/src/test/kotlin/IntEnumGeneratorTests.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ public enum Abcs: Swift.Equatable, Swift.RawRepresentable, Swift.CaseIterable, S
2626
return [
2727
.a,
2828
.b,
29-
.c,
30-
.sdkUnknown(0)
29+
.c
3130
]
3231
}
3332

smithy-swift-codegen/src/test/kotlin/ReservedWordsGeneratorTests.kt

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ extension ExampleClientTypes {
2626
.any,
2727
.open,
2828
.self,
29-
.protocol,
30-
.sdkUnknown("")
29+
.protocol
3130
]
3231
}
3332
@@ -66,8 +65,7 @@ extension ExampleClientTypes {
6665
public static var allCases: [ModelType] {
6766
return [
6867
.foo,
69-
.test,
70-
.sdkUnknown("")
68+
.test
7169
]
7270
}
7371
@@ -104,8 +102,7 @@ extension ExampleClientTypes {
104102
public static var allCases: [ModelProtocol] {
105103
return [
106104
.bar,
107-
.foo,
108-
.sdkUnknown("")
105+
.foo
109106
]
110107
}
111108

0 commit comments

Comments
 (0)