Need to filter properties from object diffs

[sailingKieler]

Hi friends of vitest,

I'm working a lot on and with Langium and Vitest has been huge benefit there.

I just ran into a problem that I would like to compare two objects for equality like

  expect(A).toBe(B);

A and B be a part of a rather large spider web of objects, und in case of a mismatch the creation of the diff visualization makes the test execution crash after some time of polluting the memory.

I saw that you have a config/customization hook for sorting the properties of objects, but no opportunity for filtering them. Ignoring a few known properties being prone to causing problems would help me a lot.

vitest/packages/utils/src/diff/index.ts

Lines 186 to 190 in 4e60333

	return {
	...formatOptions,
	compareKeys,
	printBasicPrototype,
	}

Ideally I would love if I could contribute a custom plugin to the list of pretty-format plugins:

vitest/packages/utils/src/diff/index.ts

Lines 37 to 57 in 4e60333

	const {
	AsymmetricMatcher,
	DOMCollection,
	DOMElement,
	Immutable,
	ReactElement,
	ReactTestComponent,
	} = prettyFormatPlugins
	const PLUGINS = [
	ReactTestComponent,
	ReactElement,
	DOMElement,
	DOMCollection,
	Immutable,
	AsymmetricMatcher,
	prettyFormatPlugins.Error,
	]
	const FORMAT_OPTIONS = {
	plugins: PLUGINS,
	}

Realizing that getFormatOptions(...) simply ignores my custom plugin contribution almost made me biting my table 😖

vitest/packages/utils/src/diff/index.ts

Lines 180 to 191 in 4e60333

	function getFormatOptions(
	formatOptions: PrettyFormatOptions,
	options?: DiffOptions,
	): PrettyFormatOptions {
	const { compareKeys, printBasicPrototype } = normalizeDiffOptions(options)
	return {
	...formatOptions,
	compareKeys,
	printBasicPrototype,
	}
	}

What do think about opening something in that corner?

[hi-ogawa]

A and B be a part of a rather large spider web of objects, und in case of a mismatch the creation of the diff visualization makes the test execution crash after some time of polluting the memory.

Do you have reproduction? I've also seen oom crash when equality fails #7009
I was guessing this is a bug of printer and cyclic references, or even if it's genuinely too big, it shouldn't crash and need a way to bail out printing early somehow.

Then if this turns out to be not fixable, perhaps we can consider adding user-land workaround.

[sailingKieler]

We've public examples of Langium-based projects (see also our playground https://langium.org/showcase/), I'll see whether I can provide you something. I also saw your cyclic ref detection and wondered why it didn't cut it (literally), but I didn't go deeper there.

[hi-ogawa]

FYI, I'm making a PR to help some cases #7133

Not sure about your case, but it might also help avoiding a large object diff by implementing toJSON as it's prioritized in some cases.