# Re: \[PATCH\] Add testcase for merging in a CRLF repo, showing that conflict file is in LF only

- From: Johannes Sixt \<hidden\>
- Date: 2016-06-15 22:44:43 
- List: git
- Page: https://ratatoskr.run/git/2016/06/7350813
- Thread: https://ratatoskr.run/git/2016/06/7350809/t.md

```text
Marius Storm-Olsen schrieb:
> An LF only conflict file results in the resolved file being in LF,
> the commit is in LF and a warning saying that LF will be replaced
> by CRLF, and the working dir ends up with a mix of CRLF and LF files.

After reading these 3 lines I've no idea what you are talking about. Can
you translate this to English, please? ;-)

>  Sorry, no patch to actually *fix* the problem.

Then you should use test_expect_failure instead of test_expect_success.
And maybe also mention it in the commit message.

> +test_expect_success 'Check that conflict file is CRLF' '
> +	git reset --hard a &&
> +	! git merge side &&

	test_must_fail git merge side &&

> +	cat file | remove_cr | append_cr >file.temp &&
> +	test_cmp file file.temp
> +'

-- Hannes
```