test: parse arguments before any other stuff

If the args are bad, we don't need to check for root and whatnot.

Only exception here is the debugger check because it changes defaults that we
may want to override with commandline arguments later.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

diff --git a/test/litest.c b/test/litest.c
index a73ff6f..dab62c4 100644 (file)
--- a/test/litest.c
+++ b/test/litest.c
@@ -3988,6 +3988,14 @@ main(int argc, char **argv)
        int failed_tests;
        int rc;
 
+       in_debugger = is_debugger_attached();
+       if (in_debugger)
+               setenv("CK_FORK", "no", 0);
+
+       mode = litest_parse_argv(argc, argv);
+       if (mode == LITEST_MODE_ERROR)
+               return EXIT_FAILURE;
+
        rc = check_device_access();
        if (rc != 0)
                return rc;
@@ -3999,14 +4007,6 @@ main(int argc, char **argv)
        setenv("CK_DEFAULT_TIMEOUT", "30", 0);
        setenv("LIBINPUT_RUNNING_TEST_SUITE", "1", 1);
 
-       in_debugger = is_debugger_attached();
-       if (in_debugger)
-               setenv("CK_FORK", "no", 0);
-
-       mode = litest_parse_argv(argc, argv);
-       if (mode == LITEST_MODE_ERROR)
-               return EXIT_FAILURE;
-
        setup_tests();
 
        if (mode == LITEST_MODE_LIST) {