[PATCH wayland v2] tests: rename env vars for tests

Pekka Paalanen ppaalanen at gmail.com
Fri Nov 28 03:28:56 PST 2014


On Fri, 28 Nov 2014 09:41:50 +0100
Marek Chalupa <mchqwerty at gmail.com> wrote:

> Rename WAYLAND_TESTS_NO_TIMEOUTS to WAYLAND_TEST_NO_TIMEOUTS.
> Further rename NO_ASSERT_LEAK_CHECK to WAYLAND_TEST_NO_LEAK_CHECK.
> Now the naming is consistent not only here, in Wayland, but even with
> naming of weston env varibles related to testing.
> 
> This is version 2 of the patch. The first version just renamed
> NO_ASSERT_LEAK_CHECK to WAYLAND_TEST_NO_LEAK_CHECK.
> 
> Signed-off-by: Marek Chalupa <mchqwerty at gmail.com>
> ---
>  tests/test-runner.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/test-runner.c b/tests/test-runner.c
> index d965a43..67763b3 100644
> --- a/tests/test-runner.c
> +++ b/tests/test-runner.c
> @@ -42,11 +42,14 @@ static void (*sys_free)(void*);
>  static void* (*sys_realloc)(void*, size_t);
>  static void* (*sys_calloc)(size_t, size_t);
>  
> +/* when set to 1, check if tests are not leaking memory and opened files.
> + * It is turned on by default. It can be turned off by
> + * WAYLAND_TEST_NO_LEAK_CHECK environment variable. */
>  int leak_check_enabled;
>  
>  /* when this var is set to 0, every call to test_set_timeout() is
>   * suppressed - handy when debugging the test. Can be set by
> - * WAYLAND_TESTS_NO_TIMEOUTS evnironment var */
> + * WAYLAND_TEST_NO_TIMEOUTS evnironment var */
>  static int timeouts_enabled = 1;
>  
>  extern const struct test __start_test_section, __stop_test_section;
> @@ -234,8 +237,8 @@ int main(int argc, char *argv[])
>  	sys_malloc = dlsym(RTLD_NEXT, "malloc");
>  	sys_free = dlsym(RTLD_NEXT, "free");
>  
> -	leak_check_enabled = !getenv("NO_ASSERT_LEAK_CHECK");
> -	timeouts_enabled = !getenv("WAYLAND_TESTS_NO_TIMEOUTS");
> +	leak_check_enabled = !getenv("WAYLAND_TEST_NO_LEAK_CHECK");
> +	timeouts_enabled = !getenv("WAYLAND_TEST_NO_TIMEOUTS");
>  
>  	if (argc == 2 && strcmp(argv[1], "--help") == 0)
>  		usage(argv[0], EXIT_SUCCESS);

Pushed.

Thanks,
pq


More information about the wayland-devel mailing list