store/postgres: Add test for inserts when all fulltext fields are null

Author: fordN

store/postgres/src/relational_queries.rs

@@ -1055,10 +1055,7 @@ impl<'a> InsertQuery<'a> {
                         .cloned()
                         .collect::<Vec<Value>>();
                     if !fulltext_field_values.is_empty() {
-                        entity.insert(
-                            column.field.to_string(),
-                            Value::List(fulltext_field_values),
-                        );
+                        entity.insert(column.field.to_string(), Value::List(fulltext_field_values));
                     }
                 }
                 None => (),

store/postgres/tests/relational.rs

@@ -32,7 +32,22 @@ const THINGS_GQL: &str = r#"
                 ]
             }
         ]
+    ) @fulltext(
+        name: "nullableStringsSearch"
+        language: en
+        algorithm: rank
+        include: [
+            {
+                entity: "NullableStrings",
+                fields: [
+                    { name: "name"},
+                    { name: "description"},
+                    { name: "test"},
+                ]
+            }
+        ]
     )
+
     type Thing @entity {
         id: ID!
         bigThing: Thing!
@@ -87,6 +102,13 @@ const THINGS_GQL: &str = r#"
         favorite_color: Color,
         drinks: [String!]
     }
+
+    type NullableStrings @entity {
+        id: ID!,
+        name: String,
+        description: String,
+        test: String
+    }
 "#;
 
 const SCHEMA_NAME: &str = "layout";
@@ -143,6 +165,12 @@ lazy_static! {
         entity.set("__typename", "Scalar");
         entity
     };
+    static ref EMPTY_NULLABLESTRINGS_ENTITY: Entity = {
+        let mut entity = Entity::new();
+        entity.set("id", "one");
+        entity.set("__typename", "NullableStrings");
+        entity
+    };
 }
 
 /// Removes test data from the database behind the store.
@@ -418,6 +446,27 @@ fn find() {
     });
 }
 
+#[test]
+fn insert_null_fulltext_fields() {
+    run_test(|conn, layout| -> Result<(), ()> {
+        insert_entity(
+            &conn,
+            &layout,
+            "NullableStrings",
+            EMPTY_NULLABLESTRINGS_ENTITY.clone(),
+        );
+
+        // Find entity with null string values
+        let entity = layout
+            .find(conn, "NullableStrings", "one", BLOCK_NUMBER_MAX)
+            .expect("Failed to read NullableStrings[one]")
+            .unwrap();
+        assert_entity_eq!(scrub(&*EMPTY_NULLABLESTRINGS_ENTITY), entity);
+
+        Ok(())
+    });
+}
+
 #[test]
 fn update() {
     run_test(|conn, layout| -> Result<(), ()> {