From 646b6ce27f6c78d26a0397912c35b2fa0dd53f6e Mon Sep 17 00:00:00 2001 From: Clare Macrae Date: Tue, 9 Apr 2024 21:34:08 +0100 Subject: [PATCH] test: - Update two tests that I could have updated in #2762 --- tests/Query/Filter/BooleanPreprocessor.test.ts | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/tests/Query/Filter/BooleanPreprocessor.test.ts b/tests/Query/Filter/BooleanPreprocessor.test.ts index 10f003ae19..a7cee47185 100644 --- a/tests/Query/Filter/BooleanPreprocessor.test.ts +++ b/tests/Query/Filter/BooleanPreprocessor.test.ts @@ -2,7 +2,7 @@ import { BooleanPreprocessor } from '../../../src/Query/Filter/BooleanPreprocess import { BooleanDelimiters } from '../../../src/Query/Filter/BooleanDelimiters'; function preprocess(line: string) { - return BooleanPreprocessor.preprocessExpression(line, BooleanDelimiters.allSupportedDelimiters()); + return BooleanPreprocessor.preprocessExpression(line, BooleanDelimiters.fromInstructionLine(line)); } describe('BooleanPreprocessor', () => { @@ -130,16 +130,14 @@ describe('BooleanPreprocessor', () => { describe('filters ending with delimiters', () => { it('swallows last character if filter ends with closing delimiter character )', () => { const result = preprocess('"description includes (maybe)"'); - // TODO Fix imbalanced delimiters by requiring the same delimiter set to be used in Boolean lines. - expect(result.simplifiedLine).toEqual('"f1)"'); - expect(result.filters['f1']).toEqual('description includes (maybe'); + expect(result.simplifiedLine).toEqual('"f1"'); + expect(result.filters['f1']).toEqual('description includes (maybe)'); }); it('swallows last character if filter ends with closing delimiter character "', () => { const result = preprocess('(description includes "maybe")'); - // TODO Fix imbalanced delimiters by requiring the same delimiter set to be used in Boolean lines. - expect(result.simplifiedLine).toEqual('(f1")'); - expect(result.filters['f1']).toEqual('description includes "maybe'); + expect(result.simplifiedLine).toEqual('(f1)'); + expect(result.filters['f1']).toEqual('description includes "maybe"'); }); });