fix: show diff correctly (#3620)

packages/utils/src/diff/index.ts

@@ -60,6 +60,12 @@ const FALLBACK_FORMAT_OPTIONS = {
 // Generate a string that will highlight the difference between two values
 // with green and red. (similar to how github does code diffing)
 
+/**
+ * @param a Expected value
+ * @param b Received value
+ * @param options Diff options
+ * @returns
+ */
 export function diff(a: any, b: any, options?: DiffOptions): string | null {
   if (Object.is(a, b))
     return ''

packages/utils/src/error.ts

@@ -97,7 +97,7 @@ export function processError(err: any) {
     err.nameStr = String(err.name)
 
   if (err.showDiff || (err.showDiff === undefined && err.expected !== undefined && err.actual !== undefined))
-    err.diff = diff(err.actual, err.expected)
+    err.diff = diff(err.expected, err.actual)
 
   if (typeof err.expected !== 'string')
     err.expected = stringify(err.expected, 10)

test/core/test/jest-expect.test.ts

@@ -1,7 +1,9 @@
 /* eslint-disable no-sparse-arrays */
 import { AssertionError } from 'node:assert'
 import { describe, expect, it, vi } from 'vitest'
-import { generateToBeMessage } from '@vitest/expect'
+import { generateToBeMessage, setupColors } from '@vitest/expect'
+import { processError } from '@vitest/utils/error'
+import { getDefaultColors } from '@vitest/utils'
 
 class TestError extends Error {}
 
@@ -753,4 +755,17 @@ it('compatible with jest', () => {
   expect(state.assertionCalls).toBe(2)
 })
 
+it('correctly prints diff', () => {
+  try {
+    expect({ a: 1 }).toEqual({ a: 2 })
+    expect.unreachable()
+  }
+  catch (err) {
+    setupColors(getDefaultColors())
+    const error = processError(err)
+    expect(error.diff).toContain('-   "a": 2')
+    expect(error.diff).toContain('+   "a": 1')
+  }
+})
+
 it('timeout', () => new Promise(resolve => setTimeout(resolve, 500)))

test/reporters/tests/__snapshots__/html.test.ts.snap

@@ -48,8 +48,8 @@ exports[`html reporter > resolves to "failing" status for test file "json-fail"
               "diff": "- Expected
 + Received
 
-- 2
-+ 1",
+- 1
++ 2",
               "expected": "1",
               "message": "expected 2 to deeply equal 1",
               "name": "AssertionError",
@@ -68,8 +68,8 @@ exports[`html reporter > resolves to "failing" status for test file "json-fail"
                 "diff": "- Expected
 + Received
 
-- 2
-+ 1",
+- 1
++ 2",
                 "expected": "1",
                 "message": "expected 2 to deeply equal 1",
                 "name": "AssertionError",