From 865c107cea87972e4c7b140b3aad5343542ab47a Mon Sep 17 00:00:00 2001
From: Andrew Heard <andrewheard@google.com>
Date: Tue, 12 Dec 2023 11:51:42 -0500
Subject: [PATCH] Refactor `testGenerateContent_failure_emptyContent` with
 `catch let`

---
 .../GoogleAITests/GenerativeModelTests.swift  | 32 +++++++------------
 1 file changed, 11 insertions(+), 21 deletions(-)

diff --git a/Tests/GoogleAITests/GenerativeModelTests.swift b/Tests/GoogleAITests/GenerativeModelTests.swift
index 82e2fb6..45590db 100644
--- a/Tests/GoogleAITests/GenerativeModelTests.swift
+++ b/Tests/GoogleAITests/GenerativeModelTests.swift
@@ -160,30 +160,20 @@ final class GenerativeModelTests: XCTestCase {
         withExtension: "json"
       )
 
-    var responseError: Error?
-    var content: GenerateContentResponse?
     do {
-      content = try await model.generateContent(testPrompt)
+      _ = try await model.generateContent(testPrompt)
+      XCTFail("Should throw GenerateContentError.internalError; no error thrown.")
+    } catch let GenerateContentError.internalError(underlying: underlyingError) {
+      guard case let .emptyContent(decodingError) =
+        try XCTUnwrap(underlyingError as? InvalidCandidateError) else {
+        XCTFail("Not an InvalidCandidateError.emptyContent error: \(underlyingError)")
+        return
+      }
+      _ = try XCTUnwrap(decodingError as? DecodingError,
+                        "Not a DecodingError: \(decodingError)")
     } catch {
-      responseError = error
+      XCTFail("Should throw GenerateContentError.internalError; error thrown: \(error)")
     }
-
-    XCTAssertNil(content)
-    XCTAssertNotNil(responseError)
-    let generateContentError = try XCTUnwrap(responseError as? GenerateContentError)
-    guard case let .internalError(underlyingError) = generateContentError else {
-      XCTFail("Not an internal error: \(generateContentError)")
-      return
-    }
-    let invalidCandidateError = try XCTUnwrap(underlyingError as? InvalidCandidateError)
-    guard case let .emptyContent(emptyContentUnderlyingError) = invalidCandidateError else {
-      XCTFail("Not an empty content error: \(invalidCandidateError)")
-      return
-    }
-    _ = try XCTUnwrap(
-      emptyContentUnderlyingError as? DecodingError,
-      "Not a decoding error: \(emptyContentUnderlyingError)"
-    )
   }
 
   func testGenerateContent_failure_finishReasonSafety() async throws {