Skip to content

test: assert exact CRLF diagnostics - #39

Open
PandaHUN777 wants to merge 1 commit into
opensource-nepal:mainfrom
PandaHUN777:test/crlf-diagnostics-32
Open

PandaHUN777 wants to merge 1 commit into
opensource-nepal:mainfrom
PandaHUN777:test/crlf-diagnostics-32

Conversation

@PandaHUN777

Copy link
Copy Markdown

Summary

  • Assert the exact diagnostic for CRLF commit messages read by --hash, --from-hash, and --file.
  • Update the command-line newline normalization note to match current behavior.
  • Add a lone-CR --file regression that detects removal of file newline normalization.

Closes #32

Validation

  • cargo +stable build
  • cargo +stable test
  • cargo +stable fmt --all --check
  • cargo +stable clippy --all-targets --all-features -- -D warnings
  • git diff --check
  • Confirmed the lone-CR test fails when file newline normalization is removed.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: CRLF parity tests assert exit codes, so they pass on both sides of the fix

1 participant