Skip to content

Commit

Permalink
Fix #1417, generated coverage stubs for CFE core
Browse files Browse the repository at this point in the history
Update CFE core stub libraries to use generated stubs, using the generator
script now part of UT Assert.  All existing stub logic is converted to
a default handler routine.

This also entailed moving all of the internal API prototypes to be
in the "core_private" interface lib rather than in "core_api".  The
only reason this was not done earlier was due to issues with the stubs,
but that is alleviated when using generated stubs.
  • Loading branch information
jphickey committed Apr 30, 2021
1 parent c8b5e00 commit b35bc34
Show file tree
Hide file tree
Showing 43 changed files with 6,263 additions and 4,409 deletions.
1 change: 1 addition & 0 deletions modules/core_api/fsw/inc/cfe_fs.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
#include "cfe_error.h"
#include "cfe_fs_api_typedefs.h"
#include "cfe_fs_extern_typedefs.h"
#include "cfe_time_api_typedefs.h"

/** @defgroup CFEAPIFSHeader cFE File Header Management APIs
* @{
Expand Down
24 changes: 16 additions & 8 deletions modules/core_api/ut-stubs/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,22 @@ include_directories(${osal_MISSION_DIR}/ut_assert/inc)
# Create the generic stubs library
#
add_library(ut_core_api_stubs STATIC
src/ut_es_stubs.c
src/ut_evs_stubs.c
src/ut_msg_stubs.c
src/ut_resourceid_stubs.c
src/ut_sb_stubs.c
src/ut_tbl_stubs.c
src/ut_time_stubs.c
src/ut_fs_stubs.c
src/cfe_es_hooks.c
src/cfe_es_stubs.c
src/cfe_evs_hooks.c
src/cfe_evs_stubs.c
src/cfe_fs_hooks.c
src/cfe_fs_stubs.c
src/cfe_msg_hooks.c
src/cfe_msg_stubs.c
src/cfe_resourceid_hooks.c
src/cfe_resourceid_stubs.c
src/cfe_sb_hooks.c
src/cfe_sb_stubs.c
src/cfe_tbl_hooks.c
src/cfe_tbl_stubs.c
src/cfe_time_hooks.c
src/cfe_time_stubs.c
)

# Define _CFE_CORE_ within stubs to also reveal internal APIs in header
Expand Down
Loading

0 comments on commit b35bc34

Please sign in to comment.