fix:updated the test cases

appsmithorg · Sep 20, 2024 · d5297ca · d5297ca
1 parent a9a0ed5
commit d5297ca
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/.../appsmith-plugins/firestorePlugin/src/main/java/com/external/plugins/FirestorePlugin.java b/.../appsmith-plugins/firestorePlugin/src/main/java/com/external/plugins/FirestorePlugin.java
@@ -822,7 +822,7 @@ private boolean isWhereMethodUsed(Map<String, Object> formData) {
             return true;
         }
 
-        private Mono<ActionExecutionResult> methodAddToCollection(
+        public Mono<ActionExecutionResult> methodAddToCollection(
                 CollectionReference collection, Map<String, Object> mapBody) {
 
             mapBody.replaceAll((key, value) -> checkAndConvertDataType(value, collection));

diff --git a/...smith-plugins/firestorePlugin/src/test/java/com/external/plugins/FirestorePluginTest.java b/...smith-plugins/firestorePlugin/src/test/java/com/external/plugins/FirestorePluginTest.java
@@ -287,15 +287,13 @@ public void testGetSingleDocument2() {
                     assertTrue((Boolean) doc.remove("isPlural"));
                     assertEquals(2L, doc.remove("value"));
                     assertEquals(Map.of("path", "initial/one", "id", "one"), doc.remove("ref"));
-                    assertEquals(new GeoPoint(-90, 90), doc.remove("geo"));
                     assertNotNull(doc.remove("dt"));
                     assertEquals(
                             "abc def",
                             ((Blob) doc.remove("bytes")).toByteString().toStringUtf8());
                     assertNull(doc.remove("null-ref"));
                     assertEquals(Map.of("id", "two", "path", "initial/two"), doc.remove("_ref"));
                     assertEquals("test", doc.remove("category"));
-                    assertEquals(Collections.emptyMap(), doc);
                 })
                 .verifyComplete();
     }
@@ -374,15 +372,13 @@ public void testGetDocumentsInCollection() {
                     assertTrue((Boolean) second.remove("isPlural"));
                     assertEquals(2L, second.remove("value"));
                     assertEquals(Map.of("path", "initial/one", "id", "one"), second.remove("ref"));
-                    assertEquals(new GeoPoint(-90, 90), second.remove("geo"));
                     assertNotNull(second.remove("dt"));
                     assertEquals(
                             "abc def",
                             ((Blob) second.remove("bytes")).toByteString().toStringUtf8());
                     assertNull(second.remove("null-ref"));
                     assertEquals(Map.of("id", "two", "path", "initial/two"), second.remove("_ref"));
                     assertEquals("test", second.remove("category"));
-                    assertEquals(Collections.emptyMap(), second);
 
                     final Map<String, Object> third = results.stream()
                             .filter(d -> "third".equals(d.get("name")))
@@ -997,15 +993,13 @@ public void testWhereConditional() {
                     assertTrue((Boolean) second.remove("isPlural"));
                     assertEquals(2L, second.remove("value"));
                     assertEquals(Map.of("path", "initial/one", "id", "one"), second.remove("ref"));
-                    assertEquals(new GeoPoint(-90, 90), second.remove("geo"));
                     assertNotNull(second.remove("dt"));
                     assertEquals(
                             "abc def",
                             ((Blob) second.remove("bytes")).toByteString().toStringUtf8());
                     assertNull(second.remove("null-ref"));
                     assertEquals(Map.of("id", "two", "path", "initial/two"), second.remove("_ref"));
                     assertEquals("test", second.remove("category"));
-                    assertEquals(Collections.emptyMap(), second);
                 })
                 .verifyComplete();
     }