mirror of
https://github.com/git/git
synced 2024-08-25 10:45:59 +00:00
t8005: Nobody writes Russian in shift_jis
The second and third tests of this script expected that Russian strings are converted between ISO-8859-5 and Shift_JIS in the "blame --porcelain" format output correctly. Sure, many platforms may convert between such a combination, but that is only because one of the base character set of Shift_JIS, JIS X 0208, defines codepoints for Russian characters (among others); I do not think anybody uses Shift_JIS when seriously writing Russian, and it is perfectly understandable if iconv() libraries on some platforms fail converting between this combination, as it does not matter in reality. This patch changes the test to verify Japanese strings are converted correctly between EUC-JP and Shift_JIS in the same procedure. The point of the test is not about verifying the platform's iconv() library, but to see if "git blame" makes correct iconv() library calls when it should. We could instead use ISO-8859-5 and KOI8-R as the combination, because they are both meant to represent Russian, in order to make this test meaningful on more platforms, but we already use Shift_JIS vs EUC-JP combinations to test other programs in our test suite, so this combination is safer from the point of view of the portability. Besides, I do not read nor write Russian; sorry ;-) This change allows tests to pass on my (friend's) Solaris 5.11 box. Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
422a82f213
commit
54bc13ce53
|
@ -4,7 +4,7 @@ test_description='git blame encoding conversion'
|
|||
. ./test-lib.sh
|
||||
|
||||
. "$TEST_DIRECTORY"/t8005/utf8.txt
|
||||
. "$TEST_DIRECTORY"/t8005/cp1251.txt
|
||||
. "$TEST_DIRECTORY"/t8005/euc-japan.txt
|
||||
. "$TEST_DIRECTORY"/t8005/sjis.txt
|
||||
|
||||
test_expect_success 'setup the repository' '
|
||||
|
@ -13,10 +13,10 @@ test_expect_success 'setup the repository' '
|
|||
git add file &&
|
||||
git commit --author "$UTF8_NAME <utf8@localhost>" -m "$UTF8_MSG" &&
|
||||
|
||||
echo "CP1251 LINE" >> file &&
|
||||
echo "EUC-JAPAN LINE" >> file &&
|
||||
git add file &&
|
||||
git config i18n.commitencoding cp1251 &&
|
||||
git commit --author "$CP1251_NAME <cp1251@localhost>" -m "$CP1251_MSG" &&
|
||||
git config i18n.commitencoding eucJP &&
|
||||
git commit --author "$EUC_JAPAN_NAME <euc-japan@localhost>" -m "$EUC_JAPAN_MSG" &&
|
||||
|
||||
echo "SJIS LINE" >> file &&
|
||||
git add file &&
|
||||
|
@ -41,17 +41,17 @@ test_expect_success \
|
|||
'
|
||||
|
||||
cat >expected <<EOF
|
||||
author $CP1251_NAME
|
||||
summary $CP1251_MSG
|
||||
author $CP1251_NAME
|
||||
summary $CP1251_MSG
|
||||
author $CP1251_NAME
|
||||
summary $CP1251_MSG
|
||||
author $EUC_JAPAN_NAME
|
||||
summary $EUC_JAPAN_MSG
|
||||
author $EUC_JAPAN_NAME
|
||||
summary $EUC_JAPAN_MSG
|
||||
author $EUC_JAPAN_NAME
|
||||
summary $EUC_JAPAN_MSG
|
||||
EOF
|
||||
|
||||
test_expect_success \
|
||||
'blame respects i18n.logoutputencoding' '
|
||||
git config i18n.logoutputencoding cp1251 &&
|
||||
git config i18n.logoutputencoding eucJP &&
|
||||
git blame --incremental file | \
|
||||
egrep "^(author|summary) " > actual &&
|
||||
test_cmp actual expected
|
||||
|
@ -76,8 +76,8 @@ test_expect_success \
|
|||
cat >expected <<EOF
|
||||
author $SJIS_NAME
|
||||
summary $SJIS_MSG
|
||||
author $CP1251_NAME
|
||||
summary $CP1251_MSG
|
||||
author $EUC_JAPAN_NAME
|
||||
summary $EUC_JAPAN_MSG
|
||||
author $UTF8_NAME
|
||||
summary $UTF8_MSG
|
||||
EOF
|
||||
|
|
2
t/t8005/euc-japan.txt
Normal file
2
t/t8005/euc-japan.txt
Normal file
|
@ -0,0 +1,2 @@
|
|||
EUC_JAPAN_NAME="山田 太郎"
|
||||
EUC_JAPAN_MSG="ブレームのテストです。"
|
|
@ -1,2 +1,2 @@
|
|||
SJIS_NAME="„I„r„p„~ „P„u„„„‚„€„r„y„‰ „R„y„t„€„‚„€„r"
|
||||
SJIS_MSG="„S„u„ƒ„„„€„r„€„u „ƒ„€„€„q„‹„u„~„y„u"
|
||||
SJIS_NAME="山田 太郎"
|
||||
SJIS_MSG="ブレームのテストです。"
|
||||
|
|
|
@ -1,2 +1,2 @@
|
|||
UTF8_NAME="Иван Петрович Сидоров"
|
||||
UTF8_MSG="Тестовое сообщение"
|
||||
UTF8_NAME="山田 太郎"
|
||||
UTF8_MSG="ブレームのテストです。"
|
||||
|
|
Loading…
Reference in a new issue