Running RTD from Travis

[AdrienLemaire]

When running only a few minimal unit tests, no acceptance tests and no coverage check, the grunt command doesn't fail.

Hence I tried to configure Travis, to run tests after each commit on github, as follow:

.travis.yml:

language: node_js
node_js:
  - "0.10"
before_install:
  - curl https://install.meteor.com | /bin/sh
  - npm -g coffee-script meteorite
  - npm i -g karma phantomjs istanbul grunt-cli selenium-webdriver mocha karma-chai
services:
    - mongodb
notifications:
  email:
    - coinsmanager@gmail.com

Makefile

test:
    cd test/rtd && ./rtd --debug

.PHONY: test

.gitmodules (replace git → https)

[submodule "test/rtd"]
    path = test/rtd
    url = https://github.com/xolvio/rtd.git

But there are 2 problems:

1. From https://travis-ci.org/CoinsManager/CoinsManager/builds/18244442:

   ./rtd: line 5: cd: ./node_modules/.bin: No such file or directory
   

2. Because rtd is a ci tool in itself and never stops, the travis build will never finish. How can I specify RTD to run a single set of tests and quit ?

Thank you for your help. I'll set up a new module travis-rtd-meteor or something, to make it easier for other rtd users to integrate Travis as well.

[treybean]

Give grunt runOnce a try.

[AdrienLemaire]

Ok. First problem solved at https://travis-ci.org/CoinsManager/CoinsManager/builds/18246964

The problem is that the rtd script is trying to pwd from node_modules/.bin before doing the npm install. So I forced a npm install before running rtd.
New problem is cp: cannot stat with grunt-bgshell

Running grunt with runOnce in local doesn't finish:

But at least it doesn't watch for new changes in files. will try it in Travis after I get the first issue solved

[AdrienLemaire]

cp: cannot stat issue is because I didn't version the test/acceptance/fixtures/empty.js file.

Bug when running grunt runOnce, jshint is executed even when disabled. Fixing it in a pull request

[AdrienLemaire]

Finally solved several issues of packages dependencies, and tests are run successfully on Travis. But as I feared before @treybean the build does not finish and is pending.

Build can be seen here: https://travis-ci.org/CoinsManager/CoinsManager/builds/18313136

@SamHatoum @Nomeasmo Seems to be a bug from #85

[AdrienLemaire]

Temporary solution is to comment tasks.push('closeWebdriverSessions'); from constructRunOnceTasks in the Gruntfile.js from a fork of rtd that I submodule instead of upstream.

the task should be smarter and verify that sessions are open before trying to close them

[SamHatoum]

without the closeWebdriverSessions you would get orphan browsers. Do you experience this with Travis with the line commented out?

[Nomeasmo]

Hi Sam,

I do not use Travis, anyway runOnce now activated.

Refer to the changes here: Nomeasmo@45e8132

Oliver

Am 12.02.2014 um 03:39 schrieb Sam Hatoum notifications@github.com:

without the closeWebdriverSessions you would get orphan browsers. Do you experience this with Travis with the line commented out?

—
Reply to this email directly or view it on GitHub.

[AdrienLemaire]

@SamHatoum travis is smart enough to kill processes itself apparently, not seeing any problem.

But in local, if I run make test, then a bunch of processes keep running and I have to kill them manually, or meteor run would fail. Looking for a solution to fix that.

@Nomeasmo can you do a pull request to RTD with your code ? Just modifying the files manually doesn't work for me (actually there isn't even a webdrivers/webdriver-helper.js file, seems like your repo differs quite a bit from upstream).

[AdrienLemaire]

I rebased my code with rtd upstream (noticing that my PR #130 has been removed), and the problem is still there: When you start grunt runOnce, it will never finish. You need to ctrl-c to get back to the shell.

[Nomeasmo]

Please refer to the pr #143

Let me know what you think about the limitation.