Skip to content
This repository has been archived by the owner on Sep 6, 2022. It is now read-only.

More test cases for meteor-jade #163

Open
teintinu opened this issue Sep 1, 2015 · 2 comments
Open

More test cases for meteor-jade #163

teintinu opened this issue Sep 1, 2015 · 2 comments

Comments

@teintinu
Copy link

teintinu commented Sep 1, 2015

Hi @mquandalle, and uses of meteor-jade.

I'm thinking use meteor-jade and trying make more tests, and features.

For tests I'm started it, I wonder if you can tell want you think about and, if help me with hints, for cases.

I used md files for describe tests 'cause it's a simple way to create new cases.

The basic test of meteor-jade it's running green https://github.com/thr0w/meteor-jade/blob/master/packages/jade-compiler/tests/cases/hello_world.md

There is 25 more tests pending for compiler. https://github.com/thr0w/meteor-jade/blob/master/packages/jade-compiler/tests/cases/

Thanks.

@mquandalle
Copy link
Owner

Hi @thr0w,

Thanks for that effort, that's interesting. I use a slightly different strategy for testing jade templates, I use both a meteor-jade and a spacebars template and I verify that they compile to the same render function (instead of directly specifying the blaze code like you do). That is useful because the low level Blaze API isn't specified, and sometimes changes between meteor releases.

Adding more tests is a good idea, did you take a loot at the current test suite?

@teintinu
Copy link
Author

teintinu commented Sep 1, 2015

Thanks you @mquandalle, I will open another issue to explain the feature I want (I think it's better) and why I need more tests in compiler.

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

No branches or pull requests

2 participants