Skip to content

Added Cricket Provider - #417

Merged
bodiam merged 5 commits into
datafaker-net:mainfrom
ChiragMangukia:cricket
Sep 30, 2022
Merged

bodiam merged 5 commits into
datafaker-net:mainfrom
ChiragMangukia:cricket

Conversation

@ChiragMangukia

Copy link
Copy Markdown
Contributor

Added a new Cricket Provider.

@codecov-commenter

codecov-commenter commented Sep 29, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #417 (edc9696) into main (afe9f7b) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #417      +/-   ##
============================================
- Coverage     93.91%   93.87%   -0.04%     
- Complexity     1985     1990       +5     
============================================
  Files           218      219       +1     
  Lines          3996     4003       +7     
  Branches        384      384              
============================================
+ Hits           3753     3758       +5     
- Misses          146      147       +1     
- Partials         97       98       +1     
Impacted Files Coverage Δ
.../main/java/net/datafaker/service/files/EnFile.java 93.33% <ø> (ø)
src/main/java/net/datafaker/sport/Cricket.java 100.00% <100.00%> (ø)
.../main/java/net/datafaker/sport/SportProviders.java 100.00% <100.00%> (ø)
...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java 92.59% <0.00%> (-3.71%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Comment thread src/test/java/net/datafaker/sport/CricketTest.java Outdated
Comment thread src/test/java/net/datafaker/sport/CricketTest.java Outdated
Comment thread src/test/java/net/datafaker/sport/CricketTest.java Outdated
Comment thread src/test/java/net/datafaker/sport/CricketTest.java Outdated
ChiragMangukia and others added 4 commits September 30, 2022 09:04
Co-authored-by: Sergey Nuyanzin <sergey.nuyanzin@aiven.io>
Co-authored-by: Sergey Nuyanzin <sergey.nuyanzin@aiven.io>
Co-authored-by: Sergey Nuyanzin <sergey.nuyanzin@aiven.io>
Co-authored-by: Sergey Nuyanzin <sergey.nuyanzin@aiven.io>
@ChiragMangukia

Copy link
Copy Markdown
Contributor Author

Thank You @snuyanzin for the review. I have committed suggested changes. Actually, the reference was taken from FootballTest which has public test methods.

@bodiam

bodiam commented Sep 30, 2022

Copy link
Copy Markdown
Contributor

Thanks @ChiragMangukia for this PR, looks great!

@bodiam
bodiam merged commit e455403 into datafaker-net:main Sep 30, 2022
@bodiam

bodiam commented Sep 30, 2022

Copy link
Copy Markdown
Contributor

@ChiragMangukia thanks for letting us know about the FootballTest. I've made all the test methods a bit more consistent now: #418

@ChiragMangukia
ChiragMangukia deleted the cricket branch September 30, 2022 04:57
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.

4 participants