Move tests to actual function implementing the parsing

directus · Apr 30, 2024 · 745c735 · 745c735
1 parent 57e7dba
commit 745c735
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 24 deletions.
diff --git a/api/src/utils/sanitize-query.test.ts b/api/src/utils/sanitize-query.test.ts
@@ -210,30 +210,6 @@ describe('filter', () => {
 		expect(sanitizedQuery.filter).toEqual({ field_a: { _eq: 'test' } });
 	});
 
-	test('should accept empty filter object', () => {
-		const filter = {};
-
-		const sanitizedQuery = sanitizeQuery({ filter });
-
-		expect(sanitizedQuery.filter).toEqual({});
-	});
-
-	test('should accept empty object for key', () => {
-		const filter = { field_a: {} };
-
-		const sanitizedQuery = sanitizeQuery({ filter });
-
-		expect(sanitizedQuery.filter).toEqual({ field_a: {} });
-	});
-
-	test('should accept empty array for key', () => {
-		const filter = { field_a: [] };
-
-		const sanitizedQuery = sanitizeQuery({ filter });
-
-		expect(sanitizedQuery.filter).toEqual({ field_a: [] });
-	});
-
 	test('should throw error on invalid filter', () => {
 		const filter = { field_a: null };
 

diff --git a/packages/utils/shared/parse-filter.test.ts b/packages/utils/shared/parse-filter.test.ts
@@ -12,6 +12,22 @@ describe('#parseFilter', () => {
 		vi.useRealTimers();
 	});
 
+	it('should accept empty filter object', () => {
+		const filter = {};
+
+		const parsedFilter = parseFilter(filter, null);
+
+		expect(parsedFilter).toEqual({});
+	});
+
+	it('should accept empty object for key', () => {
+		const filter = { field_a: {} };
+
+		const parsedFilter = parseFilter(filter, null);
+
+		expect(parsedFilter).toEqual({ field_a: {} });
+	});
+
 	it('returns the filter when passed accountability with only a role', () => {
 		const mockFilter = { _and: [{ field: { _eq: 'field' } }] } as Filter;
 		const mockAccountability = { role: 'admin' };