fix(autocomplete): use Tuple.getElement() for payload lookup in autocomplete (#673)

When both withScore() and withPayload() options are enabled, the code
incorrectly passed the Tuple object directly to the hash lookup instead
of extracting the string element first.

This caused a ClassCastException: "class redis.clients.jedis.resps.Tuple
cannot be cast to class java.lang.String"

Closes #673

Author: bsbodden

redis-om-spring/src/main/java/com/redis/om/spring/ops/search/SearchOperationsImpl.java

@@ -135,7 +135,7 @@ public List<Suggestion> getSuggestion(String key, String prefix, AutoCompleteOpt
           String[] keyParts = key.split(":");
           String payLoadKey = String.format("sugg:payload:%s:%s", keyParts[keyParts.length - 2],
               keyParts[keyParts.length - 1]);
-          Object payload = template.opsForHash().get(payLoadKey, suggestion);
+          Object payload = template.opsForHash().get(payLoadKey, suggestion.getElement());
           String json = payload != null ? payload.toString() : "{}";
           Map<String, Object> payloadMap = gson.fromJson(json, new TypeToken<Map<String, Object>>() {
           }.getType());

tests/src/test/java/com/redis/om/spring/annotations/autocompletable/AutoCompleteTest.java

@@ -98,4 +98,33 @@ void testGetAutocompleteSuggestionsWithScores() {
         0.27));
   }
 
+  /**
+   * Test for Issue #673: ClassCastException when using withScore() and withPayload() together.
+   *
+   * When both options are enabled, the code incorrectly passes the Tuple object
+   * (instead of its string element) to the hash lookup for payload retrieval.
+   *
+   * @see <a href="https://github.com/redis/redis-om-spring/issues/673">Issue #673</a>
+   */
+  @Test
+  void testGetAutocompleteSuggestionsWithScoresAndPayload() {
+    Map<String, Object> columbusPayload = Map.of("code", "CMH", "state", "OH");
+    Map<String, Object> columbiaPayload = Map.of("code", "CAE", "state", "SC");
+    Map<String, Object> coloradoSpringsPayload = Map.of("code", "COS", "state", "CO");
+
+    // This combination caused ClassCastException before the fix:
+    // "class redis.clients.jedis.resps.Tuple cannot be cast to class java.lang.String"
+    List<Suggestion> suggestions = repository.autoCompleteName("col",
+        AutoCompleteOptions.get().withScore().withPayload());
+
+    List<String> suggestionsString = suggestions.stream().map(Suggestion::getValue).collect(Collectors.toList());
+    List<Double> scores = suggestions.stream().map(Suggestion::getScore).collect(Collectors.toList());
+    List<Map<String, Object>> payloads = suggestions.stream().map(Suggestion::getPayload).collect(Collectors.toList());
+
+    assertThat(suggestionsString).containsAll(List.of("Columbia", "Columbus", "Colorado Springs"));
+    assertThat(scores).usingComparatorForType(new DoubleComparator(0.1), Double.class).containsAll(List.of(0.41, 0.41,
+        0.27));
+    assertThat(payloads).hasSize(3).contains(columbusPayload, columbiaPayload, coloradoSpringsPayload);
+  }
+
 }