Common code between exit tests and the fallback handler

Author: grynspan

Sources/Testing/Events/Event+FallbackHandler.swift

@@ -11,31 +11,50 @@
 private import _Testing_ExperimentalInfrastructure
 
 extension Event {
-  /// The implementation of ``fallbackEventHandler``.
+  /// Attempt to handle an event encoded as JSON as if it had been generated in
+  /// the current testing context.
   ///
   /// - Parameters:
-  ///   - abi: The ABI version to use for decoding `recordJSON`.
   ///   - recordJSON: The JSON encoding of an event record.
+  ///   - abi: The ABI version to use for decoding `recordJSON`.
   ///
   /// - Throws: Any error that prevented handling the encoded record.
-  private static func _fallbackEventHandler<V>(_ abi: V.Type, _ recordJSON: UnsafeRawBufferPointer) throws where V: ABI.Version {
-    let record = try JSON.decode(ABI.Record<ABI.CurrentVersion>.self, from: recordJSON)
+  ///
+  /// - Important: This function only handles a subset of event kinds.
+  static func handle<V>(_ recordJSON: UnsafeRawBufferPointer, encodedWith abi: V.Type) throws where V: ABI.Version {
+    let record = try JSON.decode(ABI.Record<V>.self, from: recordJSON)
     guard case let .event(event) = record.kind else {
       return
     }
-    switch event.kind {
-    case .issueRecorded:
-      Issue(event)?.record()
-    case .valueAttached:
-      if let attachment = event.attachment {
-        Attachment.record(attachment, sourceLocation: attachment._sourceLocation ?? .__here())
-      }
-    default:
-      // Not handled here.
-      break
+
+    lazy var comments: [Comment] = event._comments?.map(Comment.init(rawValue:)) ?? []
+    lazy var sourceContext = SourceContext(
+      backtrace: nil, // A backtrace from the child process will have the wrong address space.
+      sourceLocation: event._sourceLocation
+    )
+    lazy var skipInfo = SkipInfo(comment: comments.first, sourceContext: sourceContext)
+    if let issue = Issue(event) {
+      issue.record()
+    } else if let attachment = event.attachment {
+      Attachment.record(attachment, sourceLocation: event._sourceLocation!)
+    } else if case .testCancelled = event.kind {
+      _ = try? Test.cancel(with: skipInfo)
+    } else if case .testCaseCancelled = event.kind {
+      _ = try? Test.Case.cancel(with: skipInfo)
     }
   }
 
+  /// The implementation of ``fallbackEventHandler``.
+  ///
+  /// - Parameters:
+  ///   - abi: The ABI version to use for decoding `recordJSON`.
+  ///   - recordJSON: The JSON encoding of an event record.
+  ///
+  /// - Throws: Any error that prevented handling the encoded record.
+  private static func _fallbackEventHandler<V>(_ abi: V.Type, _ recordJSON: UnsafeRawBufferPointer) throws where V: ABI.Version {
+    try handle(recordJSON, encodedWith: abi)
+  }
+
   /// The fallback event handler to set when Swift Testing is the active testing
   /// library.
   private static let _fallbackEventHandler: FallbackEventHandler = { recordJSONSchemaVersionNumber, recordJSONBaseAddress, recordJSONByteCount, _ in

Sources/Testing/ExitTests/ExitTest.swift

@@ -1031,26 +1031,7 @@ extension ExitTest {
   ///
   /// - Throws: Any error encountered attempting to decode or process the JSON.
   private static func _processRecord(_ recordJSON: UnsafeRawBufferPointer, fromBackChannel backChannel: borrowing FileHandle) throws {
-    let record = try JSON.decode(ABI.Record<ABI.BackChannelVersion>.self, from: recordJSON)
-    guard case let .event(event) = record.kind else {
-      return
-    }
-
-    lazy var comments: [Comment] = event._comments?.map(Comment.init(rawValue:)) ?? []
-    lazy var sourceContext = SourceContext(
-      backtrace: nil, // A backtrace from the child process will have the wrong address space.
-      sourceLocation: event._sourceLocation
-    )
-    lazy var skipInfo = SkipInfo(comment: comments.first, sourceContext: sourceContext)
-    if let issue = Issue(event) {
-      issue.record()
-    } else if let attachment = event.attachment {
-      Attachment.record(attachment, sourceLocation: event._sourceLocation!)
-    } else if case .testCancelled = event.kind {
-      _ = try? Test.cancel(with: skipInfo)
-    } else if case .testCaseCancelled = event.kind {
-      _ = try? Test.Case.cancel(with: skipInfo)
-    }
+    try Event.handle(recordJSON, encodedWith: ABI.BackChannelVersion.self)
   }
 
   /// Decode this exit test's captured values and update its ``capturedValues``