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

Enhancement: Add DataProvider\StringProvider::withWhitespace() #374

Merged
merged 1 commit into from
Nov 17, 2020

Conversation

localheinz
Copy link
Member

@localheinz localheinz commented Nov 17, 2020

This PR

  • adds DataProvider\StringProvider::withWhitespace()

💁‍♂️ This can be useful for testing a method that should reject strings containing any whitespace characters.

@codecov
Copy link

codecov bot commented Nov 17, 2020

Codecov Report

Merging #374 (cf91aa3) into main (3249d21) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##                main      #374   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity        85        87    +2     
===========================================
  Files             12        12           
  Lines            288       302   +14     
===========================================
+ Hits             288       302   +14     
Impacted Files Coverage Δ Complexity Δ
src/DataProvider/StringProvider.php 100.00% <100.00%> (ø) 12.00 <2.00> (+2.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 3249d21...cf91aa3. Read the comment docs.

@localheinz localheinz marked this pull request as draft November 17, 2020 17:55
@ergebnis-bot ergebnis-bot self-assigned this Nov 17, 2020
ergebnis-bot
ergebnis-bot previously approved these changes Nov 17, 2020
@localheinz localheinz marked this pull request as ready for review November 17, 2020 17:56
@ergebnis-bot ergebnis-bot merged commit 825611c into main Nov 17, 2020
@ergebnis-bot ergebnis-bot deleted the feature/with-whitespace branch November 17, 2020 17:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants