Skip to content
Snippets Groups Projects
Commit 0beec633 authored by squidfunk's avatar squidfunk
Browse files

Ensure Gemini terminates with exit status 1 on failing build

parent 57871449
No related branches found
No related tags found
No related merge requests found
...@@ -41,7 +41,7 @@ matrix: ...@@ -41,7 +41,7 @@ matrix:
env: env:
- CXX=g++-4.8 - CXX=g++-4.8
install: yarn install install: yarn install
script: yarn run test:visual:run --no-clean script: yarn run test:visual:run -- --no-clean
# Limit clone depth to 5, to speed up build # Limit clone depth to 5, to speed up build
git: git:
......
...@@ -39,6 +39,7 @@ export default (gulp, config, args) => { ...@@ -39,6 +39,7 @@ export default (gulp, config, args) => {
return done => { return done => {
/* Start static file server */ /* Start static file server */
let error = false
new Promise(resolve => { new Promise(resolve => {
ecstatic.start(`${config.tests.visual}/data`, 8000, resolve) ecstatic.start(`${config.tests.visual}/data`, 8000, resolve)
...@@ -96,7 +97,8 @@ export default (gulp, config, args) => { ...@@ -96,7 +97,8 @@ export default (gulp, config, args) => {
}) })
/* Return runner for graceful stop */ /* Return runner for graceful stop */
.then(() => { .then(status => {
error = status.errored > 0
return runner return runner
}) })
}) })
...@@ -111,9 +113,13 @@ export default (gulp, config, args) => { ...@@ -111,9 +113,13 @@ export default (gulp, config, args) => {
/* Stop static file server */ /* Stop static file server */
}) })
.then(() => { .then(() => {
ecstatic.stop(done) ecstatic.stop(() => {
return error
? done(new Error("Gemini terminated with errors"))
: done()
})
}, err => { }, err => {
return done(err) return done(new Error(err))
}) })
} }
} }
...@@ -10,7 +10,7 @@ ...@@ -10,7 +10,7 @@
"screenResolution": "1280x1024", "screenResolution": "1280x1024",
"recordVideo": "false" "recordVideo": "false"
}, },
"retry": 1 "retry": 3
}, },
"firefox": { "firefox": {
"desiredCapabilities": { "desiredCapabilities": {
...@@ -20,7 +20,7 @@ ...@@ -20,7 +20,7 @@
"screenResolution": "1280x1024", "screenResolution": "1280x1024",
"recordVideo": "false" "recordVideo": "false"
}, },
"retry": 1 "retry": 3
}, },
"edge": { "edge": {
"desiredCapabilities": { "desiredCapabilities": {
...@@ -29,7 +29,7 @@ ...@@ -29,7 +29,7 @@
"screenResolution": "1280x1024", "screenResolution": "1280x1024",
"recordVideo": "false" "recordVideo": "false"
}, },
"retry": 1 "retry": 3
}, },
"ie11": { "ie11": {
"desiredCapabilities": { "desiredCapabilities": {
...@@ -39,7 +39,7 @@ ...@@ -39,7 +39,7 @@
"screenResolution": "1280x1024", "screenResolution": "1280x1024",
"recordVideo": "false" "recordVideo": "false"
}, },
"retry": 1 "retry": 3
}, },
"ie10": { "ie10": {
"desiredCapabilities": { "desiredCapabilities": {
...@@ -49,7 +49,7 @@ ...@@ -49,7 +49,7 @@
"screenResolution": "1280x1024", "screenResolution": "1280x1024",
"recordVideo": "false" "recordVideo": "false"
}, },
"retry": 1 "retry": 3
} }
}, },
"system": { "system": {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment