Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Add missing unit test #9051

Merged
merged 2 commits into from
Oct 4, 2023
Merged

Add missing unit test #9051

merged 2 commits into from
Oct 4, 2023

Conversation

mosmartin
Copy link
Contributor

What was the problem?

This PR resolves #8180

How was it solved?

Added missing unit tests

How was it tested?

Added unit tests

@mosmartin mosmartin requested review from shuse2 and has5aan September 28, 2023 12:50
@shuse2 shuse2 changed the title ✅ Add missing unit test Add missing unit test Sep 28, 2023
@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

Merging #9051 (e999381) into release/6.1.0 (79b8644) will increase coverage by 0.02%.
The diff coverage is n/a.

❗ Current head e999381 differs from pull request most recent head 504f00b. Consider uploading reports for the commit 504f00b to get more accurate results

Impacted file tree graph

@@                Coverage Diff                @@
##           release/6.1.0    #9051      +/-   ##
=================================================
+ Coverage          84.34%   84.36%   +0.02%     
=================================================
  Files                651      651              
  Lines              23985    24016      +31     
  Branches            3490     3493       +3     
=================================================
+ Hits               20229    20260      +31     
  Misses              3756     3756              

see 5 files with indirect coverage changes

@mosmartin mosmartin self-assigned this Sep 29, 2023
@shuse2 shuse2 requested review from AnanyaSshri and removed request for shuse2 September 29, 2023 10:16
@shuse2 shuse2 enabled auto-merge (squash) October 4, 2023 06:25
@shuse2 shuse2 merged commit fe90652 into release/6.1.0 Oct 4, 2023
@shuse2 shuse2 deleted the 8180-add-mising-unit-test branch October 4, 2023 06:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants