Skip to content

Commit 2fd09fc

Browse files
☔ test(append): Prevent regression on append's pureness.
1 parent 196675c commit 2fd09fc

File tree

2 files changed

+33
-20
lines changed

2 files changed

+33
-20
lines changed

test/src/regression/fast/fill_right.js

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,13 @@ import {ABSTRACT_COUNTER as COUNTER, measure} from '../../_fixtures.js';
77
import {empty} from '../../../../src/index.js';
88

99
test('left instanceof Four', (t) => {
10-
const tree = empty(COUNTER)
11-
.append(range(2))
12-
.cons(-1)
13-
.cons(-2)
14-
.cons(-3)
15-
.append(range(2, 4));
16-
t.deepEqual(Array.from(tree), Array.from(range(-3, 4)));
17-
t.is(tree.measure(), measure(COUNTER, range(-3, 4)));
10+
const before = empty(COUNTER).append(range(2)).cons(-1).cons(-2).cons(-3);
11+
12+
const after = before.append(range(2, 4));
13+
14+
t.deepEqual(Array.from(after), Array.from(range(-3, 4)));
15+
t.is(after.measure(), measure(COUNTER, range(-3, 4)));
16+
17+
t.deepEqual(Array.from(before), Array.from(range(-3, 2)));
18+
t.is(before.measure(), measure(COUNTER, range(-3, 2)));
1819
});

test/src/regression/tree/implementations/Deep/append.js

Lines changed: 24 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -9,25 +9,37 @@ import {from} from '../../../../../../src/index.js';
99
test('empty', (t) => {
1010
const r0 = range(0);
1111
const r200 = range(200);
12-
let tree = from(COUNTER, r200);
13-
tree = tree.append(r0);
14-
t.deepEqual(Array.from(tree), Array.from(r200).concat(Array.from(r0)));
15-
t.is(tree.measure(), 200);
12+
const before = from(COUNTER, r200);
13+
const after = before.append(r0);
14+
15+
t.deepEqual(Array.from(after), Array.from(r200).concat(Array.from(r0)));
16+
t.is(after.measure(), 200);
17+
18+
t.deepEqual(Array.from(before), Array.from(r200));
19+
t.is(before.measure(), 200);
1620
});
1721

1822
test('range(1)', (t) => {
1923
const r1 = range(1);
2024
const r200 = range(200);
21-
let tree = from(COUNTER, r200);
22-
tree = tree.append(r1);
23-
t.deepEqual(Array.from(tree), Array.from(r200).concat(Array.from(r1)));
24-
t.is(tree.measure(), 201);
25+
const before = from(COUNTER, r200);
26+
const after = before.append(r1);
27+
28+
t.deepEqual(Array.from(after), Array.from(r200).concat(Array.from(r1)));
29+
t.is(after.measure(), 201);
30+
31+
t.deepEqual(Array.from(before), Array.from(r200));
32+
t.is(before.measure(), 200);
2533
});
2634

2735
test('r200', (t) => {
2836
const r200 = range(200);
29-
let tree = from(COUNTER, r200);
30-
tree = tree.append(r200);
31-
t.deepEqual(Array.from(tree), Array.from(r200).concat(Array.from(r200)));
32-
t.is(tree.measure(), 400);
37+
const before = from(COUNTER, r200);
38+
const after = before.append(r200);
39+
40+
t.deepEqual(Array.from(after), Array.from(r200).concat(Array.from(r200)));
41+
t.is(after.measure(), 400);
42+
43+
t.deepEqual(Array.from(before), Array.from(r200));
44+
t.is(before.measure(), 200);
3345
});

0 commit comments

Comments
 (0)