summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2014-06-13 18:03:50 +0200
committerDaniel Vetter <daniel.vetter@ffwll.ch>2014-06-13 18:09:05 +0200
commitac0e606677823d27836708300a2c5ee05d24b458 (patch)
tree1816b8ba71276903b80c14214145ebbb0b3383de
parentbfede4ed2b401069c3130399ff5c2580e6ccbf77 (diff)
Revert "tests: Run igt.cocci over tests"
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>
-rw-r--r--tests/drv_hangman.c14
-rw-r--r--tests/kms_psr_sink_crc.c4
2 files changed, 12 insertions, 6 deletions
diff --git a/tests/drv_hangman.c b/tests/drv_hangman.c
index 667491ba..d74ed483 100644
--- a/tests/drv_hangman.c
+++ b/tests/drv_hangman.c
@@ -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);
+ }
}
}
diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
index a36ae6c1..999a0bfd 100644
--- a/tests/kms_psr_sink_crc.c
+++ b/tests/kms_psr_sink_crc.c
@@ -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;