Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Adding missing tests to FileUtils class #660

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

Sweetdevil144
Copy link
Contributor

Fixes #653

  • Improved code coverage from 70% to 95%
  • Added test cases for NullFile exception, NotYamlExtension exception and Unexpected exception

Added more Unit tests to FileUtils class
Improved code coverage from 70% to 95%
@Sweetdevil144 Sweetdevil144 changed the title Added more Unit tests to FileUtils class test : Add more Unit tests to FileUtils class Oct 11, 2023
@Sweetdevil144 Sweetdevil144 changed the title test : Add more Unit tests to FileUtils class test: Adding missing tests to FileUtils class Oct 11, 2023
@andrii-bodnar andrii-bodnar added the hacktoberfest This issue welcomes contributions for Hacktoberfest label Oct 11, 2023
@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

Merging #660 (76ce81c) into main (0a56e8f) will increase coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #660      +/-   ##
============================================
+ Coverage     62.41%   62.51%   +0.11%     
- Complexity     1325     1328       +3     
============================================
  Files           200      200              
  Lines          5580     5580              
  Branches        841      841              
============================================
+ Hits           3482     3488       +6     
+ Misses         1646     1642       -4     
+ Partials        452      450       -2     

see 1 file with indirect coverage changes

Copy link
Member

@andrii-bodnar andrii-bodnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Sweetdevil144 thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest This issue welcomes contributions for Hacktoberfest hacktoberfest-accepted tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add more Unit tests for the FileUtils class
2 participants