From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pranit Bauva Subject: [PATCH v15 1/7] t0040-test-parse-options.sh: fix style issues Date: Sun, 1 May 2016 01:33:30 +0530 Message-ID: <1462046616-2582-1-git-send-email-pranit.bauva@gmail.com> Cc: sunshine@sunshineco.com, gitster@pobox.com, Pranit Bauva To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Sat Apr 30 22:04:11 2016 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1awb7a-0000vP-WE for gcvg-git-2@plane.gmane.org; Sat, 30 Apr 2016 22:04:11 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751111AbcD3UEF (ORCPT ); Sat, 30 Apr 2016 16:04:05 -0400 Received: from mail-pa0-f67.google.com ([209.85.220.67]:34253 "EHLO mail-pa0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750759AbcD3UEE (ORCPT ); Sat, 30 Apr 2016 16:04:04 -0400 Received: by mail-pa0-f67.google.com with SMTP id yl2so15299528pac.1 for ; Sat, 30 Apr 2016 13:04:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id; bh=6+HdgjmpI74vOajYsvGaSBKl6iqT9vgWQ6UY+0VRUos=; b=z1qZk6LnYz1tM8rLSD12b+aEddoDAmxL93+I75dMWyTka9iIDmFgo7N254oftMZjt8 e+cKUYI52JfWrtPRNNtthZiPRArq4Z8Fn1bbjY88NqtETlE/dbwFl0cK0lw0QfouMJIa JpR9Or2uO5ySmiP1uKcZTKAVlOEVPQaOJmTEKmZUyGoI/4zvVAbSa5tQVe/crFANOMcR TncG9Xj6a5BrcGE6R3dbyRBiyB2uPU9bLx5Ax+cPglIW5d2EpChmLZ1WQNOx2sbxTY4U o5E33qlr6S2EAZxQOSVyv5a65xK06ODPddbegd6VUWM9Y2ZF/diIsN6TY0r7FybcW5vS 4kLA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=6+HdgjmpI74vOajYsvGaSBKl6iqT9vgWQ6UY+0VRUos=; b=E74j4U6bAZV7Up5iooBWwV3x4lRFkjdcKlBQRRnHtNFDjlGBb2eiemqTr8xcRrv0s5 dmzer64Vx2waIctuKwu3bKBk1TgwDwEaDeStGIckO9pTI7b4UXqqo5TcGO9v0/OpDgub eeFJr0j+NVJSkN2hJ+5ETwGgkDAUMmZBpwlF3HEAYUb8nxDAnwh7ZCRnMFdmA+xzzeYh VeCN+ezjaxdLd01yeVgd42HLaw3V98rvIvhZkXd7Mi2aNsoIa3b4jB90v/bjZgCKxWa2 YmMDARGbEOWtdYb19FcrQ2z+znyeEBUsyAIef7jXpPq91gn2ovC2e06WOCIPmkUl2XTd bdxA== X-Gm-Message-State: AOPr4FVWPji0vTpX4DVCVsVNNxuG3KT7TXWmhMLmFMvMrjTM3tSQQs4yI4icaVA49ztCWg== X-Received: by 10.66.122.100 with SMTP id lr4mr37718842pab.99.1462046642672; Sat, 30 Apr 2016 13:04:02 -0700 (PDT) Received: from localhost.localdomain ([27.106.101.191]) by smtp.gmail.com with ESMTPSA id h85sm33329025pfj.52.2016.04.30.13.03.57 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Sat, 30 Apr 2016 13:04:02 -0700 (PDT) X-Mailer: git-send-email 2.8.1 Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Signed-off-by: Pranit Bauva --- Changes wrt previous version (v12): - Use '\' when interpolation isn't required Signed-off-by: Pranit Bauva --- t/t0040-parse-options.sh | 76 ++++++++++++++++++++++++------------------------ 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/t/t0040-parse-options.sh b/t/t0040-parse-options.sh index 9be6411..477fcff 100755 --- a/t/t0040-parse-options.sh +++ b/t/t0040-parse-options.sh @@ -7,7 +7,7 @@ test_description='our own option parser' . ./test-lib.sh -cat > expect << EOF +cat >expect <<\EOF usage: test-parse-options --yes get a boolean @@ -49,14 +49,14 @@ Standard options EOF test_expect_success 'test help' ' - test_must_fail test-parse-options -h > output 2> output.err && + test_must_fail test-parse-options -h >output 2>output.err && test_must_be_empty output.err && test_i18ncmp expect output ' mv expect expect.err -cat >expect.template <expect.template <<\EOF boolean: 0 integer: 0 magnitude: 0 @@ -156,7 +156,7 @@ test_expect_success 'OPT_MAGNITUDE() 3giga' ' check magnitude: 3221225472 -m 3g ' -cat > expect << EOF +cat >expect <<\EOF boolean: 2 integer: 1729 magnitude: 16384 @@ -176,7 +176,7 @@ test_expect_success 'short options' ' test_must_be_empty output.err ' -cat > expect << EOF +cat >expect <<\EOF boolean: 2 integer: 1729 magnitude: 16384 @@ -204,7 +204,7 @@ test_expect_success 'missing required value' ' test_expect_code 129 test-parse-options --file ' -cat > expect << EOF +cat >expect <<\EOF boolean: 1 integer: 13 magnitude: 0 @@ -222,12 +222,12 @@ EOF test_expect_success 'intermingled arguments' ' test-parse-options a1 --string 123 b1 --boolean -j 13 -- --boolean \ - > output 2> output.err && + >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' -cat > expect << EOF +cat >expect <<\EOF boolean: 0 integer: 2 magnitude: 0 @@ -241,13 +241,13 @@ file: (not set) EOF test_expect_success 'unambiguously abbreviated option' ' - test-parse-options --int 2 --boolean --no-bo > output 2> output.err && + test-parse-options --int 2 --boolean --no-bo >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' test_expect_success 'unambiguously abbreviated option with "="' ' - test-parse-options --int=2 > output 2> output.err && + test-parse-options --int=2 >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' @@ -256,7 +256,7 @@ test_expect_success 'ambiguously abbreviated option' ' test_expect_code 129 test-parse-options --strin 123 ' -cat > expect << EOF +cat >expect <<\EOF boolean: 0 integer: 0 magnitude: 0 @@ -270,32 +270,32 @@ file: (not set) EOF test_expect_success 'non ambiguous option (after two options it abbreviates)' ' - test-parse-options --st 123 > output 2> output.err && + test-parse-options --st 123 >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' -cat > typo.err << EOF -error: did you mean \`--boolean\` (with two dashes ?) +cat >typo.err <<\EOF +error: did you mean `--boolean` (with two dashes ?) EOF test_expect_success 'detect possible typos' ' - test_must_fail test-parse-options -boolean > output 2> output.err && + test_must_fail test-parse-options -boolean >output 2>output.err && test_must_be_empty output && test_cmp typo.err output.err ' -cat > typo.err << EOF -error: did you mean \`--ambiguous\` (with two dashes ?) +cat >typo.err <<\EOF +error: did you mean `--ambiguous` (with two dashes ?) EOF test_expect_success 'detect possible typos' ' - test_must_fail test-parse-options -ambiguous > output 2> output.err && + test_must_fail test-parse-options -ambiguous >output 2>output.err && test_must_be_empty output && test_cmp typo.err output.err ' -cat > expect <expect <<\EOF boolean: 0 integer: 0 magnitude: 0 @@ -310,12 +310,12 @@ arg 00: --quux EOF test_expect_success 'keep some options as arguments' ' - test-parse-options --quux > output 2> output.err && + test-parse-options --quux >output 2>output.err && test_must_be_empty output.err && - test_cmp expect output + test_cmp expect output ' -cat > expect <expect <<\EOF boolean: 0 integer: 0 magnitude: 0 @@ -331,12 +331,12 @@ EOF test_expect_success 'OPT_DATE() works' ' test-parse-options -t "1970-01-01 00:00:01 +0000" \ - foo -q > output 2> output.err && + foo -q >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' -cat > expect <expect <<\EOF Callback: "four", 0 boolean: 5 integer: 4 @@ -351,22 +351,22 @@ file: (not set) EOF test_expect_success 'OPT_CALLBACK() and OPT_BIT() work' ' - test-parse-options --length=four -b -4 > output 2> output.err && + test-parse-options --length=four -b -4 >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' -cat > expect <expect <<\EOF Callback: "not set", 1 EOF test_expect_success 'OPT_CALLBACK() and callback errors work' ' - test_must_fail test-parse-options --no-length > output 2> output.err && + test_must_fail test-parse-options --no-length >output 2>output.err && test_i18ncmp expect output && test_i18ncmp expect.err output.err ' -cat > expect <expect <<\EOF boolean: 1 integer: 23 magnitude: 0 @@ -380,18 +380,18 @@ file: (not set) EOF test_expect_success 'OPT_BIT() and OPT_SET_INT() work' ' - test-parse-options --set23 -bbbbb --no-or4 > output 2> output.err && + test-parse-options --set23 -bbbbb --no-or4 >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' test_expect_success 'OPT_NEGBIT() and OPT_SET_INT() work' ' - test-parse-options --set23 -bbbbb --neg-or4 > output 2> output.err && + test-parse-options --set23 -bbbbb --neg-or4 >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' -cat > expect <expect <<\EOF boolean: 6 integer: 0 magnitude: 0 @@ -405,24 +405,24 @@ file: (not set) EOF test_expect_success 'OPT_BIT() works' ' - test-parse-options -bb --or4 > output 2> output.err && + test-parse-options -bb --or4 >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' test_expect_success 'OPT_NEGBIT() works' ' - test-parse-options -bb --no-neg-or4 > output 2> output.err && + test-parse-options -bb --no-neg-or4 >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' test_expect_success 'OPT_COUNTUP() with PARSE_OPT_NODASH works' ' - test-parse-options + + + + + + > output 2> output.err && + test-parse-options + + + + + + >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' -cat > expect <expect <<\EOF boolean: 0 integer: 12345 magnitude: 0 @@ -436,12 +436,12 @@ file: (not set) EOF test_expect_success 'OPT_NUMBER_CALLBACK() works' ' - test-parse-options -12345 > output 2> output.err && + test-parse-options -12345 >output 2>output.err && test_must_be_empty output.err && test_cmp expect output ' -cat >expect <expect <<\EOF boolean: 0 integer: 0 magnitude: 0 @@ -460,7 +460,7 @@ test_expect_success 'negation of OPT_NONEG flags is not ambiguous' ' test_cmp expect output ' -cat >>expect <<'EOF' +cat >>expect <<\EOF list: foo list: bar list: baz -- 2.8.1