fix: should not expand diff by default (#516)

Author: 9aoy

e2e/snapshot/diff.test.ts

@@ -0,0 +1,28 @@
+import { dirname } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { expect, it } from '@rstest/core';
+import { runRstestCli } from '../scripts';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+
+it('should show diff as expected', async () => {
+  const { cli } = await runRstestCli({
+    command: 'rstest',
+    args: ['run', 'fixtures/diff.test.ts'],
+    options: {
+      nodeOptions: {
+        cwd: __dirname,
+      },
+    },
+  });
+
+  await cli.exec;
+  expect(cli.exec.process?.exitCode).toBe(1);
+
+  const logs = cli.stdout.split('\n').filter(Boolean);
+
+  expect(logs.length).toBeLessThan(100);
+  expect(logs.find((log) => log.includes('-     99"'))).toBeTruthy();
+  expect(logs.find((log) => log.includes('+     100"'))).toBeTruthy();
+});

e2e/snapshot/fixtures/__snapshots__/diff.test.ts.snap

@@ -0,0 +1,105 @@
+// Rstest Snapshot v1
+
+exports[`test snapshot > should failed when snapshot unmatched 1`] = `
+"
+    0
+    1
+    2
+    3
+    4
+    5
+    6
+    7
+    8
+    9
+    10
+    11
+    12
+    13
+    14
+    15
+    16
+    17
+    18
+    19
+    20
+    21
+    22
+    23
+    24
+    25
+    26
+    27
+    28
+    29
+    30
+    31
+    32
+    33
+    34
+    35
+    36
+    37
+    38
+    39
+    40
+    41
+    42
+    43
+    44
+    45
+    46
+    47
+    48
+    49
+    50
+    51
+    52
+    53
+    54
+    55
+    56
+    57
+    58
+    59
+    60
+    61
+    62
+    63
+    64
+    65
+    66
+    67
+    68
+    69
+    70
+    71
+    72
+    73
+    74
+    75
+    76
+    77
+    78
+    79
+    80
+    81
+    82
+    83
+    84
+    85
+    86
+    87
+    88
+    89
+    90
+    91
+    92
+    93
+    94
+    95
+    96
+    97
+    98
+    99"
+`;

e2e/snapshot/fixtures/diff.test.ts

@@ -0,0 +1,14 @@
+import { describe, expect, it } from '@rstest/core';
+
+describe('test snapshot', () => {
+  it('should failed when snapshot unmatched', () => {
+    let content = '';
+
+    for (let i = 0; i < 101; i++) {
+      content += `
+    ${i}`;
+    }
+
+    expect(content).toMatchSnapshot();
+  });
+});

packages/core/src/runtime/util.ts

@@ -37,7 +37,9 @@ export const formatTestError = (err: any, test?: Test): FormattedError[] => {
         error.expected !== undefined &&
         error.actual !== undefined)
     ) {
-      errObj.diff = diff(err.expected, err.actual)!;
+      errObj.diff = diff(err.expected, err.actual, {
+        expand: false,
+      })!;
     }
 
     for (const key of ['actual', 'expected'] as const) {