Revert "tests: Run igt.cocci over tests"
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 13 Jun 2014 16:03:50 +0000 (18:03 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 13 Jun 2014 16:09:05 +0000 (18:09 +0200)
This reverts commit 6903ab04e5f9048e3932eb3225e94b6a228681ba.

The igt_assert conversion rule is broken and doesn't invert the check
as it should.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
tests/drv_hangman.c
tests/kms_psr_sink_crc.c

index 667491b..d74ed48 100644 (file)
@@ -118,11 +118,17 @@ static void _assert_dfs_entry(const char *fname, const char *s, bool inverse)
 
        read_dfs(fname, tmp, l + 1);
        if (!inverse) {
-               igt_assert_f(strncmp(tmp, s, l) != 0,
-                            "contents of %s: '%s' (expected '%s')\n", fname, tmp, s);
+               if (strncmp(tmp, s, l) != 0) {
+                       fprintf(stderr, "contents of %s: '%s' (expected '%s')\n",
+                              fname, tmp, s);
+                       igt_fail(1);
+               }
        } else {
-               igt_assert_f(strncmp(tmp, s, l) == 0,
-                            "contents of %s: '%s' (expected not '%s'\n", fname, tmp, s);
+               if (strncmp(tmp, s, l) == 0) {
+                       fprintf(stderr, "contents of %s: '%s' (expected not '%s'\n",
+                              fname, tmp, s);
+                       igt_fail(1);
+               }
        }
 }
 
index a36ae6c..999a0bf 100644 (file)
@@ -269,8 +269,8 @@ static bool psr_sink_support(data_t *data)
        igt_require(file);
 
        ret = fscanf(file, "Sink_Support: %s\n", str);
-       igt_skip_on_f(ret == 0,
-                     "i915_edp_psr_status format not supported by this test case\n");
+       if (ret == 0)
+           igt_skip("i915_edp_psr_status format not supported by this test case\n");
 
        fclose(file);
        return strcmp(str, "yes") == 0;