Pass message along in async withErrorReporting

.github/workflows/ci.yml

@@ -21,7 +21,7 @@ jobs:
           - debug
           - release
         xcode:
-          - '16.0'
+          - '16.2'
     name: macOS 15
     runs-on: macos-15
     steps:
@@ -55,7 +55,7 @@ jobs:
     steps:
     - uses: actions/checkout@v4
     - name: Select Xcode
-      run: sudo xcode-select -s /Applications/Xcode_16.0.app
+      run: sudo xcode-select -s /Applications/Xcode_16.2.app
     - name: Run tests
       run: make build-for-library-evolution
 
@@ -66,7 +66,7 @@ jobs:
           - Debug
           - Release
         xcode:
-          - '16.0'
+          - '16.2'
     name: Examples
     runs-on: macos-15
     steps:
@@ -125,8 +125,8 @@ jobs:
     steps:
       - uses: compnerd/gha-setup-swift@main
         with:
-          branch: swift-5.10-release
-          tag: 5.10-RELEASE
+          branch: swift-6.0.3-release
+          tag: 6.0.3-RELEASE
       - name: Set long paths
         run: git config --system core.longpaths true
       - uses: actions/checkout@v4

Sources/IssueReporting/ErrorReporting.swift

@@ -52,42 +52,104 @@ public func withErrorReporting<R>(
   }
 }
 
-/// Evaluates a throwing closure and automatically catches and reports any error thrown.
-///
-/// - Parameters:
-///   - message: A message describing the expectation.
-///   - reporters: Issue reporters to notify during the operation.
-///   - fileID: The source `#fileID` associated with the error reporting.
-///   - filePath: The source `#filePath` associated with the error reporting.
-///   - line: The source `#line` associated with the error reporting.
-///   - column: The source `#column` associated with the error reporting.
-///   - body: An asynchronous operation.
-/// - Returns: The optional result of the operation, or `nil` if an error was thrown.
-@_transparent
-public func withErrorReporting<R>(
-  _ message: @autoclosure () -> String? = nil,
-  to reporters: [any IssueReporter]? = nil,
-  fileID: StaticString = #fileID,
-  filePath: StaticString = #filePath,
-  line: UInt = #line,
-  column: UInt = #column,
-  catching body: () async throws -> R
-) async -> R? {
-  if let reporters {
-    return await withIssueReporters(reporters) {
+#if compiler(>=6)
+  /// Evaluates a throwing closure and automatically catches and reports any error thrown.
+  ///
+  /// - Parameters:
+  ///   - message: A message describing the expectation.
+  ///   - reporters: Issue reporters to notify during the operation.
+  ///   - fileID: The source `#fileID` associated with the error reporting.
+  ///   - filePath: The source `#filePath` associated with the error reporting.
+  ///   - line: The source `#line` associated with the error reporting.
+  ///   - column: The source `#column` associated with the error reporting.
+  ///   - isolation: The isolation associated with the error reporting.
+  ///   - body: An asynchronous operation.
+  /// - Returns: The optional result of the operation, or `nil` if an error was thrown.
+  @_transparent
+  public func withErrorReporting<R>(
+    _ message: @autoclosure () -> String? = nil,
+    to reporters: [any IssueReporter]? = nil,
+    fileID: StaticString = #fileID,
+    filePath: StaticString = #filePath,
+    line: UInt = #line,
+    column: UInt = #column,
+    isolation: isolated (any Actor)? = #isolation,
+    catching body: () async throws -> sending R
+  ) async -> R? {
+    if let reporters {
+      return await withIssueReporters(reporters) {
+        do {
+          return try await body()
+        } catch {
+          reportIssue(
+            error,
+            message(),
+            fileID: fileID,
+            filePath: filePath,
+            line: line,
+            column: column
+          )
+          return nil
+        }
+      }
+    } else {
       do {
         return try await body()
       } catch {
-        reportIssue(error, fileID: fileID, filePath: filePath, line: line, column: column)
+        reportIssue(
+          error,
+          message(),
+          fileID: fileID,
+          filePath: filePath,
+          line: line,
+          column: column
+        )
         return nil
       }
     }
-  } else {
-    do {
-      return try await body()
-    } catch {
-      reportIssue(error, fileID: fileID, filePath: filePath, line: line, column: column)
-      return nil
+  }
+#else
+  @_transparent
+  @_unsafeInheritExecutor
+  public func withErrorReporting<R>(
+    _ message: @autoclosure () -> String? = nil,
+    to reporters: [any IssueReporter]? = nil,
+    fileID: StaticString = #fileID,
+    filePath: StaticString = #filePath,
+    line: UInt = #line,
+    column: UInt = #column,
+    catching body: () async throws -> R
+  ) async -> R? {
+    if let reporters {
+      return await withIssueReporters(reporters) {
+        do {
+          return try await body()
+        } catch {
+          reportIssue(
+            error,
+            message(),
+            fileID: fileID,
+            filePath: filePath,
+            line: line,
+            column: column
+          )
+          return nil
+        }
+      }
+    } else {
+      do {
+        return try await body()
+      } catch {
+        reportIssue(
+          error,
+          message(),
+          fileID: fileID,
+          filePath: filePath,
+          line: line,
+          column: column
+        )
+        return nil
+      }
     }
   }
-}
+#endif

Sources/IssueReporting/IssueReporter.swift

@@ -183,16 +183,28 @@ public func withIssueReporters<R>(
   try IssueReporters.$_current.withValue(LockIsolated(reporters), operation: operation)
 }
 
-/// Overrides the task's issue reporters for the duration of the asynchronous operation.
-///
-/// An asynchronous version of ``withIssueReporters(_:operation:)-91179``.
-///
-/// - Parameters:
-///   - reporters: Issue reporters to notify during the operation.
-///   - operation: An asynchronous operation.
-public func withIssueReporters<R>(
-  _ reporters: [any IssueReporter],
-  operation: () async throws -> R
-) async rethrows -> R {
-  try await IssueReporters.$_current.withValue(LockIsolated(reporters), operation: operation)
-}
+#if compiler(>=6)
+  /// Overrides the task's issue reporters for the duration of the asynchronous operation.
+  ///
+  /// An asynchronous version of ``withIssueReporters(_:operation:)-91179``.
+  ///
+  /// - Parameters:
+  ///   - reporters: Issue reporters to notify during the operation.
+  ///   - isolation: The isolation associated with the operation.
+  ///   - operation: An asynchronous operation.
+  public func withIssueReporters<R>(
+    _ reporters: [any IssueReporter],
+    isolation: isolated (any Actor)? = #isolation,
+    operation: () async throws -> R
+  ) async rethrows -> R {
+    try await IssueReporters.$_current.withValue(LockIsolated(reporters), operation: operation)
+  }
+#else
+  @_unsafeInheritExecutor
+  public func withIssueReporters<R>(
+    _ reporters: [any IssueReporter],
+    operation: () async throws -> R
+  ) async rethrows -> R {
+    try await IssueReporters.$_current.withValue(LockIsolated(reporters), operation: operation)
+  }
+#endif

Tests/IssueReportingTests/SwiftTestingTests.swift

@@ -1,4 +1,4 @@
-#if canImport(Testing)
+#if canImport(Testing) && !os(Windows)
   import Testing
   import IssueReporting
 

Tests/IssueReportingTests/UnimplementedTests.swift

@@ -1,4 +1,4 @@
-#if canImport(Testing)
+#if canImport(Testing) && !os(Windows)
   import IssueReporting
   import Testing
 

Tests/IssueReportingTests/WithErrorReportingTests.swift

@@ -1,4 +1,4 @@
-#if canImport(Testing)
+#if canImport(Testing) && !os(Windows)
   import Testing
   import IssueReporting
 
@@ -21,7 +21,37 @@
         issue.description == "Caught error: SomeError(): Failed"
       }
     }
+
+    @Test func overload() async {
+      await withKnownIssue {
+        await withErrorReporting { () async throws in
+          throw SomeError()
+        }
+      } matching: { issue in
+        issue.description == "Caught error: SomeError()"
+      }
+
+      await withKnownIssue {
+        await withErrorReporting("Failed") { () async throws in
+          throw SomeError()
+        }
+      } matching: { issue in
+        issue.description == "Caught error: SomeError(): Failed"
+      }
+    }
+
+    @MainActor
+    @Test func isolation() async {
+      await withKnownIssue {
+        await withErrorReporting { () async throws in
+          throw SomeError()
+        }
+      } matching: { issue in
+        issue.description == "Caught error: SomeError()"
+      }
+    }
   }
 
   private struct SomeError: Error {}
 #endif
+