search

RFD-FHEM / SIGNALduino_TOOL

FHEM Module for the SIGNALduino project.
GNU General Public License v3.0
4 stars 4 forks source link

Automatisches Tests der JSON Datei #21

Closed sidey79 closed 4 years ago

sidey79 commented 4 years ago

Von #20:

Es soll automatisiert geprüft werden, ob die JSON Datei dem korrekten Syntax entspricht.

sidey79 commented 4 years ago

@HomeAutoUser

Schau mal ob das deinen Vorstellungen so entspricht:

https://travis-ci.com/RFD-FHEM/SIGNALduino_TOOL/builds/134932501 https://github.com/RFD-FHEM/SIGNALduino_TOOL/runs/288281510 https://github.com/RFD-FHEM/SIGNALduino_TOOL/blob/b7522aa869ce0d57b71faa267198137a89ecd95b/t/json_check.pl

Ich habe nur ein paar Demotests eingebaut, um das Prinzip zu verifizieren :)

HomeAutoUser commented 4 years ago

Ich denke, das sollte für den Anfang reichen. Abändern können wir ja immer nochmal.

HomeAutoUser commented 4 years ago

Achso, ne kleine Anmerkung habe ich noch. Können wir https://github.com/RFD-FHEM/SIGNALduino_TOOL/tree/pre-release/t den Ordner t einen "festen Namen" verpassen so wie bei den anderen Repros. Bsp: Tests oder UnitTest Ich fände es von vorteil, wenn dieser bei unseren Tests gleich ist um sofort zu erkennen AH... das ist das ;-)

sidey79 commented 4 years ago

Ich habe mich da ein wenig weiter entwickelt und mich an vorhandene Standards gehalten. Der Ordner t ist üblich um Tests abzulegen. in t/lib kommen besondere Bibliotheken.

https://github.com/Perl/perl5/blob/blead/t/README

Würde es so lassen, da es ein 100% Perl Test ist und nicht wie beim UnitTest Modul ein Test innerhalb von FHEM.

HomeAutoUser commented 4 years ago

Dnn belassen wir es so und wieder wurden wir weitergebildet :-)

HomeAutoUser commented 4 years ago

@sidey79 laufen hier im Hintergrund schon Tests wenn neue PR´s mit RAWMSG´s für die JSON Datei durchgeführt werden?

sidey79 commented 4 years ago

Im Branch pre-release laufen die.

Nachdem ich mich die Tage ein wenig mit GitHub Actions beschäftigt habe, finde ich, dass wir das umstellen sollten. :)

HomeAutoUser commented 4 years ago

"Hier würde" ich mir ja das umstellen auf GitAction ja selbst zutrauen aber ich weiß nicht, was die bisherigen angelegten Tests dann machen?

Den Vorteil in GitAction sehe ich auf jedenfall, das die controls_Datei angepasst wird und zusätzlich auch die CHANGED ;-)

sidey79 commented 4 years ago

Naja, die Controls Datei wird aktuell ja mittels travisCI aktualisiert, aber das überschreibt das letzte Ergebnis vom Check. :(

Die Anpassungen für Tests habe ich im RSL Repo ein wenig erprobt und kann es als Blaupause verwenden.

HomeAutoUser commented 4 years ago

Thema "Automatisches Tests der JSON Datei", diese laufen im Hintergrund. https://github.com/RFD-FHEM/SIGNALduino_TOOL/issues/21#issuecomment-559991850

Somit schließe ich das Thema. Was die automatische Anpassung von Dateien wie Controls oder Changed angeht, so können wir dies in einem neuen Faden erörtern sobald wir eine Lösung haben.