Gardening: Migrate test suite to GH issues: Interpreter

AnthonyLatsis · AnthonyLatsis · commit af660968473a · 2022-09-01T06:35:57.000+03:00
diff --git a/test/Interpreter/SDK/objc_swift_getObjectType.swift b/test/Interpreter/SDK/objc_swift_getObjectType.swift
@@ -24,8 +24,8 @@ extension P {
   }
 }
 
-// SR-4363
-// Test several paths through swift_getObjectType() 
+// https://github.com/apple/swift/issues/46942
+// Test several paths through 'swift_getObjectType()'.
 
 // instance of a Swift class that conforms to P
 class O: NSObject, P { }
diff --git a/test/Interpreter/SR-10600.swift b/test/Interpreter/SR-10600.swift
@@ -2,6 +2,8 @@
 
 // REQUIRES: executable_test
 
+// https://github.com/apple/swift/issues/53000
+
 public class BaseView { }
 public class GenericView<T>: BaseView { }
 public class FinalView: GenericView<ContentForTheView> { }
diff --git a/test/Interpreter/class_resilience.swift b/test/Interpreter/class_resilience.swift
@@ -324,8 +324,9 @@ ResilientClassTestSuite.test("ResilientThenEmpty") {
   expectEqual(c.resilient.i, 17)
 }
 
-// This test triggers SR-815 (rdar://problem/25318716) on macOS 10.9 and iOS 7.
-// Disable it for now when testing on those versions.
+// This test triggers https://github.com/apple/swift/issues/43427
+// (rdar://problem/25318716) on macOS 10.9 and iOS 7. Disable it for now when
+// testing on those versions.
 if #available(macOS 10.10, iOS 8, *) {
   runAllTests()
 } else {
diff --git a/test/Interpreter/classes.swift b/test/Interpreter/classes.swift
@@ -208,7 +208,7 @@ func makeOne<T : Makeable>(_: T.Type) -> T {
 
 makeOne(Child.self).doSomething() // CHECK: Heaven!
 
-// https://bugs.swift.org/browse/SR-3840
+// https://github.com/apple/swift/issues/46425
 
 class BaseProperty {
   var value: Int {
diff --git a/test/Interpreter/enum.swift b/test/Interpreter/enum.swift
@@ -554,7 +554,8 @@ presentEitherOr(EitherOr<(), String>.Right("foo")) // CHECK-NEXT: Right(foo)
 // CHECK-NEXT: Right(foo)
 presentEitherOrsOf(t: (), u: "foo")
 
-// SR-5148
+// https://github.com/apple/swift/issues/47724
+
 enum Payload {
     case email
 }
diff --git a/test/Interpreter/generic_casts.swift b/test/Interpreter/generic_casts.swift
@@ -308,15 +308,17 @@ print(#line, nongenericAnyIsPAndPCSubType(type: PS.self)) // CHECK: [[@LINE]] fa
 print(#line, genericAnyIs(type: PS.self, to: (P & PCSub).self, expected: false)) // CHECK: [[@LINE]] false
 print(#line, nongenericAnyIsPAndPCSubType(type: PE.self)) // CHECK: [[@LINE]] false
 print(#line, genericAnyIs(type: PE.self, to: (P & PCSub).self, expected: false)) // CHECK: [[@LINE]] false
-//print(#line, nongenericAnyIsPAndPCSubType(type: PC.self)) // CHECK-SR-11565: [[@LINE]] false -- FIXME: reenable this when SR-11565 is fixed
+// FIXME: reenable this when https://github.com/apple/swift/issues/53970 is fixed
+//print(#line, nongenericAnyIsPAndPCSubType(type: PC.self)) // C HECK: [[@LINE]] false
 print(#line, genericAnyIs(type: PC.self, to: (P & PCSub).self, expected: false)) // CHECK: [[@LINE]] false
 print(#line, nongenericAnyIsPAndPCSubType(type: PCSub.self)) // CHECK: [[@LINE]] true
 print(#line, genericAnyIs(type: PCSub.self, to: (P & PCSub).self, expected: false)) // CHECK: [[@LINE]] false
 print(#line, nongenericAnyAsConditionalPAndPCSubType(type: PS.self)) // CHECK: [[@LINE]] false
 print(#line, genericAnyAsConditional(type: PS.self, to: (P & PCSub).self, expected: false)) // CHECK: [[@LINE]] false
 print(#line, nongenericAnyAsConditionalPAndPCSubType(type: PE.self)) // CHECK: [[@LINE]] false
 print(#line, genericAnyAsConditional(type: PE.self, to: (P & PCSub).self, expected: false)) // CHECK: [[@LINE]] false
-// print(#line, nongenericAnyAsConditionalPAndPCSubType(type: PC.self)) // CHECK-SR-11565: [[@LINE]] false -- FIXME: reenable this when SR-11565 is fixed
+// FIXME: reenable this when https://github.com/apple/swift/issues/53970 is fixed
+// print(#line, nongenericAnyAsConditionalPAndPCSubType(type: PC.self)) // C HECK: [[@LINE]] false
 print(#line, genericAnyAsConditional(type: PC.self, to: (P & PCSub).self, expected: false)) // CHECK: [[@LINE]] false
 print(#line, nongenericAnyAsConditionalPAndPCSubType(type: PCSub.self)) // CHECK: [[@LINE]] true
 print(#line, genericAnyAsConditional(type: PCSub.self, to: (P & PCSub).self, expected: false)) // CHECK: [[@LINE]] false
diff --git a/test/Interpreter/generic_class.swift b/test/Interpreter/generic_class.swift
@@ -184,8 +184,9 @@ class MoreConcreteQuadruple : SemiConcreteTriple<State> {
   }
 }
 
-// This check triggers SR-815 (rdar://problem/25318716) on macOS 10.9 and iOS 7.
-// Disable it for now when testing on those versions.
+// This check triggers https://github.com/apple/swift/issues/43427
+// (rdar://problem/25318716) on macOS 10.9 and iOS 7. Disable it for now when
+// testing on those versions.
 if #available(macOS 10.10, iOS 8, *) {
   var u = MoreConcreteQuadruple(10, 17, State.CA, "Hella")
 
diff --git a/test/Interpreter/generic_objc_subclass.swift b/test/Interpreter/generic_objc_subclass.swift
@@ -234,8 +234,9 @@ fixedB.third = 17
 // CHECK: (101, 0, 0, 0, 16, [19, 84], 17)
 print(fixedG())
 
-// Problem with field alignment in direct generic subclass of NSObject -
-// <https://bugs.swift.org/browse/SR-2586>
+// https://github.com/apple/swift/issues/45191
+// Problem with field alignment in direct generic subclass of 'NSObject'.
+
 public class PandorasBox<T>: NSObject {
     final public var value: T
 
diff --git a/test/Interpreter/lifetime_nonmutating_address_only.swift b/test/Interpreter/lifetime_nonmutating_address_only.swift
@@ -1,7 +1,7 @@
 // RUN: %target-run-simple-swift | %FileCheck %s
 // REQUIRES: executable_test
 
-// SR-8990
+// https://github.com/apple/swift/issues/51493
 
 // CHECK: A
 // CHECK: B
diff --git a/test/Interpreter/protocol_extensions.swift b/test/Interpreter/protocol_extensions.swift
@@ -284,7 +284,8 @@ ProtocolExtensionTestSuite.test("ClassInitializer") {
   expectTrue(Sub.self == metatypes[3].1)
 }
 
-// https://bugs.swift.org/browse/SR-617
+// https://github.com/apple/swift/issues/43234
+
 protocol SelfMetadataTest {
   associatedtype T = Int
 
diff --git a/test/Interpreter/resilient_metadata_cycles.swift b/test/Interpreter/resilient_metadata_cycles.swift
@@ -20,7 +20,8 @@ import resilient_class
 
 var ResilientMetadataCycleTests = TestSuite("Resilient metadata cycle tests")
 
-// SR-7876
+// https://github.com/apple/swift/issues/50411
+
 enum test0_Node {
   case link(size: Size, children: [test0_Node])
 
@@ -29,7 +30,7 @@ enum test0_Node {
   }
 }
 
-ResilientMetadataCycleTests.test("SR-7876") {
+ResilientMetadataCycleTests.test("https://github.com/apple/swift/issues/50411") {
   _ = test0_Node.test()
 }
 
diff --git a/test/Interpreter/sr10941.swift b/test/Interpreter/sr10941.swift
@@ -4,7 +4,7 @@
 // RUN: %target-run %t/a.out
 // REQUIRES: executable_test
 
-// https://bugs.swift.org/browse/SR-10941
+// https://github.com/apple/swift/issues/53332
 
 public protocol SupportedPropertyType { }
 
diff --git a/test/Interpreter/subclass_existentials.swift b/test/Interpreter/subclass_existentials.swift
@@ -458,8 +458,9 @@ SubclassExistentialsTestSuite.test("Failing dynamic downcast to subclass existen
   }
 }
 
-// This test triggers SR-815 (rdar://problem/25318716) on macOS 10.9 and iOS 7.
-// Disable it for now when testing on those versions.
+// This test triggers https://github.com/apple/swift/issues/43427
+// (rdar://problem/25318716) on macOS 10.9 and iOS 7. Disable it for now when
+// testing on those versions.
 if #available(macOS 10.10, iOS 8, *) {
   runAllTests()
 } else {
diff --git a/test/Interpreter/subscripting.swift b/test/Interpreter/subscripting.swift
@@ -27,7 +27,7 @@ print(Foo[]) // CHECK: static subscript
 print(Foo.subscript()) // CHECK: static func
 
 
-// SR-7418
+// https://github.com/apple/swift/issues/49961
 
 protocol P {
   subscript<T : Y>(_: T) -> Int { get set }
diff --git a/test/Interpreter/throwing_initializers.swift b/test/Interpreter/throwing_initializers.swift
@@ -409,7 +409,9 @@ ThrowingInitTestSuite.test("InitFailure_Struct") {
 
 // Specific regression tests:
 
-// <https://bugs.swift.org/browse/SR-1714> - try? self.init(...)` in init? causes overrelease
+// https://github.com/apple/swift/issues/44323
+// 'try? self.init(...)' in 'init?' causes overrelease
+
 class ThrowAndFailRoot {
   let x = LifetimeTracked(0)
 }
@@ -437,6 +439,7 @@ ThrowingInitTestSuite.test("ThrowsAndFailableTest") {
   if let x = ThrowAndFailDerived(b: false) { preconditionFailure() }
 }
 
-// <https://bugs.swift.org/browse/SR-3132> - Invalid pointer dequeued from free list. Runtime crash on some weird code
+// https://github.com/apple/swift/issues/45720
+// Invalid pointer dequeued from free list: run-time crash on some weird code
 
 runAllTests()

Original file line number	Diff line number	Diff line change
`@@ -24,8 +24,8 @@ extension P {`
`24`	`24`	`}`
`25`	`25`	`}`
`26`	`26`
`27`		`-// SR-4363`
`28`		`-// Test several paths through swift_getObjectType()`
	`27`	`+// https://github.com/apple/swift/issues/46942`
	`28`	`+// Test several paths through 'swift_getObjectType()'.`
`29`	`29`
`30`	`30`	`// instance of a Swift class that conforms to P`
`31`	`31`	`class O: NSObject, P { }`
Original file line number	Diff line number	Diff line change
`@@ -554,7 +554,8 @@ presentEitherOr(EitherOr<(), String>.Right("foo")) // CHECK-NEXT: Right(foo)`
`554`	`554`	`// CHECK-NEXT: Right(foo)`
`555`	`555`	`presentEitherOrsOf(t: (), u: "foo")`
`556`	`556`
`557`		`-// SR-5148`
	`557`	`+// https://github.com/apple/swift/issues/47724`
	`558`	`+`
`558`	`559`	`enum Payload {`
`559`	`560`	`case email`
`560`	`561`	`}`
Original file line number	Diff line number	Diff line change
`@@ -184,8 +184,9 @@ class MoreConcreteQuadruple : SemiConcreteTriple<State> {`
`184`	`184`	`}`
`185`	`185`	`}`
`186`	`186`
`187`		`-// This check triggers SR-815 (rdar://problem/25318716) on macOS 10.9 and iOS 7.`
`188`		`-// Disable it for now when testing on those versions.`
	`187`	`+// This check triggers https://github.com/apple/swift/issues/43427`
	`188`	`+// (rdar://problem/25318716) on macOS 10.9 and iOS 7. Disable it for now when`
	`189`	`+// testing on those versions.`
`189`	`190`	`if #available(macOS 10.10, iOS 8, *) {`
`190`	`191`	`var u = MoreConcreteQuadruple(10, 17, State.CA, "Hella")`
`191`	`192`
Original file line number	Diff line number	Diff line change
`@@ -284,7 +284,8 @@ ProtocolExtensionTestSuite.test("ClassInitializer") {`
`284`	`284`	`expectTrue(Sub.self == metatypes[3].1)`
`285`	`285`	`}`
`286`	`286`
`287`		`-// https://bugs.swift.org/browse/SR-617`
	`287`	`+// https://github.com/apple/swift/issues/43234`
	`288`	`+`
`288`	`289`	`protocol SelfMetadataTest {`
`289`	`290`	`associatedtype T = Int`
`290`	`291`