Fix tests

src/filter.ts

@@ -58,11 +58,11 @@ export class MetadataFilter {
 	 * @throws Throw an error if an invalid field is specified
 	 */
 	filterField(field: string, fieldValue: string): string {
-		if (!(field in this.mergedFilterSet)) {
-			throw new TypeError(`Invalid filter field: ${field}`);
+		if (field in this.mergedFilterSet) {
+			return this.filterText(fieldValue, this.mergedFilterSet[field]);
 		}
 
-		return this.filterText(fieldValue, this.mergedFilterSet[field]);
+		throw new TypeError(`Invalid filter field: ${field}`);
 	}
 
 	/**

test/filter.spec.ts

@@ -111,6 +111,7 @@ describe('Test filtering empty strings', () => {
 
 	it('should not call filter function is the input is null', () => {
 		const input = null;
+		// @ts-ignore
 		const actual = filter.filterField('artist', input);
 
 		expect(actual).to.be.equal(input);
@@ -127,6 +128,7 @@ describe('Test filtering invalid filter field', () => {
 
 describe('Test invalid filter', () => {
 	it('should throw error if the filter set is not specified', () => {
+		// @ts-ignore
 		expect(() => createFilter(null)).to.throw();
 	});
 

test/helper/util.ts

@@ -1,4 +1,4 @@
-import { MetadataFilter, createFilter } from '../../src/filter';
+import { MetadataFilter, createFilter, FilterSet } from '../../src/filter';
 import { FilterFuncion } from '../../src/functions';
 
 /**
@@ -17,7 +17,7 @@ export function createFilterFromFunction(
 	const filterSet = fields.reduce((filterSet, field) => {
 		filterSet[field] = filterFunc;
 		return filterSet;
-	}, {});
+	}, {} as FilterSet);
 
 	return createFilter(filterSet);
 }