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

Remove executables from TestData folder (#132) #133

Merged
merged 6 commits into from
Apr 21, 2017

Conversation

csoltenborn
Copy link
Owner

@csoltenborn csoltenborn commented Apr 20, 2017

Build all test executables from source (#132)

@csoltenborn csoltenborn added this to the 0.10.0 milestone Apr 20, 2017
@csoltenborn csoltenborn self-assigned this Apr 20, 2017
@codecov
Copy link

codecov bot commented Apr 20, 2017

Codecov Report

Merging #133 into develop will decrease coverage by 0.12%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop    #133      +/-   ##
==========================================
- Coverage    72.62%   72.5%   -0.13%     
==========================================
  Files           78      78              
  Lines         2970    2971       +1     
  Branches       442     442              
==========================================
- Hits          2157    2154       -3     
- Misses         661     663       +2     
- Partials       152     154       +2
Flag Coverage Δ
#Integration 57.41% <100%> (-0.25%) ⬇️
#Unit 53.21% <0%> (-0.09%) ⬇️
Impacted Files Coverage Δ
GoogleTestAdapter/Core/GoogleTestDiscoverer.cs 84.5% <ø> (ø) ⬆️
GoogleTestAdapter/Core/Helpers/Utils.cs 100% <100%> (ø) ⬆️
...eTestAdapter/Core/TestCases/NewTestCaseResolver.cs 89.79% <0%> (-4.09%) ⬇️
GoogleTestAdapter/DiaResolver/DiaResolver.cs 71.28% <0%> (-1.99%) ⬇️

@csoltenborn csoltenborn merged commit 8d6e6d2 into develop Apr 21, 2017
@csoltenborn csoltenborn deleted the #132_RemoveExecutablesFromTestData branch April 21, 2017 06:21
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.

1 participant