Closed elektron-bbs closed 2 years ago
Merging #1096 (e700875) into master (0bed9e3) will increase coverage by
0.84%
. The diff coverage is64.00%
.
@@ Coverage Diff @@
## master #1096 +/- ##
==========================================
+ Coverage 63.82% 64.67% +0.84%
==========================================
Files 129 133 +4
Lines 9480 9563 +83
Branches 1507 1518 +11
==========================================
+ Hits 6051 6185 +134
+ Misses 2261 2199 -62
- Partials 1168 1179 +11
Flag | Coverage Δ | |
---|---|---|
fhem | 56.74% <64.00%> (+1.07%) |
:arrow_up: |
modules | 64.67% <64.00%> (+0.84%) |
:arrow_up: |
perl | 90.59% <ø> (+0.08%) |
:arrow_up: |
unittests | 64.67% <64.00%> (+0.84%) |
: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.47% <64.00%> (-0.13%) |
:arrow_down: |
t/FHEM/14_FLAMINGO/09_parseDatat.t | 80.00% <0.00%> (ø) |
|
t/FHEM/14_SD_WS07/09_parseDatat.t | 80.00% <0.00%> (ø) |
|
t/FHEM/14_SD_WS09/00_load.t | 100.00% <0.00%> (ø) |
|
t/FHEM/14_BresserTemeo/00_load.t | 100.00% <0.00%> (ø) |
|
FHEM/00_SIGNALduino.pm | 65.27% <0.00%> (+1.53%) |
:arrow_up: |
FHEM/10_SD_Rojaflex.pm | 71.54% <0.00%> (+2.84%) |
:arrow_up: |
FHEM/14_SD_WS09.pm | 72.35% <0.00%> (+7.69%) |
:arrow_up: |
:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more
@elektron-bbs
Ich habe keine Testdaten für diesen Sensor mit den Readings gefunden. Haben wir noch keine oder habe ich es übersehen?
Für dieses Protokoll existieren zwei Testdatensätze, allerdings ohne die Readings windDirectionDegree und windDirectionText. Wenn du diese Readings auch noch testen willst, müsstest du wahrscheinlich deine Tests anpassen, da dafür das Setzen eines neuen Attributes erforderlich ist. Eine Testnachricht für diesen Sensor habe ich in der SD_ProtocolData.pm hinterlegt.
Für dieses Protokoll existieren zwei Testdatensätze, allerdings ohne die Readings windDirectionDegree und windDirectionText. Wenn du diese Readings auch noch testen willst, müsstest du wahrscheinlich deine Tests anpassen, da dafür das Setzen eines neuen Attributes erforderlich ist. Eine Testnachricht für diesen Sensor habe ich in der SD_ProtocolData.pm hinterlegt.
Den Fall hatten wir schon mal glaube ich. Ich schaue mir das morgen an.
Es reicht aus, im Testdatensatz das zu setzende Attribut und den Wert anzugeben.
Kann der vorhandene Testdatensatz dafür erweitert werden oder braucht es einen neuen, weil dann die anderen Readings nicht mehr erzeugt werden?
Ich habe einen neuen Datensatz für den Sensor TFA 30.3251.10 Windsensor protocol 85 eingefügt.
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) Wind direction was not decoded with sensor TFA 30.3251.10. https://forum.fhem.de/index.php/topic,107998.msg1217472.html#msg1217472
What is the new behavior (if this is a feature change)? Wind direction is now decoded for sensor TFA 30.3251.10. Added CRC8 check.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) no
Other information: