Skip to content

Commit 634e5a5

Browse files
pedrottimarkSimenB
authored andcommitted
expect: fix regression in toThrow for message prop and asymmetric matchers (#7697)
1 parent 6081f7a commit 634e5a5

File tree

4 files changed

+525
-58
lines changed

4 files changed

+525
-58
lines changed

CHANGELOG.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@
125125
- `[babel-plugin-jest-hoist]` Ignore TS type annotations when looking for out-of-scope references ([#7641](https://github.com/facebook/jest/pull/7641))
126126
- `[jest-config]` Add name to project if one does not exist to pick correct resolver ([#5862](https://github.com/facebook/jest/pull/5862))
127127
- `[jest-runtime]` Pass `watchPathIgnorePatterns` to Haste instance ([#7585](https://github.com/facebook/jest/pull/7585))
128-
- `[jest-runtime]` Resolve mock files via Haste when using `require.resolve` ([#7687](https://github.com/facebook/jest/pull/7585))
128+
- `[jest-runtime]` Resolve mock files via Haste when using `require.resolve` ([#7687](https://github.com/facebook/jest/pull/7687))
129129

130130
### Chore & Maintenance
131131

packages/expect/src/__tests__/__snapshots__/toThrowMatchers.test.js.snap

+204
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,91 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

3+
exports[`.toThrow() asymmetric any-Class fail isNot false 1`] = `
4+
"<dim>expect(</><red>received</><dim>).</>toThrow<dim>(</><green>expected</><dim>)</>
5+
6+
Expected asymmetric matcher: <green>Any<Err2></>
7+
8+
Received name: <red>\\"Error\\"</>
9+
Received message: <red>\\"apple\\"</>
10+
11+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
12+
`;
13+
14+
exports[`.toThrow() asymmetric any-Class fail isNot true 1`] = `
15+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrow<dim>(</><green>expected</><dim>)</>
16+
17+
Expected asymmetric matcher: <green>Any<Err></>
18+
19+
Received name: <red>\\"Error\\"</>
20+
Received message: <red>\\"apple\\"</>
21+
22+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
23+
`;
24+
25+
exports[`.toThrow() asymmetric anything fail isNot false 1`] = `
26+
"<dim>expect(</><red>received</><dim>).</>toThrow<dim>(</><green>expected</><dim>)</>
27+
28+
Expected asymmetric matcher: <green>Anything</>
29+
30+
Thrown value: <red>null</>
31+
"
32+
`;
33+
34+
exports[`.toThrow() asymmetric anything fail isNot true 1`] = `
35+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrow<dim>(</><green>expected</><dim>)</>
36+
37+
Expected asymmetric matcher: <green>Anything</>
38+
39+
Received name: <red>\\"Error\\"</>
40+
Received message: <red>\\"apple\\"</>
41+
42+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
43+
`;
44+
45+
exports[`.toThrow() asymmetric no-symbol fail isNot false 1`] = `
46+
"<dim>expect(</><red>received</><dim>).</>toThrow<dim>(</><green>expected</><dim>)</>
47+
48+
Expected asymmetric matcher: <green>{\\"asymmetricMatch\\": [Function asymmetricMatch]}</>
49+
50+
Received name: <red>\\"Error\\"</>
51+
Received message: <red>\\"apple\\"</>
52+
53+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
54+
`;
55+
56+
exports[`.toThrow() asymmetric no-symbol fail isNot true 1`] = `
57+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrow<dim>(</><green>expected</><dim>)</>
58+
59+
Expected asymmetric matcher: <green>{\\"asymmetricMatch\\": [Function asymmetricMatch]}</>
60+
61+
Received name: <red>\\"Error\\"</>
62+
Received message: <red>\\"apple\\"</>
63+
64+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
65+
`;
66+
67+
exports[`.toThrow() asymmetric objectContaining fail isNot false 1`] = `
68+
"<dim>expect(</><red>received</><dim>).</>toThrow<dim>(</><green>expected</><dim>)</>
69+
70+
Expected asymmetric matcher: <green>ObjectContaining {\\"name\\": \\"NotError\\"}</>
71+
72+
Received name: <red>\\"Error\\"</>
73+
Received message: <red>\\"apple\\"</>
74+
75+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
76+
`;
77+
78+
exports[`.toThrow() asymmetric objectContaining fail isNot true 1`] = `
79+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrow<dim>(</><green>expected</><dim>)</>
80+
81+
Expected asymmetric matcher: <green>ObjectContaining {\\"name\\": \\"Error\\"}</>
82+
83+
Received name: <red>\\"Error\\"</>
84+
Received message: <red>\\"apple\\"</>
85+
86+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
87+
`;
88+
389
exports[`.toThrow() error class did not throw at all 1`] = `
490
"<dim>expect(</><red>received</><dim>).</>toThrow<dim>(</><green>expected</><dim>)</>
591

@@ -39,6 +125,22 @@ Received message: <red>\\"apple\\"</>
39125
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
40126
`;
41127

128+
exports[`.toThrow() error-message fail isNot false 1`] = `
129+
"<dim>expect(</><red>received</><dim>).</>toThrow<dim>(</><green>expected</><dim>)</>
130+
131+
Expected message: <green>\\"apple\\"</>
132+
Received message: <red>\\"banana\\"</>
133+
"
134+
`;
135+
136+
exports[`.toThrow() error-message fail isNot true 1`] = `
137+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrow<dim>(</><green>expected</><dim>)</>
138+
139+
Expected message: <green>\\"apple\\"</>
140+
Received message: <red>\\"apple\\"</>
141+
"
142+
`;
143+
42144
exports[`.toThrow() expected is undefined threw, but should not have (non-error falsey) 1`] = `
43145
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrow<dim>()</>
44146

@@ -174,6 +276,92 @@ Received value: <red>\\"Internal Server Error\\"</>
174276
"
175277
`;
176278

279+
exports[`.toThrowError() asymmetric any-Class fail isNot false 1`] = `
280+
"<dim>expect(</><red>received</><dim>).</>toThrowError<dim>(</><green>expected</><dim>)</>
281+
282+
Expected asymmetric matcher: <green>Any<Err2></>
283+
284+
Received name: <red>\\"Error\\"</>
285+
Received message: <red>\\"apple\\"</>
286+
287+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
288+
`;
289+
290+
exports[`.toThrowError() asymmetric any-Class fail isNot true 1`] = `
291+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrowError<dim>(</><green>expected</><dim>)</>
292+
293+
Expected asymmetric matcher: <green>Any<Err></>
294+
295+
Received name: <red>\\"Error\\"</>
296+
Received message: <red>\\"apple\\"</>
297+
298+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
299+
`;
300+
301+
exports[`.toThrowError() asymmetric anything fail isNot false 1`] = `
302+
"<dim>expect(</><red>received</><dim>).</>toThrowError<dim>(</><green>expected</><dim>)</>
303+
304+
Expected asymmetric matcher: <green>Anything</>
305+
306+
Thrown value: <red>null</>
307+
"
308+
`;
309+
310+
exports[`.toThrowError() asymmetric anything fail isNot true 1`] = `
311+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrowError<dim>(</><green>expected</><dim>)</>
312+
313+
Expected asymmetric matcher: <green>Anything</>
314+
315+
Received name: <red>\\"Error\\"</>
316+
Received message: <red>\\"apple\\"</>
317+
318+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
319+
`;
320+
321+
exports[`.toThrowError() asymmetric no-symbol fail isNot false 1`] = `
322+
"<dim>expect(</><red>received</><dim>).</>toThrowError<dim>(</><green>expected</><dim>)</>
323+
324+
Expected asymmetric matcher: <green>{\\"asymmetricMatch\\": [Function asymmetricMatch]}</>
325+
326+
Received name: <red>\\"Error\\"</>
327+
Received message: <red>\\"apple\\"</>
328+
329+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
330+
`;
331+
332+
exports[`.toThrowError() asymmetric no-symbol fail isNot true 1`] = `
333+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrowError<dim>(</><green>expected</><dim>)</>
334+
335+
Expected asymmetric matcher: <green>{\\"asymmetricMatch\\": [Function asymmetricMatch]}</>
336+
337+
Received name: <red>\\"Error\\"</>
338+
Received message: <red>\\"apple\\"</>
339+
340+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
341+
`;
342+
343+
exports[`.toThrowError() asymmetric objectContaining fail isNot false 1`] = `
344+
"<dim>expect(</><red>received</><dim>).</>toThrowError<dim>(</><green>expected</><dim>)</>
345+
346+
Expected asymmetric matcher: <green>ObjectContaining {\\"name\\": \\"NotError\\"}</>
347+
348+
Received name: <red>\\"Error\\"</>
349+
Received message: <red>\\"apple\\"</>
350+
351+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
352+
`;
353+
354+
exports[`.toThrowError() asymmetric objectContaining fail isNot true 1`] = `
355+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrowError<dim>(</><green>expected</><dim>)</>
356+
357+
Expected asymmetric matcher: <green>ObjectContaining {\\"name\\": \\"Error\\"}</>
358+
359+
Received name: <red>\\"Error\\"</>
360+
Received message: <red>\\"apple\\"</>
361+
362+
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
363+
`;
364+
177365
exports[`.toThrowError() error class did not throw at all 1`] = `
178366
"<dim>expect(</><red>received</><dim>).</>toThrowError<dim>(</><green>expected</><dim>)</>
179367

@@ -213,6 +401,22 @@ Received message: <red>\\"apple\\"</>
213401
<dim>at jestExpect (</>packages/expect/src/__tests__/toThrowMatchers-test.js<dim>:24:74)</>"
214402
`;
215403

404+
exports[`.toThrowError() error-message fail isNot false 1`] = `
405+
"<dim>expect(</><red>received</><dim>).</>toThrowError<dim>(</><green>expected</><dim>)</>
406+
407+
Expected message: <green>\\"apple\\"</>
408+
Received message: <red>\\"banana\\"</>
409+
"
410+
`;
411+
412+
exports[`.toThrowError() error-message fail isNot true 1`] = `
413+
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrowError<dim>(</><green>expected</><dim>)</>
414+
415+
Expected message: <green>\\"apple\\"</>
416+
Received message: <red>\\"apple\\"</>
417+
"
418+
`;
419+
216420
exports[`.toThrowError() expected is undefined threw, but should not have (non-error falsey) 1`] = `
217421
"<dim>expect(</><red>received</><dim>).</>not<dim>.</>toThrowError<dim>()</>
218422

0 commit comments

Comments
 (0)