提交 f995c7c7 编写于 作者: Z zhangdengyu

feat:修复使用不规范的测试用例

1、在编译kernel/liteos_a测试用例时process、extended、libc三个模块出现报错,主要分为以下几种类型:
   1.1、函数返回值为int或其它类型,错误使用了返回NULL的断言宏。修改断言宏返回正确的类型。
   1.2、函数返回值为int或其它类型,错误的return了NULL。修改成正确的返回值类型。
   1.3、waitpid函数第三个参数为int类型,错误传入了NULL。修改成正确的入参类型。
   1.4、使用的断言宏传入的参数一个为int或其他类型,一个为NULL,类似不匹配。修改成匹配的类型。
   1.5、testsuites/unittest/process/basic/pthread/smoke/pthread_test_012.cpp中调用Syscall接口,
        第二个参数为in类型,错误的传入了NULL。修改入参为正确的类型。
2、hb set选择ipcamera_hispark_taurus,编译之后使用nfs将out/hispark_taurus/ipcamera_hispark_taurus/test/unittest/kernel/bin目录中的
   9个测试用例放到单板上进行测试,运行结果全部通过,并生成对应结果从xml文件,其中liteos_a_basic_unittest.bin无xml文件生成是由于gtest框架升级引入。

Close: #I6J7QK
Signed-off-by: Nzhangdengyu <zhangdengyu2@huawei.com>
Change-Id: Ifa977658bfc47ba85484e06465735cdffef3dbe2
上级 c2b5a589
...@@ -53,7 +53,7 @@ static UINT32 TestCase(VOID) ...@@ -53,7 +53,7 @@ static UINT32 TestCase(VOID)
char sentence1[15] = "Hello World"; char sentence1[15] = "Hello World";
char a[4] = {0}; char a[4] = {0};
retValue = sprintf_s(a, sizeof(a), "%d", i); retValue = sprintf_s(a, sizeof(a), "%d", i);
ICUNIT_ASSERT_NOT_EQUAL(retValue, NULL, retValue); ICUNIT_ASSERT_NOT_EQUAL(retValue, 0, retValue);
retValue = strcat_s(sentence1, sizeof(sentence1), a); retValue = strcat_s(sentence1, sizeof(sentence1), a);
ICUNIT_ASSERT_EQUAL(retValue, 0, retValue); ICUNIT_ASSERT_EQUAL(retValue, 0, retValue);
ret = write(*writeFd, sentence1, strlen(sentence1) + 1); ret = write(*writeFd, sentence1, strlen(sentence1) + 1);
......
...@@ -44,7 +44,7 @@ static UINT32 testcase(VOID) ...@@ -44,7 +44,7 @@ static UINT32 testcase(VOID)
return LOS_NOK; return LOS_NOK;
} }
confstr(_CS_PATH, pathbuf, n); confstr(_CS_PATH, pathbuf, n);
ICUNIT_ASSERT_NOT_EQUAL_NULL(pathbuf, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(pathbuf, NULL, -1);
free(pathbuf); free(pathbuf);
pathbuf = NULL; pathbuf = NULL;
...@@ -57,7 +57,7 @@ static UINT32 testcase(VOID) ...@@ -57,7 +57,7 @@ static UINT32 testcase(VOID)
return LOS_NOK; return LOS_NOK;
} }
confstr(_CS_GNU_LIBPTHREAD_VERSION, gun_libpthread_version_buf, n); confstr(_CS_GNU_LIBPTHREAD_VERSION, gun_libpthread_version_buf, n);
ICUNIT_ASSERT_NOT_EQUAL_NULL(gun_libpthread_version_buf, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(gun_libpthread_version_buf, NULL, -1);
free(gun_libpthread_version_buf); free(gun_libpthread_version_buf);
gun_libpthread_version_buf = NULL; gun_libpthread_version_buf = NULL;
...@@ -70,7 +70,7 @@ static UINT32 testcase(VOID) ...@@ -70,7 +70,7 @@ static UINT32 testcase(VOID)
return LOS_NOK; return LOS_NOK;
} }
confstr(_CS_GNU_LIBC_VERSION, gun_libc_version_buf, n); confstr(_CS_GNU_LIBC_VERSION, gun_libc_version_buf, n);
ICUNIT_ASSERT_NOT_EQUAL_NULL(gun_libc_version_buf, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(gun_libc_version_buf, NULL, -1);
free(gun_libc_version_buf); free(gun_libc_version_buf);
gun_libc_version_buf = NULL; gun_libc_version_buf = NULL;
......
...@@ -47,17 +47,17 @@ static UINT32 testcase(VOID) ...@@ -47,17 +47,17 @@ static UINT32 testcase(VOID)
printf("Locale is: %s\n", setlocale(LC_TIME, "en_US.UTF-8")); printf("Locale is: %s\n", setlocale(LC_TIME, "en_US.UTF-8"));
strftime(buffer, 80, "%c", timer); strftime(buffer, 80, "%c", timer);
printf("Date is: %s\n", buffer); printf("Date is: %s\n", buffer);
ICUNIT_ASSERT_NOT_EQUAL_NULL(buffer, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(buffer, NULL, -1);
printf("Locale is: %s\n", setlocale(LC_TIME, "zh_CN.UTF-8")); printf("Locale is: %s\n", setlocale(LC_TIME, "zh_CN.UTF-8"));
strftime(buffer, 80, "%c", timer); strftime(buffer, 80, "%c", timer);
printf("Date is: %s\n", buffer); printf("Date is: %s\n", buffer);
ICUNIT_ASSERT_NOT_EQUAL_NULL(buffer, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(buffer, NULL, -1);
printf("Locale is: %s\n", setlocale(LC_TIME, "")); printf("Locale is: %s\n", setlocale(LC_TIME, ""));
strftime(buffer, 80, "%c", timer); strftime(buffer, 80, "%c", timer);
printf("Date is: %s\n", buffer); printf("Date is: %s\n", buffer);
ICUNIT_ASSERT_NOT_EQUAL_NULL(buffer, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(buffer, NULL, -1);
setlocale(LC_ALL, "C"); setlocale(LC_ALL, "C");
return LOS_OK; return LOS_OK;
......
...@@ -47,17 +47,17 @@ static UINT32 testcase(VOID) ...@@ -47,17 +47,17 @@ static UINT32 testcase(VOID)
printf("Locale is: %s\n", setlocale(LC_TIME, "en_US.UTF-8")); printf("Locale is: %s\n", setlocale(LC_TIME, "en_US.UTF-8"));
(void)strftime(buffer, sizeof(buffer), "%c", timer); (void)strftime(buffer, sizeof(buffer), "%c", timer);
printf("Date is: %s\n", buffer); printf("Date is: %s\n", buffer);
ICUNIT_ASSERT_NOT_EQUAL_NULL(buffer, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(buffer, NULL, -1);
printf("Locale is: %s\n", setlocale(LC_TIME, "zh_CN.UTF-8")); printf("Locale is: %s\n", setlocale(LC_TIME, "zh_CN.UTF-8"));
(void)strftime(buffer, sizeof(buffer), "%c", timer); (void)strftime(buffer, sizeof(buffer), "%c", timer);
printf("Date is: %s\n", buffer); printf("Date is: %s\n", buffer);
ICUNIT_ASSERT_NOT_EQUAL_NULL(buffer, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(buffer, NULL, -1);
printf("Locale is: %s\n", setlocale(LC_TIME, "")); printf("Locale is: %s\n", setlocale(LC_TIME, ""));
(void)strftime(buffer, sizeof(buffer), "%c", timer); (void)strftime(buffer, sizeof(buffer), "%c", timer);
printf("Date is: %s\n", buffer); printf("Date is: %s\n", buffer);
ICUNIT_ASSERT_NOT_EQUAL_NULL(buffer, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(buffer, NULL, -1);
setlocale(LC_ALL, "C"); setlocale(LC_ALL, "C");
return LOS_OK; return LOS_OK;
......
...@@ -89,7 +89,7 @@ static UINT32 testcase(VOID) ...@@ -89,7 +89,7 @@ static UINT32 testcase(VOID)
printf("%s\n", nl_langinfo(CRNCYSTR)); printf("%s\n", nl_langinfo(CRNCYSTR));
char *string = nl_langinfo(CRNCYSTR); char *string = nl_langinfo(CRNCYSTR);
ICUNIT_ASSERT_NOT_EQUAL_NULL(string, NULL, string); ICUNIT_ASSERT_NOT_EQUAL(string, NULL, string);
setlocale(LC_ALL, "C"); setlocale(LC_ALL, "C");
return LOS_OK; return LOS_OK;
......
...@@ -57,7 +57,7 @@ static UINT32 testcase(VOID) { ...@@ -57,7 +57,7 @@ static UINT32 testcase(VOID) {
printf("%s\n", nl_langinfo_l(RADIXCHAR, reinterpret_cast<locale_t>(const_cast<char *>("zh_CN.UTF-8")))); printf("%s\n", nl_langinfo_l(RADIXCHAR, reinterpret_cast<locale_t>(const_cast<char *>("zh_CN.UTF-8"))));
char *string = nl_langinfo_l(CRNCYSTR, reinterpret_cast<locale_t>(const_cast<char *>("zh_CN.UTF-8"))); char *string = nl_langinfo_l(CRNCYSTR, reinterpret_cast<locale_t>(const_cast<char *>("zh_CN.UTF-8")));
ICUNIT_ASSERT_NOT_EQUAL_NULL(string, NULL, string); ICUNIT_ASSERT_NOT_EQUAL(string, NULL, string);
setlocale(LC_ALL, "C"); setlocale(LC_ALL, "C");
return LOS_OK; return LOS_OK;
......
...@@ -77,7 +77,7 @@ static UINT32 testcase(VOID) ...@@ -77,7 +77,7 @@ static UINT32 testcase(VOID)
if (mnt && !(feof(fp) || ferror(fp))) { if (mnt && !(feof(fp) || ferror(fp))) {
ret = hasmntopt(mnt, opt); ret = hasmntopt(mnt, opt);
printf("hasmntopt=%s\n", ret); printf("hasmntopt=%s\n", ret);
ICUNIT_ASSERT_NOT_EQUAL_NULL(ret, NULL, -1); ICUNIT_ASSERT_NOT_EQUAL(ret, NULL, -1);
mnt = getmntent(fp); mnt = getmntent(fp);
} }
......
...@@ -45,11 +45,11 @@ static int TestThread(void *arg) ...@@ -45,11 +45,11 @@ static int TestThread(void *arg)
int ret = 0; int ret = 0;
pthread_t gTh; pthread_t gTh;
ret = pthread_create(&gTh, NULL, PthreadTest115, NULL); ret = pthread_create(&gTh, NULL, PthreadTest115, NULL);
ICUNIT_ASSERT_EQUAL_NULL(ret, 0, ret); ICUNIT_ASSERT_EQUAL(ret, 0, ret);
ret = pthread_join(gTh, NULL); ret = pthread_join(gTh, NULL);
ICUNIT_ASSERT_EQUAL_NULL(ret, 0, ret); ICUNIT_ASSERT_EQUAL(ret, 0, ret);
ICUNIT_ASSERT_EQUAL_NULL(g_usetTestCount, 1, g_usetTestCount); ICUNIT_ASSERT_EQUAL(g_usetTestCount, 1, g_usetTestCount);
*g_shmptr = 100; // 100, set shared num. *g_shmptr = 100; // 100, set shared num.
...@@ -66,11 +66,11 @@ static int TestThread(void *arg) ...@@ -66,11 +66,11 @@ static int TestThread(void *arg)
ret = waitpid(pid, NULL, 0); ret = waitpid(pid, NULL, 0);
printf("waitpid ret : %d errno : %d pid : %d getpid : %d\n", ret, errno, pid, getpid()); printf("waitpid ret : %d errno : %d pid : %d getpid : %d\n", ret, errno, pid, getpid());
ICUNIT_ASSERT_EQUAL_NULL(ret, pid, ret); ICUNIT_ASSERT_EQUAL(ret, pid, ret);
*g_shmptr = data; *g_shmptr = data;
EXIT: EXIT:
return NULL; return 0;
} }
// This testcase us used for undefination of LOSCFG_USER_TEST_SMP // This testcase us used for undefination of LOSCFG_USER_TEST_SMP
...@@ -100,7 +100,7 @@ static int Testcase(void) ...@@ -100,7 +100,7 @@ static int Testcase(void)
ICUNIT_GOTO_EQUAL(*g_shmptr, arg, *g_shmptr, EXIT2); ICUNIT_GOTO_EQUAL(*g_shmptr, arg, *g_shmptr, EXIT2);
ret = waitpid(pid, &status, NULL); ret = waitpid(pid, &status, 0);
ICUNIT_GOTO_EQUAL(ret, pid, ret, EXIT2); ICUNIT_GOTO_EQUAL(ret, pid, ret, EXIT2);
EXIT2: EXIT2:
......
...@@ -39,10 +39,10 @@ static int TestThread(void *arg) ...@@ -39,10 +39,10 @@ static int TestThread(void *arg)
printf("TestThread ppid : %d g_ppid : %d\n", ppid, g_ppid); printf("TestThread ppid : %d g_ppid : %d\n", ppid, g_ppid);
ICUNIT_ASSERT_EQUAL_NULL(ppid, g_ppid, g_ppid); ICUNIT_ASSERT_EQUAL(ppid, g_ppid, g_ppid);
*g_shmptr = 100; // 100, set shared num. *g_shmptr = 100; // 100, set shared num.
return NULL; return 0;
} }
// This testcase us used for undefination of LOSCFG_USER_TEST_SMP // This testcase us used for undefination of LOSCFG_USER_TEST_SMP
...@@ -55,7 +55,7 @@ static int Testcase(void) ...@@ -55,7 +55,7 @@ static int Testcase(void)
int ret; int ret;
int count; int count;
pid_t pid; pid_t pid;
int shmid = shmget(IPC_PRIVATE, sizeof(int), IPC_CREAT | 0600); // 0600 config of shmget int shmid = shmget(IPC_PRIVATE, sizeof(int), IPC_CREAT | 0600); // 0600 config of shmget
ICUNIT_ASSERT_NOT_EQUAL(shmid, -1, shmid); ICUNIT_ASSERT_NOT_EQUAL(shmid, -1, shmid);
...@@ -73,7 +73,7 @@ static int Testcase(void) ...@@ -73,7 +73,7 @@ static int Testcase(void)
stackTop = reinterpret_cast<char *>(reinterpret_cast<unsigned long>(stack) + arg); stackTop = reinterpret_cast<char *>(reinterpret_cast<unsigned long>(stack) + arg);
pid = clone(TestThread, (void *)stackTop, CLONE_PARENT, &arg); pid = clone(TestThread, (void *)stackTop, CLONE_PARENT, &arg);
ret = waitpid(pid, &status, NULL); ret = waitpid(pid, &status, 0);
ICUNIT_GOTO_EQUAL(ret, -1, ret, EXIT2); ICUNIT_GOTO_EQUAL(ret, -1, ret, EXIT2);
count = 0; count = 0;
......
...@@ -41,7 +41,7 @@ static int TestThread(void *arg) ...@@ -41,7 +41,7 @@ static int TestThread(void *arg)
printf("TestThread ppid : %d g_ppid : %d\n", ppid, g_ppid); printf("TestThread ppid : %d g_ppid : %d\n", ppid, g_ppid);
ICUNIT_ASSERT_EQUAL_NULL(ppid, g_ppid, g_ppid); ICUNIT_ASSERT_EQUAL(ppid, g_ppid, g_ppid);
*g_shmptr = 1000; // 1000, set shared num. *g_shmptr = 1000; // 1000, set shared num.
...@@ -54,11 +54,11 @@ static int TestThread(void *arg) ...@@ -54,11 +54,11 @@ static int TestThread(void *arg)
} }
ret = waitpid(pid, NULL, 0); ret = waitpid(pid, NULL, 0);
ICUNIT_ASSERT_EQUAL_NULL(ret, pid, ret); ICUNIT_ASSERT_EQUAL(ret, pid, ret);
*g_shmptr = 100; // 100, set shared num. *g_shmptr = 100; // 100, set shared num.
EXIT: EXIT:
return NULL; return 0;
} }
// This testcase us used for undefination of LOSCFG_USER_TEST_SMP // This testcase us used for undefination of LOSCFG_USER_TEST_SMP
...@@ -70,7 +70,7 @@ static int Testcase(void) ...@@ -70,7 +70,7 @@ static int Testcase(void)
void *stack; void *stack;
char *stackTop; char *stackTop;
int ret; int ret;
int shmid = shmget(IPC_PRIVATE, sizeof(int), IPC_CREAT | 0600); // 0600 config of shmget int shmid = shmget(IPC_PRIVATE, sizeof(int), IPC_CREAT | 0600); // 0600 config of shmget
ICUNIT_ASSERT_NOT_EQUAL(shmid, -1, shmid); ICUNIT_ASSERT_NOT_EQUAL(shmid, -1, shmid);
...@@ -90,7 +90,7 @@ static int Testcase(void) ...@@ -90,7 +90,7 @@ static int Testcase(void)
ICUNIT_GOTO_EQUAL(*g_shmptr, 100, *g_shmptr, EXIT2); // 100, assert g_shmptr equal to this. ICUNIT_GOTO_EQUAL(*g_shmptr, 100, *g_shmptr, EXIT2); // 100, assert g_shmptr equal to this.
ret = waitpid(pid, &status, NULL); ret = waitpid(pid, &status, 0);
ICUNIT_GOTO_EQUAL(ret, -1, ret, EXIT2); ICUNIT_GOTO_EQUAL(ret, -1, ret, EXIT2);
EXIT2: EXIT2:
......
...@@ -105,7 +105,7 @@ static int Testcase(void) ...@@ -105,7 +105,7 @@ static int Testcase(void)
exit(Child()); exit(Child());
} else if (ret > 0) { } else if (ret > 0) {
pid = ret; pid = ret;
ret = waitpid(pid, &status, NULL); ret = waitpid(pid, &status, 0);
status = WEXITSTATUS(status); status = WEXITSTATUS(status);
ICUNIT_ASSERT_EQUAL(ret, pid, ret); ICUNIT_ASSERT_EQUAL(ret, pid, ret);
ICUNIT_ASSERT_EQUAL(status, 0, status); ICUNIT_ASSERT_EQUAL(status, 0, status);
......
...@@ -33,7 +33,7 @@ ...@@ -33,7 +33,7 @@
static int Testcase(void) static int Testcase(void)
{ {
int ret; int ret;
ret = Syscall(SYS_set_thread_area, NULL, 0, 0, 0); ret = Syscall(SYS_set_thread_area, 0x0, 0, 0, 0);
ICUNIT_ASSERT_EQUAL(ret, EINVAL, ret); ICUNIT_ASSERT_EQUAL(ret, EINVAL, ret);
ret = Syscall(SYS_set_thread_area, 0x100, 0, 0, 0); ret = Syscall(SYS_set_thread_area, 0x100, 0, 0, 0);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册