Fix PostgresIndexQueryBuilder null pointer exception (#116)

* Throw IllegalArgumentException when query string does not match regex

* Unit test for exception

postgres-persistence/src/main/java/com/netflix/conductor/postgres/util/PostgresIndexQueryBuilder.java

@@ -75,6 +75,8 @@ public Condition(String query) {
                 if (this.attribute.endsWith("_time")) {
                     values.set(0, millisToUtc(values.get(0)));
                 }
+            } else {
+                throw new IllegalArgumentException("Incorrectly formatted query string: " + query);
             }
         }
 

postgres-persistence/src/test/java/com/netflix/conductor/postgres/util/PostgresIndexQueryBuilderTest.java

@@ -24,6 +24,7 @@
 import com.netflix.conductor.postgres.config.PostgresProperties;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.fail;
 import static org.mockito.Mockito.*;
 
 public class PostgresIndexQueryBuilderTest {
@@ -288,4 +289,19 @@ void shouldAllowJsonSearch() throws SQLException {
                 "SELECT json_data::TEXT FROM table_name WHERE json_data @> ?::JSONB LIMIT ? OFFSET ?";
         assertEquals(expectedQuery, builder.getQuery());
     }
+
+    @Test()
+    void shouldThrowIllegalArgumentExceptionWhenQueryStringIsInvalid() {
+        String inputQuery =
+                "workflowType IN (one,two) AND status IN (COMPLETED,RUNNING) AND startTime>1675701498000 AND xyz";
+
+        try {
+            new PostgresIndexQueryBuilder(
+                    "table_name", inputQuery, "", 0, 15, new ArrayList<>(), properties);
+
+            fail("should have failed since xyz does not conform to expected format");
+        } catch (IllegalArgumentException e) {
+            assertEquals("Incorrectly formatted query string: xyz", e.getMessage());
+        }
+    }
 }