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

Include filbert_loose.js testing in 'grunt test' #30

Open
differentmatt opened this issue Jul 17, 2014 · 3 comments
Open

Include filbert_loose.js testing in 'grunt test' #30

differentmatt opened this issue Jul 17, 2014 · 3 comments

Comments

@differentmatt
Copy link
Owner

Running grunt test should test filbert.js and filbert_loose.js.

Currently it only runs tests against filbert.js. To test filbert_loose.js, you have to
hack test/spec/util.js to target filbert_loose.js instead.

@bhargavvader
Copy link

Is this bug fixed?
I printed a check in the filbert_loose file and it appeared when grunt test was executed.

@differentmatt
Copy link
Owner Author

I don't think so. I tried putting a print statement in filbert_loose parse_dammit and did not see it print after running grunt test.

How exactly did you check this?

@bhargavvader
Copy link

My bad, I seem to have checked by printing above parse_dammit (line 40 in the filbert_loose file).
Will try and find a fix.

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

2 participants