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

Re arrange helper tests #716

Merged
merged 2 commits into from
Jun 6, 2016
Merged

Conversation

DanPurdy
Copy link
Member

While filtering through our open issues and at the same time writing a new helper test i realised we never actually did what we proposed in #322 completely. This splits them into their separate files which makes them clearer to get to imo.

closes #322

<DCO 1.1 Signed-off-by: Dan Purdy dan@danpurdy.co.uk>

@DanPurdy DanPurdy changed the title ✅ re arrange helper tests closes #322 Re arrange helper tests May 17, 2016
@coveralls
Copy link

coveralls commented May 17, 2016

Coverage Status

Coverage remained the same at 97.271% when pulling 041db13 on DanPurdy:feature/split-helpers into b2f8523 on sasstools:develop.

@DanPurdy DanPurdy added this to the 1.8.0 milestone May 17, 2016
@coveralls
Copy link

coveralls commented Jun 3, 2016

Coverage Status

Coverage remained the same at 97.271% when pulling 4bdbc94 on DanPurdy:feature/split-helpers into ff4f5f1 on sasstools:develop.

@bgriffith bgriffith merged commit 4970c3c into sasstools:develop Jun 6, 2016
@DanPurdy DanPurdy deleted the feature/split-helpers branch June 6, 2016 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactoring the helpers test suite
3 participants