add tests for generative content response and change #text to return … (

#144)

…all text parts

---------

Co-authored-by: David Motsonashvili <davidmotson@google.com>

generativeai/src/main/java/com/google/ai/client/generativeai/type/GenerateContentResponse.kt

@@ -30,8 +30,10 @@ class GenerateContentResponse(
   val promptFeedback: PromptFeedback?,
   val usageMetadata: UsageMetadata?
 ) {
-  /** Convenience field representing the first text part in the response, if it exists. */
-  val text: String? by lazy { firstPartAs<TextPart>()?.text }
+  /** Convenience field representing all the text parts in the response, if they exists. */
+  val text: String? by lazy {
+    candidates.first().content.parts.filterIsInstance<TextPart>().joinToString(" ") { it.text }
+  }
 
   /** Convenience field representing the first function call part in the request, if it exists */
   @Deprecated("Use functionCalls instead", ReplaceWith("functionCalls"))

generativeai/src/test/java/com/google/ai/client/generativeai/GenerateContentResponseTest.kt

@@ -0,0 +1,101 @@
+/*
+ * Copyright 2024 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.google.ai.client.generativeai
+
+import com.google.ai.client.generativeai.type.Candidate
+import com.google.ai.client.generativeai.type.FunctionCallPart
+import com.google.ai.client.generativeai.type.GenerateContentResponse
+import com.google.ai.client.generativeai.type.content
+import io.kotest.matchers.collections.shouldHaveSize
+import io.kotest.matchers.shouldBe
+import org.junit.Test
+
+internal class GenerateContentResponseTest {
+
+  @Test
+  fun `generate response should pull all functions requests`() {
+    val response =
+      GenerateContentResponse(
+        candidates =
+          listOf(
+            Candidate(
+              content {
+                part(FunctionCallPart("blah", mapOf()))
+                part(FunctionCallPart("blah2", mapOf()))
+                text("This is a textPart")
+              },
+              listOf(),
+              listOf(),
+              null
+            )
+          ),
+        null,
+        null
+      )
+
+    response.functionCalls shouldHaveSize 2
+  }
+
+  @Test
+  fun `generate response should get strings even if they are not the first part`() {
+    val response =
+      GenerateContentResponse(
+        candidates =
+          listOf(
+            Candidate(
+              content {
+                part(FunctionCallPart("blah", mapOf()))
+                part(FunctionCallPart("blah2", mapOf()))
+                text("This is a textPart")
+              },
+              listOf(),
+              listOf(),
+              null
+            )
+          ),
+        null,
+        null
+      )
+
+    response.text shouldBe "This is a textPart"
+  }
+
+  @Test
+  fun `generate response should get strings and concatenate them together`() {
+    val response =
+      GenerateContentResponse(
+        candidates =
+          listOf(
+            Candidate(
+              content {
+                part(FunctionCallPart("blah", mapOf()))
+                part(FunctionCallPart("blah2", mapOf()))
+                text("This is a textPart")
+                text("This is another textPart")
+              },
+              listOf(),
+              listOf(),
+              null
+            )
+          ),
+        null,
+        null
+      )
+
+    response.text shouldBe "This is a textPart This is another textPart"
+  }
+}