Skip to content

Commit 93f8946

Browse files
authored
Merge pull request #209 from microbit-foundation/192-add-architecture-test-to-test-for-implementation-in-filters-enum
Added test to ensure all filters are implemented in determineFilter()
2 parents 1f0394b + 91c2953 commit 93f8946

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
/**
2+
* @jest-environment jsdom
3+
*/
4+
5+
import fs from 'fs';
6+
import { determineFilter, Filters } from '../script/datafunctions';
7+
import { doesNotThrow } from 'assert';
8+
function enumKeys<O extends object, K extends keyof O = keyof O>(obj: O): K[] {
9+
return Object.keys(obj).filter(k => Number.isNaN(+k)) as K[];
10+
}
11+
describe('Data functions architecture test', () => {
12+
test('All filters should be implemented in determineFilter', () => {
13+
for (const filter of enumKeys(Filters)) {
14+
expect(() => determineFilter(Filters[filter])).not.toThrow();
15+
}
16+
});
17+
});

0 commit comments

Comments
 (0)