Removed 'reversed' from intToLittleEndianArray (#10)

* Removed 'reversed' from intToLittleEndianArray in order to send correct values to indoor trainer, it was not vorking correctly with the reversed array * Fixed 'reversed' and wrote test for target power case
Malte2036 · Mar 17, 2024 · b6963fb · b6963fb
1 parent 0117d46
commit b6963fb
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/lib/src/ftms/characteristic/machine/control_point/machine_control_point.dart b/lib/src/ftms/characteristic/machine/control_point/machine_control_point.dart
@@ -72,7 +72,7 @@ class MachineControlPoint {
     List<int> writeData = [opCode.value];
     if (parameter != null) {
       writeData.addAll(
-          Utils.intToLittleEndianArray(parameter!.value, parameter!.size));
+          Utils.intToLittleEndianArray(parameter!.value, parameter!.size).reversed);
     }
     return writeData;
   }

diff --git a/test/ftms/characteristic/machine/control_point/machine_control_point_test.dart b/test/ftms/characteristic/machine/control_point/machine_control_point_test.dart
@@ -18,4 +18,12 @@ void main() {
     expect(controlPoint.opCode.type, MachineControlPointOpcodeType.stopOrPause);
     expect(controlPoint.getWriteData(), [0x8, 0x2]);
   });
+
+  test("test machine control point with Target Power", () {
+    var controlPoint = MachineControlPoint.setTargetPower(power: 300);
+
+    expect(controlPoint.opCode.value, 0x5);
+    expect(controlPoint.opCode.type, MachineControlPointOpcodeType.setTargetPower);
+    expect(controlPoint.getWriteData(), [0x5, 0x2C,0x01]);
+  });
 }