Do not consider this patch, it is bogus.
The test plan is already defined inside the notification executable runned in
kernel_test.
Thanks
On Thu, Mar 28, 2019 at 05:02:21PM -0400, Jonathan Rajotte wrote:
> Otherwise tap parsers can complain that there was no test plan and
> report a failure.
>
> Signed-off-by: Jonathan Rajotte <jonathan.rajotte-jul...@efficios.com>
> ---
> tests/regression/tools/notification/test_notification_kernel | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/regression/tools/notification/test_notification_kernel
> b/tests/regression/tools/notification/test_notification_kernel
> index e7368df26..604cc48cf 100755
> --- a/tests/regression/tools/notification/test_notification_kernel
> +++ b/tests/regression/tools/notification/test_notification_kernel
> @@ -107,12 +107,13 @@ function kernel_test
> rm -rf ${consumerd_pipe[@]} 2> /dev/null
> }
>
> +plan_tests $NUM_TESTS
> +
> if [ "$(id -u)" == "0" ]; then
> validate_lttng_modules_present
> kernel_test
> else
> # Kernel tests are skipped.
> - plan_tests $NUM_TESTS
> skip 0 "Root access is needed. Skipping all kernel notification tests."
> $NUM_TESTS
> fi
>
> --
> 2.17.1
>
--
Jonathan Rajotte-Julien
EfficiOS
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
