Skip to content

[WIP] enhance unit testing - #4597

Draft
ChristianBeer wants to merge 8 commits into
masterfrom
cb_enhance_unit-testing
Draft

[WIP] enhance unit testing#4597
ChristianBeer wants to merge 8 commits into
masterfrom
cb_enhance_unit-testing

Conversation

@ChristianBeer

Copy link
Copy Markdown
Member

Some simple enhancements to unit testing. Still WIP.

Description of the Change
More unit testing of library functions.

Alternate Designs
Alternatively one could add the reference files to git and copy them into the directory where tests are executed but that seemed more work than reading and writing directly from within the test.

Release Notes
N/A

@ChristianBeer ChristianBeer added the C: Testing Integration and Unit testing label Dec 26, 2021
@ChristianBeer
ChristianBeer marked this pull request as draft December 26, 2021 23:38
@codecov

codecov Bot commented Dec 27, 2021

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 11.16%. Comparing base (6f8fba3) to head (8275531).
⚠️ Report is 4151 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4597      +/-   ##
============================================
+ Coverage     10.90%   11.16%   +0.25%     
  Complexity     1044     1044              
============================================
  Files           277      277              
  Lines         35626    35627       +1     
  Branches       7852     7852              
============================================
+ Hits           3886     3976      +90     
+ Misses        31520    31431      -89     
  Partials        220      220              

see 9 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Rearranged the lines alphabetically to better compare it with the source files.
Also uses EXPECT_instead of ASSERT_ because with ASSERT_ the test immediately stops not executing the remaining tests of a suite. It's better to execute all tests and get a better picture what is failing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C: Testing Integration and Unit testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant