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

fix(app, react-api-client): fix tests after weird merges #12710

Merged
merged 1 commit into from
May 16, 2023

Conversation

shlokamin
Copy link
Member

Overview

At the end of the day yesterday we merged in a flurry of PRs that resulted in some frontend test failures. This PR should fix those failures and make edge green again.

Review Requests

Make sure all frontend tests pass on this PR

Risk assessment

Low

@shlokamin shlokamin requested review from a team as code owners May 16, 2023 15:51
@shlokamin shlokamin requested review from b-cooper, koji and a team and removed request for a team May 16, 2023 15:51
@shlokamin shlokamin changed the base branch from edge to internal-release_0.8.0 May 16, 2023 15:51
@shlokamin shlokamin changed the base branch from internal-release_0.8.0 to edge May 16, 2023 15:52
@shlokamin shlokamin force-pushed the fix_odd-protocol-setup-test branch from 187539f to 4488891 Compare May 16, 2023 15:56
@shlokamin shlokamin requested a review from a team as a code owner May 16, 2023 15:56
@shlokamin shlokamin changed the base branch from edge to internal-release_0.8.0 May 16, 2023 15:56
@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Merging #12710 (4488891) into internal-release_0.8.0 (9ac04fa) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@                    Coverage Diff                     @@
##           internal-release_0.8.0   #12710      +/-   ##
==========================================================
- Coverage                   73.04%   73.03%   -0.01%     
==========================================================
  Files                        1482     1483       +1     
  Lines                       48172    48184      +12     
  Branches                     3021     3022       +1     
==========================================================
+ Hits                        35188    35192       +4     
- Misses                      12520    12528       +8     
  Partials                      464      464              
Flag Coverage Δ
app 44.04% <ø> (ø)
g-code-testing 96.44% <ø> (+<0.01%) ⬆️
notify-server 89.13% <ø> (ø)
react-api-client 72.85% <ø> (-1.48%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 3 files with indirect coverage changes

Copy link
Contributor

@koji koji left a comment

Choose a reason for hiding this comment

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

passed all tests on local

Copy link
Contributor

@b-cooper b-cooper left a comment

Choose a reason for hiding this comment

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

⚒️

@shlokamin shlokamin merged commit 33ebc2e into internal-release_0.8.0 May 16, 2023
@shlokamin shlokamin deleted the fix_odd-protocol-setup-test branch May 16, 2023 16:34
b-cooper added a commit that referenced this pull request May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants