{{ message }}
BridgeJS: show unified diff on snapshot mismatch#600
Merged
kateinoigakukun merged 1 commit intoFeb 8, 2026
Merged
Conversation
Member
Sorry, something went wrong.
Contributor
There was a problem hiding this comment.
Pull request overview
Improves BridgeJS snapshot test failure diagnostics by emitting an .actual file and including a unified diff in the test failure message to make mismatches easier to inspect.
Changes:
- Write
<snapshot>.actualon mismatch and include adiff -uunified diff in the expectation message. - Add a helper to run
diffviaProcessand capture its output.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
Hide details
View details
kateinoigakukun
merged commit
kateinoigakukun
merged commit 180c010
into
main
Feb 8, 2026
16 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.
Motivation
Snapshot failures only reported file paths, making it slow to see what changed.
Overview
Enhance BridgeJS snapshot assertion to write the .actual file, run
diff -u, and include the unified diff in the expectation message.