Skip to content
This repository has been archived by the owner on Oct 13, 2022. It is now read-only.

Command chaining & cy.within() #33

Merged
merged 5 commits into from
May 2, 2019
Merged

Command chaining & cy.within() #33

merged 5 commits into from
May 2, 2019

Conversation

badeball
Copy link
Contributor

This patch adds the foundation for supporting chaining off of other commands, as well as support for using with cy.within().

Beware, I am completely new to Cypress and have gathered most of my inspiration from this repository and kentcdodds/cypress-testing-library.

@CLAassistant
Copy link

CLAassistant commented Apr 25, 2019

CLA assistant check
All committers have signed the CLA.

@bahmutov
Copy link
Contributor

@badeball can you take a look - I have merged your previous PR #35 so now this code needs to be updated a little

@badeball
Copy link
Contributor Author

badeball commented May 2, 2019

@bahmutov, I've fixed the failing tests with 1bbdfaf. Turns our that I had somewhat misunderstood some assertions, which happened to work before 5afe28d was introduced.

The empty assertion doesn't do what I thought it did. A jquery-wrapped
empty array isn't "empty" per said assertion, hence our tests failed in
context of 5afe28d.
@bahmutov bahmutov merged commit 515edc0 into cypress-io:master May 2, 2019
@bahmutov
Copy link
Contributor

bahmutov commented May 2, 2019

@badeball this is so so good, thank you 👏

@bahmutov
Copy link
Contributor

bahmutov commented May 2, 2019

🎉 This PR is included in version 1.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants