Skip to content

Commit 6d2c709

Browse files
committed
resolve more comments
Signed-off-by: Santiago Figueroa Manrique <figueroa1395@gmail.com>
1 parent eef1fd4 commit 6d2c709

File tree

1 file changed

+13
-25
lines changed

1 file changed

+13
-25
lines changed

tests/cpp_unit_tests/test_current_sensor.cpp

Lines changed: 13 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -72,18 +72,18 @@ TEST_CASE("Test current sensor") {
7272
CurrentSensorCalcParam<symmetric_t> sym_sensor_param = sym_current_sensor.calc_param<symmetric_t>();
7373
CurrentSensorCalcParam<asymmetric_t> asym_sensor_param = sym_current_sensor.calc_param<asymmetric_t>();
7474

75-
CurrentSensorOutput<symmetric_t> const sym_sensor_output =
76-
sym_current_sensor.get_output<symmetric_t>(i_sym, ComplexValue<symmetric_t>{1.0});
77-
CurrentSensorOutput<asymmetric_t> sym_sensor_output_asym_param =
78-
sym_current_sensor.get_output<asymmetric_t>(i_asym, ComplexValue<asymmetric_t>{1.0});
79-
8075
// These two are only to test the residuals for local angle measurements.
8176
// conj(i) simulates the phase shift for local angle output residuals when the reference voltage is
8277
// real.
83-
CurrentSensorOutput<symmetric_t> const sym_sensor_output_local_residuals =
84-
sym_current_sensor.get_output<symmetric_t>(conj(i_sym), ComplexValue<symmetric_t>{1.0});
85-
CurrentSensorOutput<asymmetric_t> sym_sensor_output_asym_param_local_residuals =
86-
sym_current_sensor.get_output<asymmetric_t>(conj(i_asym_local), ComplexValue<asymmetric_t>{1.0});
78+
CurrentSensorOutput<symmetric_t> const sym_sensor_output =
79+
(angle_measurement_type == AngleMeasurementType::global_angle)
80+
? sym_current_sensor.get_output<symmetric_t>(i_sym, ComplexValue<symmetric_t>{1.0})
81+
: sym_current_sensor.get_output<symmetric_t>(conj(i_sym), ComplexValue<symmetric_t>{1.0});
82+
CurrentSensorOutput<asymmetric_t> const sym_sensor_output_asym_param =
83+
(angle_measurement_type == AngleMeasurementType::global_angle)
84+
? sym_current_sensor.get_output<asymmetric_t>(i_asym, ComplexValue<asymmetric_t>{1.0})
85+
: sym_current_sensor.get_output<asymmetric_t>(conj(i_asym_local),
86+
ComplexValue<asymmetric_t>{1.0});
8787

8888
// Check symmetric sensor output for symmetric parameters
8989
CHECK(sym_sensor_param.angle_measurement_type == angle_measurement_type);
@@ -99,13 +99,8 @@ TEST_CASE("Test current sensor") {
9999

100100
CHECK(sym_sensor_output.id == 0);
101101
CHECK(sym_sensor_output.energized == 1);
102-
if (angle_measurement_type == AngleMeasurementType::global_angle) {
103-
CHECK(sym_sensor_output.i_residual == doctest::Approx(0.0));
104-
CHECK(sym_sensor_output.i_angle_residual == doctest::Approx(0.0));
105-
} else {
106-
CHECK(sym_sensor_output_local_residuals.i_residual == doctest::Approx(0.0));
107-
CHECK(sym_sensor_output_local_residuals.i_angle_residual == doctest::Approx(0.0));
108-
}
102+
CHECK(sym_sensor_output.i_residual == doctest::Approx(0.0));
103+
CHECK(sym_sensor_output.i_angle_residual == doctest::Approx(0.0));
109104

110105
// Check symmetric sensor output for asymmetric parameters
111106
CHECK(asym_sensor_param.measurement.real_component.variance[0] ==
@@ -121,16 +116,9 @@ TEST_CASE("Test current sensor") {
121116
CHECK(sym_sensor_output_asym_param.energized == 1);
122117
for (auto phase = 0; phase < 3; ++phase) {
123118
CAPTURE(phase);
124-
if (angle_measurement_type == AngleMeasurementType::global_angle) {
125-
CHECK(sym_sensor_output_asym_param.i_residual[phase] == doctest::Approx(0.0));
126-
CHECK(sym_sensor_output_asym_param.i_angle_residual[phase] == doctest::Approx(0.0));
127-
} else {
128-
CHECK(sym_sensor_output_asym_param_local_residuals.i_residual[phase] == doctest::Approx(0.0));
129-
CHECK(sym_sensor_output_asym_param_local_residuals.i_angle_residual[phase] ==
130-
doctest::Approx(0.0));
131-
}
119+
CHECK(sym_sensor_output_asym_param.i_residual[phase] == doctest::Approx(0.0));
120+
CHECK(sym_sensor_output_asym_param.i_angle_residual[phase] == doctest::Approx(0.0));
132121
}
133-
134122
CHECK(sym_current_sensor.get_terminal_type() == terminal_type);
135123
CHECK(sym_current_sensor.get_angle_measurement_type() == angle_measurement_type);
136124
}

0 commit comments

Comments
 (0)