From 55ab880823ff84a274caae9a290ab1a36f97d0c8 Mon Sep 17 00:00:00 2001 From: Jacob Dahl Date: Tue, 17 Mar 2026 21:55:35 -0800 Subject: [PATCH] refactor(systemcmds/tests): convert params.c to module.yaml Convert 1 parameter file(s) from legacy C format to YAML module configuration. --- src/systemcmds/tests/CMakeLists.txt | 2 + src/systemcmds/tests/params.c | 62 ----------------------------- src/systemcmds/tests/params.yaml | 34 ++++++++++++++++ 3 files changed, 36 insertions(+), 62 deletions(-) delete mode 100644 src/systemcmds/tests/params.c create mode 100644 src/systemcmds/tests/params.yaml diff --git a/src/systemcmds/tests/CMakeLists.txt b/src/systemcmds/tests/CMakeLists.txt index 2beab87943..2fdec3c070 100644 --- a/src/systemcmds/tests/CMakeLists.txt +++ b/src/systemcmds/tests/CMakeLists.txt @@ -91,6 +91,8 @@ px4_add_module( -O2 # optimization needed to keep matrix code size reasonable SRCS ${srcs} + MODULE_CONFIG + params.yaml DEPENDS dataman_client version diff --git a/src/systemcmds/tests/params.c b/src/systemcmds/tests/params.c deleted file mode 100644 index 26adf3b905..0000000000 --- a/src/systemcmds/tests/params.c +++ /dev/null @@ -1,62 +0,0 @@ -/**************************************************************************** - * - * Copyright (C) 2017 PX4 Development Team. All rights reserved. - * - * Redistribution and use in source and binary forms, with or without - * modification, are permitted provided that the following conditions - * are met: - * - * 1. Redistributions of source code must retain the above copyright - * notice, this list of conditions and the following disclaimer. - * 2. Redistributions in binary form must reproduce the above copyright - * notice, this list of conditions and the following disclaimer in - * the documentation and/or other materials provided with the - * distribution. - * 3. Neither the name PX4 nor the names of its contributors may be - * used to endorse or promote products derived from this software - * without specific prior written permission. - * - * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS - * AS IS AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT - * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS - * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE - * COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, - * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, - * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS - * OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED - * AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT - * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN - * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE - * POSSIBILITY OF SUCH DAMAGE. - * - ****************************************************************************/ - -/** - * @group Testing - */ -PARAM_DEFINE_INT32(TEST_1, 2); - -/** - * @group Testing - */ -PARAM_DEFINE_INT32(TEST_2, 4); - -/** - * @group Testing - */ -PARAM_DEFINE_FLOAT(TEST_3, 5.0f); - -/** - * @group Testing - */ -PARAM_DEFINE_INT32(TEST_RC_X, 8); - -/** - * @group Testing - */ -PARAM_DEFINE_INT32(TEST_RC2_X, 16); - -/** - * @group Testing - */ -PARAM_DEFINE_INT32(TEST_PARAMS, 12345678); diff --git a/src/systemcmds/tests/params.yaml b/src/systemcmds/tests/params.yaml new file mode 100644 index 0000000000..04f9746259 --- /dev/null +++ b/src/systemcmds/tests/params.yaml @@ -0,0 +1,34 @@ +module_name: tests +parameters: +- group: Testing + definitions: + TEST_1: + description: + short: TEST_1 + type: int32 + default: 2 + TEST_2: + description: + short: TEST_2 + type: int32 + default: 4 + TEST_3: + description: + short: TEST_3 + type: float + default: 5.0 + TEST_RC_X: + description: + short: TEST_RC_X + type: int32 + default: 8 + TEST_RC2_X: + description: + short: TEST_RC2_X + type: int32 + default: 16 + TEST_PARAMS: + description: + short: TEST_PARAMS + type: int32 + default: 12345678