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

Travis CI - Test should fail #130

Closed
anthonny opened this issue Oct 8, 2015 · 4 comments
Closed

Travis CI - Test should fail #130

anthonny opened this issue Oct 8, 2015 · 4 comments
Assignees
Milestone

Comments

@anthonny
Copy link
Member

anthonny commented Oct 8, 2015

Test should fail because Opal is undefined but the job pass

image

https://travis-ci.org/asciidoctor/asciidoctor.js/builds/83552848

@mojavelinux
Copy link
Member

Probably the return value of the build is not making it back to the shell?

@ggrossetie ggrossetie self-assigned this Oct 10, 2015
@ggrossetie
Copy link
Member

mhevery/jasmine-node#212

@ggrossetie
Copy link
Member

I tried to update grunt-jasmine-node to 0.3.1 but this version is trying to load Bower spec files:

Exception loading: /home/guillaume/workspace/opensource/asciidoctor.js/spec/bower/bower.spec.js
[ReferenceError: commonSpec is not defined]
Failed to execute "jasmine.executeSpecsInFolder": ReferenceError: commonSpec is not defined

The config all: ['spec/npm'] is somehow ignored.
With this version ReferenceError terminates Grunt build with exit code 0...

@ggrossetie
Copy link
Member

I think this issue is now outdated 😉

See #166 #167

@ggrossetie ggrossetie added this to the v1.5.4 milestone Jan 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants