Re: [PATCHv2 06/13] credential: apply helper config
On Tue, Dec 06, 2011 at 07:45:11PM -0500, Jeff King wrote:
Show 19 quoted lines
> On Tue, Dec 06, 2011 at 03:58:35PM -0800, Junio C Hamano wrote:
>
> > > +test_expect_success 'respect configured credentials' '
> > > + test_config credential.helper "$HELPER" &&
> > > + check fill <<-\EOF
> > > + --
> > > + username=foo
> > > + password=bar
> > > + --
> > > + EOF
> > > +'
> >
> > Hmm, why do I get ask-ass-{username,password} from this one?
>
> Ugh. Because apparently one of my re-roll tweaks from patch 03 regressed
> this. Sorry, I should have been more careful about running the full
> suite, not just the tests in the commits I tweaked.
>
> Let me investigate and get back to you.Brown paper bag time.
This needs squashed in, due to the changes in patch 03/13:
---
diff --git a/t/t0300-credentials.sh b/t/t0300-credentials.sh
index e3f61f4..42d0f5b 100755
--- a/t/t0300-credentials.sh
+++ b/t/t0300-credentials.sh
@@ -192,7 +192,7 @@ test_expect_success 'internal getpass does not ask for known username' '
EOF
'
-HELPER="f() {
+HELPER="!f() {
cat >/dev/null
echo username=foo
echo password=bar
diff --git a/t/t5550-http-fetch.sh b/t/t5550-http-fetch.sh
index 21e2f5d..c59908f 100755
--- a/t/t5550-http-fetch.sh
+++ b/t/t5550-http-fetch.sh
@@ -102,7 +102,7 @@ test_expect_success 'http auth can request both user and pass' '
'
test_expect_success 'http auth respects credential helper config' '
- test_config_global credential.helper "f() {
+ test_config_global credential.helper "!f() {
cat >/dev/null
echo username=user@host
echo password=user@host
Jeff King <peff@peff.net>