utest: fix wrong tests for tdm_vblank.c 09/163609/1
authorRoman Marchenko <r.marchenko@samsung.com>
Mon, 11 Dec 2017 12:11:48 +0000 (14:11 +0200)
committerRoman Marchenko <r.marchenko@samsung.com>
Tue, 12 Dec 2017 10:06:02 +0000 (12:06 +0200)
   - VblankDestroyWrongVblankPtr:       use exit(0) in a ASSERT_EXIT block
   - VblankGetOffsetFailWrongOffsetPtr: was deleted as libtdm can't check an offset pointer
   - VblankGetOffsetSucces:             error has to be equel TDM_ERROR_NONE

Change-Id: If346f47e01de9fe884563f9ddaa12dcd26142884
Signed-off-by: Roman Marchenko <r.marchenko@samsung.com>
utests/src/ut_tdm_vblank.cpp

index 8f61790..fe4368d 100644 (file)
@@ -460,7 +460,7 @@ TEST_F(TDMVblank, VblankDestroyWrongVblankPtr)
 {
        SKIP_FLAG(has_output);
 
-       ASSERT_EXIT({tdm_vblank_destroy((tdm_vblank *)0xFFFFFFFF);},
+       ASSERT_EXIT({tdm_vblank_destroy((tdm_vblank *)0xFFFFFFFF); exit(0);},
                                ::testing::ExitedWithCode(0), "");
 }
 
@@ -872,19 +872,6 @@ TEST_F(TDMVblank, VblankGetOffsetFailWrongVblankPtr)
                                ::testing::ExitedWithCode(0), "");
 }
 
-TEST_F(TDMVblank, VblankGetOffsetFailWrongOffsetPtr)
-{
-       tdm_error error = TDM_ERROR_BAD_MODULE;
-       SKIP_FLAG(has_output);
-
-       ASSERT_EXIT({error = tdm_vblank_get_offset(default_vblank, (int *)0xFFFFFFFF);
-                               if (error == TDM_ERROR_NONE)
-                                       exit(1);
-                               else
-                                       exit(0);},
-                               ::testing::ExitedWithCode(0), "");
-}
-
 TEST_F(TDMVblank, VblankGetOffsetSuccesWithoutSet)
 {
        tdm_error error;
@@ -905,10 +892,10 @@ TEST_F(TDMVblank, VblankGetOffsetSucces)
        SKIP_FLAG(has_output);
 
        error = tdm_vblank_set_offset(default_vblank, set_offset);
-       ASSERT_TRUE(error != TDM_ERROR_NONE);
+       ASSERT_TRUE(error == TDM_ERROR_NONE);
 
        error = tdm_vblank_get_offset(default_vblank, &ret_offset);
-       ASSERT_TRUE(error != TDM_ERROR_NONE);
+       ASSERT_TRUE(error == TDM_ERROR_NONE);
 
        ASSERT_TRUE(set_offset == ret_offset);
 }