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

Feature/testing moved sources to src added tests #257

Merged
merged 17 commits into from Dec 14, 2016
Merged

Feature/testing moved sources to src added tests #257

merged 17 commits into from Dec 14, 2016

Conversation

@annejan
Copy link
Member

@annejan annejan commented Dec 14, 2016

Currently only one test, with 2 assertions on Util::normalizeFolderPath

annejan added 13 commits Dec 12, 2016
@jounathaen
Copy link
Member

@jounathaen jounathaen commented Dec 14, 2016

Dat commit messages :-D

@annejan
Copy link
Member Author

@annejan annejan commented Dec 14, 2016

It was late ;)

For some reason coverage with lcov doesn't work (well) on my mac yet . .
Testing some more (and will add test(s))

@YoshiMan
Copy link

@YoshiMan YoshiMan commented Dec 14, 2016

Awesome. the first step for tests :)

annejan added 4 commits Dec 14, 2016
@annejan annejan merged commit 0fbf476 into master Dec 14, 2016
4 of 6 checks passed
4 of 6 checks passed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push The Travis CI build could not complete due to an error
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/codeship Build succeeded
Details
linthub Linthub has cleared the pull request, no code style suggestions found.
Details
@annejan annejan deleted the feature/testing branch Dec 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants