From a4dd6721f2846a1052c4517076e6c6e1b88041a7 Mon Sep 17 00:00:00 2001 From: "Jose F. Martinez Pedraza" Date: Tue, 10 Aug 2021 15:54:42 -0400 Subject: [PATCH] Fix #812 Add Perf API functional tests and add Perf Trigger and Perf Start commands to functional tests workflow --- .github/workflows/functional-tests.yml | 5 +- modules/cfe_testcase/CMakeLists.txt | 1 + modules/cfe_testcase/src/cfe_test.c | 1 + modules/cfe_testcase/src/cfe_test.h | 1 + modules/cfe_testcase/src/es_perf_test.c | 74 +++++++++++++++++++++++++ 5 files changed, 81 insertions(+), 1 deletion(-) create mode 100644 modules/cfe_testcase/src/es_perf_test.c diff --git a/.github/workflows/functional-tests.yml b/.github/workflows/functional-tests.yml index 7f0ff0965..0241f2b1e 100644 --- a/.github/workflows/functional-tests.yml +++ b/.github/workflows/functional-tests.yml @@ -63,11 +63,14 @@ jobs: - name: List cpu1 run: ls build/exe/cpu1/ + # Run cFS, send commands to set perf trigger and start perf data, and run functional tests - name: Run cFS run: | ./core-cpu1 & sleep 10 - ../host/cmdUtil --pktid=0x1806 --cmdcode=4 --endian=LE --string="20:CFE_TEST_APP" --string="20:CFE_TestMain" --string="64:cfe_testcase" --uint32=16384 --uint8=0 --uint8=0 --uint16=100 & + ../host/cmdUtil --pktid=0x1806 --cmdcode=17 --endian=LE --uint32=3 --uint32=0x40000000 + ../host/cmdUtil --pktid=0x1806 --cmdcode=14 --endian=LE --uint32=2 + ../host/cmdUtil --pktid=0x1806 --cmdcode=4 --endian=LE --string="20:CFE_TEST_APP" --string="20:CFE_TestMain" --string="64:cfe_testcase" --uint32=16384 --uint8=0 --uint8=0 --uint16=100 sleep 30 counter=0 diff --git a/modules/cfe_testcase/CMakeLists.txt b/modules/cfe_testcase/CMakeLists.txt index c5abc909e..cb3c3c111 100644 --- a/modules/cfe_testcase/CMakeLists.txt +++ b/modules/cfe_testcase/CMakeLists.txt @@ -11,6 +11,7 @@ add_cfe_app(cfe_testcase src/es_cds_test.c src/es_counter_test.c src/es_misc_test.c + src/es_perf_test.c src/es_mempool_test.c src/es_resource_id_test.c src/evs_filters_test.c diff --git a/modules/cfe_testcase/src/cfe_test.c b/modules/cfe_testcase/src/cfe_test.c index 580074d4f..52d9b8efc 100644 --- a/modules/cfe_testcase/src/cfe_test.c +++ b/modules/cfe_testcase/src/cfe_test.c @@ -65,6 +65,7 @@ void CFE_TestMain(void) ESMemPoolTestSetup(); ESMiscTestSetup(); ESResourceIDTestSetup(); + ESPerfTestSetup(); ESTaskTestSetup(); EVSFiltersTestSetup(); EVSSendTestSetup(); diff --git a/modules/cfe_testcase/src/cfe_test.h b/modules/cfe_testcase/src/cfe_test.h index 09614a2c5..b3c196ffc 100644 --- a/modules/cfe_testcase/src/cfe_test.h +++ b/modules/cfe_testcase/src/cfe_test.h @@ -96,6 +96,7 @@ void ESInfoTestSetup(void); void ESMemPoolTestSetup(void); void ESMiscTestSetup(void); void ESResourceIDTestSetup(void); +void ESPerfTestSetup(void); void ESTaskTestSetup(void); void EVSFiltersTestSetup(void); void EVSSendTestSetup(void); diff --git a/modules/cfe_testcase/src/es_perf_test.c b/modules/cfe_testcase/src/es_perf_test.c new file mode 100644 index 000000000..ebba792b4 --- /dev/null +++ b/modules/cfe_testcase/src/es_perf_test.c @@ -0,0 +1,74 @@ +/************************************************************************* +** +** GSC-18128-1, "Core Flight Executive Version 6.7" +** +** Copyright (c) 2006-2019 United States Government as represented by +** the Administrator of the National Aeronautics and Space Administration. +** All Rights Reserved. +** +** Licensed under the Apache License, Version 2.0 (the "License"); +** you may not use this file except in compliance with the License. +** You may obtain a copy of the License at +** +** http://www.apache.org/licenses/LICENSE-2.0 +** +** Unless required by applicable law or agreed to in writing, software +** distributed under the License is distributed on an "AS IS" BASIS, +** WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +** See the License for the specific language governing permissions and +** limitations under the License. +** +** File: es_perf_test.c +** +** Purpose: +** Functional test of ES Performance APIs +** +*************************************************************************/ + +/* + * Includes + */ + +#include "cfe_test.h" + +void TestPerfLogEntry(void) +{ + UtAssert_VOIDCALL(CFE_ES_PerfLogEntry(0)); + UtAssert_VOIDCALL(CFE_ES_PerfLogEntry(CFE_MISSION_ES_PERF_MAX_IDS-1)); + UtAssert_VOIDCALL(CFE_ES_PerfLogEntry(CFE_MISSION_ES_PERF_MAX_IDS)); +} + +void TestPerfLogExit(void) +{ + UtAssert_VOIDCALL(CFE_ES_PerfLogExit(0)); + UtAssert_VOIDCALL(CFE_ES_PerfLogExit(CFE_MISSION_ES_PERF_MAX_IDS-1)); + UtAssert_VOIDCALL(CFE_ES_PerfLogExit(CFE_MISSION_ES_PERF_MAX_IDS)); +} + +void TestPerfLogAdd(void) +{ + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(0, 0)); + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(0, 1)); + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(0, 0xFFFFFFFF)); + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(CFE_MISSION_ES_PERF_MAX_IDS-1, 0)); + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(CFE_MISSION_ES_PERF_MAX_IDS-1, 1)); + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(CFE_MISSION_ES_PERF_MAX_IDS-1, 0xFFFFFFFF)); + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(CFE_MISSION_ES_PERF_MAX_IDS, 0)); + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(CFE_MISSION_ES_PERF_MAX_IDS, 1)); + UtAssert_VOIDCALL(CFE_ES_PerfLogAdd(CFE_MISSION_ES_PERF_MAX_IDS, 0xFFFFFFFF)); +} + +/* These commands should trigger and stop perf data (based on value commanded in functional test workflow) */ +void TestPerfLogTrigger(void) +{ + UtAssert_VOIDCALL(CFE_ES_PerfLogEntry(126)); + UtAssert_VOIDCALL(CFE_ES_PerfLogExit(126)); +} + +void ESPerfTestSetup(void) +{ + UtTest_Add(TestPerfLogEntry, NULL, NULL, "Test PerfLogEntry"); + UtTest_Add(TestPerfLogExit, NULL, NULL, "Test PerfLogExit"); + UtTest_Add(TestPerfLogAdd, NULL, NULL, "Test PerfLogAdd"); + UtTest_Add(TestPerfLogTrigger, NULL, NULL, "Test Perf Trigger"); +}