Skip to content

Conversation

@cacieprins
Copy link
Contributor

  • Closes

Additional details

A previous attempt to modernize cloud API requests introduced a bug in how requests are made to the Cloud API under certain proxy contexts. This adds a system test to verify the behavior of the cloud API client when recording under a few different scenarios when proxies are configured.

Steps to test

Run the record_through_proxy system test.

How has the user experience changed?

N/A

PR Tasks

@cacieprins cacieprins marked this pull request as ready for review April 28, 2025 16:35
@cypress
Copy link

cypress bot commented Apr 28, 2025

cypress    Run #62866

Run Properties:  status check passed Passed #62866  •  git commit 2fd41bfe24: Merge branch 'develop' into system-test-record-through-proxy
Project cypress
Branch Review system-test-record-through-proxy
Run status status check passed Passed #62866
Run duration 19m 12s
Commit git commit 2fd41bfe24: Merge branch 'develop' into system-test-record-through-proxy
Committer Cacie Prins
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 8
Tests that did not run due to a developer annotating a test with .skip  Pending 1232
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 32182
View all changes introduced in this branch ↗︎
UI Coverage  46.11%
  Untested elements 189  
  Tested elements 166  
Accessibility  92.73%
  Failed rules  3 critical   9 serious   2 moderate   2 minor
  Failed elements 697  

Copy link
Contributor

@AtofStryker AtofStryker left a comment

Choose a reason for hiding this comment

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

Do we have primary vs preliminary defined anywhere? I'm not super familiar with this domain and curious what the differences are.

@mschile mschile closed this Sep 24, 2025
@mschile mschile deleted the system-test-record-through-proxy branch September 24, 2025 15:29
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.

4 participants