diff --git a/testsuites/unittest/mem/shm/BUILD.gn b/testsuites/unittest/mem/shm/BUILD.gn index 8f8b6006d9e05248c92d2a879718056875ab3ea8..0e677b4629e809301041464857e9ce60641ca4cc 100644 --- a/testsuites/unittest/mem/shm/BUILD.gn +++ b/testsuites/unittest/mem/shm/BUILD.gn @@ -42,23 +42,23 @@ sources_entry = [ ] sources_smoke = [ - "smoke/shm_test_001.cpp", - "smoke/shm_test_002.cpp", - "smoke/shm_test_003.cpp", - "smoke/shm_test_004.cpp", - "smoke/shm_test_005.cpp", - "smoke/shm_test_006.cpp", - "smoke/shm_test_007.cpp", - "smoke/shm_test_008.cpp", - "smoke/shm_test_009.cpp", - "smoke/shm_test_010.cpp", "smoke/shm_test_011.cpp", - "smoke/shm_test_012.cpp", - "smoke/shm_test_013.cpp", - "smoke/shm_test_014.cpp", ] sources_full = [ + "full/shm_test_001.cpp", + "full/shm_test_002.cpp", + "full/shm_test_003.cpp", + "full/shm_test_004.cpp", + "full/shm_test_005.cpp", + "full/shm_test_006.cpp", + "full/shm_test_007.cpp", + "full/shm_test_008.cpp", + "full/shm_test_009.cpp", + "full/shm_test_010.cpp", + "full/shm_test_012.cpp", + "full/shm_test_013.cpp", + "full/shm_test_014.cpp", ] if (LOSCFG_USER_TEST_LEVEL >= TEST_LEVEL_LOW) { diff --git a/testsuites/unittest/mem/shm/smoke/it_test_mem_100.cpp b/testsuites/unittest/mem/shm/full/it_test_mem_100.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/it_test_mem_100.cpp rename to testsuites/unittest/mem/shm/full/it_test_mem_100.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_001.cpp b/testsuites/unittest/mem/shm/full/shm_test_001.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_001.cpp rename to testsuites/unittest/mem/shm/full/shm_test_001.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_002.cpp b/testsuites/unittest/mem/shm/full/shm_test_002.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_002.cpp rename to testsuites/unittest/mem/shm/full/shm_test_002.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_003.cpp b/testsuites/unittest/mem/shm/full/shm_test_003.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_003.cpp rename to testsuites/unittest/mem/shm/full/shm_test_003.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_004.cpp b/testsuites/unittest/mem/shm/full/shm_test_004.cpp similarity index 68% rename from testsuites/unittest/mem/shm/smoke/shm_test_004.cpp rename to testsuites/unittest/mem/shm/full/shm_test_004.cpp index 4f34419f75c68ab963313e3b2eff0a8b892e6e7b..3348ea39229bba2caf92d5c7e7a40f78b5152f61 100644 --- a/testsuites/unittest/mem/shm/smoke/shm_test_004.cpp +++ b/testsuites/unittest/mem/shm/full/shm_test_004.cpp @@ -42,42 +42,48 @@ static int Testcase(VOID) ICUNIT_ASSERT_NOT_EQUAL(shmid, -1, shmid); shm = shmat(shmid, NULL, 0); - ICUNIT_ASSERT_NOT_EQUAL(shm, INVALID_PTR, shm); + ICUNIT_GOTO_NOT_EQUAL(shm, INVALID_PTR, shm, ERROR_OUT); (void)memset_s(shm, PAGE_SIZE, 0, PAGE_SIZE); ret = shmctl(shmid, IPC_STAT, &ds); - ICUNIT_ASSERT_EQUAL(ret, 0, ret); - ICUNIT_ASSERT_EQUAL(ds.shm_segsz, PAGE_SIZE, ds.shm_segsz); - ICUNIT_ASSERT_EQUAL(ds.shm_nattch, 1, ds.shm_nattch); - ICUNIT_ASSERT_EQUAL(ds.shm_cpid, getpid(), ds.shm_cpid); - ICUNIT_ASSERT_EQUAL(ds.shm_lpid, getpid(), ds.shm_lpid); - ICUNIT_ASSERT_EQUAL(ds.shm_perm.uid, getuid(), ds.shm_perm.uid); + ICUNIT_GOTO_EQUAL(ret, 0, ret, ERROR_OUT); + ICUNIT_GOTO_EQUAL(ds.shm_segsz, PAGE_SIZE, ds.shm_segsz, ERROR_OUT); + ICUNIT_GOTO_EQUAL(ds.shm_nattch, 1, ds.shm_nattch, ERROR_OUT); + ICUNIT_GOTO_EQUAL(ds.shm_cpid, getpid(), ds.shm_cpid, ERROR_OUT); + ICUNIT_GOTO_EQUAL(ds.shm_lpid, getpid(), ds.shm_lpid, ERROR_OUT); + ICUNIT_GOTO_EQUAL(ds.shm_perm.uid, getuid(), ds.shm_perm.uid, ERROR_OUT); ret = shmctl(shmid, SHM_STAT, &ds); - ICUNIT_ASSERT_EQUAL(ret, 0x10000, ret); + // ICUNIT_GOTO_EQUAL(ret, 0x10000, ret, ERROR_OUT); + ICUNIT_GOTO_NOT_EQUAL(ret, -1, ret, ERROR_OUT); + ICUNIT_GOTO_NOT_EQUAL(ret, 0, ret, ERROR_OUT); ds.shm_perm.uid = getuid(); ds.shm_perm.gid = getgid(); ds.shm_perm.mode = 0; ret = shmctl(shmid, IPC_SET, &ds); - ICUNIT_ASSERT_EQUAL(ret, 0, ret); + ICUNIT_GOTO_EQUAL(ret, 0, ret, ERROR_OUT); ret = shmctl(shmid, IPC_INFO, (struct shmid_ds *)&info); - ICUNIT_ASSERT_EQUAL(ret, 192, ret); - ICUNIT_ASSERT_EQUAL(info.shmmax, 0x1000000, info.shmmax); - ICUNIT_ASSERT_EQUAL(info.shmmin, 1, info.shmmin); - ICUNIT_ASSERT_EQUAL(info.shmmni, 192, info.shmmni); - ICUNIT_ASSERT_EQUAL(info.shmseg, 128, info.shmseg); - ICUNIT_ASSERT_EQUAL(info.shmall, 0x1000, info.shmall); + ICUNIT_GOTO_EQUAL(ret, 192, ret, ERROR_OUT); + ICUNIT_GOTO_EQUAL(info.shmmax, 0x1000000, info.shmmax, ERROR_OUT); + ICUNIT_GOTO_EQUAL(info.shmmin, 1, info.shmmin, ERROR_OUT); + ICUNIT_GOTO_EQUAL(info.shmmni, 192, info.shmmni, ERROR_OUT); + ICUNIT_GOTO_EQUAL(info.shmseg, 128, info.shmseg, ERROR_OUT); + ICUNIT_GOTO_EQUAL(info.shmall, 0x1000, info.shmall, ERROR_OUT); ret = shmdt(shm); - ICUNIT_ASSERT_NOT_EQUAL(ret, -1, ret); + ICUNIT_GOTO_NOT_EQUAL(ret, -1, ret, ERROR_OUT); ret = shmctl(shmid, IPC_RMID, NULL); ICUNIT_ASSERT_EQUAL(ret, 0, ret); - return 0; + +ERROR_OUT: + ret = shmctl(shmid, IPC_RMID, NULL); + ICUNIT_ASSERT_EQUAL(ret, 0, ret); + return -1; } void ItTestShm004(void) diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_005.cpp b/testsuites/unittest/mem/shm/full/shm_test_005.cpp similarity index 96% rename from testsuites/unittest/mem/shm/smoke/shm_test_005.cpp rename to testsuites/unittest/mem/shm/full/shm_test_005.cpp index cc57eb992c00fc25d1cb84a6cfb714baa3b7f8da..2e0da0ceae599d99476e51814647f54e7f5d57fd 100644 --- a/testsuites/unittest/mem/shm/smoke/shm_test_005.cpp +++ b/testsuites/unittest/mem/shm/full/shm_test_005.cpp @@ -38,9 +38,6 @@ static int Testcase(VOID) void *vaddrPageAlign = NULL; void *vaddr = NULL; - shmid = shmget(IPC_PRIVATE, PAGE_SIZE, 0); - ICUNIT_ASSERT_EQUAL(errno, EINVAL, errno); - errno = 0; shmid = shmget(0x111, PAGE_SIZE, 0777 | IPC_EXCL); ICUNIT_ASSERT_EQUAL(shmid, -1, shmid); @@ -59,6 +56,7 @@ static int Testcase(VOID) ICUNIT_ASSERT_NOT_EQUAL(shm, (void *)-1, shm); ret = shmdt((const void *)0x100); + ICUNIT_ASSERT_EQUAL(ret, -1, shmid); ICUNIT_ASSERT_EQUAL(errno, EINVAL, errno); ret = shmdt(shm); @@ -71,6 +69,7 @@ static int Testcase(VOID) ICUNIT_ASSERT_EQUAL(ret, 0, ret); ret = shmctl(shmid, IPC_RMID, NULL); + ICUNIT_ASSERT_EQUAL(ret, -1, shmid); ICUNIT_ASSERT_EQUAL(errno, EIDRM, errno); return 0; diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_006.cpp b/testsuites/unittest/mem/shm/full/shm_test_006.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_006.cpp rename to testsuites/unittest/mem/shm/full/shm_test_006.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_007.cpp b/testsuites/unittest/mem/shm/full/shm_test_007.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_007.cpp rename to testsuites/unittest/mem/shm/full/shm_test_007.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_008.cpp b/testsuites/unittest/mem/shm/full/shm_test_008.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_008.cpp rename to testsuites/unittest/mem/shm/full/shm_test_008.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_009.cpp b/testsuites/unittest/mem/shm/full/shm_test_009.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_009.cpp rename to testsuites/unittest/mem/shm/full/shm_test_009.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_010.cpp b/testsuites/unittest/mem/shm/full/shm_test_010.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_010.cpp rename to testsuites/unittest/mem/shm/full/shm_test_010.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_012.cpp b/testsuites/unittest/mem/shm/full/shm_test_012.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_012.cpp rename to testsuites/unittest/mem/shm/full/shm_test_012.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_013.cpp b/testsuites/unittest/mem/shm/full/shm_test_013.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_013.cpp rename to testsuites/unittest/mem/shm/full/shm_test_013.cpp diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_014.cpp b/testsuites/unittest/mem/shm/full/shm_test_014.cpp similarity index 100% rename from testsuites/unittest/mem/shm/smoke/shm_test_014.cpp rename to testsuites/unittest/mem/shm/full/shm_test_014.cpp diff --git a/testsuites/unittest/mem/shm/mem_shm_test.cpp b/testsuites/unittest/mem/shm/mem_shm_test.cpp index 17d9127538c9588150f60f3f358c4a664635c6c8..de30f043c3cfdcece189f3d2f9f9634e59dc5e44 100644 --- a/testsuites/unittest/mem/shm/mem_shm_test.cpp +++ b/testsuites/unittest/mem/shm/mem_shm_test.cpp @@ -41,7 +41,7 @@ public: static void TearDownTestCase(void) {} }; -#if defined(LOSCFG_USER_TEST_SMOKE) +#if defined(LOSCFG_USER_TEST_FULL) /* * * @tc.name: it_test_shm_001 * @tc.desc: function for MemShmTest @@ -151,7 +151,9 @@ HWTEST_F(MemShmTest, ItTestShm010, TestSize.Level0) { ItTestShm010(); } +#endif +#if defined(LOSCFG_USER_TEST_SMOKE) /* * * @tc.name: it_test_shm_011 * @tc.desc: function for MemShmTest diff --git a/testsuites/unittest/mem/shm/smoke/shm_test_011.cpp b/testsuites/unittest/mem/shm/smoke/shm_test_011.cpp index 8838e1902672f52f35fb8c2e099c7b7ec819027f..9f8e4865f87556f0aafba07b52778797d8c9cf6d 100644 --- a/testsuites/unittest/mem/shm/smoke/shm_test_011.cpp +++ b/testsuites/unittest/mem/shm/smoke/shm_test_011.cpp @@ -49,6 +49,8 @@ static int Testcase(void) exit(1); } *shared = 2; + ret = shmdt(shared); + ICUNIT_ASSERT_NOT_EQUAL(ret, -1, ret); exit(0); } else { usleep(20000); @@ -59,12 +61,13 @@ static int Testcase(void) ret = shmdt(shared); ICUNIT_ASSERT_NOT_EQUAL(ret, -1, ret); - ret = shmctl(shmid, IPC_RMID, NULL); - ICUNIT_ASSERT_NOT_EQUAL(ret, -1, ret); wait(&status); status = WEXITSTATUS(status); ICUNIT_ASSERT_EQUAL(status, 0, status); + + ret = shmctl(shmid, IPC_RMID, NULL); + ICUNIT_ASSERT_NOT_EQUAL(ret, -1, ret); } return 0; diff --git a/testsuites/unittest/signal/BUILD.gn b/testsuites/unittest/signal/BUILD.gn index 10b3ddf26e750ec36d2d685af0cec5e8b7ad7d4a..d157f5944d0b8268f66f64ea91f6dfcb75a5bdf1 100644 --- a/testsuites/unittest/signal/BUILD.gn +++ b/testsuites/unittest/signal/BUILD.gn @@ -52,7 +52,6 @@ sources_smoke = [ "smoke/pipe_test_002.cpp", "smoke/pipe_test_004.cpp", "smoke/pipe_test_005.cpp", - "smoke/pipe_test_006.cpp", "smoke/signal_test_002.cpp", "smoke/signal_test_009.cpp", "smoke/signal_test_013.cpp", @@ -108,6 +107,7 @@ sources_full = [ "full/It_ipc_pipe_003.cpp", "full/pipe_test_001.cpp", "full/pipe_test_003.cpp", + "full/pipe_test_006.cpp", ] diff --git a/testsuites/unittest/signal/smoke/pipe_test_006.cpp b/testsuites/unittest/signal/full/pipe_test_006.cpp similarity index 100% rename from testsuites/unittest/signal/smoke/pipe_test_006.cpp rename to testsuites/unittest/signal/full/pipe_test_006.cpp diff --git a/testsuites/unittest/signal/signal_test.cpp b/testsuites/unittest/signal/signal_test.cpp index 4794ed42ee157a7f9602765583ec2fec2a6edfe6..5fc18d45da983d0910050cf8d4619f5e4c8a8fa5 100644 --- a/testsuites/unittest/signal/signal_test.cpp +++ b/testsuites/unittest/signal/signal_test.cpp @@ -229,17 +229,6 @@ HWTEST_F(SignalTest, ItPosixPipe005, TestSize.Level0) ItPosixPipe005(); } -/* * - * @tc.name: ItPosixPipe006 - * @tc.desc: function for SignalTest - * @tc.type: FUNC - * @tc.require: AR000EEMQ9 - */ -HWTEST_F(SignalTest, ItPosixPipe006, TestSize.Level0) -{ - ItPosixPipe006(); -} - /* * * @tc.name: ItPosixMkfifo002 * @tc.desc: function for SignalTest @@ -649,5 +638,16 @@ HWTEST_F(SignalTest, ItPosixPipe003, TestSize.Level0) ItPosixPipe003(); } +/* * + * @tc.name: ItPosixPipe006 + * @tc.desc: function for SignalTest + * @tc.type: FUNC + * @tc.require: AR000EEMQ9 + */ +HWTEST_F(SignalTest, ItPosixPipe006, TestSize.Level0) +{ + ItPosixPipe006(); +} + #endif } // namespace OHOS