Skip to content

Commit e621d88

Browse files
committed
Fix member linking test
1 parent b46b3c1 commit e621d88

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

src/read-models/shared-state/member-linking.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@ export class MemberLinking {
1818
for (const group of this.grouping) {
1919
for (const memberNumber of numbers) {
2020
if (group.has(memberNumber)) {
21-
numbers.forEach(group.add);
21+
numbers.forEach(v => group.add(v));
22+
return;
2223
}
23-
return;
2424
}
2525
}
2626
this.grouping.push(new Set(numbers));

tests/read-models/shared-state/group-membership-numbers.test.ts

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,21 @@
1-
import {ReadonlyNonEmptyArray} from 'fp-ts/lib/ReadonlyNonEmptyArray';
21
import {MemberLinking} from '../../../src/read-models/shared-state/member-linking';
32

43
describe('Group membership numbers', () => {
54
const tests: {
65
input: number[][];
7-
output: ReadonlyArray<ReadonlyNonEmptyArray<number>>;
6+
output: ReadonlyArray<ReadonlySet<number>>;
87
}[] = [
98
{
109
input: [[1, 2]],
11-
output: [[1, 2]],
10+
output: [new Set([1, 2])],
1211
},
1312
{
1413
input: [
1514
[1, 2],
1615
[2, 3],
1716
[4, 5],
1817
],
19-
output: [
20-
[1, 2, 3],
21-
[4, 5],
22-
],
18+
output: [new Set([1, 2, 3]), new Set([4, 5])],
2319
},
2420
];
2521

0 commit comments

Comments
 (0)