Skip to content

Commit 1b3fda2

Browse files
committed
test(combine): use expect extensions for Result
1 parent 55de237 commit 1b3fda2

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

test/result/combine.spec.ts

+8-10
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,15 @@ describe('Result', () => {
88

99
const result = Result.combine({ success, failure });
1010

11-
expect(result.isFailure).toBe(true);
11+
expect(result).toFailWith('1st Error');
1212
});
1313

1414
test('succeeds if one results succeed', () => {
1515
const success = Result.success(1);
1616

1717
const result = Result.combine({ success });
1818

19-
expect(result.isSuccess).toBe(true);
19+
expect(result).toSucceed();
2020
});
2121

2222
test('succeeds if all results succeed', () => {
@@ -25,7 +25,7 @@ describe('Result', () => {
2525

2626
const result = Result.combine({ success_1, success_2 });
2727

28-
expect(result.isSuccess).toBe(true);
28+
expect(result).toSucceed();
2929
});
3030

3131
test('yields all result values on success', () => {
@@ -34,10 +34,10 @@ describe('Result', () => {
3434

3535
const result = Result.combine({ success_1, success_2 });
3636

37-
const values = result.getValueOrThrow();
38-
39-
expect(values.success_1).toBe(1);
40-
expect(values.success_2).toEqual({ name: 'Arthur' });
37+
expect(result).toSucceedWith({
38+
success_1: 1,
39+
success_2: { name: 'Arthur' },
40+
});
4141
});
4242

4343
test('concatenates error messages', () => {
@@ -46,11 +46,9 @@ describe('Result', () => {
4646
const failure_2_message = '2nd Error';
4747
const failure_2 = Result.failure(failure_2_message);
4848

49-
const expected_message = `${failure_1_message}, ${failure_2_message}`;
50-
5149
const result = Result.combine({ failure_1, failure_2 });
5250

53-
expect(result.getErrorOrThrow()).toBe(expected_message);
51+
expect(result).toFailWith(`${failure_1_message}, ${failure_2_message}`);
5452
});
5553
});
5654
});

0 commit comments

Comments
 (0)