Skip to content

Commit

Permalink
Fix broken cmake tests
Browse files Browse the repository at this point in the history
Move install test into basic test to test both cases with only a single build
Remove UT test because it's tested elsewhere as part of the build process and takes a significant amount of time
  • Loading branch information
Joshua-Anderson committed Aug 12, 2021
1 parent 8b0d1c9 commit 470c680
Show file tree
Hide file tree
Showing 10 changed files with 72 additions and 112 deletions.
6 changes: 5 additions & 1 deletion Drv/Ip/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,11 @@ set(SOURCE_FILES
"${CMAKE_CURRENT_LIST_DIR}/UdpSocket.cpp"
"${CMAKE_CURRENT_LIST_DIR}/SocketReadTask.cpp"
)
set(MOD_DEPS Os)

set(MOD_DEPS
Os
Fw/Buffer
)

register_fprime_module()

Expand Down
5 changes: 5 additions & 0 deletions Fw/Buffer/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,11 @@ set(SOURCE_FILES
"${CMAKE_CURRENT_LIST_DIR}/BufferSendPortAi.xml"
"${CMAKE_CURRENT_LIST_DIR}/Buffer.cpp"
)

set(MOD_DEPS
Fw/Types
)

register_fprime_module()

set(UT_SOURCE_FILES
Expand Down
3 changes: 1 addition & 2 deletions Svc/Framer/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,9 @@ set(SOURCE_FILES
)

set(MOD_DEPS
Svc_FramingProtocol
Svc/FramingProtocol
)


register_fprime_module()

#### UTS ###
Expand Down
7 changes: 7 additions & 0 deletions Svc/FramingProtocol/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,12 @@ set(SOURCE_FILES
"${CMAKE_CURRENT_LIST_DIR}/FprimeProtocol.cpp"
)

set(MOD_DEPS
Fw/Buffer
Fw/Time
Utils/Hash
Utils/Types
)

register_fprime_module()

2 changes: 1 addition & 1 deletion cmake/test/src/cmake.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
# Constants to supplied to the calls to subprocess
CMAKE = "cmake"
MAKE_CALL = "make"
MAKE_ARGS = ["-j128"]
MAKE_ARGS = ["-j2"]


def register_test(
Expand Down
55 changes: 53 additions & 2 deletions cmake/test/src/test_basic.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
BUILD_DIR = os.path.join("<FPRIME>", "Ref")
EXPECTED = [
os.path.join("bin", platform.system(), "Ref"),
os.path.join("<FPRIME_INSTALL>", "dict", "RefTopologyAppDictionary.xml"),
os.path.join("lib", platform.system(), "libDrv_BlockDriver.a"),
os.path.join("lib", platform.system(), "libSvc_FileUplink.a"),
os.path.join("lib", platform.system(), "libSvc_CmdSequencer.a"),
Expand Down Expand Up @@ -52,7 +51,6 @@
os.path.join("lib", platform.system(), "libSvc_AssertFatalAdapter.a"),
os.path.join("lib", platform.system(), "libSvc_PolyIf.a"),
os.path.join("lib", platform.system(), "libSvc_FileDownlink.a"),
os.path.join("lib", platform.system(), "libFw_ComFile.a"),
os.path.join("lib", platform.system(), "libRef_RecvBuffApp.a"),
os.path.join("lib", platform.system(), "libSvc_Time.a"),
os.path.join("lib", platform.system(), "libFw_Cfg.a"),
Expand All @@ -63,5 +61,58 @@
os.path.join("lib", platform.system(), "libCFDP_Checksum.a"),
os.path.join("lib", platform.system(), "libSvc_PrmDb.a"),
os.path.join("lib", platform.system(), "libSvc_PassiveConsoleTextLogger.a"),
os.path.join("<FPRIME_INSTALL>", "dict", "RefTopologyAppDictionary.xml"),
os.path.join("<FPRIME_INSTALL>", "bin", "Ref"),
os.path.join("<FPRIME_INSTALL>", "dict", "RefTopologyAppDictionary.xml"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libDrv_BlockDriver.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_FileUplink.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_CmdSequencer.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Time.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_FileManager.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libRef_SendBuffApp.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_FilePacket.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_Ping.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Types.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libUtils_Hash.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Port.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_RateGroupDriver.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_FatalHandler.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_TlmChan.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Obj.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Prm.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_ActiveRateGroup.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_Cycle.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_Health.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_LinuxTime.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libRef_PingReceiver.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Com.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libDrv_DataTypes.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Buffer.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Comp.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_CmdDispatcher.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Log.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Tlm.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libOs.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libRef_SignalGen.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_Sched.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_Seq.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_WatchDog.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_AssertFatalAdapter.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_FileDownlink.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libRef_RecvBuffApp.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_Time.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Cfg.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_Fatal.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_ActiveLogger.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_BufferManager.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libFw_Cmd.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libCFDP_Checksum.a"),
os.path.join("<FPRIME_INSTALL>", "lib", "static", "libSvc_PrmDb.a"),
os.path.join(
"<FPRIME_INSTALL>",
"lib",
"static",
"libSvc_PassiveConsoleTextLogger.a",
),
]
cmake.register_test(__name__, "ref-standard")
1 change: 0 additions & 1 deletion cmake/test/src/test_custom_target.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@
os.path.join("awesome", "Svc_AssertFatalAdapter_is_awesome"),
os.path.join("awesome", "Svc_PolyIf_is_awesome"),
os.path.join("awesome", "Svc_FileDownlink_is_awesome"),
os.path.join("awesome", "Fw_ComFile_is_awesome"),
os.path.join("awesome", "Svc_Time_is_awesome"),
os.path.join("awesome", "Fw_Cfg_is_awesome"),
os.path.join("awesome", "Svc_Fatal_is_awesome"),
Expand Down
66 changes: 0 additions & 66 deletions cmake/test/src/test_install.py

This file was deleted.

1 change: 0 additions & 1 deletion cmake/test/src/test_shared.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@
os.path.join("lib", platform.system(), "libSvc_AssertFatalAdapter." + ext),
os.path.join("lib", platform.system(), "libSvc_PolyIf." + ext),
os.path.join("lib", platform.system(), "libSvc_FileDownlink." + ext),
os.path.join("lib", platform.system(), "libFw_ComFile." + ext),
os.path.join("lib", platform.system(), "libRef_RecvBuffApp." + ext),
os.path.join("lib", platform.system(), "libSvc_Time." + ext),
os.path.join("lib", platform.system(), "libFw_Cfg." + ext),
Expand Down
38 changes: 0 additions & 38 deletions cmake/test/src/test_ut.py

This file was deleted.

0 comments on commit 470c680

Please sign in to comment.