Fix nasa#1445, add conditional around priority sem test

The multi-thread sem tests (count + bin) have a test that relies on the
priority taking effect, that the highest priority thread will get the
sem first.  However when running as non-root and in permissive mode, the
priority is not applied and thus this becomes a race condtion.

For now, just skip this test if configured for permissive mode.

Author: jphickey

src/tests/bin-sem-test/bin-sem-test.c

@@ -184,18 +184,26 @@ void Test_BinSem(void)
         OS_SUCCESS);
     OS_TaskDelay(100);
     UtAssert_UINT32_EQ(task_counter[0] + task_counter[1] + task_counter[2], 0);
-    UtAssert_INT32_EQ(OS_BinSemGive(sem_id[0]), OS_SUCCESS);
-    OS_TaskDelay(100);
-    UtAssert_UINT32_EQ(task_counter[0], 1);
-    UtAssert_UINT32_EQ(task_counter[1] + task_counter[2], 0);
     UtAssert_INT32_EQ(OS_BinSemFlush(sem_id[0]), OS_SUCCESS);
     OS_TaskDelay(100);
-    UtAssert_UINT32_EQ(task_counter[0], 2);
+    UtAssert_UINT32_EQ(task_counter[0], 1);
     UtAssert_UINT32_EQ(task_counter[1], 1);
     UtAssert_UINT32_EQ(task_counter[2], 1);
 
+    test_val = 3;
+
+    /* In permissive mode, priorities are not enforced, and thus there is no
+     * guarantee of which task will run first.  If priorities are enabled then
+     * it should be guaranteed that Task_0 will get the sem first. */
+#ifndef OSAL_CONFIG_DEBUG_PERMISSIVE_MODE
+    UtAssert_INT32_EQ(OS_BinSemGive(sem_id[0]), OS_SUCCESS);
+    OS_TaskDelay(100);
+    UtAssert_UINT32_EQ(task_counter[0], 2);
+    UtAssert_UINT32_EQ(task_counter[1] + task_counter[2], 2);
+    ++test_val;
+#endif
+
     /* Give loop for tasks to complete */
-    test_val = 4;
     while (test_val < 9)
     {
         test_val++;

src/tests/count-sem-test/count-sem-test.c

@@ -186,13 +186,20 @@ void Test_CountSem(void)
         OS_SUCCESS);
     OS_TaskDelay(100);
     UtAssert_UINT32_EQ(task_counter[0] + task_counter[1] + task_counter[2], 0);
+    test_val = 0;
+
+    /* In permissive mode, priorities are not enforced, and thus there is no
+     * guarantee of which task will run first.  If priorities are enabled then
+     * it should be guaranteed that Task_0 will get the sem first. */
+#ifndef OSAL_CONFIG_DEBUG_PERMISSIVE_MODE
     UtAssert_INT32_EQ(OS_CountSemGive(sem_id[0]), OS_SUCCESS);
     OS_TaskDelay(100);
     UtAssert_UINT32_EQ(task_counter[0], 1);
     UtAssert_UINT32_EQ(task_counter[1] + task_counter[2], 0);
+    ++test_val;
+#endif
 
     /* Give loop for tasks to complete */
-    test_val = 1;
     while (test_val < 9)
     {
         test_val++;