14_SD_WS - protocol 27 new sensor EFS-3110A

[elektron-bbs]

• Please check if the PR fulfills these requirements

• Tests for the changes have been added / modified (needed for for bug fixes / features)
• commandref has been added / updated (needed for bug fixes / features)
• CHANGED has been updated (needed for bug fixes / features)

• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  feature

• What is the current behavior? (You can also link to an open issue here)
  no decoding sensor EFS-3110A
  Eurochron Zusatzsensor für EFS-3110A #889

• What is the new behavior (if this is a feature change)?
  decoding sensor EFS-3110A

• Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
  no

• Other information:

[coveralls]

Pull Request Test Coverage Report for Build 3759

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.1%) to 53.239%

---

Totals
Change from base Build 3724:	-0.1%
Covered Lines:	3304
Relevant Lines:	6206

---

💛 - Coveralls

[elektron-bbs]

Die Tests zeigen Fehler und ich erkenne wieder mal nicht warum :-(

[sidey79]

Ach, das ist dieser blöde Mist mit :(

Dass es in test_SD_WS_devices-definition.txt liegt weisst Du sicher schon.

            # Failed test 'check reading type'
            # at (eval 44) line 55.
            # +---------------------+----+----------+
            # | GOT                 | OP | CHECK    |
            # +---------------------+----+----------+
            # | EFTH-800, EFS-3110A | eq | EFTH-800 |
            # +---------------------+----+----------+
        # Failed test 'Verify readings'
        # at (eval 44) line 58.
    # Failed test 'Checking module: SD_WS device: EFTH-800 TestNo: 0 '
    # at (eval 44) line 70.
            # Failed test 'check reading type'
            # at (eval 44) line 55.
            # +---------------------+----+----------+
            # | GOT                 | OP | CHECK    |
            # +---------------------+----+----------+
            # | EFTH-800, EFS-3110A | eq | EFTH-800 |
            # +---------------------+----+----------+
        # Failed test 'Verify readings'
        # at (eval 44) line 58.
    # Failed test 'Checking module: SD_WS device: EFTH-800 TestNo: 1 '
    # at (eval 44) line 70.
            # Failed test 'check reading type'
            # at (eval 44) line 55.
            # +---------------------+----+----------+
            # | GOT                 | OP | CHECK    |
            # +---------------------+----+----------+
            # | EFTH-800, EFS-3110A | eq | EFTH-800 |
            # +---------------------+----+----------+
        # Failed test 'Verify readings'
        # at (eval 44) line 58.
    # Failed test 'Checking module: SD_WS device: EFTH-800 TestNo: 2 '
    # at (eval 44) line 70.
# Failed test 'Verify sensor´s from JSON'

Ich denke, der Name des Sensors im Reading passt nicht und deshalb schlägt der Test fehl.

[elektron-bbs]

Danke, das war es.
Warum mir allerdings Travis die Fehler nicht anzeigt, so wie du sie siehst, ist mir schleierhaft. Bei den Fehlern bekam ich nur folgendes zu sehen:

--------- Starting test test_SD_WS_devices-definition.txt: ---------
Output of test_SD_WS_devices:
<BINARY>
<BINARY>
Errors of test test_SD_WS_devices:
"<BINARY>"
FHEM Log snippet from test run:
--------- Test test_SD_WS_devices-definition.txt: error ---------
UnitTest/makefile:38: recipe for target 'test_SD_WS_devices' failed
make[1]: *** [test_SD_WS_devices] Error 1
make[1]: Leaving directory '/home/travis/build/RFD-FHEM/RFFHEM'
Makefile:19: recipe for target 'test' failed
make: *** [test] Error 2
The command "make test" exited with 2.
0.02s$ sudo pkill "^perl" || true

Da rätselt man dann schon, wie es zu so einem "gravierenden" Fehler kommen kann, das sang- und klanglos abgebrochen wird.
Auch jetzt, nachdem es fehlerfrei durchgelaufen ist, ist die Ausgabe wenig aussagekräftig:

--------- Starting test test_SD_WS_devices-definition.txt: ---------
Output of test_SD_WS_devices:
<BINARY>
--------- Test test_SD_WS_devices-definition.txt: ok ---------

[sidey79]

@elektron-bbs

Prinzipiell alles fein mit dem PR, aber in der Commandref habe ich gesehen, dass die Sensoren dort benannt werden

RFFHEM/FHEM/14_SD_WS.pm

Line 1222 in 791ce18

<li>EuroChron EFTH-800 (temperature and humidity sensor)</li>

Ich denke es wäre sinnvoll den neuen Sensor dort noch zu hinterlegen. Siehst Du das ebenso?

[elektron-bbs]

Sehe ich genau so und habe es ergänzt.

[sidey79]

vermutlich muss ich den merge machen? richtig?

[elektron-bbs]

So ist es, ich bekomme keinen Button zu sehen.