Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update grunt / jasmine testing framework to support external build #40

Closed
aashish24 opened this issue Apr 3, 2014 · 4 comments
Closed
Labels

Comments

@aashish24
Copy link
Member

No description provided.

@jbeezley
Copy link
Contributor

jbeezley commented Apr 8, 2014

I have an experimental revamp of the unit testing framework here: https://github.com/OpenGeoscience/geojs/tree/fix-build-testing

It allows you to do for example:

cmake -DBUILD_GEOJS_TESTING=ON ${GEOJS_SOURCE_DIR}
make
ctest

The jasmine tests are added as individual ctest targets. The method for doing this was ruthlessly stolen from @ronichoudhury's code in Kitware/tangelo.

@jbeezley
Copy link
Contributor

This, #31, and #19 are all related, so I'm pinging them here.

I updated fix-build-testing to fix the server port issue. I also added an option for node path in the CMakeLists. Grunt expects node to be in the PATH, so I am just adding the given path to the environment variable for each test. A little ugly, but it seems to work.

@waxlamp
Copy link
Contributor

waxlamp commented Apr 21, 2014

Ruthlessness appreciated :)

Let me know if you run into problems with the general setup, or if you come
up with enhancements etc.

On Tue, Apr 8, 2014 at 11:10 AM, Jonathan Beezley
notifications@github.comwrote:

I have an experimental revamp of the unit testing framework here:
https://github.com/OpenGeoscience/geojs/tree/fix-build-testing

It allows you to do for example:

cmake -DBUILD_GEOJS_TESTING=ON ${GEOJS_SOURCE_DIR}
make
ctest

The jasmine tests are added as individual ctest targets. The method for
doing this was ruthlessly stolen from @ronichoudhuryhttps://github.com/ronichoudhury's
code in Kitware/tangelo https://github.com/Kitware/tangelo.


Reply to this email directly or view it on GitHubhttps://github.com//issues/40#issuecomment-39859737
.

@jbeezley
Copy link
Contributor

As it turns out we need to use selenium for webgl support, so I am running the tests now through a grunt plugin, grunt-webdriver-jasmine-runner, rather than phantomjs.

@aashish24 aashish24 added bug and removed bug labels May 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants