From 32ef5c2ecaae42fe5d6262c5fae8a9918db20988 Mon Sep 17 00:00:00 2001
From: Jonathan Grynspan <jgrynspan@apple.com>
Date: Fri, 19 Jul 2024 15:09:05 -0400
Subject: [PATCH] Remove `Test.Case`'s conformance to `Identifiable`.

With further consideration after #548, we've decided to remove conformance to
`Identifiable` from `Test.Case` rather than promoting it to API. The identity of
a test case is fuzzy because it depends on the arguments to a test, not all of
which may be uniquely identifiable. We need more time to consider the design of
this part of the interface before making it a permanent part of our API.
---
 Sources/Testing/Parameterization/Test.Case.ID.swift | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Sources/Testing/Parameterization/Test.Case.ID.swift b/Sources/Testing/Parameterization/Test.Case.ID.swift
index 97bece8aa..26b57fdf8 100644
--- a/Sources/Testing/Parameterization/Test.Case.ID.swift
+++ b/Sources/Testing/Parameterization/Test.Case.ID.swift
@@ -8,27 +8,27 @@
 // See https://swift.org/CONTRIBUTORS.txt for Swift project authors
 //
 
-extension Test.Case: Identifiable {
+extension Test.Case {
   /// The ID of a test case.
   ///
   /// Instances of this type are considered unique within the scope of a given
   /// parameterized test function. They are not necessarily unique across two
   /// different ``Test`` instances.
+  @_spi(ForToolsIntegrationOnly)
   public struct ID: Sendable, Equatable, Hashable {
     /// The IDs of the arguments of this instance's associated ``Test/Case``, in
     /// the order they appear in ``Test/Case/arguments``.
     ///
     /// The value of this property is `nil` if _any_ of the associated test
     /// case's arguments has a `nil` ID.
-    @_spi(ForToolsIntegrationOnly)
     public var argumentIDs: [Argument.ID]?
 
-    @_spi(ForToolsIntegrationOnly)
     public init(argumentIDs: [Argument.ID]?) {
       self.argumentIDs = argumentIDs
     }
   }
 
+  @_spi(ForToolsIntegrationOnly)
   public var id: ID {
     let argumentIDs = arguments.compactMap(\.id)
     guard argumentIDs.count == arguments.count else {