RFD-FHEM / RFFHEM

Counterpart of SIGNALDuino, it's the code for FHEM to work with the data received from the uC
GNU General Public License v3.0
44 stars 33 forks source link

Update tests | for the functional reliability of the module SD_UT #1214

Closed HomeAutoUser closed 9 months ago

HomeAutoUser commented 9 months ago

added data for Tedsen_SKX1xx

codecov[bot] commented 9 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (2a5ebf5) 67.40% compared to head (3c8c8bd) 69.57%.

Files Patch % Lines
t/FHEM/14_SD_UT/03_set.t 75.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1214 +/- ## ========================================== + Coverage 67.40% 69.57% +2.16% ========================================== Files 135 135 Lines 10021 10008 -13 Branches 1597 1596 -1 ========================================== + Hits 6755 6963 +208 + Misses 1953 1724 -229 - Partials 1313 1321 +8 ``` | [Flag](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1214/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | Coverage Δ | | |---|---|---| | [fhem](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1214/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | `59.77% <87.50%> (+2.52%)` | :arrow_up: | | [modules](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1214/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | `69.57% <87.50%> (+2.16%)` | :arrow_up: | | [perl](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1214/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | `89.63% <ø> (ø)` | | | [unittests](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1214/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | `69.57% <87.50%> (+2.16%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sidey79 commented 9 months ago

@HomeAutoUser

Ich hab den Test für SD_UT_Test_DC_1961_TG_1846 mal angepasst, die sendMsg war leicht abweichend von der die Du hattest.

HomeAutoUser commented 9 months ago

@HomeAutoUser

Ich hab den Test für SD_UT_Test_DC_1961_TG_1846 mal angepasst, die sendMsg war leicht abweichend von der die Du hattest.

vielen Dank @sidey79. Die muss ich gleich nochmal vergleichen. Ich habe sie dann vermutlich falsch kopiert ... hm

sidey79 commented 9 months ago

Du legst ja einen Test nach dem anderen nach :)

Ist denn schon Weihnachten?

HomeAutoUser commented 9 months ago

Du legst ja einen Test nach dem anderen nach :)

Ist denn schon Weihnachten?

Ganz Weihnachten ist noch nicht, aber mit dem Hintergedanken, das Modul etwas zu überarbeiten :-) Somit möchten wir gern auch sicherstellen, das auch in Zukunft die Funktion gegeben ist.

PS: Ich möchte nur noch die "essfreie Zeit" nutzen smile

sidey79 commented 9 months ago

Du legst ja einen Test nach dem anderen nach :) Ist denn schon Weihnachten?

Ganz Weihnachten ist noch nicht, aber mit dem Hintergedanken, das Modul etwas zu überarbeiten :-) Somit möchten wir gern auch sicherstellen, das auch in Zukunft die Funktion gegeben ist.

PS: Ich möchte nur noch die "essfreie Zeit" nutzen smile

Ja, so war das auch immer Gedacht :)

Sollen die If/Else Abfragen ersetzt werden?

HomeAutoUser commented 9 months ago

Ja, so war das auch immer Gedacht :)

Sollen die If/Else Abfragen ersetzt werden?

Wir wollen erstmal den großteil der doppelten Schreibweise bei den Modellen "kürzen". Was danach noch folgt, mal schauen. Soeben bin ich auf der Meinung, sogar noch einen Bug entdeckt zu haben. Sowas markiere ich erstmal im Testfile und somit wissen wir, wo wir ansetzen müssen.

HomeAutoUser commented 9 months ago

@sidey79 @elektron-bbs Ich wünsche Euch eine schöne Weihnachtszeit :-) Bis die Tage wink