Gedare Bloom started a new discussion on testsuites/psxtests/psxmsgq05/init.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1066#note_142877

 > +  queue = mq_open( QUEUE_NAME, O_CREAT | O_RDWR, 0777, &attr );
 > +  rtems_test_assert( queue != (mqd_t) -1 );
 > +
 > +  puts( "Init - Validate accepted priorities 0..MQ_PRIO_MAX" );
 > +  for ( p = 0; p <= MQ_PRIO_MAX; p++ ) {
 > +    send_msg = (int) p;
 > +    sc = mq_send( queue, (const char *) &send_msg, MSG_SIZE, p );
 > +    rtems_test_assert( sc == 0 );
 > +
 > +    recv_msg = -1;
 > +    recv_prio = 0;
 > +    n = mq_receive( queue, (char *) &recv_msg, MSG_SIZE, &recv_prio );
 > +    rtems_test_assert( n == (ssize_t) MSG_SIZE );
 > +    rtems_test_assert( recv_msg == send_msg );
 > +    rtems_test_assert( recv_prio == p );
 > +  }

This test would be cleaner if you put each of these logically separate test 
cases in separate functions.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1066#note_142877
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to