From: Jeff King <peff@peff.net>
To: git@vger.kernel.org
Cc: "Lukas Sandström" <luksan@gmail.com>,
"Jay Soffian" <jaysoffian@gmail.com>,
"John Szakmeister" <john@szakmeister.net>,
"Erik Faye-Lund" <kusmabite@gmail.com>,
"Ted Zlatanov" <tzz@lifelogs.com>
Subject: credential helper tests
Date: Fri, 23 Sep 2011 18:15:13 -0400 [thread overview]
Message-ID: <20110923221513.GA3087@sigill.intra.peff.net> (raw)
Since we've had a few credential helpers posted to the list recently, I
really want to try them all out. This can be a little bit tricky for
automated testing, though, for two reasons:
- they run on lots of platforms with lots of dependencies
- they interact with parts of the systems that are opaque to git. So
we can't make a test that reliably simulates "and then the user
types 'foo' into a dialog box" across all platforms.
Instead, I came up with a separate test script that is intended to be
run interactively with the user. It runs the helpers through a battery
of tests, and tells the user what to expect and what to input to any
dialogs or prompts.
I've run it already on the helpers I've written. I plan on running it
with the helpers that have been posted, as well. But I also wanted to
make it public so that authors could use it as a development aid.
It's not integrated with git's tests at all. In theory it could be part
of t/, but disabled unless the user asks for it. However, I'm not sure
that makes much sense. It's intended to test helpers that aren't
necessarily even shipped with git, and wouldn't necessarily even need
git to run.
Also, it is by no means a strict set of tests. A helper that did not
store credentials, but only presented dialogs in a different way, or one
that was about accessing a read-only store of credentials would not
pass. So think of it as a best-practices guide and an exercise script
for certain types of helpers, not necessarily as a set of tests that
must be passed.
-- >8 --
#!/bin/sh
# e.g., "cache"
helper=$1
say() {
echo >&2 "==> $*"
}
check() {
for i in username password; do
v=$1; shift
case "$v" in
auto:*)
v=${v#auto:}
say " $i should be automatic ($v)"
;;
*)
say " Input $i=$v"
;;
esac
echo $i=$v
done >expect
if git credential-$helper "$@" >actual &&
git --no-pager diff --no-index expect actual; then
say OK
else
say FAIL
fi
}
reject() {
git credential-$helper --reject "$@" || exit 1
}
say 'Cleaning old invocations...'
reject --unique=https:foo.tld
reject --unique=https:bar.tld
say 'No context (initial, should ask)'
check user pass
say 'No context (again, should ask)'
check user2 pass2
say 'Context foo.tld (initial, should ask)'
check foo-user foo-pass --unique=https:foo.tld
say 'Context foo.tld (again)'
check auto:foo-user auto:foo-pass --unique=https:foo.tld
say 'Context bar.tld (should ask)'
check bar-user bar-pass --unique=https:bar.tld
say 'Context bar.tld (again)'
check auto:bar-user auto:bar-pass --unique=https:bar.tld
say 'Context foo.tld (should still remember)'
check auto:foo-user auto:foo-pass --unique=https:foo.tld
say 'Forget foo.tld (should ask)'
reject --unique=https:foo.tld
check foo-user2 foo-pass2 --unique=https:foo.tld
say 'Context foo.tld (again)'
check auto:foo-user2 auto:foo-pass2 --unique=https:foo.tld
say 'Context bar.tld (should still remember)'
check auto:bar-user auto:bar-pass --unique=https:bar.tld
say 'Alternate user at foo.tld (should ask)'
check auto:foo-user3 foo-pass3 --unique=https:foo.tld --username=foo-user3
say 'Remember new user'
check auto:foo-user3 auto:foo-pass3 --unique=https:foo.tld --username=foo-user3
say 'Remember old user'
check auto:foo-user2 auto:foo-pass2 --unique=https:foo.tld --username=foo-user2
say 'Forget new user (should ask)'
reject --unique=https:foo.tld --username=foo-user3
check auto:foo-user3 foo-pass4 --unique=https:foo.tld --username=foo-user3
say 'New user is now remembered'
check auto:foo-user3 auto:foo-pass4 --unique=https:foo.tld --username=foo-user3
say 'Remember old user'
check auto:foo-user2 auto:foo-pass2 --unique=https:foo.tld --username=foo-user2
next reply other threads:[~2011-09-23 22:15 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-23 22:15 Jeff King [this message]
2011-09-23 22:26 ` credential helper tests Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110923221513.GA3087@sigill.intra.peff.net \
--to=peff@peff.net \
--cc=git@vger.kernel.org \
--cc=jaysoffian@gmail.com \
--cc=john@szakmeister.net \
--cc=kusmabite@gmail.com \
--cc=luksan@gmail.com \
--cc=tzz@lifelogs.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).