From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-5.9 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_HI,RP_MATCHES_RCVD shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id 5E86A207DF for ; Mon, 19 Sep 2016 18:55:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752603AbcISSzE (ORCPT ); Mon, 19 Sep 2016 14:55:04 -0400 Received: from ikke.info ([178.21.113.177]:38816 "EHLO vps892.directvps.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751543AbcISSzD (ORCPT ); Mon, 19 Sep 2016 14:55:03 -0400 Received: by vps892.directvps.nl (Postfix, from userid 182) id 923D04400BF; Mon, 19 Sep 2016 20:55:01 +0200 (CEST) Received: from io.ikke (unknown [10.8.0.30]) by vps892.directvps.nl (Postfix) with ESMTP id 4C0474400B9; Mon, 19 Sep 2016 20:54:57 +0200 (CEST) From: Kevin Daudt To: git@vger.kernel.org Cc: Kevin Daudt , Swift Geek , Jeff King , Junio C Hamano Subject: [PATCH v2 1/2] t5100-mailinfo: replace common path prefix with variable Date: Mon, 19 Sep 2016 20:54:39 +0200 Message-Id: <20160919185440.18234-2-me@ikke.info> X-Mailer: git-send-email 2.10.0.86.g6ffa4f1.dirty In-Reply-To: <20160919185440.18234-1-me@ikke.info> References: <20160919185440.18234-1-me@ikke.info> In-Reply-To: <20160916210204.31282-1-me@ikke.info> References: <20160916210204.31282-1-me@ikke.info> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Many tests need to store data in a file, and repeat the same pattern to refer to that path: "$TEST_DATA"/t5100/ Create a variable that contains this path, and use that instead. Signed-off-by: Kevin Daudt --- t/t5100-mailinfo.sh | 56 +++++++++++++++++++++++++++-------------------------- 1 file changed, 29 insertions(+), 27 deletions(-) diff --git a/t/t5100-mailinfo.sh b/t/t5100-mailinfo.sh index 1a5a546..27bf3b8 100755 --- a/t/t5100-mailinfo.sh +++ b/t/t5100-mailinfo.sh @@ -7,8 +7,10 @@ test_description='git mailinfo and git mailsplit test' . ./test-lib.sh +DATA="$TEST_DIRECTORY/t5100" + test_expect_success 'split sample box' \ - 'git mailsplit -o. "$TEST_DIRECTORY"/t5100/sample.mbox >last && + 'git mailsplit -o. "$DATA"/sample.mbox >last && last=$(cat last) && echo total is $last && test $(cat last) = 17' @@ -17,9 +19,9 @@ check_mailinfo () { mail=$1 opt=$2 mo="$mail$opt" git mailinfo -u $opt msg$mo patch$mo <$mail >info$mo && - test_cmp "$TEST_DIRECTORY"/t5100/msg$mo msg$mo && - test_cmp "$TEST_DIRECTORY"/t5100/patch$mo patch$mo && - test_cmp "$TEST_DIRECTORY"/t5100/info$mo info$mo + test_cmp "$DATA"/msg$mo msg$mo && + test_cmp "$DATA"/patch$mo patch$mo && + test_cmp "$DATA"/info$mo info$mo } @@ -27,15 +29,15 @@ for mail in 00* do test_expect_success "mailinfo $mail" ' check_mailinfo $mail "" && - if test -f "$TEST_DIRECTORY"/t5100/msg$mail--scissors + if test -f "$DATA"/msg$mail--scissors then check_mailinfo $mail --scissors fi && - if test -f "$TEST_DIRECTORY"/t5100/msg$mail--no-inbody-headers + if test -f "$DATA"/msg$mail--no-inbody-headers then check_mailinfo $mail --no-inbody-headers fi && - if test -f "$TEST_DIRECTORY"/t5100/msg$mail--message-id + if test -f "$DATA"/msg$mail--message-id then check_mailinfo $mail --message-id fi @@ -45,7 +47,7 @@ done test_expect_success 'split box with rfc2047 samples' \ 'mkdir rfc2047 && - git mailsplit -orfc2047 "$TEST_DIRECTORY"/t5100/rfc2047-samples.mbox \ + git mailsplit -orfc2047 "$DATA"/rfc2047-samples.mbox \ >rfc2047/last && last=$(cat rfc2047/last) && echo total is $last && @@ -56,18 +58,18 @@ do test_expect_success "mailinfo $mail" ' git mailinfo -u $mail-msg $mail-patch <$mail >$mail-info && echo msg && - test_cmp "$TEST_DIRECTORY"/t5100/empty $mail-msg && + test_cmp "$DATA"/empty $mail-msg && echo patch && - test_cmp "$TEST_DIRECTORY"/t5100/empty $mail-patch && + test_cmp "$DATA"/empty $mail-patch && echo info && - test_cmp "$TEST_DIRECTORY"/t5100/rfc2047-info-$(basename $mail) $mail-info + test_cmp "$DATA"/rfc2047-info-$(basename $mail) $mail-info ' done test_expect_success 'respect NULs' ' - git mailsplit -d3 -o. "$TEST_DIRECTORY"/t5100/nul-plain && - test_cmp "$TEST_DIRECTORY"/t5100/nul-plain 001 && + git mailsplit -d3 -o. "$DATA"/nul-plain && + test_cmp "$DATA"/nul-plain 001 && (cat 001 | git mailinfo msg patch) && test_line_count = 4 patch @@ -75,52 +77,52 @@ test_expect_success 'respect NULs' ' test_expect_success 'Preserve NULs out of MIME encoded message' ' - git mailsplit -d5 -o. "$TEST_DIRECTORY"/t5100/nul-b64.in && - test_cmp "$TEST_DIRECTORY"/t5100/nul-b64.in 00001 && + git mailsplit -d5 -o. "$DATA"/nul-b64.in && + test_cmp "$DATA"/nul-b64.in 00001 && git mailinfo msg patch <00001 && - test_cmp "$TEST_DIRECTORY"/t5100/nul-b64.expect patch + test_cmp "$DATA"/nul-b64.expect patch ' test_expect_success 'mailinfo on from header without name works' ' mkdir info-from && - git mailsplit -oinfo-from "$TEST_DIRECTORY"/t5100/info-from.in && - test_cmp "$TEST_DIRECTORY"/t5100/info-from.in info-from/0001 && + git mailsplit -oinfo-from "$DATA"/info-from.in && + test_cmp "$DATA"/info-from.in info-from/0001 && git mailinfo info-from/msg info-from/patch \ info-from/out && - test_cmp "$TEST_DIRECTORY"/t5100/info-from.expect info-from/out + test_cmp "$DATA"/info-from.expect info-from/out ' test_expect_success 'mailinfo finds headers after embedded From line' ' mkdir embed-from && - git mailsplit -oembed-from "$TEST_DIRECTORY"/t5100/embed-from.in && - test_cmp "$TEST_DIRECTORY"/t5100/embed-from.in embed-from/0001 && + git mailsplit -oembed-from "$DATA"/embed-from.in && + test_cmp "$DATA"/embed-from.in embed-from/0001 && git mailinfo embed-from/msg embed-from/patch \ embed-from/out && - test_cmp "$TEST_DIRECTORY"/t5100/embed-from.expect embed-from/out + test_cmp "$DATA"/embed-from.expect embed-from/out ' test_expect_success 'mailinfo on message with quoted >From' ' mkdir quoted-from && - git mailsplit -oquoted-from "$TEST_DIRECTORY"/t5100/quoted-from.in && - test_cmp "$TEST_DIRECTORY"/t5100/quoted-from.in quoted-from/0001 && + git mailsplit -oquoted-from "$DATA"/quoted-from.in && + test_cmp "$DATA"/quoted-from.in quoted-from/0001 && git mailinfo quoted-from/msg quoted-from/patch \ quoted-from/out && - test_cmp "$TEST_DIRECTORY"/t5100/quoted-from.expect quoted-from/msg + test_cmp "$DATA"/quoted-from.expect quoted-from/msg ' test_expect_success 'mailinfo unescapes with --mboxrd' ' mkdir mboxrd && git mailsplit -omboxrd --mboxrd \ - "$TEST_DIRECTORY"/t5100/sample.mboxrd >last && + "$DATA"/sample.mboxrd >last && test x"$(cat last)" = x2 && for i in 0001 0002 do git mailinfo mboxrd/msg mboxrd/patch \ mboxrd/out && - test_cmp "$TEST_DIRECTORY"/t5100/${i}mboxrd mboxrd/msg + test_cmp "$DATA"/${i}mboxrd mboxrd/msg done && sp=" " && echo "From " >expect && -- 2.10.0.86.g6ffa4f1.dirty