Re: [PATCH v2 1/2] t4205, t6006: Add failing tests for the case when i18n.logOutputEncoding is set
On Fri, May 16, 2014 at 11:49:40AM -0700, Junio C Hamano wrote: > Alexey Shumkin writes: > > > Pretty format string %<(N,[ml]trunc)>%s truncates subject to a given > > length with an appropriate padding. This works for non-ASCII texts when > > i18n.logOutputEncoding is UTF-8 only (independently of a printed commit > > message encoding) but does not work when i18n.logOutputEncoding is NOT > > UTF-8. > > > > There were no breakages as far as were no tests for the case > > when both a commit message and logOutputEncoding are not UTF-8. > > > > Add failing tests for that which will be fixed in the next patch. > > > > Signed-off-by: Alexey Shumkin > > Reviewed-by: Nguyễn Thái Ngọc Duy > > --- > > t/t4205-log-pretty-formats.sh | 169 > > ++ > > t/t6006-rev-list-format.sh| 75 ++- > > 2 files changed, 242 insertions(+), 2 deletions(-) > > > > diff --git a/t/t4205-log-pretty-formats.sh b/t/t4205-log-pretty-formats.sh > > index 2a6278b..6791e0d 100755 > > --- a/t/t4205-log-pretty-formats.sh > > +++ b/t/t4205-log-pretty-formats.sh > > @@ -153,6 +153,19 @@ EOF > > test_cmp expected actual > > ' > > > > +test_expect_success 'left alignment formatting. i18n.logOutputEncoding' ' > > + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(40)%s" > > >actual && > > + # complete the incomplete line at the end > > + echo >>actual && > > Would it change the meaning of the test if you used tformat: instead > of format: (or --format="%<(40)%s")? If it doesn't, it would make > it unnecessary to append an extra LF and explain why you do so. Well, actually, I just copied previous tests and added i18n.logOutputEncoding. But as I can see in the code - no, tformat will not change the meaning. so, may be there is a reason to change that (initial) tests from format to tformat first? And then add mine new. > > > + qz_to_tab_space> It is minor but many existing uses of iconv in our tests spell these > as UTF-8 and ISO8859-1 in uppercase. I vaguely recall there was a > portability concern to favor the ones that are used in existing > tests, but probably it no longer matters (I see you added the > lowercase one with de6029a2 mid last year), so I am fine if these > stay lowercase. I've grep'ed for lowercase iso8859-1 in test, and found almost all of them is code added by me. Grep for uppercase gives more results. I can refactor that first for uniformity. > > > + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(1)%s" > > >actual && > > + # complete the incomplete line at the end > > + echo >>actual && > > Likewise for all the other "--pretty=format:" followed by an echo. the same copy-paste-modify. > > Thanks. -- Alexey Shumkin -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 1/2] t4205, t6006: Add failing tests for the case when i18n.logOutputEncoding is set
Alexey Shumkin writes: > Pretty format string %<(N,[ml]trunc)>%s truncates subject to a given > length with an appropriate padding. This works for non-ASCII texts when > i18n.logOutputEncoding is UTF-8 only (independently of a printed commit > message encoding) but does not work when i18n.logOutputEncoding is NOT > UTF-8. > > There were no breakages as far as were no tests for the case > when both a commit message and logOutputEncoding are not UTF-8. > > Add failing tests for that which will be fixed in the next patch. > > Signed-off-by: Alexey Shumkin > Reviewed-by: Nguyễn Thái Ngọc Duy > --- > t/t4205-log-pretty-formats.sh | 169 > ++ > t/t6006-rev-list-format.sh| 75 ++- > 2 files changed, 242 insertions(+), 2 deletions(-) > > diff --git a/t/t4205-log-pretty-formats.sh b/t/t4205-log-pretty-formats.sh > index 2a6278b..6791e0d 100755 > --- a/t/t4205-log-pretty-formats.sh > +++ b/t/t4205-log-pretty-formats.sh > @@ -153,6 +153,19 @@ EOF > test_cmp expected actual > ' > > +test_expect_success 'left alignment formatting. i18n.logOutputEncoding' ' > + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(40)%s" > >actual && > + # complete the incomplete line at the end > + echo >>actual && Would it change the meaning of the test if you used tformat: instead of format: (or --format="%<(40)%s")? If it doesn't, it would make it unnecessary to append an extra LF and explain why you do so. > + qz_to_tab_space+ git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(1)%s" > >actual && > + # complete the incomplete line at the end > + echo >>actual && Likewise for all the other "--pretty=format:" followed by an echo. Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v2 1/2] t4205, t6006: Add failing tests for the case when i18n.logOutputEncoding is set
Pretty format string %<(N,[ml]trunc)>%s truncates subject to a given length with an appropriate padding. This works for non-ASCII texts when i18n.logOutputEncoding is UTF-8 only (independently of a printed commit message encoding) but does not work when i18n.logOutputEncoding is NOT UTF-8. There were no breakages as far as were no tests for the case when both a commit message and logOutputEncoding are not UTF-8. Add failing tests for that which will be fixed in the next patch. Signed-off-by: Alexey Shumkin Reviewed-by: Nguyễn Thái Ngọc Duy --- t/t4205-log-pretty-formats.sh | 169 ++ t/t6006-rev-list-format.sh| 75 ++- 2 files changed, 242 insertions(+), 2 deletions(-) diff --git a/t/t4205-log-pretty-formats.sh b/t/t4205-log-pretty-formats.sh index 2a6278b..6791e0d 100755 --- a/t/t4205-log-pretty-formats.sh +++ b/t/t4205-log-pretty-formats.sh @@ -153,6 +153,19 @@ EOF test_cmp expected actual ' +test_expect_success 'left alignment formatting. i18n.logOutputEncoding' ' + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(40)%s" >actual && + # complete the incomplete line at the end + echo >>actual && + qz_to_tab_spaceactual && # complete the incomplete line at the end @@ -166,6 +179,19 @@ EOF test_cmp expected actual ' +test_expect_success 'left alignment formatting at the nth column. i18n.logOutputEncoding' ' + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%h %<|(40)%s" >actual && + # complete the incomplete line at the end + echo >>actual && + qz_to_tab_space actual && # complete the incomplete line at the end @@ -179,6 +205,19 @@ EOF test_cmp expected actual ' +test_expect_success 'left alignment formatting with no padding. i18n.logOutputEncoding' ' + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(1)%s" >actual && + # complete the incomplete line at the end + echo >>actual && + cat actual && # complete the incomplete line at the end @@ -192,6 +231,19 @@ EOF test_cmp expected actual ' +test_expect_failure 'left alignment formatting with trunc. i18n.logOutputEncoding' ' + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(10,trunc)%s" >actual && + # complete the incomplete line at the end + echo >>actual && + qz_to_tab_space actual && # complete the incomplete line at the end @@ -205,6 +257,19 @@ EOF test_cmp expected actual ' +test_expect_failure 'left alignment formatting with ltrunc. i18n.logOutputEncoding' ' + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(10,ltrunc)%s" >actual && + # complete the incomplete line at the end + echo >>actual && + qz_to_tab_space actual && # complete the incomplete line at the end @@ -218,6 +283,19 @@ EOF test_cmp expected actual ' +test_expect_failure 'left alignment formatting with mtrunc. i18n.logOutputEncoding' ' + git -c i18n.logOutputEncoding=iso8859-1 log --pretty="format:%<(10,mtrunc)%s" >actual && + # complete the incomplete line at the end + echo >>actual && + qz_to_tab_space (40)%s" >actual && # complete the incomplete line at the end @@ -231,6 +309,19 @@ EOF test_cmp expected actual ' +test_expect_success 'right alignment formatting. i18n.logOutputEncoding'