]> git.ipfire.org Git - thirdparty/git.git/commitdiff
test-tool: extend trace2 helper with 400ancestry
authorMatthew John Cheetham <mjcheetham@outlook.com>
Fri, 13 Feb 2026 19:54:59 +0000 (19:54 +0000)
committerJunio C Hamano <gitster@pobox.com>
Fri, 13 Feb 2026 20:18:32 +0000 (12:18 -0800)
Add a new test helper "400ancestry" to the trace2 test-tool that
spawns a child process with a controlled trace2 environment, capturing
only the child's trace2 output (including cmd_ancestry events) in
isolation.

The helper clears all inherited GIT_TRACE2* variables in the child
and enables only the requested target (normal, perf, or event),
directing output to a specified file. This gives the test suite a
reliable way to capture cmd_ancestry events: the child always sees
"test-tool" as its immediate parent in the process ancestry, providing
a predictable value to verify in tests.

Signed-off-by: Matthew John Cheetham <mjcheetham@outlook.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/helper/test-trace2.c

index 415df078c1638a3a1fdf5b555efbef63e68be43b..3b12f4173e4c093e9293645a6cac0e49830e30f7 100644 (file)
@@ -466,6 +466,63 @@ static int ut_303redact_def_param(int argc, const char **argv)
        return 0;
 }
 
+/*
+ * Run a child process with specific trace2 environment settings so that
+ * we can capture its trace2 output (including cmd_ancestry) in isolation.
+ *
+ * test-tool trace2 400ancestry <target> <output_file> [<child_command_line>]
+ *
+ * <target> is one of: normal, perf, event
+ *
+ * For example:
+ *     test-tool trace2 400ancestry normal out.normal test-tool trace2 001return 0
+ *
+ * The child process inherits a controlled trace2 environment where only
+ * the specified target is directed to <output_file>. The parent's trace2
+ * environment variables are cleared in the child so that only the child's
+ * events are captured.
+ *
+ * This is used by t0213-trace2-ancestry.sh to test cmd_ancestry events.
+ * The child process will see "test-tool" as its immediate parent in the
+ * process ancestry, giving us a predictable value to verify.
+ */
+static int ut_400ancestry(int argc, const char **argv)
+{
+       struct child_process cmd = CHILD_PROCESS_INIT;
+       const char *target;
+       const char *outfile;
+       int result;
+
+       if (argc < 3)
+               die("expect <target> <output_file> <child_command_line>");
+
+       target = argv[0];
+       outfile = argv[1];
+       argv += 2;
+       argc -= 2;
+
+       /* Clear all trace2 environment variables in the child. */
+       strvec_push(&cmd.env, "GIT_TRACE2=");
+       strvec_push(&cmd.env, "GIT_TRACE2_PERF=");
+       strvec_push(&cmd.env, "GIT_TRACE2_EVENT=");
+       strvec_push(&cmd.env, "GIT_TRACE2_BRIEF=1");
+
+       /* Set only the requested target. */
+       if (!strcmp(target, "normal"))
+               strvec_pushf(&cmd.env, "GIT_TRACE2=%s", outfile);
+       else if (!strcmp(target, "perf"))
+               strvec_pushf(&cmd.env, "GIT_TRACE2_PERF=%s", outfile);
+       else if (!strcmp(target, "event"))
+               strvec_pushf(&cmd.env, "GIT_TRACE2_EVENT=%s", outfile);
+       else
+               die("invalid target '%s', expected: normal, perf, event",
+                   target);
+
+       strvec_pushv(&cmd.args, argv);
+       result = run_command(&cmd);
+       exit(result);
+}
+
 /*
  * Usage:
  *     test-tool trace2 <ut_name_1> <ut_usage_1>
@@ -497,6 +554,8 @@ static struct unit_test ut_table[] = {
        { ut_301redact_child_start, "301redact_child_start", "<argv...>" },
        { ut_302redact_exec,        "302redact_exec",        "<exe> <argv...>" },
        { ut_303redact_def_param,   "303redact_def_param",   "<key> <value>" },
+
+       { ut_400ancestry,           "400ancestry",           "<target> <output_file> [<child_command_line>]" },
 };
 /* clang-format on */