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

Update unit test of removeSupportAllNFTs method #9018

Merged
merged 2 commits into from
Sep 21, 2023

Conversation

mosmartin
Copy link
Contributor

What was the problem?

This PR resolves #8982

How was it solved?

Updated the removeSupportAllNFTs unit test

How was it tested?

Updated unit test passes

@mosmartin mosmartin requested review from Incede and has5aan September 20, 2023 10:44
@mosmartin mosmartin self-assigned this Sep 20, 2023
@codecov
Copy link

codecov bot commented Sep 20, 2023

Codecov Report

Merging #9018 (8de5405) into release/6.1.0 (22213a7) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head 8de5405 differs from pull request most recent head f5ee856. Consider uploading reports for the commit f5ee856 to get more accurate results

Impacted file tree graph

@@                Coverage Diff                @@
##           release/6.1.0    #9018      +/-   ##
=================================================
- Coverage          84.36%   84.36%   -0.01%     
=================================================
  Files                651      650       -1     
  Lines              24016    24022       +6     
  Branches            3493     3490       -3     
=================================================
+ Hits               20261    20266       +5     
- Misses              3755     3756       +1     

@mosmartin mosmartin requested review from shuse2 and removed request for has5aan September 20, 2023 11:19
@shuse2 shuse2 removed their request for review September 20, 2023 11:48
@shuse2 shuse2 enabled auto-merge (squash) September 21, 2023 12:53
@shuse2 shuse2 merged commit a251bb3 into release/6.1.0 Sep 21, 2023
@shuse2 shuse2 deleted the 8982-update-unit-test branch September 21, 2023 14:08
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