Expose issue severity and isFailure as API

Author: suzannaratcliff

Sources/Testing/Issues/Issue+Recording.swift

@@ -58,25 +58,6 @@ extension Issue {
     return self
   }
 
-  /// Record an issue when a running test fails unexpectedly.
-  ///
-  /// - Parameters:
-  ///   - comment: A comment describing the expectation.
-  ///   - sourceLocation: The source location to which the issue should be
-  ///     attributed.
-  ///
-  /// - Returns: The issue that was recorded.
-  ///
-  /// Use this function if, while running a test, an issue occurs that cannot be
-  /// represented as an expectation (using the ``expect(_:_:sourceLocation:)``
-  /// or ``require(_:_:sourceLocation:)-5l63q`` macros.)
-  @discardableResult public static func record(
-    _ comment: Comment? = nil,
-    sourceLocation: SourceLocation = #_sourceLocation
-  ) -> Self {
-    record(comment, severity: .error, sourceLocation: sourceLocation)
-  }
-
   /// Record an issue when a running test fails unexpectedly.
   ///
   /// - Parameters:
@@ -90,7 +71,6 @@ extension Issue {
   /// Use this function if, while running a test, an issue occurs that cannot be
   /// represented as an expectation (using the ``expect(_:_:sourceLocation:)``
   /// or ``require(_:_:sourceLocation:)-5l63q`` macros.)
-  @_spi(Experimental)
   @discardableResult public static func record(
     _ comment: Comment? = nil,
     severity: Severity,
@@ -105,28 +85,7 @@ extension Issue {
 // MARK: - Recording issues for errors
 
 extension Issue {
-  /// Record a new issue when a running test unexpectedly catches an error.
-  ///
-  /// - Parameters:
-  ///   - error: The error that caused the issue.
-  ///   - comment: A comment describing the expectation.
-  ///   - sourceLocation: The source location to which the issue should be
-  ///     attributed.
-  ///
-  /// - Returns: The issue that was recorded.
-  ///
-  /// This function can be used if an unexpected error is caught while running a
-  /// test and it should be treated as a test failure. If an error is thrown
-  /// from a test function, it is automatically recorded as an issue and this
-  /// function does not need to be used.
-  @discardableResult public static func record(
-    _ error: any Error,
-    _ comment: Comment? = nil,
-    sourceLocation: SourceLocation = #_sourceLocation
-  ) -> Self {
-    record(error, comment, severity: .error, sourceLocation: sourceLocation)
-  }
-  
+
   /// Record a new issue when a running test unexpectedly catches an error.
   ///
   /// - Parameters:
@@ -142,7 +101,6 @@ extension Issue {
   /// test and it should be treated as a test failure. If an error is thrown
   /// from a test function, it is automatically recorded as an issue and this
   /// function does not need to be used.
-  @_spi(Experimental)
   @discardableResult public static func record(
     _ error: any Error,
     _ comment: Comment? = nil,

Sources/Testing/Issues/Issue.swift

@@ -84,7 +84,6 @@ public struct Issue: Sendable {
   ///
   /// - ``warning``
   /// - ``error``
-  @_spi(Experimental)
   public enum Severity: Sendable {
     /// The severity level for an issue which should be noted but is not
     /// necessarily an error.
@@ -101,8 +100,12 @@ public struct Issue: Sendable {
   }
 
   /// The severity of this issue.
-  @_spi(Experimental)
   public var severity: Severity
+  
+  /// If the issues is a failing issue.
+  public var isFailure: Bool {
+    severity == .error
+  }
 
   /// Any comments provided by the developer and associated with this issue.
   ///