Refactor bluetooth reading

[DSIW]

• Extract ScooterReader
• Extract BatteryReader
• Extract CharacteristicRepository
• Remove unused CBB capacity reading

QA checklist

• Read state (state & power state)
• Read primary battery SOC
• Read primary battery cycles
• Read secondary battery SOC
• Read secondary battery cycles
• Read seat closed
• Read handlebar
• Read AUX SOC
• Read CBB SOC
• Read CBB charging state
• Update ping
• SOCs are cached (after app restart)
• Commands can be sent (locking & hibernation)

[TheFreal]

Gerade mal ein bisschen durch geguckt...ich weiß, clean code und so, aber fühlt sich für mich persönlich irgendwie etwas arg verschachtelt an? Brauchen sowas wie CacheManager oder StateOfChargeReader wirklich eigene Klassen und Dateien?

[DSIW]

Kein Problem, ich schaue mir die Abstraktionen nochmals kritisch an und werde es hoffentlich noch weiter vereinfachen können.

Grundsätzlich bin ich ein Freund von eher kleineren Klassen, um sie leichter testen zu können.

Den CacheManager hatte ich hinzugefügt, um das Schreiben und Auslesen an einer Stelle zu haben.

Den StateOfChargeReader finde ich gerade sinnvoll, weil er den Ping und das Caching beinhaltet. Ggf. brauchen wir den extra CycleReader nicht und kann es in einen allgemeineren BatteryReader umfunktionieren.

[DSIW]

Ich habe nun einige Komponenten zu dem BatteryReader hinzugefügt inkl. Unit-Tests.

[TheFreal]

Might make sense to rebase your branch first to make sure all changes make it over