Aufruf an alle: Travis-CI/AppVeyor für alle Adapter …

pix

Hallo,

bei mir (tvspielfim-Adapter) kommt auch ein Fehler. Die Anzeige deutet aber auf ein Problem in der testAdapter.js Datei.

! ````
iobroker.tvspielfilm@0.2.2 test /home/travis/build/Pix---/ioBroker.tvspielfilm
node node_modules/mocha/bin/mocha
/home/travis/build/Pix---/ioBroker.tvspielfilm/test/testAdapter.js:12
var adapterShortName = setup.adapterName.substring(setup.adapterName.indexOf('.')+1);
^
TypeError: Cannot read property 'substring' of undefined
at Object. <anonymous>(/home/travis/build/Pix---/ioBroker.tvspielfilm/test/testAdapter.js:12:41)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at /home/travis/build/Pix---/ioBroker.tvspielfilm/node_modules/mocha/lib/mocha.js:220:27
at Array.forEach (native)
at Mocha.loadFiles (/home/travis/build/Pix---/ioBroker.tvspielfilm/node_modules/mocha/lib/mocha.js:217:14)
at Mocha.run (/home/travis/build/Pix---/ioBroker.tvspielfilm/node_modules/mocha/lib/mocha.js:469:10)
at Object. <anonymous>(/home/travis/build/Pix---/ioBroker.tvspielfilm/node_modules/mocha/bin/_mocha:404:18)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.runMain (module.js:604:10)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:509:3
npm ERR! Test failed. See above for more details.</anonymous></anonymous>


Mir raucht der Kopf, ist ganz schön Arbeit. Dazu noch die Appveyor Geschichte und dann noch das "news"-Feld in io-packages :o :roll:

Gruß

Pix

EDIT: Überhaupt ist das noch alles ziemliches Neuland für mich. Einzig der Tankerkönig-Adapter ist grün.

apollon77

Hey,

@Pix: Hast nen Pull-Request … ich hab mal die Setup.js aktualisiert ... jetzt kommt ein Parsing-Fehler (denke mal das du keine Feeds angegeben hast oder so)

https://travis-ci.org/Apollon77/ioBroke ... /189337696

@Jay Cee: In der package.json muss das "main"-File angegeben werden. Ich hab Dir auch nen Pull-Request im Github gegeben. Dann ist alles aktuell :-) Und du hattest auch testing fr nde 0.x dabei, was aber wegen der ES6 Features die Du verwendest (z.B. const) nicht unterstützt sind.

Für beides Gilt: Appveyor tests laufen noch, rechne aber nicht mit Problemen :-) Schicke noch nen Pull-Request hinterher falls da nochwas rausfällt

apollon77

Hier ein kleines Update:

Inzwischen werden die meisten Adapter auch mit node 7 getestet um zu wissen wo wir stehen. Und es gibt eine neue Version vom setup.js mit einem kleinen Fix für Sonderfälle.

https://github.com/Apollon77/ioBroker.n … b/setup.js

Have fun and happy testing :-)

apollon77

Es gab weitere Updates. Aktuellste Test-Files gibts ggf von mir als Pull-Request oder in den Template-Adaptern (z.B. https://github.com/ioBroker/ioBroker.template)

AlCalzone

Ich hätte da mal ein Problem…

Meine Builds laufen auf allen getesteten Maschinen in der Regel problemlos durch, mit einer Ausnahme: Mac OSX mit NodeJS v4.

Installation/Kompilierung läuft durch, dann knallts (in 80% der Builds, ab und zu geht's doch).

Das letzte, was der Log zeigt, sind die folgenden Zeilen:

checkIsControllerInstalled...
checkIsControllerInstalled: installed!
Setup finished.
Copy adapter...

Dann passiert 10 Minuten lang nichts, bevor Travis den Job abbricht (keine Rückmeldung seit 10 Minuten). Diese Konfiguration erscheint mir allgemein etwas langsamer als die anderen, möglicherweise verschluckt sie sich beim sychronen rekursiven Kopieren.

Hast du eine Idee, was man da machen könnte?

apollon77

Hm interessant, ja die MacOS Infrastruktur von Travis-CI ist teilweise nicht 100% stabil, aber so ein nachvollziehbares Thema hatte ich noch nicht. hast Du mal ein Beispiel?

Kannst ja mal in die Funktion die das kopieren in setup.js macht debug einbauen das Du siehst Ob er einfach nur ewig braucht (wäre dann ein I/O Thema der Travis-Maschine) oder ob er irgendwo mittendrin aufhört …

AlCalzone

Kann nachvollzogen werden, wenn hier https://github.com/AlCalzone/iobroker.t … travis.yml die Zeilen 12-15 auskommentiert werden. Ich bau mal debug-logs ein und schaue.

apollon77

Du kannst auch "allow_failures" oder so nutzen anstelle exclude dann lässt er es laufen der Gesamtstatus ist aber grün auch wenn das fehlschlägt, vllt einfacher um zu sehen ob es irgendwann wieder tut :-)

AlCalzone

Interessant, diesmal hakts an ner anderen Stelle (nach dem erfolgreichen Kopieren):

Adapter copied.
Install adapter...
  1) "before all" hook: Test tradfri adapter: Start js-controller
Controller is not running!
Adapter normal terminated: true
  Test package.json and io-package.json
  ✓ Test package files
  1 passing (28m)
  1 failing
  1) Test tradfri adapter "before all" hook: Test tradfri adapter: Start js-controller:
     Error: timeout of 600000ms exceeded. Ensure the done() callback is being called in this test.

apollon77

Sehr interessant … erhöhe mal den timeout ...

AlCalzone

~~@apollon77:~~

Sehr interessant … erhöhe mal den timeout ... `
Werd ich mal probieren.

Aber: Es ist zum Mäusemelken… allow_failures aktiviert, und der Job läuft sofort problemlos durch, wenn auch in der dreifachen Zeit...

https://travis-ci.org/AlCalzone/iobroke ... /229009845

apollon77

LOL

AlGu

Hallo Apollon,

ich habe mich auch mal versucht in der Adapter-Entwicklung und wollte jetzt diese Tests auch einbauen. Leider kommt bei mir aber ein Fehler den ich nicht Verstehe.

Mein Adapter ist dieser hier: https://github.com/AlGu1/hombot und noch nicht im Forum angekündigt, da ich noch mind. die Tests einbauen wollte.

Die Fehler bei Travis-CI sind z.B.: (Edit: https://travis-ci.org/AlGu1/hombot/jobs/344045512)

! ````

Test hombot adapter
"before all" hook: Test hombot adapter: Start js-controller:
Uncaught Error: connect ECONNREFUSED 127.0.0.1:9000
at Object.exports._errnoException (util.js:907:11)
at exports._exceptionWithHostPort (util.js:930:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1081:14)

und bei AppVeyor: (Edit: [https://ci.appveyor.com/project/AlGu1/h … gewji5f5rc](https://ci.appveyor.com/project/AlGu1/hombot/build/job/9ru2qdgewji5f5rc))

>! ````
 1) Test hombot adapter
       "before all" hook: Test hombot adapter: Start js-controller:
     Uncaught Error: connect ECONNREFUSED 127.0.0.1:9000
      at Object.exports._errnoException (util.js:907:11)
      at exports._exceptionWithHostPort (util.js:930:20)
      at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1081:14)
  2) Test hombot adapter
       "after all" hook: Test hombot adapter: Stop js-controller:
     Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
      at test\testAdapter.js:137:13
      at test\lib\setup.js:649:13
      at null._onTimeout (test\lib\setup.js:594:17)
  3) Test hombot adapter
       "after all" hook: Test hombot adapter: Stop js-controller:
     Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
      at test\testAdapter.js:137:13
      at test\lib\setup.js:649:13
      at null._onTimeout (test\lib\setup.js:594:17)
>! ````

Sagen dir die Meldungen etwas? Ich habe versucht mich an die Anleitung für das Adapter.Template zu halten. Und bin mir nicht sicher ob ich das auch alles richtig gemacht habe.

Vielen Dank und Gruß

AlGu

apollon77

Sehr interessant …

bau mal in https://github.com/AlGu1/hombot/blob/ma ... up.js#L308 um das client.connect() ein try/catch ein um die exception abzufangen. geht es dann ?!

AlGu

Ich glaube ich weiß woran es liegt. Ich habe den Adapter ja noch unter "hombot" im Github und nicht unter "ioBroker.hombot". Denn der "appName" wird aus dem "__dirname" gelesen (getAppName). Dadurch kommt dann z.B. bei AppVeyor folgender Fehler:

Command failed: npm install https://github.com/hombot/hombot.js-controller/tarball/master --prefix ./  --production

zustande und die Url gibt es so ja nicht.

Kannst du mir sagen wie ich den Adapter in "ioBroker/hombot" verschieben? Oder kann das nur Bluefox?

Vielen Dank und Gruß

AlGu

AlCalzone

~~@AlGu:~~

Kannst du mir sagen wie ich den Adapter in "ioBroker/hombot" verschieben? `
Eigentlich musst du nur dein Repo umbenennen zu ioBroker.hombot. Meine Adapter sind auch alle unter meinem Namen, z.B. https://github.com/AlCalzone/ioBroker.tradfri und da funktionieren die Tests.

apollon77

Das interesanteste ist ja das folgende: Ich habe gerade einen fork gemacht und travis-ci läuft bei mir …

https://travis-ci.org/Apollon77/ioBroke ... /344201789

Einziger Fehler weil node 4.x Tests fehlschlagen ist das dort "request" noch nicht in nodejs drin war und es in den "dependecies" fehlt. Sobald Du die aktuelle "request" dep in die package schreibst klappt bei mir alles.

Und ich habe NICHTS verändert am Code. Es ist dein aktueller Git-Stand.

Hast Du irgendwas spezielles bei travis-ci eingestellt bei Dir?

PS: Hast Du das mit dem try/catch mal versucht?

AlCalzone

~~@apollon77:~~

https://travis-ci.org/Apollon77/ioBroke … /344201789 `
Da steht jetzt "ioBroker." im Repo-Namen. Daran wirds liegen.

apollon77

aaaahhhh jetzt seh ich das … ja na klar wenn "ioBroker" im Reponamen komplett fehlt dann gehts schieff :-)) klatsch (das war die hand vor die Stirn und so)

AlGu

Danke jetzt klappt es. Ich muss nun nur noch vernünftige Tests in die Datei einbauen.

Viele Grüße

AlGu

Community Forum

NEWS

Aufruf an alle: Travis-CI/AppVeyor für alle Adapter …

Support us

637

32.4k

81.4k

1.3m