Skip to content

Potsbo/refactor for testability - #23

Open
potsbo wants to merge 8 commits into
masterfrom
potsbo/refactor-for-testability
Open

Potsbo/refactor for testability#23
potsbo wants to merge 8 commits into
masterfrom
potsbo/refactor-for-testability

Conversation

@potsbo

@potsbo potsbo commented Aug 24, 2020

Copy link
Copy Markdown
Owner

No description provided.

@codecov-commenter

codecov-commenter commented Aug 24, 2020

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 59.25926% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.16%. Comparing base (694de19) to head (2173e30).
⚠️ Report is 13 commits behind head on master.

Files with missing lines Patch % Lines
KeyType/EventConverter.swift 39.39% 20 Missing ⚠️
KeyType/KeyEventWatcher.swift 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
+ Coverage   57.75%   62.16%   +4.41%     
==========================================
  Files          11       12       +1     
  Lines         258      259       +1     
==========================================
+ Hits          149      161      +12     
+ Misses        109       98      -11     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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