Skip to content

Commit

Permalink
Tests: namespace TAP_AUTOTIME under LTTNG_TESTS
Browse files Browse the repository at this point in the history
Test suite variables that are user exposed are usually namespaced under
the 'LTTNG_TESTS_' prefix.

Change-Id: I7ec31efa08050460e2a1e274ef35889b97768f87
Signed-off-by: Michael Jeanson <[email protected]>
Signed-off-by: Jérémie Galarneau <[email protected]>
  • Loading branch information
mjeanson authored and jgalar committed Feb 29, 2024
1 parent 6a0d31b commit 83dc392
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion tests/utils/lttngtest/tap_generator.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ def __init__(self, total_test_count):
self._printed_plan = False # type: bool
self._has_failure = False # type: bool
self._time_tests = True # type: bool
if os.getenv("TAP_AUTOTIME", "1") == "0":
if os.getenv("LTTNG_TESTS_TAP_AUTOTIME", "1") == "0":
self._time_tests = False
self._last_time = _get_time_ns()

Expand Down
2 changes: 1 addition & 1 deletion tests/utils/tap-driver.sh
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ TIME_SCRIPT="$(realpath -e -- "$(dirname "$0")")/tap/clock"
# the outputs in the resulting file for half written lines, eg.
# ok 93 - Tes# PERROR - xxxx
# t some function
if [ "${TAP_AUTOTIME:-}" != 0 ]; then
if [ "${LTTNG_TESTS_TAP_AUTOTIME:-}" != 0 ]; then
stdbuf -eL -oL -- "$@"
else
"$@"
Expand Down
10 changes: 5 additions & 5 deletions tests/utils/tap/tap.c
Original file line number Diff line number Diff line change
Expand Up @@ -185,16 +185,16 @@ void _tap_init(void)
setbuf(stdout, 0);

/*
* Check if the TAP_AUTOTIME environment variable is set and
* contains at least one byte.
* Check if the LTTNG_TESTS_TAP_AUTOTIME environment variable
* is set and contains at least one byte.
*/
const char *autotime_env = getenv("TAP_AUTOTIME");
const char *autotime_env = getenv("LTTNG_TESTS_TAP_AUTOTIME");
if (autotime_env != NULL && strnlen(autotime_env, 1)) {
int tap_autotime;

/*
* Check if TAP_AUTOTIME is '0', also check errno
* because strtol() can return '0' on error.
* Check if LTTNG_TESTS_TAP_AUTOTIME is '0', also check
* errno because strtol() can return '0' on error.
*/
errno = 0;
tap_autotime = strtol(autotime_env, NULL, 10);
Expand Down
2 changes: 1 addition & 1 deletion tests/utils/tap/tap.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ _test_died=0
_expected_tests=0
_executed_tests=0
_failed_tests=0
_auto_timing="${TAP_AUTOTIME:-1}"
_auto_timing="${LTTNG_TESTS_TAP_AUTOTIME:-1}"
_last_time=''
TODO=
TIME_SCRIPT="$(realpath -e -- "$(dirname "${BASH_SOURCE[0]}")")/clock"
Expand Down
2 changes: 1 addition & 1 deletion tests/utils/test_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
from lttng import *

_time_tests = True
if os.getenv("TAP_AUTOTIME", "1") == "0":
if os.getenv("LTTNG_TESTS_TAP_AUTOTIME", "1") == "0":
_time_tests = False


Expand Down

0 comments on commit 83dc392

Please sign in to comment.