assert: --assert-full-diff

[redyetidev]

Fixes #51902

This PR adds a --assert-full-diff option to have AssertionErrors print the full diff, rather than a truncated version.

[MoLow]

i'd rather we add an options object to assert.deepStrictEqual and assert.strictEqual instead of a flag

[redyetidev]

i'd rather we add an options object to assert.deepStrictEqual and assert.strictEqual instead of a flag

I'm worried that users may misunderstand the options object for an object to be asserted

[aduh95]

The commit message does not adhere to the guidelines, the tests are not passing: I'm converting this PR to draft. I think you can that reviewing PRs takes time, and as it is currently it doesn't seem like reviewing this PR would be a good use of my time. Have in mind that adding a CLI flag is a higher bar than extending the API.

Sorry if my comment sounds dismissive, I know you spent time making this PR, and I would prefer show more respect of that time you spend, but honestly I don't feel that you sending this PR as is is very respectful of the reviewers' time either.

[redyetidev]

The commit message does not adhere to the guidelines, the tests are not passing: I'm converting this PR to draft. I think you can that reviewing PRs takes time, and as it is currently it doesn't seem like reviewing this PR would be a good use of my time. Have in mind that adding a CLI flag is a higher bar than extending the API.

No problem! I'll fix the PR and verify everything before I resubmit

Sorry if my comment sounds dismissive, I know you spent time making this PR, and I would prefer show more respect of that time you spend, but honestly I don't feel that you sending this PR as is is very respectful of the reviewers' time either.

I'm sorry, I'll fix the PR, I didn't realize it was failing the tests (sorry!).

[redyetidev]

(forgot to un-draft this)

Note: The tests are passing

[redyetidev]

Can this land (once conflicts are resolved), or is it not even worth it?