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

chore: Bind this correctly when setting response headers with cy.route() #18859

Merged
merged 1 commit into from
Nov 9, 2021

Conversation

chrisbreiding
Copy link
Contributor

User facing changelog

N/A - this bug is only in develop, not released yet

Additional details

How has the user experience changed?

PR Tasks

  • Have tests been added/updated?
  • N/A Has the original issue or this PR been tagged with a release in ZenHub?
  • N/A Has a PR for user-facing changes been opened in cypress-documentation?
  • N/A Have API changes been updated in the type definitions?
  • N/A Have new configuration options been added to the cypress.schema.json?

@chrisbreiding chrisbreiding requested a review from a team as a code owner November 9, 2021 20:07
@chrisbreiding chrisbreiding requested review from jennifer-shehane and removed request for a team November 9, 2021 20:07
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Nov 9, 2021

Thanks for taking the time to open a PR!

@chrisbreiding chrisbreiding requested a review from flotwig November 9, 2021 20:08
@cypress
Copy link

cypress bot commented Nov 9, 2021



Test summary

18654 2 214 7Flakiness 3


Run details

Project cypress
Status Failed
Commit 6b851e9
Started Nov 9, 2021 8:09 PM
Ended Nov 9, 2021 8:20 PM
Duration 11:32 💡
OS Linux Debian - 10.9
Browser Multiple

View run in Cypress Dashboard ➡️


Failures

navigation_spec.js Failed
1 ... > throws on: 'Infinity'
assertions_spec.js Failed
1 ... > rejects any element not in the document

Flakiness

commands/net_stubbing_spec.ts Flakiness
1 network stubbing > waiting and aliasing > can timeout waiting on a single request using "alias.request"
commands/xhr_spec.js Flakiness
1 ... > no status when request isnt forced 404
e2e/redirects_spec.js Flakiness
1 redirection > meta > binds to the new page after a timeout

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@chrisbreiding chrisbreiding changed the title fix: Bind this correctly when setting response headers with cy.route() chore: Bind this correctly when setting response headers with cy.route() Nov 9, 2021
@chrisbreiding chrisbreiding merged commit e66734d into develop Nov 9, 2021
@chrisbreiding chrisbreiding deleted the issue-18858-cy-route-headers-bug branch November 9, 2021 20:30
tgriesser added a commit that referenced this pull request Nov 18, 2021
* develop: (329 commits)
  chore: Update Chrome (stable) to 96.0.4664.45 (#18931)
  fix: Loading of specs with % in the filename (#18877)
  chore: refactor `create` into class `$Cy` (#18715)
  chore: Update Chrome (beta) to 96.0.4664.45 (#18891)
  fix: flaky `system-tests-firefox` job (#18848)
  release 9.0.0
  feat: ensure major release
  have conduit app wait on localhost:3000
  fix install-required-node
  use --legacy-peer-deps
  feat: ensure major release
  fix darwin node install
  chore(driver): fix integration test retry configuration (#18643)
  feat(deps): update dependency electron to v15 🌟 (#18317)
  chore: Bind this correctly when setting response headers with cy.route() (#18859)
  feat: create config package for config validation (#18589)
  chore: patch `winston` to suppress `padLevels` warning (#18824)
  chore: test out major release build
  fix: remove outdated npm registry links (#18727)
  fix: Adding an existing command with `Cypress.Commands.add()` will throw an error (#18587)
  ...
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.

Error when stubbing headers with cy.route()
3 participants