Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add kernel testcases #6177

Closed
wants to merge 3 commits into from
Closed

Add kernel testcases #6177

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c5852bb
add itc_environ changes
NayanaNagaraj23 May 3, 2024
0e73d08
Modification of function names as positive and negative and addition …
NayanaNagaraj23 May 6, 2024
ad2513e
added some negative test cases for kernel modules
NayanaNagaraj23 May 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 4 additions & 4 deletions apps/examples/testcase/le_tc/kernel/itc_environ.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
* Preconditions :none
* Postconditions :none
*/
static void itc_environ_setenv_p(void)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We check _p and _n internally in the automation tool. Do you have any reason to change?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We check _p and _n internally in the automation tool. Do you have any reason to change?

In some functions the _p and _n are at the middle and others are at the end of the function name, this is not identified properly in the TCM tool. So, I have changed the names as _pos and _neg at the end of the function for better identification and uniformity.

static void itc_environ_setenv_pos(void)
{
char *buffer = "";
char *result;
Expand Down Expand Up @@ -74,7 +74,7 @@ static void itc_environ_setenv_p(void)
* Preconditions :none
* Postconditions :none
*/
static void itc_environ_getenv_n(void)
static void itc_environ_getenv_neg(void)
{
int exec_index;
char var_buf[BUFF_LEN];
Expand Down Expand Up @@ -103,8 +103,8 @@ static void itc_environ_getenv_n(void)

int itc_environ_main(void)
{
itc_environ_setenv_p();
itc_environ_getenv_n();
itc_environ_setenv_pos();
itc_environ_getenv_neg();

return 0;
}
4 changes: 2 additions & 2 deletions apps/examples/testcase/le_tc/kernel/itc_libc_semaphore.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
* @Preconditions :NA
* @Postconditions :NA
*/
static void itc_libc_semaphore_getprotocol_n_no_sem_init(void)
static void itc_libc_semaphore_getprotocol_no_sem_init_neg(void)
{
sem_t sem;
int protocol;
Expand All @@ -62,6 +62,6 @@ static void itc_libc_semaphore_getprotocol_n_no_sem_init(void)

int itc_libc_semaphore_main(void)
{
itc_libc_semaphore_getprotocol_n_no_sem_init();
itc_libc_semaphore_getprotocol_no_sem_init_neg();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as previous

u/itc_module_feature_function_n/p_details

This is what we use in the naming and automation tool parses.
Please let me know why you change the naming.

return 0;
}
8 changes: 4 additions & 4 deletions apps/examples/testcase/le_tc/kernel/itc_sched.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ static void thread_cleanup(pthread_t* array, int thread_cnt)
* Preconditions :none
* Postconditions :none
*/
static void itc_sched_setget_scheduler_param_p_all_priority(void)
static void itc_sched_setget_scheduler_param_all_priority_pos(void)
{
int ret_chk = ERROR;
struct sched_param st_setparam;
Expand Down Expand Up @@ -148,7 +148,7 @@ static void itc_sched_setget_scheduler_param_p_all_priority(void)
* Preconditions :none
* Postconditions :none
*/
static void itc_sched_self_p(void)
static void itc_sched_self_pos(void)
{
pthread_t thread_id[THREAD_CNT];
int ret_chk;
Expand All @@ -171,8 +171,8 @@ static void itc_sched_self_p(void)

int itc_sched_main(void)
{
itc_sched_setget_scheduler_param_p_all_priority();
itc_sched_self_p();
itc_sched_setget_scheduler_param_all_priority_pos();
itc_sched_self_pos();

return 0;
}
4 changes: 2 additions & 2 deletions apps/examples/testcase/le_tc/kernel/itc_semaphore.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
* Preconditions :none
* Postconditions :none
*/
static void itc_semaphore_sem_init_destroy_p_multitime(void)
static void itc_semaphore_sem_init_destroy_multitime_pos(void)
{
sem_t sem;
int ret_chk;
Expand All @@ -62,7 +62,7 @@ static void itc_semaphore_sem_init_destroy_p_multitime(void)

int itc_semaphore_main(void)
{
itc_semaphore_sem_init_destroy_p_multitime();
itc_semaphore_sem_init_destroy_multitime_pos();

return 0;
}
8 changes: 4 additions & 4 deletions apps/examples/testcase/le_tc/kernel/itc_timer.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
* Preconditions :Creation of timer_id(timer_create)
* Postconditions :none
*/
static void itc_timer_set_time_n_after_delete(void)
static void itc_timer_set_time_after_delete_neg(void)
{
int ret_chk = ERROR;
clockid_t clockid = CLOCK_REALTIME;
Expand Down Expand Up @@ -72,7 +72,7 @@ static void itc_timer_set_time_n_after_delete(void)
* Preconditions :Creation of timer_id(timer_create)
* Postconditions :none
*/
static void itc_timer_get_time_n_after_delete(void)
static void itc_timer_get_time_after_delete_neg(void)
{
int ret_chk = ERROR;
clockid_t clockid = CLOCK_REALTIME;
Expand Down Expand Up @@ -111,8 +111,8 @@ static void itc_timer_get_time_n_after_delete(void)

int itc_timer_main(void)
{
itc_timer_set_time_n_after_delete();
itc_timer_get_time_n_after_delete();
itc_timer_set_time_after_delete_neg();
itc_timer_get_time_after_delete_neg();

return 0;
}
79 changes: 64 additions & 15 deletions apps/examples/testcase/le_tc/kernel/tc_clock.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ extern struct timespec g_basetime;
* Postconditions :none
* @return :void
*/
static void tc_clock_clock_getres(void)
static void tc_clock_clock_getres_pos(void)
{
int ret_chk;
struct timespec st_res;
Expand All @@ -66,6 +66,14 @@ static void tc_clock_clock_getres(void)
TC_ASSERT_EQ("clock_getres", st_res.tv_sec, 0);
TC_ASSERT_EQ("clock_getres", st_res.tv_nsec, NSEC_PER_TICK);

TC_SUCCESS_RESULT();
}

static void tc_clock_clock_getres_neg(void)
{
int ret_chk;
struct timespec st_res;

ret_chk = clock_getres(33 , &st_res);
TC_ASSERT_EQ("clock_getres", ret_chk, ERROR);

Expand All @@ -81,15 +89,25 @@ static void tc_clock_clock_getres(void)
* Postconditions :none
* @return :void
*/
static void tc_clock_clock_set_get_time(void)


static void tc_clock_clock_get_time_pos(void)
{
int ret_chk;
struct timespec stime;
struct timespec gtime;

ret_chk = clock_gettime(CLOCK_ERR, &stime);
TC_ASSERT_EQ("clock_gettime", ret_chk, ERROR);
TC_ASSERT_EQ("clock_gettime", errno, EINVAL);
ret_chk = clock_gettime(CLOCK_REALTIME, &stime);
TC_ASSERT_EQ("clock_gettime", ret_chk, OK);

TC_SUCCESS_RESULT();
}

static void tc_clock_clock_set_time_pos(void)
{
int ret_chk;
struct timespec stime;
struct timespec gtime;

ret_chk = clock_gettime(CLOCK_REALTIME, &stime);
TC_ASSERT_EQ("clock_gettime", ret_chk, OK);
Expand All @@ -111,12 +129,30 @@ static void tc_clock_clock_set_get_time(void)

stime.tv_sec -= l_day; /* Setting original time to system */

ret_chk = clock_settime(CLOCK_REALTIME, &stime);
TC_ASSERT_EQ("clock_setime", ret_chk, OK);

TC_SUCCESS_RESULT();
}

static void tc_clock_clock_set_time_neg(void)
{
int ret_chk;
struct timespec stime;

ret_chk = clock_settime(CLOCK_ERR, &stime);
TC_ASSERT_EQ("clock_settime", ret_chk, ERROR);
TC_ASSERT_EQ("clock_settime", errno, EINVAL);

ret_chk = clock_settime(CLOCK_REALTIME, &stime);
TC_ASSERT_EQ("clock_setime", ret_chk, OK);
TC_SUCCESS_RESULT();
}

static void tc_clock_clock_get_time_invalid_clock_neg(void)
{
int ret_chk;
struct timespec stime;

ret_chk = clock_gettime(CLOCK_ERR, &stime);
TC_ASSERT_EQ("clock_gettime", errno, EINVAL);

TC_SUCCESS_RESULT();
}
Expand All @@ -130,7 +166,7 @@ static void tc_clock_clock_set_get_time(void)
* Postconditions :none
* @return :void
*/
static void tc_clock_clock_gettimeofday(void)
static void tc_clock_clock_gettimeofday_pos(void)
{
int ret_chk;
struct timeval tv1;
Expand All @@ -148,6 +184,15 @@ static void tc_clock_clock_gettimeofday(void)
if (tv2.tv_sec == tv1.tv_sec + SEC_2) {
TC_ASSERT_GEQ("gettimeofday", tv2.tv_usec, tv1.tv_usec);
}

TC_SUCCESS_RESULT();
}


static void tc_clock_clock_gettimeofday_neg(void)
{
int ret_chk;

#ifdef CONFIG_DEBUG
ret_chk = gettimeofday(NULL, NULL);
TC_ASSERT_EQ("gettimeofday", ret_chk, ERROR);
Expand All @@ -165,7 +210,7 @@ static void tc_clock_clock_gettimeofday(void)
* Postconditions :none
* @return :void
*/
static void tc_clock_clock_abstime2ticks(void)
static void tc_clock_clock_abstime2ticks_pos(void)
{
int fd;
int ret;
Expand All @@ -184,10 +229,14 @@ static void tc_clock_clock_abstime2ticks(void)

int clock_main(void)
{
tc_clock_clock_gettimeofday();
tc_clock_clock_set_get_time();
tc_clock_clock_getres();
tc_clock_clock_abstime2ticks();

tc_clock_clock_gettimeofday_pos();
tc_clock_clock_gettimeofday_neg();
tc_clock_clock_get_time_invalid_clock_neg();
tc_clock_clock_get_time_pos();
tc_clock_clock_set_time_pos();
tc_clock_clock_set_time_neg();
tc_clock_clock_getres_pos();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check indentation

tc_clock_clock_getres_neg();
tc_clock_clock_abstime2ticks_pos();
return 0;
}