Skip to content

Commit

Permalink
Merge pull request #170 from cben/fix-sauce
Browse files Browse the repository at this point in the history
Resolved consistent errors in IE, Chrome, Safari; disabled Android test.
  • Loading branch information
cben authored Sep 29, 2017
2 parents b3f94b3 + f73b233 commit 52f935a
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 8 deletions.
6 changes: 5 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,12 @@ branches:
- /.*/

language: node_js
# Should test the Node version that runs in production.
# https://docs.travis-ci.com/user/languages/javascript-with-nodejs/#Specifying-Node.js-versions
# https://devcenter.heroku.com/articles/nodejs-support#specifying-a-node-js-version
# I believe RHcloud (openshift 2) was running 0.11 but it's being sunset anyway.
node_js:
- "0.11" # Latest http://docs.travis-ci.com/user/languages/javascript-with-nodejs/#Provided-Node.js-Versions
- "8"

# http://blog.travis-ci.com/2013-12-05-speed-up-your-builds-cache-your-dependencies/
cache:
Expand Down
3 changes: 3 additions & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@
"sync-exec": "~0.6.2",
"wd": "~1.0.0"
},
"engines": {
"node": "8.x"
},
"main": "server.coffee",
"scripts": {
"start": "coffee server.coffee",
Expand Down
14 changes: 7 additions & 7 deletions test/browser-on-saucelabs.spec.coffee
Original file line number Diff line number Diff line change
Expand Up @@ -45,17 +45,17 @@ timeouts = {
desiredBrowsers = [
# Generated with https://docs.saucelabs.com/reference/platforms-configurator/
# Desktop:
{browserName: 'internet explorer', version: '8.0', platform: 'Windows XP'}
{browserName: 'internet explorer', version: '8.0', platform: 'Windows 7'}
{browserName: 'internet explorer', version: '11.0', platform: 'Windows 10'}
{browserName: 'MicrosoftEdge'}
# arbitrary somewhat old - but not ancient - FF and Chrome versions.
{browserName: 'firefox', version: '30.0', platform: 'Linux'}
{browserName: 'chrome', version: '30.0', platform: 'Linux'}
{browserName: 'safari', version: '9.0'}
{browserName: 'safari', version: '6.0'}
{browserName: 'chrome', version: '35.0', platform: 'Linux'}
{browserName: 'Safari', version: '10.0', platform: 'macOS 10.12'}
{browserName: 'Safari', version: '7.0', platform: 'OS X 10.9'}
# Mobile (doesn't mean it's usable though):
{browserName: 'safari', deviceName: 'iPad Simulator', platformVersion: '8.4'}
{browserName: 'android', deviceName: 'Android Emulator', platformVersion: '4.0'}
{browserName: 'Safari', deviceName: 'iPad Simulator', platformName: 'iOS', platformVersion: '8.4'}
# {browserName: 'Browser', deviceName: 'Android Emulator', platformName: 'Android', platformVersion: '4.4'}
]

getCommonDesired = ->
Expand Down Expand Up @@ -142,7 +142,7 @@ describeBrowserTest = (browserName, getDesired, getSite) ->
expect(err).to.be(null)
browser.waitFor wd.asserters.jsCondition('document.title.match(/smoke test/)'), 10*sec, (err, value) ->
expect(err).to.be(null)
browser.waitForElementByCss '.MathJax_Display', 15*sec, (err, el) ->
browser.waitForElementByCss '.MathJax_Display', 30*sec, (err, el) ->
expect(err).to.be(null)
el.text (err, text) ->
expect(err).to.be(null)
Expand Down

0 comments on commit 52f935a

Please sign in to comment.