Fix issue with conditional in query value (#2781)

* Add failing unit test

* Add is conditional unit test

* Is that really the fix? Just check for space or parthensis if midconditional. Add unit test for join

* Formatting

Author: Alex-Tideman

src/lib/utilities/query/tokenize.test.ts

@@ -12,6 +12,7 @@ const combinedQuery =
 const valuesWithSpacesQuery =
   '`Custom Key Word`="Hello there world" AND `WorkflowId`="one and two = three" OR `WorkflowType`="example=\'one\'"';
 const keywordListQuery = '`CustomKeywordListField`in("Hello", "World")';
+const startsWithQuery = '`WorkflowType` STARTS_WITH "Hello"';
 
 describe('tokenize', () => {
   it('should eliminate spaces', () => {
@@ -145,4 +146,28 @@ describe('tokenize', () => {
       '2022-04-20T18:09:49-06:00',
     ]);
   });
+
+  it('should tokenize the startsWithQuery', () => {
+    const query = startsWithQuery;
+
+    expect(tokenize(query)).toEqual(['WorkflowType', 'STARTS_WITH', 'Hello']);
+  });
+
+  it('should tokenize the startsWithInQuery for values with IN conditional in them', () => {
+    const query = '`WorkflowType` STARTS_WITH "Inspect"';
+
+    expect(tokenize(query)).toEqual(['WorkflowType', 'STARTS_WITH', 'Inspect']);
+  });
+
+  it('should tokenize the startsWithInQuery for values with IS conditional in them', () => {
+    const query = '`WorkflowType` STARTS_WITH "issue"';
+
+    expect(tokenize(query)).toEqual(['WorkflowType', 'STARTS_WITH', 'issue']);
+  });
+
+  it('should tokenize the startsWithInQuery for values with JOIN conditional in them', () => {
+    const query = '`WorkflowType` STARTS_WITH "Joiner"';
+
+    expect(tokenize(query)).toEqual(['WorkflowType', 'STARTS_WITH', 'Joiner']);
+  });
 });

src/lib/utilities/query/tokenize.ts

@@ -82,7 +82,11 @@ export const tokenize = (string: string): Tokens => {
       addBufferToTokens();
       cursor += 3;
       continue;
-    } else if (isConditional(midConditional)) {
+    } else if (
+      isConditional(midConditional) &&
+      (isSpace(string[cursor + 2]) || isParenthesis(string[cursor + 2]))
+    ) {
+      // To prevent false positives like "inspect" being a "in" conditional, check for space or parenthesis after the midConditional
       buffer += midConditional;
       addBufferToTokens();
       cursor += 2;