Skip to content
This repository has been archived by the owner on Jan 4, 2022. It is now read-only.

Fix: Test data providers directly #332

Merged
merged 1 commit into from Oct 3, 2020
Merged

Fix: Test data providers directly #332

merged 1 commit into from Oct 3, 2020

Conversation

localheinz
Copy link
Member

This PR

  • tests data providers directly

Follows #326.

@localheinz localheinz self-assigned this Oct 3, 2020
@codecov
Copy link

codecov bot commented Oct 3, 2020

Codecov Report

Merging #332 into main will increase coverage by 31.02%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##               main      #332       +/-   ##
==============================================
+ Coverage     68.97%   100.00%   +31.02%     
  Complexity       65        65               
==============================================
  Files             8         8               
  Lines           245       227       -18     
==============================================
+ Hits            169       227       +58     
+ Misses           76         0       -76     
Impacted Files Coverage Δ Complexity Δ
src/DataProvider/NullProvider.php 100.00% <0.00%> (+100.00%) 1.00% <0.00%> (ø%)
src/DataProvider/StringProvider.php 100.00% <0.00%> (+100.00%) 8.00% <0.00%> (ø%)
src/DataProvider/BooleanProvider.php 100.00% <0.00%> (+100.00%) 4.00% <0.00%> (ø%)

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 45507c1...269bb69. Read the comment docs.

@ergebnis-bot ergebnis-bot self-requested a review October 3, 2020 12:26
@ergebnis-bot ergebnis-bot self-assigned this Oct 3, 2020
@ergebnis-bot ergebnis-bot merged commit 75cdb94 into main Oct 3, 2020
@ergebnis-bot ergebnis-bot deleted the fix/test branch October 3, 2020 12:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants