Re: [PATCH v2 2/2] t0028: add more tests

2019-09-24 Thread Alexandr Miloslavskiy
On 24.09.2019 8:21, Johannes Sixt wrote: What are we testing here? Is there some back-and-forth conversion going on, and are we testing that the conversion happens at all, or that the correct conversion/encoding is picked, or that the conversion that is finally chosen is correct? Why does it help

Re: [PATCH v2 2/2] t0028: add more tests

2019-09-24 Thread Alexandr Miloslavskiy
On 24.09.2019 6:06, Torsten Bögershausen wrote: Would this make more sense: After I discovered that UTF-16-LE-BOM test was bugged, I decided that better tests are required OK > Looking at the other test cases, should utf-8 be written as UTF-8 > for consistency ? OK > General remark: > Do we

Re: [PATCH v2 2/2] t0028: add more tests

2019-09-23 Thread Johannes Sixt
Am 23.09.19 um 12:04 schrieb Alexandr Miloslavskiy via GitGitGadget: > From: Alexandr Miloslavskiy > > After I discovered that UTF-16-LE-BOM test was bugged and still > succeeded, I decided that better tests are required. Possibly the best > option here is to compare git results against hardcoded

Re: [PATCH v2 2/2] t0028: add more tests

2019-09-23 Thread Torsten Bögershausen
On Mon, Sep 23, 2019 at 03:04:19AM -0700, Alexandr Miloslavskiy via GitGitGadget wrote: > From: Alexandr Miloslavskiy Thanks for the tests, some nit-picks inline. > > After I discovered that UTF-16-LE-BOM test was bugged and still > succeeded... My interpretation is that the \000\000 must be h

[PATCH v2 2/2] t0028: add more tests

2019-09-23 Thread Alexandr Miloslavskiy via GitGitGadget
From: Alexandr Miloslavskiy After I discovered that UTF-16-LE-BOM test was bugged and still succeeded, I decided that better tests are required. Possibly the best option here is to compare git results against hardcoded ground truth. The new tests also cover more interesting chars where (ANSI !=