Re: [PATCH v2 08/10] t6050: add tests for listing with --format
- From
Christian Couder <chriscool@tuxfamily.org>
- Date
- Dec 9, 2013, 05:58 UTC
- Message-ID
- <20131209.065831.1058089388384173102.chriscool@tuxfamily.org>
- In-Reply-To
- <CAPig+cTcacoNGstYeYRd3_WKAP6=ga7gV4puSsr7c2sO_OMNTQ@mail.gmail.com>
From: Eric Sunshine <sunshine@sunshineco.com>
Show 15 quoted lines
> > On Sat, Dec 7, 2013 at 11:21 AM, Christian Couder > <chriscool@tuxfamily.org> wrote: >> This patch adds tests for "git replace -l --format=<fmt>". >> Only tests when <fmt> is 'medium' and 'full' are needed >> because 'short' is the same as with no --format option. > > Nevertheless, don't you want to test that it behaves in the expected > manner when given --format=short and other legal or illegal > combinations? > > --format > --format short > --format bogus > ...etc...
Ok, I will add more tests.
Thanks, Christian.