2019-02-23 06:25:10 +08:00
|
|
|
#!/bin/sh
|
|
|
|
|
|
|
|
test_description='test trace2 facility (normal target)'
|
2021-10-12 21:56:40 +08:00
|
|
|
|
2024-08-14 14:51:52 +08:00
|
|
|
TEST_PASSES_SANITIZE_LEAK=true
|
2019-02-23 06:25:10 +08:00
|
|
|
. ./test-lib.sh
|
|
|
|
|
2019-04-16 04:39:47 +08:00
|
|
|
# Turn off any inherited trace2 settings for this test.
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
sane_unset GIT_TRACE2 GIT_TRACE2_PERF GIT_TRACE2_EVENT
|
|
|
|
sane_unset GIT_TRACE2_BRIEF
|
|
|
|
sane_unset GIT_TRACE2_CONFIG_PARAMS
|
2019-04-16 04:39:47 +08:00
|
|
|
|
2019-02-23 06:25:10 +08:00
|
|
|
# Add t/helper directory to PATH so that we can use a relative
|
|
|
|
# path to run nested instances of test-tool.exe (see 004child).
|
|
|
|
# This helps with HEREDOC comparisons later.
|
|
|
|
TTDIR="$GIT_BUILD_DIR/t/helper/" && export TTDIR
|
|
|
|
PATH="$TTDIR:$PATH" && export PATH
|
|
|
|
|
|
|
|
# Warning: use of 'test_cmp' may run test-tool.exe and/or git.exe
|
|
|
|
# Warning: to do the actual diff/comparison, so the HEREDOCs here
|
|
|
|
# Warning: only cover our actual calls to test-tool and/or git.
|
|
|
|
# Warning: So you may see extra lines in artifact files when
|
|
|
|
# Warning: interactively debugging.
|
|
|
|
|
|
|
|
V=$(git version | sed -e 's/^git version //') && export V
|
|
|
|
|
|
|
|
# There are multiple trace2 targets: normal, perf, and event.
|
|
|
|
# Trace2 events will/can be written to each active target (subject
|
|
|
|
# to whatever filtering that target decides to do).
|
|
|
|
# This script tests the normal target in isolation.
|
|
|
|
#
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
# Defer setting GIT_TRACE2 until the actual command line we want to test
|
2019-02-23 06:25:10 +08:00
|
|
|
# because hidden git and test-tool commands run by the test harness
|
|
|
|
# can contaminate our output.
|
|
|
|
|
|
|
|
# Enable "brief" feature which turns off "<clock> <file>:<line> " prefix.
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
GIT_TRACE2_BRIEF=1 && export GIT_TRACE2_BRIEF
|
2019-02-23 06:25:10 +08:00
|
|
|
|
|
|
|
# Basic tests of the trace2 normal stream. Since this stream is used
|
|
|
|
# primarily with printf-style debugging/tracing, we do limited testing
|
|
|
|
# here.
|
|
|
|
#
|
|
|
|
# We do confirm the following API features:
|
|
|
|
# [] the 'version <v>' event
|
|
|
|
# [] the 'start <argv>' event
|
|
|
|
# [] the 'cmd_name <name>' event
|
|
|
|
# [] the 'exit <time> code:<code>' event
|
|
|
|
# [] the 'atexit <time> code:<code>' event
|
|
|
|
#
|
|
|
|
# Fields of the form _FIELD_ are tokens that have been replaced (such
|
|
|
|
# as the elapsed time).
|
|
|
|
|
|
|
|
# Verb 001return
|
|
|
|
#
|
|
|
|
# Implicit return from cmd_<verb> function propagates <code>.
|
|
|
|
|
|
|
|
test_expect_success 'normal stream, return code 0' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
GIT_TRACE2="$(pwd)/trace.normal" test-tool trace2 001return 0 &&
|
2019-02-23 06:25:10 +08:00
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 001return 0
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
exit elapsed:_TIME_ code:0
|
|
|
|
atexit elapsed:_TIME_ code:0
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'normal stream, return code 1' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
test_must_fail env GIT_TRACE2="$(pwd)/trace.normal" test-tool trace2 001return 1 &&
|
2019-02-23 06:25:10 +08:00
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 001return 1
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
exit elapsed:_TIME_ code:1
|
|
|
|
atexit elapsed:_TIME_ code:1
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2019-03-22 05:09:51 +08:00
|
|
|
test_expect_success 'automatic filename' '
|
|
|
|
test_when_finished "rm -r traces actual expect" &&
|
|
|
|
mkdir traces &&
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
GIT_TRACE2="$(pwd)/traces" test-tool trace2 001return 0 &&
|
2019-03-22 05:09:51 +08:00
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <"$(ls traces/*)" >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 001return 0
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
exit elapsed:_TIME_ code:0
|
|
|
|
atexit elapsed:_TIME_ code:0
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2019-02-23 06:25:10 +08:00
|
|
|
# Verb 002exit
|
|
|
|
#
|
|
|
|
# Explicit exit(code) from within cmd_<verb> propagates <code>.
|
|
|
|
|
|
|
|
test_expect_success 'normal stream, exit code 0' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
GIT_TRACE2="$(pwd)/trace.normal" test-tool trace2 002exit 0 &&
|
2019-02-23 06:25:10 +08:00
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 002exit 0
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
exit elapsed:_TIME_ code:0
|
|
|
|
atexit elapsed:_TIME_ code:0
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'normal stream, exit code 1' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
test_must_fail env GIT_TRACE2="$(pwd)/trace.normal" test-tool trace2 002exit 1 &&
|
2019-02-23 06:25:10 +08:00
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 002exit 1
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
exit elapsed:_TIME_ code:1
|
|
|
|
atexit elapsed:_TIME_ code:1
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
# Verb 003error
|
|
|
|
#
|
|
|
|
# To the above, add multiple 'error <msg>' events
|
|
|
|
|
|
|
|
test_expect_success 'normal stream, error event' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
GIT_TRACE2="$(pwd)/trace.normal" test-tool trace2 003error "hello world" "this is a test" &&
|
2019-02-23 06:25:10 +08:00
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 003error '\''hello world'\'' '\''this is a test'\''
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
error hello world
|
|
|
|
error this is a test
|
|
|
|
exit elapsed:_TIME_ code:0
|
|
|
|
atexit elapsed:_TIME_ code:0
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2021-02-06 04:09:08 +08:00
|
|
|
# Verb 007bug
|
|
|
|
#
|
|
|
|
# Check that BUG writes to trace2
|
|
|
|
|
|
|
|
test_expect_success 'BUG messages are written to trace2' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
|
|
|
test_must_fail env GIT_TRACE2="$(pwd)/trace.normal" test-tool trace2 007bug &&
|
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 007bug
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
error the bug message
|
|
|
|
exit elapsed:_TIME_ code:99
|
|
|
|
atexit elapsed:_TIME_ code:99
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2022-06-02 20:25:33 +08:00
|
|
|
test_expect_success 'bug messages with BUG_if_bug() are written to trace2' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
|
|
|
test_expect_code 99 env GIT_TRACE2="$(pwd)/trace.normal" \
|
|
|
|
test-tool trace2 008bug 2>err &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
a bug message
|
|
|
|
another bug message
|
|
|
|
an explicit BUG_if_bug() following bug() call(s) is nice, but not required
|
|
|
|
EOF
|
|
|
|
sed "s/^.*: //" <err >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 008bug
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
error a bug message
|
|
|
|
error another bug message
|
|
|
|
error an explicit BUG_if_bug() following bug() call(s) is nice, but not required
|
|
|
|
exit elapsed:_TIME_ code:99
|
|
|
|
atexit elapsed:_TIME_ code:99
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'bug messages without explicit BUG_if_bug() are written to trace2' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
|
|
|
test_expect_code 99 env GIT_TRACE2="$(pwd)/trace.normal" \
|
|
|
|
test-tool trace2 009bug_BUG 2>err &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
a bug message
|
|
|
|
another bug message
|
|
|
|
had bug() call(s) in this process without explicit BUG_if_bug()
|
|
|
|
EOF
|
|
|
|
sed "s/^.*: //" <err >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 009bug_BUG
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
error a bug message
|
|
|
|
error another bug message
|
|
|
|
error on exit(): had bug() call(s) in this process without explicit BUG_if_bug()
|
|
|
|
exit elapsed:_TIME_ code:99
|
|
|
|
atexit elapsed:_TIME_ code:99
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'bug messages followed by BUG() are written to trace2' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
|
|
|
test_expect_code 99 env GIT_TRACE2="$(pwd)/trace.normal" \
|
|
|
|
test-tool trace2 010bug_BUG 2>err &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
a bug message
|
|
|
|
a BUG message
|
|
|
|
EOF
|
|
|
|
sed "s/^.*: //" <err >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 010bug_BUG
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
error a bug message
|
|
|
|
error a BUG message
|
|
|
|
exit elapsed:_TIME_ code:99
|
|
|
|
atexit elapsed:_TIME_ code:99
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
trace2: rename environment variables to GIT_TRACE2*
For an environment variable that is supposed to be set by users, the
GIT_TR2* env vars are just too unclear, inconsistent, and ugly.
Most of the established GIT_* environment variables don't use
abbreviations, and in case of the few that do (GIT_DIR,
GIT_COMMON_DIR, GIT_DIFF_OPTS) it's quite obvious what the
abbreviations (DIR and OPTS) stand for. But what does TR stand for?
Track, traditional, trailer, transaction, transfer, transformation,
transition, translation, transplant, transport, traversal, tree,
trigger, truncate, trust, or ...?!
The trace2 facility, as the '2' suffix in its name suggests, is
supposed to eventually supercede Git's original trace facility. It's
reasonable to expect that the corresponding environment variables
follow suit, and after the original GIT_TRACE variables they are
called GIT_TRACE2; there is no such thing is 'GIT_TR'.
All trace2-specific config variables are, very sensibly, in the
'trace2' section, not in 'tr2'.
OTOH, we don't gain anything at all by omitting the last three
characters of "trace" from the names of these environment variables.
So let's rename all GIT_TR2* environment variables to GIT_TRACE2*,
before they make their way into a stable release.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-05-19 22:43:08 +08:00
|
|
|
sane_unset GIT_TRACE2_BRIEF
|
2019-04-16 04:39:47 +08:00
|
|
|
|
|
|
|
# Now test without environment variables and get all Trace2 settings
|
|
|
|
# from the global config.
|
|
|
|
|
|
|
|
test_expect_success 'using global config, normal stream, return code 0' '
|
|
|
|
test_when_finished "rm trace.normal actual expect" &&
|
|
|
|
test_config_global trace2.normalBrief 1 &&
|
|
|
|
test_config_global trace2.normalTarget "$(pwd)/trace.normal" &&
|
|
|
|
test-tool trace2 001return 0 &&
|
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 001return 0
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
exit elapsed:_TIME_ code:0
|
|
|
|
atexit elapsed:_TIME_ code:0
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'using global config with include' '
|
|
|
|
test_when_finished "rm trace.normal actual expect real.gitconfig" &&
|
|
|
|
test_config_global trace2.normalBrief 1 &&
|
|
|
|
test_config_global trace2.normalTarget "$(pwd)/trace.normal" &&
|
|
|
|
mv "$(pwd)/.gitconfig" "$(pwd)/real.gitconfig" &&
|
|
|
|
test_config_global include.path "$(pwd)/real.gitconfig" &&
|
|
|
|
test-tool trace2 001return 0 &&
|
|
|
|
perl "$TEST_DIRECTORY/t0210/scrub_normal.perl" <trace.normal >actual &&
|
|
|
|
cat >expect <<-EOF &&
|
|
|
|
version $V
|
|
|
|
start _EXE_ trace2 001return 0
|
|
|
|
cmd_name trace2 (trace2)
|
|
|
|
exit elapsed:_TIME_ code:0
|
|
|
|
atexit elapsed:_TIME_ code:0
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2023-11-23 03:18:35 +08:00
|
|
|
test_expect_success 'unsafe URLs are redacted by default' '
|
|
|
|
test_when_finished \
|
|
|
|
"rm -r trace.normal unredacted.normal clone clone2" &&
|
|
|
|
|
|
|
|
test_config_global \
|
|
|
|
"url.$(pwd).insteadOf" https://user:pwd@example.com/ &&
|
|
|
|
test_config_global trace2.configParams "core.*,remote.*.url" &&
|
|
|
|
|
|
|
|
GIT_TRACE2="$(pwd)/trace.normal" \
|
|
|
|
git clone https://user:pwd@example.com/ clone &&
|
|
|
|
! grep user:pwd trace.normal &&
|
|
|
|
|
|
|
|
GIT_TRACE2_REDACT=0 GIT_TRACE2="$(pwd)/unredacted.normal" \
|
|
|
|
git clone https://user:pwd@example.com/ clone2 &&
|
|
|
|
grep "start .* clone https://user:pwd@example.com" unredacted.normal &&
|
|
|
|
grep "remote.origin.url=https://user:pwd@example.com" unredacted.normal
|
|
|
|
'
|
|
|
|
|
2019-02-23 06:25:10 +08:00
|
|
|
test_done
|