Skip to content

CNPJ test was flaky, fixed now.#507

Merged
bodiam merged 1 commit intomainfrom
fix_cnpj_test
Nov 1, 2022
Merged

CNPJ test was flaky, fixed now.#507
bodiam merged 1 commit intomainfrom
fix_cnpj_test

Conversation

@bodiam
Copy link
Contributor

@bodiam bodiam commented Nov 1, 2022

@NarekDW fixed the flaky test

@codecov-commenter
Copy link

codecov-commenter commented Nov 1, 2022

Codecov Report

Merging #507 (158ae81) into main (0ed44b8) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #507      +/-   ##
============================================
+ Coverage     93.29%   93.38%   +0.08%     
- Complexity     2200     2217      +17     
============================================
  Files           236      238       +2     
  Lines          4637     4654      +17     
  Branches        493      493              
============================================
+ Hits           4326     4346      +20     
+ Misses          190      185       -5     
- Partials        121      123       +2     
Impacted Files Coverage Δ
.../main/java/net/datafaker/service/files/EnFile.java 93.33% <ø> (ø)
...n/java/net/datafaker/providers/movie/Doraemon.java 100.00% <100.00%> (ø)
.../net/datafaker/providers/movie/MovieProviders.java 100.00% <100.00%> (ø)
...ava/net/datafaker/providers/videogame/Control.java 100.00% <100.00%> (ø)
...afaker/providers/videogame/VideoGameProviders.java 100.00% <100.00%> (ø)
.../main/java/net/datafaker/service/FakerContext.java 87.17% <0.00%> (+2.56%) ⬆️
...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java 96.29% <0.00%> (+3.70%) ⬆️

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

@bodiam bodiam merged commit 3fc2637 into main Nov 1, 2022
@bodiam bodiam deleted the fix_cnpj_test branch November 15, 2022 14:07
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