Skip to content

Add tests!#12

Open
jarydkrish wants to merge 4 commits intomasterfrom
#11-tests
Open

Add tests!#12
jarydkrish wants to merge 4 commits intomasterfrom
#11-tests

Conversation

@jarydkrish
Copy link

Co-Authored-By: ptdixon701 ptdixon701@users.noreply.github.com

Closes #11

Co-Authored-By: ptdixon701 <ptdixon701@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Apr 2, 2018

Codecov Report

Merging #12 into master will increase coverage by 0.91%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   44.52%   45.43%   +0.91%     
==========================================
  Files          39       39              
  Lines         438      438              
  Branches       74       74              
==========================================
+ Hits          195      199       +4     
+ Misses        224      220       -4     
  Partials       19       19
Flag Coverage Δ
#javascript 45.43% <100%> (+0.91%) ⬆️
Impacted Files Coverage Δ
src/utilities/analysis.js 100% <100%> (+7.27%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 99893d6...e39a8ea. Read the comment docs.

@jarydkrish jarydkrish changed the title Add analysis.test.js test file for analysis.js Add tests! Apr 2, 2018
Co-Authored-By: ptdixon701 <ptdixon701@users.noreply.github.com>
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.

1 participant