mirror of
https://github.com/git/git
synced 2024-08-27 11:39:22 +00:00
Merge branch 'jc/test-parse-options-expect'
t0040 had too many unnecessary repetitions in its test data. Teach test-parse-options program so that a caller can tell what it expects in its output, so that these repetitions can be cleaned up. * jc/test-parse-options-expect: t0040: convert a few tests to use test-parse-options --expect t0040: remove unused test helpers test-parse-options: --expect=<string> option to simplify tests test-parse-options: fix output when callback option fails
This commit is contained in:
commit
2997ea960f
|
@ -14,10 +14,18 @@ static char *file = NULL;
|
|||
static int ambiguous;
|
||||
static struct string_list list;
|
||||
|
||||
static struct {
|
||||
int called;
|
||||
const char *arg;
|
||||
int unset;
|
||||
} length_cb;
|
||||
|
||||
static int length_callback(const struct option *opt, const char *arg, int unset)
|
||||
{
|
||||
printf("Callback: \"%s\", %d\n",
|
||||
(arg ? arg : "not set"), unset);
|
||||
length_cb.called = 1;
|
||||
length_cb.arg = arg;
|
||||
length_cb.unset = unset;
|
||||
|
||||
if (unset)
|
||||
return 1; /* do not support unset */
|
||||
|
||||
|
@ -31,6 +39,61 @@ static int number_callback(const struct option *opt, const char *arg, int unset)
|
|||
return 0;
|
||||
}
|
||||
|
||||
static int collect_expect(const struct option *opt, const char *arg, int unset)
|
||||
{
|
||||
struct string_list *expect;
|
||||
struct string_list_item *item;
|
||||
struct strbuf label = STRBUF_INIT;
|
||||
const char *colon;
|
||||
|
||||
if (!arg || unset)
|
||||
die("malformed --expect option");
|
||||
|
||||
expect = (struct string_list *)opt->value;
|
||||
colon = strchr(arg, ':');
|
||||
if (!colon)
|
||||
die("malformed --expect option, lacking a colon");
|
||||
strbuf_add(&label, arg, colon - arg);
|
||||
item = string_list_insert(expect, strbuf_detach(&label, NULL));
|
||||
if (item->util)
|
||||
die("malformed --expect option, duplicate %s", label.buf);
|
||||
item->util = (void *)arg;
|
||||
return 0;
|
||||
}
|
||||
|
||||
__attribute__((format (printf,3,4)))
|
||||
static void show(struct string_list *expect, int *status, const char *fmt, ...)
|
||||
{
|
||||
struct string_list_item *item;
|
||||
struct strbuf buf = STRBUF_INIT;
|
||||
va_list args;
|
||||
|
||||
va_start(args, fmt);
|
||||
strbuf_vaddf(&buf, fmt, args);
|
||||
va_end(args);
|
||||
|
||||
if (!expect->nr)
|
||||
printf("%s\n", buf.buf);
|
||||
else {
|
||||
char *colon = strchr(buf.buf, ':');
|
||||
if (!colon)
|
||||
die("malformed output format, output lacking colon: %s", fmt);
|
||||
*colon = '\0';
|
||||
item = string_list_lookup(expect, buf.buf);
|
||||
*colon = ':';
|
||||
if (!item)
|
||||
; /* not among entries being checked */
|
||||
else {
|
||||
if (strcmp((const char *)item->util, buf.buf)) {
|
||||
printf("-%s\n", (char *)item->util);
|
||||
printf("+%s\n", buf.buf);
|
||||
*status = 1;
|
||||
}
|
||||
}
|
||||
}
|
||||
strbuf_release(&buf);
|
||||
}
|
||||
|
||||
int main(int argc, char **argv)
|
||||
{
|
||||
const char *prefix = "prefix/";
|
||||
|
@ -38,6 +101,7 @@ int main(int argc, char **argv)
|
|||
"test-parse-options <options>",
|
||||
NULL
|
||||
};
|
||||
struct string_list expect = STRING_LIST_INIT_NODUP;
|
||||
struct option options[] = {
|
||||
OPT_BOOL(0, "yes", &boolean, "get a boolean"),
|
||||
OPT_BOOL('D', "no-doubt", &boolean, "begins with 'no-'"),
|
||||
|
@ -78,28 +142,38 @@ int main(int argc, char **argv)
|
|||
OPT__VERBOSE(&verbose, "be verbose"),
|
||||
OPT__DRY_RUN(&dry_run, "dry run"),
|
||||
OPT__QUIET(&quiet, "be quiet"),
|
||||
OPT_CALLBACK(0, "expect", &expect, "string",
|
||||
"expected output in the variable dump",
|
||||
collect_expect),
|
||||
OPT_END(),
|
||||
};
|
||||
int i;
|
||||
int ret = 0;
|
||||
|
||||
argc = parse_options(argc, (const char **)argv, prefix, options, usage, 0);
|
||||
|
||||
printf("boolean: %d\n", boolean);
|
||||
printf("integer: %d\n", integer);
|
||||
printf("magnitude: %lu\n", magnitude);
|
||||
printf("timestamp: %lu\n", timestamp);
|
||||
printf("string: %s\n", string ? string : "(not set)");
|
||||
printf("abbrev: %d\n", abbrev);
|
||||
printf("verbose: %d\n", verbose);
|
||||
printf("quiet: %d\n", quiet);
|
||||
printf("dry run: %s\n", dry_run ? "yes" : "no");
|
||||
printf("file: %s\n", file ? file : "(not set)");
|
||||
if (length_cb.called) {
|
||||
const char *arg = length_cb.arg;
|
||||
int unset = length_cb.unset;
|
||||
show(&expect, &ret, "Callback: \"%s\", %d",
|
||||
(arg ? arg : "not set"), unset);
|
||||
}
|
||||
show(&expect, &ret, "boolean: %d", boolean);
|
||||
show(&expect, &ret, "integer: %d", integer);
|
||||
show(&expect, &ret, "magnitude: %lu", magnitude);
|
||||
show(&expect, &ret, "timestamp: %lu", timestamp);
|
||||
show(&expect, &ret, "string: %s", string ? string : "(not set)");
|
||||
show(&expect, &ret, "abbrev: %d", abbrev);
|
||||
show(&expect, &ret, "verbose: %d", verbose);
|
||||
show(&expect, &ret, "quiet: %d", quiet);
|
||||
show(&expect, &ret, "dry run: %s", dry_run ? "yes" : "no");
|
||||
show(&expect, &ret, "file: %s", file ? file : "(not set)");
|
||||
|
||||
for (i = 0; i < list.nr; i++)
|
||||
printf("list: %s\n", list.items[i].string);
|
||||
show(&expect, &ret, "list: %s", list.items[i].string);
|
||||
|
||||
for (i = 0; i < argc; i++)
|
||||
printf("arg %02d: %s\n", i, argv[i]);
|
||||
show(&expect, &ret, "arg %02d: %s", i, argv[i]);
|
||||
|
||||
return 0;
|
||||
return ret;
|
||||
}
|
||||
|
|
|
@ -45,6 +45,7 @@ Standard options
|
|||
-v, --verbose be verbose
|
||||
-n, --dry-run dry run
|
||||
-q, --quiet be quiet
|
||||
--expect <string> expected output in the variable dump
|
||||
|
||||
EOF
|
||||
|
||||
|
@ -56,52 +57,12 @@ test_expect_success 'test help' '
|
|||
|
||||
mv expect expect.err
|
||||
|
||||
cat >expect.template <<\EOF
|
||||
boolean: 0
|
||||
integer: 0
|
||||
magnitude: 0
|
||||
timestamp: 0
|
||||
string: (not set)
|
||||
abbrev: 7
|
||||
verbose: -1
|
||||
quiet: 0
|
||||
dry run: no
|
||||
file: (not set)
|
||||
EOF
|
||||
|
||||
check() {
|
||||
check () {
|
||||
what="$1" &&
|
||||
shift &&
|
||||
expect="$1" &&
|
||||
shift &&
|
||||
sed "s/^$what .*/$what $expect/" <expect.template >expect &&
|
||||
test-parse-options $* >output 2>output.err &&
|
||||
test_must_be_empty output.err &&
|
||||
test_cmp expect output
|
||||
}
|
||||
|
||||
check_i18n() {
|
||||
what="$1" &&
|
||||
shift &&
|
||||
expect="$1" &&
|
||||
shift &&
|
||||
sed "s/^$what .*/$what $expect/" <expect.template >expect &&
|
||||
test-parse-options $* >output 2>output.err &&
|
||||
test_must_be_empty output.err &&
|
||||
test_i18ncmp expect output
|
||||
}
|
||||
|
||||
check_unknown() {
|
||||
case "$1" in
|
||||
--*)
|
||||
echo error: unknown option \`${1#--}\' >expect ;;
|
||||
-*)
|
||||
echo error: unknown switch \`${1#-}\' >expect ;;
|
||||
esac &&
|
||||
cat expect.err >>expect &&
|
||||
test_must_fail test-parse-options $* >output 2>output.err &&
|
||||
test_must_be_empty output &&
|
||||
test_cmp expect output.err
|
||||
test-parse-options --expect="$what $expect" "$@"
|
||||
}
|
||||
|
||||
check_unknown_i18n() {
|
||||
|
@ -356,9 +317,7 @@ test_expect_success 'OPT_CALLBACK() and OPT_BIT() work' '
|
|||
test_cmp expect output
|
||||
'
|
||||
|
||||
cat >expect <<\EOF
|
||||
Callback: "not set", 1
|
||||
EOF
|
||||
>expect
|
||||
|
||||
test_expect_success 'OPT_CALLBACK() and callback errors work' '
|
||||
test_must_fail test-parse-options --no-length >output 2>output.err &&
|
||||
|
|
Loading…
Reference in a new issue