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

Laika doesn't work with velocity tests. #148

Open
frozeman opened this issue Nov 23, 2014 · 1 comment
Open

Laika doesn't work with velocity tests. #148

frozeman opened this issue Nov 23, 2014 · 1 comment

Comments

@frozeman
Copy link

I tried to add laike to an app where i have also the sanjo:jasmine velocity package add. It fails stating:

    [server log] TypeError: Object #<Object> has no method 'registerTestingFramework'
  at [object Object]._.extend.registerWithVelocity (/Users/frozeman/.../my-meteor-blog/.meteor/local/build/programs/server/packages/sanjo_jasmine.js:200:14)

If i remove sanjo:jasmine it fails, because it want to read also the jasmine test files:

ReferenceError: describe is not defined

Could you add an option to specify the folder in which laika is looking for the tests?
All velocity frameworks must be placed inside a folder named after the testing framework like tests/jasmine or tests/mocha. Could we add the option to only look in tests/laika?

I'm currently finishing up my book (https://www.packtpub.com/web-development/building-single-page-web-apps-meteor) and would like to present laika in the testing chapter, next to velocity.

If you can make it work with velocity would be great, otherwise, if you at least add the option to specify the folder, so it could work along velocity (though one has to remove sanjo:jasmine before, which is also not ideal) i could leave laika in the chapter, otherwise i have to leave it out as its incompatible to meteors official testing framework :(

I'm kind of in a hurry as the book will be published soon, so i would appreciate if you could look into this asp.

@apendua
Copy link
Collaborator

apendua commented Nov 23, 2014

@frozeman I think this option is already there:

https://github.com/arunoda/laika/issues#L8

Can you please verify if it fulfills your needs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants