aboutsummaryrefslogtreecommitdiff
path: root/unittests/param_test.cpp
diff options
context:
space:
mode:
authorDaniel Agar <daniel@agar.ca>2015-03-02 14:32:39 -0500
committerLorenz Meier <lm@inf.ethz.ch>2015-03-19 23:49:35 +0100
commitb55fe24161ef60c7329494ab741263b9b01fe19c (patch)
treedade57dc2b36e5db8be1d53a4db71cc1963ac2ca /unittests/param_test.cpp
parentfb040f91174a5369aa7dd5b272015995ba01acba (diff)
downloadpx4-firmware-b55fe24161ef60c7329494ab741263b9b01fe19c.tar.gz
px4-firmware-b55fe24161ef60c7329494ab741263b9b01fe19c.tar.bz2
px4-firmware-b55fe24161ef60c7329494ab741263b9b01fe19c.zip
unittests fix code style
Diffstat (limited to 'unittests/param_test.cpp')
-rw-r--r--unittests/param_test.cpp39
1 files changed, 24 insertions, 15 deletions
diff --git a/unittests/param_test.cpp b/unittests/param_test.cpp
index 5ea6bcc60..44ae4df06 100644
--- a/unittests/param_test.cpp
+++ b/unittests/param_test.cpp
@@ -13,7 +13,8 @@ struct param_info_s *param_info_limit;
/*
* Adds test parameters
*/
-void _add_parameters() {
+void _add_parameters()
+{
struct param_info_s test_1 = {
"TEST_1",
PARAM_TYPE_INT32
@@ -44,17 +45,19 @@ void _add_parameters() {
param_array[3] = rc2_x;
param_info_base = (struct param_info_s *) &param_array[0];
param_info_limit = (struct param_info_s *) &param_array[4]; // needs to point at the end of the data,
- // therefore number of params + 1
+ // therefore number of params + 1
}
-void _assert_parameter_int_value(param_t param, int32_t expected) {
+void _assert_parameter_int_value(param_t param, int32_t expected)
+{
int32_t value;
int result = param_get(param, &value);
ASSERT_EQ(0, result) << printf("param_get (%i) did not return parameter\n", param);
ASSERT_EQ(expected, value) << printf("value for param (%i) doesn't match default value\n", param);
}
-void _set_all_int_parameters_to(int32_t value) {
+void _set_all_int_parameters_to(int32_t value)
+{
param_set((param_t)0, &value);
param_set((param_t)1, &value);
param_set((param_t)2, &value);
@@ -66,9 +69,10 @@ void _set_all_int_parameters_to(int32_t value) {
_assert_parameter_int_value((param_t)3, value);
}
-TEST(ParamTest, SimpleFind) {
+TEST(ParamTest, SimpleFind)
+{
_add_parameters();
-
+
param_t param = param_find("TEST_2");
ASSERT_NE(PARAM_INVALID, param) << "param_find did not find parameter";
@@ -78,7 +82,8 @@ TEST(ParamTest, SimpleFind) {
ASSERT_EQ(4, value) << "value of returned parameter does not match";
}
-TEST(ParamTest, ResetAll) {
+TEST(ParamTest, ResetAll)
+{
_add_parameters();
_set_all_int_parameters_to(50);
@@ -90,11 +95,12 @@ TEST(ParamTest, ResetAll) {
_assert_parameter_int_value((param_t)3, 16);
}
-TEST(ParamTest, ResetAllExcludesOne) {
+TEST(ParamTest, ResetAllExcludesOne)
+{
_add_parameters();
_set_all_int_parameters_to(50);
- const char* excludes[] = {"RC_X"};
+ const char *excludes[] = {"RC_X"};
param_reset_excludes(excludes, 1);
_assert_parameter_int_value((param_t)0, 2);
@@ -103,11 +109,12 @@ TEST(ParamTest, ResetAllExcludesOne) {
_assert_parameter_int_value((param_t)3, 16);
}
-TEST(ParamTest, ResetAllExcludesTwo) {
+TEST(ParamTest, ResetAllExcludesTwo)
+{
_add_parameters();
_set_all_int_parameters_to(50);
- const char* excludes[] = {"RC_X", "TEST_1"};
+ const char *excludes[] = {"RC_X", "TEST_1"};
param_reset_excludes(excludes, 2);
_assert_parameter_int_value((param_t)0, 50);
@@ -116,11 +123,12 @@ TEST(ParamTest, ResetAllExcludesTwo) {
_assert_parameter_int_value((param_t)3, 16);
}
-TEST(ParamTest, ResetAllExcludesBoundaryCheck) {
+TEST(ParamTest, ResetAllExcludesBoundaryCheck)
+{
_add_parameters();
_set_all_int_parameters_to(50);
- const char* excludes[] = {"RC_X", "TEST_1"};
+ const char *excludes[] = {"RC_X", "TEST_1"};
param_reset_excludes(excludes, 1);
_assert_parameter_int_value((param_t)0, 2);
@@ -129,11 +137,12 @@ TEST(ParamTest, ResetAllExcludesBoundaryCheck) {
_assert_parameter_int_value((param_t)3, 16);
}
-TEST(ParamTest, ResetAllExcludesWildcard) {
+TEST(ParamTest, ResetAllExcludesWildcard)
+{
_add_parameters();
_set_all_int_parameters_to(50);
- const char* excludes[] = {"RC*"};
+ const char *excludes[] = {"RC*"};
param_reset_excludes(excludes, 1);
_assert_parameter_int_value((param_t)0, 2);