Closed elektron-bbs closed 1 year ago
Merging #1116 (32dd450) into master (82d8b5d) will increase coverage by
0.03%
. The diff coverage is62.79%
.
@@ Coverage Diff @@
## master #1116 +/- ##
==========================================
+ Coverage 64.95% 64.99% +0.03%
==========================================
Files 133 135 +2
Lines 9671 9723 +52
Branches 1533 1544 +11
==========================================
+ Hits 6282 6319 +37
- Misses 2185 2189 +4
- Partials 1204 1215 +11
Flag | Coverage Δ | |
---|---|---|
fhem | 56.99% <62.79%> (+0.08%) |
:arrow_up: |
modules | 64.99% <62.79%> (+0.03%) |
:arrow_up: |
perl | 90.37% <ø> (+0.08%) |
:arrow_up: |
unittests | 64.99% <62.79%> (+0.03%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
FHEM/lib/SD_ProtocolData.pm | 100.00% <ø> (ø) |
|
FHEM/14_SD_WS.pm | 65.41% <62.79%> (-0.06%) |
:arrow_down: |
t/FHEM/14_SD_BELL/09_parseDatat.t | ||
t/FHEM/14_FLAMINGO/00_load.t | 100.00% <0.00%> (ø) |
|
t/FHEM/14_SD_BELL/00_load.t | 100.00% <0.00%> (ø) |
|
t/FHEM/14_BresserTemeo/09_parseDatat.t | 80.00% <0.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Hinsichtlich dcf
sind keine Testdaten hinterlegt.
Wie hast Du das herausbekommen und verifiziert dass es stimmt? Hast Du eventuell Testdaten dafür?
Testdaten dafür habe ich natürlich, sonst hätte ich das Reading ja nicht einbauen können.
MU;P0=-13168;P1=469;P2=-1000;P3=1450;D=01212121212121232121232121212121212123212321232123232123212123232323232323212323232323232321232323212323232123232123232123232123232323232321212323232321232123;CP=1;R=79;
Verifizieren kann ich das, in dem ich den Logeintrag
2022.09.03 16:02:02 4: signalduino1: Read, msg: MU;P0=-13168;P1=469;P2=-1000;P3=1450;D=01212121212121232121232121212121212123212321232123232123212123232323232323212323232323232321232323212323232123232123232123232123232323232321212323232321232123;CP=1;R=79;
mit dem Reading vergleiche
READINGS:
2022-09-10 21:54:57 batteryState ok
2022-09-10 21:54:57 dcf 2022-09-03 16:02:02
2022-09-10 13:34:38 humidity 43
2022-09-10 13:34:38 rain 530.4
2022-09-10 13:34:38 rawRainCounter 1768
2022-09-10 21:54:57 state
2022-09-10 13:34:38 temperature 22
2022-09-10 21:54:57 type TFA_35.1077
2022-09-10 13:34:38 windGust 0.7
2022-09-10 13:34:38 windSpeed 0.3
Mir gelingt es jetzt nur nicht mit dem SIGNALduino_TOOL ein Update für das JSON zu erzeugen. Ich vermute, weil state leer bleibt. Das war allerdings Absicht.
Ich habe jetzt einen Test-Datensatz mit dem Reading dcf erstellt.
Wenn Du den Branch nicht mehr benötigst, dann lösche ihn bitte, wir haben ziemlich viele branches dieser Art. :)
Sooo viele sind es ja nun auch nicht... Ich lösche den Branch lieber immer erst etwas später, bis der oder die User begriffen haben, das das jetzt im Master ist :-)
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
[ ] Bugfix (please link issue)
[x] Feature enhancement
[ ] Documentation update
[ ] Unittest enhancement
[ ] other
What is the current behavior? (You can also link to an open issue here, if this describes the current behavior) No decoding of weather sensors of type TFA 35.1077.54.S2 (weather station) with 30.3151 (T/H-transmitter), 30.3152 (rain gauge), 30.3153 (anemometer) https://forum.fhem.de/index.php/topic,119335.msg1221926.html#msg1221926
What is the new behavior (if this is a feature change)? Decoding of the sensor data
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) no
Other information: