Enable headless testing through casperjs. #214
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This enables us to test 1010 of a total of 1062 single tests without opening the test suite in a browser. We skip some tests as we currently are not able to mock-up certain calls of the ancient Test.AnotherWay suite. The total running time is usually reduced by about 50%. Call the script from the root of a GeoExt checkout as follows: ./tests/headless/casperjs-1.1-beta.1/bin/casperjs test tests/headless/run-testsuite.js This also marks the first important step towards continous integration tests, e.g. through Travis.
This is actually more precise than what we had before.
|
Hi @marcjansen , thanks for this. I thinks it is an importatnt step to have an automized test suite in order to bring GeoExt towards continous integration. Really good idea to bring this up. Test tests within the existing testsuite in the browser still pass all in FF 25, Chrome 31 and IE 11. What should be documented is the fact that casperjs is not running out of the box on every system without the installation of dependencies. If you want to please merge. |
|
Thanks for taking a look at this big diff. You are right; we should add a small readme listing the requirements and also we should possibly try to also run/pass the other tests. Next: Travis! |
marcjansen
added a commit
that referenced
this pull request
Nov 24, 2013
Enable headless testing through casperjs. (r=@chrismayer)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This enables us to test 1010 of a total of 1062 single tests without opening the
test suite in a browser. We skip some tests as we currently are not able to mock-up
certain calls of the ancient Test.AnotherWay suite. The total running time is usually
reduced by about 50%.
Call the script from the root of a GeoExt checkout as follows:
./tests/headless/casperjs-1.1-beta.1/bin/casperjs test tests/headless/run-testsuite.js
This also marks the first important step towards continous integration tests, e.g. through
Travis.