diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-05-22 09:34:36 +0200 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-05-22 09:36:31 +0200 |
commit | e2ac7a15f195b67a44c474920cd220baab92f6b1 (patch) | |
tree | dcdfd5a7714f9b64ae1e3fc02c58d9f4fd3f1851 /tests | |
parent | 0675765eff4a7409afa98734aa0c8fbc02ce5ff6 (diff) |
tests/kms_sink_crc_basic: Use igt_assert
Less verbose code makes for clearer test logic.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/kms_sink_crc_basic.c | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/tests/kms_sink_crc_basic.c b/tests/kms_sink_crc_basic.c index 4be115c2..087b79db 100644 --- a/tests/kms_sink_crc_basic.c +++ b/tests/kms_sink_crc_basic.c @@ -134,28 +134,19 @@ static void basic_sink_crc_check(data_t *data, uint32_t connector_id) /* get reference crc for black color */ get_crc(ref_crc_black); - if (strcmp(ref_crc_black, ref_crc_white) == 0) { - fprintf(stderr, "CRC Black should differ from CRC White\n"); - igt_fail(1); - } + igt_assert(strcmp(ref_crc_black, ref_crc_white) != 0); /*Go White again*/ connector_set_mode(data, &connector, &connector.config.default_mode, WHITE); get_crc(crc_check); - if (strcmp(crc_check, ref_crc_white) != 0) { - fprintf(stderr, "CRC check for white differ from reference\n"); - igt_fail(1); - } + igt_assert(strcmp(crc_check, ref_crc_white) == 0); /* Go Black again */ connector_set_mode(data, &connector, &connector.config.default_mode, BLACK); get_crc(crc_check); - if (strcmp(crc_check, ref_crc_black) != 0) { - fprintf(stderr, "CRC check for black differ from reference\n"); - igt_fail(1); - } + igt_assert(strcmp(crc_check, ref_crc_black) == 0); kmstest_free_connector_config(&connector.config); } |