Re: [PATCH] completion: fix completing unstuck email alias arguments

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Quoting Jacob Keller <jacob.keller@xxxxxxxxx>:

On Fri, Dec 11, 2015 at 4:18 PM, SZEDER Gábor <szeder@xxxxxxxxxx> wrote:
Completing unstuck form of email aliases doesn't quite work:

$ git send-email --to <TAB>
alice   bob     cecil
$ git send-email --to a<TAB>
alice   bob     cecil


Woops. Is it possible to add a test for this case? I honestly don't
know how the tests for the completion works.

It's possible, but I'm not sure we should.

It's a common pattern throughout the completion script to call
__gitcomp() (or __gitcomp_nl()) this way, out of the almost 200
callsites many look like this one.  We don't test all those, because
it would require a lot of tests for not that much gain in coverage
in my opinion.  The interesting thing that's definitely worth testing
is what happens inside the command substitution that supplies the
possible completion words, but your tests added to the send-email
testsuite already cover that thoroughly.

Gábor

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]