search

nodeschool / admin

CLI tool for setting up and maintaining a nodeschool chapters and other things.
17 stars 7 forks source link

Testing #5

Closed dinodsaurus closed 8 years ago

dinodsaurus commented 8 years ago

I was thinking about adding tests to this cli tool. The question is should we go for mocha or tape? What are your preferences ?

martinheidegger commented 8 years ago

:sparkles: 'O' :sparkles: I like tape better because it runs faster and I like mocha better because it produces nicer output. You can choose whatever you feel more comfortable with.

martinheidegger commented 8 years ago

@fforres What do you think?

fforres commented 8 years ago

I've been wanting to use tape some more :+1: I'm up for it :D

martinheidegger commented 8 years ago

tape it is! @dinodsaurus Do you mind to open a PR so we can collaborate on the tests?

dinodsaurus commented 8 years ago

Unfortunately I have started with mocha/chai :dancer: I will create a PR so you can check it out and if you dont like it we can rewrite it to tape :+1:

fforres commented 8 years ago

As long as we can Test @dinodsaurus. I'm more familiar with the mocha/chai/sinon mix :)

(Also, if you folks ever have the time, AVA sounds really cool, no globals though)

martinheidegger commented 8 years ago

AVA indeed looks interesting. For the time being plain-old mocha should suffice. I think this issue has been answered lets continue in #16