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

Update spec tests (RTM) #81

Merged
merged 2 commits into from
Sep 19, 2017
Merged

Conversation

Phil-Friderici
Copy link
Contributor

Refactor the spec tests to follow the standard. Ensure all functionality is tested properly.

This eleminates the special treatment that was needed for strings in tests.
@ghoneycutt
Copy link
Owner

Thanks @Phil-Friderici

@ghoneycutt
Copy link
Owner

Noticed the WIP, let me know when this is ready for merge.

@Phil-Friderici Phil-Friderici changed the title Update spec tests (WIP) Update spec tests (RTM) Sep 19, 2017
@ghoneycutt
Copy link
Owner

Is that commit saying I should Read The Manual? :>

@ghoneycutt
Copy link
Owner

Thanks @Phil-Friderici

@ghoneycutt ghoneycutt merged commit 4bf0b3a into ghoneycutt:master Sep 19, 2017
@Phil-Friderici
Copy link
Contributor Author

sry 4 bd inglish, 4got the F

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

Successfully merging this pull request may close these issues.

2 participants