Skip to content

Run tests on Travis CI - #1043

Open
jedateach wants to merge 7 commits into
CreateJS:masterfrom
jedateach:travis-ci
Open

Run tests on Travis CI#1043
jedateach wants to merge 7 commits into
CreateJS:masterfrom
jedateach:travis-ci

Conversation

@jedateach

@jedateach jedateach commented Mar 12, 2020

Copy link
Copy Markdown

These changes get travis up and running.

There are a couple of test groups disabled. Any help to figure out the issues with them is appreciated.

Here you can see the first pass: https://travis-ci.org/jedateach/EaselJS/builds/661221048

xdescribe("Deprecated Methods and Properties", function () {


describe("Deprecated Functions", function(done) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What was the done doing?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this case... Nothing. Which is why I removed it.

The version of jasmine that I installed gave errors that one of these functions didn't actually accept any arguments... I think it was the done callback in the describe test function argument.

Usually it is intended as a callback for asynchronous tests to end.
https://jasmine.github.io/tutorials/async

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

Labels

None yet

2 participants