search

citmp2015 / flink-visual-programming

2 stars 9 forks source link

E2E and unit testing structure #157

Closed MonsJovis closed 8 years ago

MonsJovis commented 8 years ago

Solves #38 and #172

MonsJovis commented 8 years ago

Der Build failed, da Protractor Komponenten benutzt, die mit node-gyp gebuilded werden und Abhängigkeiten zu build-essential haben, welches nicht installiert ist. Müssen wir Lauritz fragen um das zu installieren, @Fabsi110, @whoww?

Hier noch der Log-Output von Jetkins:

[ERROR] gyp ERR! build error 
[ERROR] gyp ERR! stack Error: not found: make
[ERROR] gyp ERR! stack     at F (/home/vs_pj/.jenkins/jobs/flink-visual-pull-requests/workspace/client/node/node_modules/npm/node_modules/which/which.js:78:19)
[ERROR] gyp ERR! stack     at E (/home/vs_pj/.jenkins/jobs/flink-visual-pull-requests/workspace/client/node/node_modules/npm/node_modules/which/which.js:82:29)
[ERROR] gyp ERR! stack     at /home/vs_pj/.jenkins/jobs/flink-visual-pull-requests/workspace/client/node/node_modules/npm/node_modules/which/which.js:93:16
[ERROR] gyp ERR! stack     at FSReqWrap.oncomplete (fs.js:82:15)
[ERROR] gyp ERR! System Linux 3.16.0-41-generic
[ERROR] gyp ERR! command "/home/vs_pj/.jenkins/jobs/flink-visual-pull-requests/workspace/client/node/node" "/home/vs_pj/.jenkins/jobs/flink-visual-pull-requests/workspace/client/node/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
[ERROR] gyp ERR! cwd /home/vs_pj/.jenkins/jobs/flink-visual-pull-requests/workspace/client/node_modules/grunt-protractor-runner/node_modules/protractor/node_modules/selenium-webdriver/node_modules/ws/node_modules/utf-8-validate
[ERROR] gyp ERR! node -v v4.2.4
[ERROR] gyp ERR! node-gyp -v v3.0.3
[ERROR] gyp ERR! not ok

http://stackoverflow.com/questions/14772508/npm-failed-to-install-time-with-make-not-found-error

MonsJovis commented 8 years ago

retest this please

MonsJovis commented 8 years ago

retest this please

MonsJovis commented 8 years ago

retest this please

MonsJovis commented 8 years ago

@IwoTens hmm... da es bei mir und auf asok16 läuft, kann ich kaum weiterhelfen. aber du könntest probieren mit der version von 'selenium-webdriver' zu spielen.

IwoTens commented 8 years ago

Ich habs jetzt auch zum laufen bekommen.

npm install phantomjs ./node_modules/protractor/bin/webdriver-manager update

Moeglicherweise wird also irgendwo phantomjs erwartet statt phantomjs-prebuilt.

IwoTens commented 8 years ago

Der e2e test scheint nicht ausgefuehrt zu werden.

MonsJovis commented 8 years ago

@IwoTens bzgl. der phantomjs abhängigkeit müssen wir uns das vielleicht im detail noch bei dir anschauen oder evtl auch mit shrinkwrap probieren (issue #169). auf asok16 läufts, bei mir auf windows 10 läufts auch.

Auch der E2E test wird auf asok16 ausgeführt:

[INFO] Running "protractor:test" (protractor) task
[INFO] Starting selenium standalone server...
[INFO] [launcher] Running 1 instances of WebDriver
[INFO] Selenium standalone server started at http://130.149.248.45:41571/wd/hub
[INFO] Started
[INFO] .
[INFO] 
[INFO] 
[INFO] 1 spec, 0 failures
[INFO] Finished in 2.298 seconds
[INFO] Shutting down selenium standalone server.
[INFO] [launcher] 0 instance(s) of WebDriver still running
[INFO] [launcher] phantomjs #1 passed
[INFO] 
[INFO] Done, without errors.

Hab die neu hinzugekommenen files nochmal gebeautified. Tabbing sollte nun also passen.

IwoTens commented 8 years ago

Ich hatte bei dem E2E test einfach mal das not rausgenommen und es lief immer noch durch.

MonsJovis commented 8 years ago

grad nochmal bei mir getestet. wenn ich das .not rausnehme, läufts nicht mehr durch. es muss wirklich an deinem system liegen :unamused:

MonsJovis commented 8 years ago

Würde frei nach "Better done than perfect" vorschlagen, den PR trotzdem zu mergen. Das wichtigste ist, dass es auf asok16 läuft.