Opt repetition event handlers out of being considered for all

Author: harlanhaskins

Sources/Testing/Running/Runner.RuntimeState.swift

@@ -74,15 +74,22 @@ extension Configuration {
   ///
   /// - Parameters:
   ///   - configuration: The new value to set for ``Configuration/current``.
+  ///   - addingToAll: Whether to add this configuration to the set of implicitly-tracked ``Configuration``s that are told about detached Issues.
   ///   - body: A function to call.
   ///
   /// - Returns: Whatever is returned by `body`.
   ///
   /// - Throws: Whatever is thrown by `body`.
-  static func withCurrent<R>(_ configuration: Self, perform body: () throws -> R) rethrows -> R {
-    let id = configuration._addToAll()
+  static func withCurrent<R>(
+    _ configuration: Self,
+    addingToAll: Bool = true,
+    perform body: () throws -> R
+  ) rethrows -> R {
+    let id = addingToAll ? configuration._addToAll() : nil
     defer {
-      configuration._removeFromAll(identifiedBy: id)
+      if let id {
+        configuration._removeFromAll(identifiedBy: id)
+      }
     }
 
     var runtimeState = Runner.RuntimeState.current ?? .init()
@@ -95,15 +102,22 @@ extension Configuration {
   ///
   /// - Parameters:
   ///   - configuration: The new value to set for ``Configuration/current``.
+  ///   - addingToAll: Whether to add this configuration to the set of implicitly-tracked ``Configuration``s that are told about detached Issues.
   ///   - body: A function to call.
   ///
   /// - Returns: Whatever is returned by `body`.
   ///
   /// - Throws: Whatever is thrown by `body`.
-  static func withCurrent<R>(_ configuration: Self, perform body: () async throws -> R) async rethrows -> R {
-    let id = configuration._addToAll()
+  static func withCurrent<R>(
+    _ configuration: Self,
+    addingToAll: Bool = true,
+    perform body: () async throws -> R
+  ) async rethrows -> R {
+    let id = addingToAll ? configuration._addToAll() : nil
     defer {
-      configuration._removeFromAll(identifiedBy: id)
+      if let id {
+        configuration._removeFromAll(identifiedBy: id)
+      }
     }
 
     var runtimeState = Runner.RuntimeState.current ?? .init()

Sources/Testing/Running/Runner.swift

@@ -226,10 +226,10 @@ extension Runner {
     // Determine what kind of event to send for this step based on its action.
     switch step.action {
     case .run:
-      Event.post(.testStarted, for: (step.test, nil), iteration: context.iteration, configuration: configuration)
+      Event.post(.testStarted, for: (step.test, nil), configuration: configuration)
       shouldSendTestEnded = true
     case let .skip(skipInfo):
-      Event.post(.testSkipped(skipInfo), for: (step.test, nil), iteration: context.iteration, configuration: configuration)
+      Event.post(.testSkipped(skipInfo), for: (step.test, nil), configuration: configuration)
       shouldSendTestEnded = false
     case let .recordIssue(issue):
       // Scope posting the issue recorded event such that issue handling
@@ -247,7 +247,7 @@ extension Runner {
     }
     defer {
       if shouldSendTestEnded {
-        Event.post(.testEnded, for: (step.test, nil), iteration: context.iteration, configuration: configuration)
+        Event.post(.testEnded, for: (step.test, nil), configuration: configuration)
       }
     }
 
@@ -506,7 +506,7 @@ extension Runner {
       }
 
       await Test.withCurrentIteration(iteration) {
-        await Configuration.withCurrent(config) {
+        await Configuration.withCurrent(config, addingToAll: false) {
           await body()
         }
       }