Skip to content
This repository has been archived by the owner on Feb 2, 2023. It is now read-only.

We need tests #9

Closed
flo-l opened this issue May 23, 2016 · 3 comments
Closed

We need tests #9

flo-l opened this issue May 23, 2016 · 3 comments

Comments

@flo-l
Copy link

flo-l commented May 23, 2016

Tests are definitely necessary for this repo, IMHO.

I'm not sure how to go about this, as it's currently not possible to run scripts as tests, see here.

But what about some ruby tests, rspec maybe? We need to test if the results on the ruby side are correct anyway ;)

@flo-l
Copy link
Author

flo-l commented May 23, 2016

OK I just discovered the original branch, which includes rspec tests.

@wycats could you merge original and master?

@wycats
Copy link
Contributor

wycats commented Jun 18, 2016

@flo-l sorry about that and how long it has taken me to reply (I've been travelling for a bit!)

Yes, I think we should merge the branches and make sure we have a good testing setup that we can use with Travis.

@wagenet
Copy link
Collaborator

wagenet commented Feb 25, 2017

The duration_wip branch has Travis support.

@wagenet wagenet closed this as completed Feb 25, 2017
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

3 participants