Problem beim integrationstest mit github-actions

[AlCalzone]

@oliverio Ich schau mir das auf Windows mal an. Weil ichs aber grade sehe: Alle Libraries, die nur im Frontend benötigt werden, haben in dependencies nix zu suchen. Die werden beim Kompilieren gebundlet und dann als eine Datei ausgeliefert.

Mindestens folgende gehören stattdessen in die devDependencies:

"@iobroker/adapter-react": "^2.0.15",
"@material-ui/icons": "^4.11.2",
"@material-ui/lab": "^4.0.0-alpha.60",
"@material-ui/pickers": "^3.3.10",
"@material-ui/core": "^4.12.3",
"react": "^17.0.2",
"react-dom": "^17.0.2",
"react-icons": "^4.3.1",
"prop-types": "^15.8.1",

[AlCalzone]

Also bei mir läuft's auch unter Windows nicht:

npm ERR! code 1
npm ERR! path C:\Users\domin\AppData\Local\Temp\test-iobroker.mytime\node_modules\iobroker.vis
npm ERR! command failed
npm ERR! command C:\WINDOWS\system32\cmd.exe /d /s /c node main.js --install
npm ERR! C:\Users\domin\AppData\Local\Temp\test-iobroker.mytime\node_modules\@iobroker\js-controller-adapter\build\lib\adapter\adapter.js:61
npm ERR!         throw new Error(`Cannot find ${configFileName}`);
npm ERR!         ^
npm ERR!
npm ERR! Error: Cannot find C:/Users/domin/AppData/Local/Temp/test-iobroker.mytime/iobroker-data/iobroker.json
npm ERR!     at new Adapter (C:\Users\domin\AppData\Local\Temp\test-iobroker.mytime\node_modules\@iobroker\js-controller-adapter\build\lib\adapter\adapter.js:61:15)
npm ERR!     at Object.<anonymous> (C:\Users\domin\AppData\Local\Temp\test-iobroker.mytime\node_modules\iobroker.vis\main.js:20:24)
npm ERR!     at Module._compile (node:internal/modules/cjs/loader:1101:14)
npm ERR!     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
npm ERR!     at Module.load (node:internal/modules/cjs/loader:981:32)
npm ERR!     at Function.Module._load (node:internal/modules/cjs/loader:822:12)
npm ERR!     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:79:12)
npm ERR!     at node:internal/main/run_main_module:17:47

Das liegt wohl wirklich daran, dass beim npm install schon die DB laufen muss. Ich nehm das in meinen PR für die nächste Testing-Version mit auf.

[apollon77]

@alcalzone sagte in Problem beim integrationstest mit github-actions:

Das liegt wohl wirklich daran, dass beim npm install schon die DB laufen muss. Ich nehm das in meinen PR für die nächste Testing-Version mit auf.

Hängen bleiben oder so einen fehler werfen sollte es aber dennoch nicht

[AlCalzone]

@apollon77 Im Testing wird alles auf einmal installiert (npm install --production), aber da läuft weder der Controller (weil er gerade auch installiert wird), noch wurde setup first ausgeführt. vis scheint aber davon auszugehen.

[apollon77]

@alcalzone ich dachte an sich das (zumindestens früher) das install auch geht wennes auch ist ... deswegen -> issue anlegen bei vis!

[AlCalzone]

@apollon77 sagte in Problem beim integrationstest mit github-actions:

deswegen -> issue anlegen bei vis!

Ich denke das ist spezifisch ein Problem im Testing-Setup, wo bei der Installation noch gar nichts aus dem ioBroker-Umfeld existiert.

[apollon77]

@alcalzone Interessant ist: Wenn ich "npm install iobroker.vis" mache dann ist dem post install Skript egal das ioBroker nicht läuft - wie es soll ... warum ist es beim testing nicht?

[OliverIO]

also,
nach dem neu sortieren der dependencies laufen die tests nun durch.
warum das so ist weiß ich nicht.
falls da was fehlte oder ein konflikt entstanden ist, dann sollten die fehlermeldungen auch irgendwie erscheinen. ansonsten ist das wirklich schwierig. rauszufinden.

ich verstehe natürlich, das wenn der test auf unterschiedlichen platformen (github/lokal) unterschiedliche ergebnisse liefert, eine fehlersuche sehr schwer ist.

[AlCalzone]

@oliverio sagte in Problem beim integrationstest mit github-actions:

nach dem neu sortieren der dependencies laufen die tests nun durch.

Lass mich das auch mal nachstellen.

Edit: Also auch mit deinem aktuellen Stand gehts bei mir lokal nicht. Das müsste daran liegen, dass versucht wird, die iobroker-Basis-Konfig anzupassen, bevor sie von js-controller erstellt wird.

[AlCalzone]

@OliverIO Hab das Problem gefunden. Wenn das Testing erst den Controller installiert und in einem separaten Schritt die Adapter, dann läuft es 1A.
Testing v2.5.4 sollte bestenfalls heute noch kommen.

[OliverIO]

@alcalzone
super, danke.

[AlCalzone]

@oliverio Ist released, magst du direkt mal testen?

[OliverIO]

@alcalzone
https://github.com/oweitman/ioBroker.mytime/actions/runs/1781578934
lasse diese action nochmal laufen.
ist für alles soweit gut, nur node14 macos schief gegangen.

[AlCalzone]

@oliverio sagte in Problem beim integrationstest mit github-actions:

nur node14 macos schief gegangen

ja, passiert ab und an

Sieht gut aus!