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

Unit test ChainEndpoint #9168

Merged
merged 12 commits into from
Dec 4, 2023
Merged

Conversation

has5aan
Copy link
Contributor

@has5aan has5aan commented Nov 30, 2023

What was the problem?

This PR resolves #7897 .

How was it solved?

Implemented missing unit tests for ChainEndpoint.

@has5aan has5aan self-assigned this Nov 30, 2023
@has5aan has5aan requested review from shuse2 and mosmartin November 30, 2023 11:43
Copy link

codecov bot commented Nov 30, 2023

Codecov Report

Merging #9168 (e06ed27) into release/6.1.0 (175f817) will increase coverage by 0.29%.
The diff coverage is 100.00%.

❗ Current head e06ed27 differs from pull request most recent head 94206b8. Consider uploading reports for the commit 94206b8 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           release/6.1.0    #9168      +/-   ##
=================================================
+ Coverage          84.36%   84.65%   +0.29%     
=================================================
  Files                655      655              
  Lines              24113    24113              
  Branches            3497     3497              
=================================================
+ Hits               20343    20414      +71     
+ Misses              3770     3699      -71     
Files Coverage Δ
framework/src/engine/endpoint/chain.ts 98.42% <100.00%> (+55.90%) ⬆️

@shuse2 shuse2 enabled auto-merge (squash) December 4, 2023 07:56
@shuse2 shuse2 merged commit f641009 into release/6.1.0 Dec 4, 2023
9 checks passed
@shuse2 shuse2 deleted the 7879-engine-endpoints-unit-tests branch December 4, 2023 08:15
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.

3 participants