search

longshotlabs / meteor-template-extension

A Meteor package: Replace already defined templates, inherit helpers and events from other templates
https://atmospherejs.com/aldeed/template-extension
MIT License
220 stars 20 forks source link

Add tests #11

Closed mitar closed 9 years ago

mitar commented 9 years ago

Please add tests. This is too critical code to be left without tests. It is using Meteor internals so it is unclear when they can change.

zimme commented 9 years ago

:+1:

aldeed commented 9 years ago

That's funny because I was thinking the same thing this morning. Will put it on the to-do list.

grabbou commented 9 years ago

@aldeed can do some basic scenarios + set up testing scheme based on your other projects ;)

mitar commented 9 years ago

In #12 I already prepared some tests so they have just to be extended.

mitar commented 9 years ago

Can you also enable Travis CI tests? I see that .travis.yaml is already there. I think it should just start working.

aldeed commented 9 years ago

Yep, just did and added readme badge

aldeed commented 9 years ago

After pulling @jgladch's tests, this is the list of things we still need tests for:

jgladch commented 9 years ago

On it