Skip to content

Commit d44b91c

Browse files
committed
Fix #1784, replace refs to cFE_FTAssert macros
Change cFE_FTAssert macros defined in cfe_testcase to use the macros now provided in cfe_assert.h instead.
1 parent 97790b2 commit d44b91c

File tree

6 files changed

+14
-30
lines changed

6 files changed

+14
-30
lines changed

modules/cfe_testcase/src/cfe_test.h

+1-17
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242

4343
#include "uttest.h"
4444
#include "utassert.h"
45+
#include "cfe_assert.h"
4546

4647
typedef struct
4748
{
@@ -57,23 +58,6 @@ typedef struct
5758
*/
5859
#define CFE_ASSERT_LOG_FILE_NAME "/cf/cfe_test.log"
5960

60-
/* Compare two Resource IDs */
61-
#define cFE_FTAssert_ResourceID_EQ(actual, expect) \
62-
UtAssert_True(CFE_RESOURCEID_TEST_EQUAL(actual, expect), "%s (%lu) == %s (%lu)", #actual, \
63-
CFE_RESOURCEID_TO_ULONG(actual), #expect, CFE_RESOURCEID_TO_ULONG(expect))
64-
65-
/* Check if a Resource ID is Undefined */
66-
#define cFE_FTAssert_ResourceID_Undefined(id) \
67-
UtAssert_True(!CFE_RESOURCEID_TEST_DEFINED(id), "%s (%lu) not defined", #id, CFE_RESOURCEID_TO_ULONG(id))
68-
69-
/* Assert a return code is not equal to cfe_success */
70-
#define cFE_FTAssert_NOT_CFE_SUCCESS(actual) \
71-
do \
72-
{ \
73-
int32 rcact = (int32)(actual); \
74-
UtAssert_True(rcact < CFE_SUCCESS, "%s == (%ld) ", #actual, (long)rcact); \
75-
} while (0)
76-
7761
/* Log calls to void functions */
7862
#define cFE_FTAssert_VOIDCALL(func) (func, UtAssert(true, #func, __FILE__, __LINE__))
7963

modules/cfe_testcase/src/es_cds_test.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ void TestCDSName(void)
8282
UtAssert_INT32_EQ(CFE_ES_GetCDSBlockName(CDSNameBuf, CDSHandlePtr, sizeof(CDSNameBuf)), CFE_SUCCESS);
8383
UtAssert_StrCmp(CDSNameBuf, CDSName, "CFE_ES_GetCDSBlockName() = %s", CDSNameBuf);
8484
UtAssert_INT32_EQ(CFE_ES_GetCDSBlockIDByName(&IdByName, CDSNameBuf), CFE_SUCCESS);
85-
cFE_FTAssert_ResourceID_EQ(CDSHandlePtr, IdByName);
85+
CFE_UtAssert_RESOURCEID_EQ(CDSHandlePtr, IdByName);
8686

8787
UtAssert_INT32_EQ(CFE_ES_GetCDSBlockName(NULL, CDSHandlePtr, sizeof(CDSNameBuf)), CFE_ES_BAD_ARGUMENT);
8888
UtAssert_INT32_EQ(CFE_ES_GetCDSBlockName(CDSNameBuf, CFE_ES_CDS_BAD_HANDLE, sizeof(CDSNameBuf)),

modules/cfe_testcase/src/es_info_test.c

+6-6
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ void TestAppInfo(void)
5252

5353
UtAssert_INT32_EQ(CFE_ES_GetAppIDByName(&AppIdByName, TEST_EXPECTED_APP_NAME), CFE_SUCCESS);
5454
UtAssert_INT32_EQ(CFE_ES_GetAppID(&TestAppId), CFE_SUCCESS);
55-
cFE_FTAssert_ResourceID_EQ(TestAppId, AppIdByName);
55+
CFE_UtAssert_RESOURCEID_EQ(TestAppId, AppIdByName);
5656
UtAssert_INT32_EQ(CFE_ES_GetAppName(AppNameBuf, TestAppId, sizeof(AppNameBuf)), CFE_SUCCESS);
5757
UtAssert_StrCmp(AppNameBuf, TEST_EXPECTED_APP_NAME, "CFE_ES_GetAppName() = %s", AppNameBuf);
5858

@@ -122,7 +122,7 @@ void TestAppInfo(void)
122122
UtAssert_True(ESAppInfo.NumOfChildTasks > 0, "ES App Info -> Child Tasks = %d", (int)ESAppInfo.NumOfChildTasks);
123123

124124
UtAssert_INT32_EQ(CFE_ES_GetAppIDByName(&AppIdByName, INVALID_APP_NAME), CFE_ES_ERR_NAME_NOT_FOUND);
125-
cFE_FTAssert_ResourceID_Undefined(AppIdByName);
125+
CFE_UtAssert_RESOURCEID_UNDEFINED(AppIdByName);
126126
UtAssert_INT32_EQ(CFE_ES_GetAppID(NULL), CFE_ES_BAD_ARGUMENT);
127127
UtAssert_INT32_EQ(CFE_ES_GetAppIDByName(NULL, TEST_EXPECTED_APP_NAME), CFE_ES_BAD_ARGUMENT);
128128
UtAssert_INT32_EQ(CFE_ES_GetAppName(AppNameBuf, CFE_ES_APPID_UNDEFINED, sizeof(AppNameBuf)),
@@ -146,15 +146,15 @@ void TestTaskInfo(void)
146146

147147
UtAssert_INT32_EQ(CFE_ES_GetTaskInfo(&TaskInfo, AppInfo.MainTaskId), CFE_SUCCESS);
148148
UtAssert_INT32_EQ(CFE_ES_GetTaskID(&TaskId), CFE_SUCCESS);
149-
cFE_FTAssert_ResourceID_EQ(TaskId, AppInfo.MainTaskId);
149+
CFE_UtAssert_RESOURCEID_EQ(TaskId, AppInfo.MainTaskId);
150150

151151
UtAssert_StrCmp(TaskInfo.AppName, AppInfo.Name, "TaskInfo.AppName (%s) = AppInfo.name (%s)", TaskInfo.AppName,
152152
AppInfo.Name);
153153
UtAssert_StrCmp(TaskInfo.TaskName, AppInfo.MainTaskName, "TaskInfo.TaskName (%s) = AppInfo.MainTaskName (%s)",
154154
TaskInfo.TaskName, AppInfo.MainTaskName);
155155

156-
cFE_FTAssert_ResourceID_EQ(TaskInfo.TaskId, AppInfo.MainTaskId);
157-
cFE_FTAssert_ResourceID_EQ(TaskInfo.AppId, AppId);
156+
CFE_UtAssert_RESOURCEID_EQ(TaskInfo.TaskId, AppInfo.MainTaskId);
157+
CFE_UtAssert_RESOURCEID_EQ(TaskInfo.AppId, AppId);
158158
UtAssert_INT32_EQ(TaskInfo.ExecutionCounter, AppInfo.ExecutionCounter);
159159

160160
UtAssert_INT32_EQ(CFE_ES_GetTaskInfo(&TaskInfo, CFE_ES_TASKID_UNDEFINED), CFE_ES_ERR_RESOURCEID_NOT_VALID);
@@ -205,7 +205,7 @@ void TestLibInfo(void)
205205

206206
UtAssert_INT32_EQ(LibInfo.ExceptionAction, 0);
207207
UtAssert_True(LibInfo.Priority == 0, "Lib Info -> Priority = %d", (int)LibInfo.Priority);
208-
cFE_FTAssert_ResourceID_Undefined(LibInfo.MainTaskId);
208+
CFE_UtAssert_RESOURCEID_UNDEFINED(LibInfo.MainTaskId);
209209
UtAssert_True(LibInfo.ExecutionCounter == 0, "Lib Info -> ExecutionCounter = %d", (int)LibInfo.ExecutionCounter);
210210
UtAssert_True(strlen(LibInfo.MainTaskName) == 0, "Lib Info -> Task Name = %s", LibInfo.MainTaskName);
211211
UtAssert_True(LibInfo.NumOfChildTasks == 0, "Lib Info -> Child Tasks = %d", (int)LibInfo.NumOfChildTasks);

modules/cfe_testcase/src/es_task_test.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -108,14 +108,14 @@ void TestChildTaskName(void)
108108
CFE_SUCCESS);
109109

110110
UtAssert_INT32_EQ(CFE_ES_GetTaskIDByName(&TaskIdByName, TaskName), CFE_SUCCESS);
111-
cFE_FTAssert_ResourceID_EQ(TaskIdByName, TaskId);
111+
CFE_UtAssert_RESOURCEID_EQ(TaskIdByName, TaskId);
112112

113113
UtAssert_INT32_EQ(CFE_ES_GetTaskName(TaskNameBuf, TaskId, sizeof(TaskNameBuf)), CFE_SUCCESS);
114114
UtAssert_StrCmp(TaskNameBuf, TaskName, "CFE_ES_GetTaskName() = %s", TaskNameBuf);
115115

116116
UtAssert_INT32_EQ(CFE_ES_GetTaskIDByName(NULL, TaskName), CFE_ES_BAD_ARGUMENT);
117117
UtAssert_INT32_EQ(CFE_ES_GetTaskIDByName(&TaskIdByName, INVALID_TASK_NAME), CFE_ES_ERR_NAME_NOT_FOUND);
118-
cFE_FTAssert_ResourceID_Undefined(TaskIdByName);
118+
CFE_UtAssert_RESOURCEID_UNDEFINED(TaskIdByName);
119119

120120
UtAssert_INT32_EQ(CFE_ES_GetTaskName(NULL, TaskId, sizeof(TaskNameBuf)), CFE_ES_BAD_ARGUMENT);
121121
UtAssert_INT32_EQ(CFE_ES_GetTaskName(TaskNameBuf, CFE_ES_TASKID_UNDEFINED, sizeof(TaskNameBuf)),

modules/cfe_testcase/src/fs_header_test.c

+3-3
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ void TestCreateHeader(void)
5959
UtAssert_INT32_EQ(OS_lseek(fd, 0, OS_SEEK_CUR), sizeof(CFE_FS_Header_t));
6060

6161
UtAssert_INT32_EQ(CFE_FS_WriteHeader(fd, NULL), CFE_FS_BAD_ARGUMENT);
62-
cFE_FTAssert_NOT_CFE_SUCCESS(CFE_FS_WriteHeader(OS_OBJECT_ID_UNDEFINED, &Header));
62+
CFE_UtAssert_NOT_SUCCESS(CFE_FS_WriteHeader(OS_OBJECT_ID_UNDEFINED, &Header));
6363

6464
cFE_FTAssert_VOIDCALL(CFE_FS_InitHeader(NULL, TestDescription, CFE_FS_SubType_ES_ERLOG));
6565
cFE_FTAssert_VOIDCALL(CFE_FS_InitHeader(&HeaderFail, NULL, CFE_FS_SubType_ES_ERLOG));
@@ -88,7 +88,7 @@ void TestReadHeader(void)
8888
UtAssert_StrCmp(TestDescription, ReadHeader.Description, "ReadHeader.Description = %s", ReadHeader.Description);
8989

9090
UtAssert_INT32_EQ(CFE_FS_ReadHeader(NULL, fd), CFE_FS_BAD_ARGUMENT);
91-
cFE_FTAssert_NOT_CFE_SUCCESS(CFE_FS_ReadHeader(&ReadHeader, OS_OBJECT_ID_UNDEFINED));
91+
CFE_UtAssert_NOT_SUCCESS(CFE_FS_ReadHeader(&ReadHeader, OS_OBJECT_ID_UNDEFINED));
9292

9393
OS_close(fd);
9494
OS_remove(OS_TEST_HEADER_FILENAME);
@@ -114,7 +114,7 @@ void TestTimeStamp(void)
114114
UtAssert_UINT32_EQ(0xFFFFFFFF, ReadHeader.TimeSeconds);
115115
UtAssert_UINT32_EQ(0xFFFFFFFF, ReadHeader.TimeSubSeconds);
116116

117-
cFE_FTAssert_NOT_CFE_SUCCESS(CFE_FS_SetTimestamp(OS_OBJECT_ID_UNDEFINED, NewTimestamp));
117+
CFE_UtAssert_NOT_SUCCESS(CFE_FS_SetTimestamp(OS_OBJECT_ID_UNDEFINED, NewTimestamp));
118118

119119
OS_close(fd);
120120
OS_remove(OS_TEST_HEADER_FILENAME);

modules/cfe_testcase/src/sb_pipe_mang_test.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ void TestPipeName(void)
112112
UtAssert_StrCmp(PipeNameBuf, PipeName, "CFE_SB_GetPipeName() = %s", PipeNameBuf);
113113

114114
UtAssert_INT32_EQ(CFE_SB_GetPipeIdByName(&PipeIdBuff, PipeName), CFE_SUCCESS);
115-
cFE_FTAssert_ResourceID_EQ(PipeId, PipeIdBuff);
115+
CFE_UtAssert_RESOURCEID_EQ(PipeId, PipeIdBuff);
116116

117117
UtAssert_INT32_EQ(CFE_SB_GetPipeName(NULL, sizeof(PipeNameBuf), PipeId), CFE_SB_BAD_ARGUMENT);
118118
UtAssert_INT32_EQ(CFE_SB_GetPipeName(PipeNameBuf, 0, PipeId), CFE_SB_BAD_ARGUMENT);

0 commit comments

Comments
 (0)