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

Enable headless testing through casperjs. #214

Merged
merged 4 commits into from Nov 24, 2013

Conversation

Projects
None yet
2 participants
@marcjansen
Member

marcjansen commented Nov 24, 2013

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.

marcjansen added some commits Nov 22, 2013

Enable headless testing through casperjs.
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.
Swapping undefined/null when conmpairing.
This is actually more precise than what we had before.
@chrismayer

This comment has been minimized.

Show comment
Hide comment
@chrismayer

chrismayer Nov 24, 2013

Contributor

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.
Also be sure the missing tests will be integrated in the future, at least make an issue for that.

If you want to please merge.

Contributor

chrismayer commented Nov 24, 2013

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.
Also be sure the missing tests will be integrated in the future, at least make an issue for that.

If you want to please merge.

@marcjansen

This comment has been minimized.

Show comment
Hide comment
@marcjansen

marcjansen Nov 24, 2013

Member

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!

Member

marcjansen commented Nov 24, 2013

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

Merge pull request #214 from marcjansen/headless
Enable headless testing through casperjs. (r=@chrismayer)

@marcjansen marcjansen merged commit e85c840 into geoext:master Nov 24, 2013

@marcjansen marcjansen deleted the marcjansen:headless branch Jun 11, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment