[prev in list] [next in list] [prev in thread] [next in thread]
List: git
Subject: Re: [PATCH] blame: add blame.showemail config option
From: Junio C Hamano <gitster () pobox ! com>
Date: 2015-04-30 21:43:06
Message-ID: xmqqk2wtxpg5.fsf () gitster ! dls ! corp ! google ! com
[Download RAW message or body]
Eric Sunshine <sunshine@sunshineco.com> writes:
> On Thu, Apr 30, 2015 at 10:03 AM, Quentin Neill <quentin.neill@gmail.com> wrote:
>> I'm thinking of a test file that will test all the git blame options,
>> but for this
>> patch it will only test the new showEmail config. I read t/README and
>> tentatively named the new test file "t/8009-blame-config.sh".
>
> I'm not sure that these new proposed tests warrant a new test script.
> Moreover, the tests presumably will be testing combinations of
> configuration options and command-line switches, so having "config" in
> the script name doesn't seem quite correct.
>
> t8002-blame.sh already contains a test for --show-email, so it may be
> logical to add the new tests there, alongside the existing one. On the
> other hand, if you do add a new test script, then perhaps the existing
> --show-email test in t8002-blame.sh should be moved to the new script.
Good thinking. I would vote for the former, unless the number of
additional tests exceed 20 (as t8002 has about 100 tests).
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic