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

test: e2e fixes for #10441 #11162

Conversation

MajorLift
Copy link
Contributor

@MajorLift MajorLift commented Sep 11, 2024

Description

A large number of e2e tests are failing on #10441, and most of them do not seem directly related to the changes in the PR.

To avoid polluting the diffs in the original PR, e2e fixes will be staged and tested here, before being merged into the original once both prs have received final approvals.

Related issues

Manual testing steps

Screenshots/Recordings

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@MajorLift MajorLift added the Run Smoke E2E Triggers smoke e2e on Bitrise label Sep 11, 2024
Copy link
Contributor

github-actions bot commented Sep 11, 2024

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 3638195
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/1186129c-d86f-4e68-a05e-f88378c8e348

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch from a8faa6f to f9537ae Compare September 11, 2024 19:59
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch 2 times, most recently from c5e900a to a272b22 Compare September 12, 2024 03:34
@MajorLift MajorLift added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Sep 12, 2024

This comment was marked as outdated.

@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from a272b22 to 4f39501 Compare September 12, 2024 03:43
@MajorLift MajorLift added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Sep 12, 2024
Copy link
Contributor

github-actions bot commented Sep 12, 2024

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 4f39501
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/144ccc9e-dab8-47d7-926b-215b966e8386

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 52.63158% with 9 lines in your changes missing coverage. Please review.

Please upload report for BASE (jongsun/fix-composable-controller@4ba9792). Learn more about missing BASE report.

Files with missing lines Patch % Lines
app/selectors/snaps/snapController.ts 42.85% 4 Missing ⚠️
app/store/persistConfig.ts 0.00% 3 Missing ⚠️
app/reducers/swaps/index.js 0.00% 2 Missing ⚠️
Additional details and impacted files
@@                         Coverage Diff                          @@
##             jongsun/fix-composable-controller   #11162   +/-   ##
====================================================================
  Coverage                                     ?   55.30%           
====================================================================
  Files                                        ?     1564           
  Lines                                        ?    37222           
  Branches                                     ?     4421           
====================================================================
  Hits                                         ?    20587           
  Misses                                       ?    16162           
  Partials                                     ?      473           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch 2 times, most recently from f9537ae to 058687b Compare September 12, 2024 06:47
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from 4f39501 to 7c6be3a Compare September 12, 2024 06:48
@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch from 058687b to b4bfd59 Compare September 12, 2024 06:48
@MajorLift MajorLift removed the Run Smoke E2E Triggers smoke e2e on Bitrise label Sep 12, 2024
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from 7c6be3a to e157ce1 Compare September 12, 2024 06:49
@MajorLift MajorLift added Run Smoke E2E Triggers smoke e2e on Bitrise DO-NOT-MERGE Pull requests that should not be merged labels Sep 12, 2024
Copy link
Contributor

github-actions bot commented Sep 12, 2024

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: e157ce1
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/02132c91-fbeb-4c57-96fc-61711d603459

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch from b4bfd59 to 69970ad Compare September 12, 2024 07:02
@MajorLift MajorLift changed the base branch from jongsun/fix-composable-controller to main September 12, 2024 07:30
@MajorLift MajorLift changed the base branch from main to jongsun/fix-composable-controller September 12, 2024 07:30
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from e157ce1 to 2941c5f Compare September 12, 2024 07:30
@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch from 2b4198b to 2b6eaac Compare November 26, 2024 12:11
… undefined errors

- `isProfileSyncingEnabled`
- `SwapsController.aggregatorMetadata`
@MajorLift MajorLift force-pushed the jongsun/test/240911-e2e-fixes-for-10441 branch from ae84a70 to 44032c8 Compare November 26, 2024 12:13
@MajorLift MajorLift force-pushed the jongsun/fix-composable-controller branch from 2b6eaac to 7e2a0ab Compare November 26, 2024 14:48
@MajorLift MajorLift closed this Dec 1, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Dec 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DO-NOT-MERGE Pull requests that should not be merged Run Smoke E2E Triggers smoke e2e on Bitrise team-wallet-framework
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants