From fe2a4dee050e2b7b0db7e4520956d27bced2ae68 Mon Sep 17 00:00:00 2001
From: Jaime Leonardo Suncin Cruz <leosuncin@gmail.com>
Date: Sun, 2 Feb 2020 17:20:40 -0600
Subject: [PATCH 1/2] test: chain commands with subject different of document,
 element or window

---
 cypress/fixtures/test-app/index.html | 14 ++++++++++++
 cypress/integration/find.spec.js     | 32 ++++++++++++++++++----------
 2 files changed, 35 insertions(+), 11 deletions(-)
diff --git a/cypress/fixtures/test-app/index.html b/cypress/fixtures/test-app/index.html
index 36a3fc3..dcd1eeb 100644
--- a/cypress/fixtures/test-app/index.html
+++ b/cypress/fixtures/test-app/index.html
@@ -15,6 +15,13 @@
     section {
       padding: 10px;
     }
+    input:valid + span {
+      display: none;
+    }
+    input:invalid + span {
+      display: block;
+      color: red;
+    }
   </style>
 </head>
 <body>
@@ -110,6 +117,13 @@ <h2>Eventual non-existence</h2>
       }, 500)
     </script>
   </section>
+  <section>
+    <h2>Chain selectors</h2>
+    <form onsubmit="return false" action="#">
+      <label>Required: <input type="text" required /><span>Error message</span></label>
+      <button type="submit">Submit</button>
+    </form>
+  </section>
 <!-- Prettier unindents the script tag below -->
 <script>
 document
diff --git a/cypress/integration/find.spec.js b/cypress/integration/find.spec.js
index 3bbf33b..96fc2a0 100644
--- a/cypress/integration/find.spec.js
+++ b/cypress/integration/find.spec.js
@@ -88,18 +88,15 @@ describe('find* dom-testing-library commands', () => {
   /* Test the behaviour around these queries */
 
   it('findByText should handle non-existence', () => {
-    cy.findByText('Does Not Exist')
-      .should('not.exist')
+    cy.findByText('Does Not Exist').should('not.exist')
   })
 
   it('findByText should handle eventual existence', () => {
-    cy.findByText('Eventually Exists')
-      .should('exist')
+    cy.findByText('Eventually Exists').should('exist')
   })
 
   it('findByText should handle eventual non-existence', () => {
-    cy.findByText('Eventually Not exists')
-      .should('not.exist')
+    cy.findByText('Eventually Not exists').should('not.exist')
   })
 
   it("findByText with should('not.exist')", () => {
@@ -111,7 +108,7 @@ describe('find* dom-testing-library commands', () => {
 
   it('findByText with a previous subject', () => {
     cy.get('#nested')
-      .findByText('Button Text 1', { fallbackRetryWithoutPreviousSubject: false })
+      .findByText('Button Text 1', {fallbackRetryWithoutPreviousSubject: false})
       .should('not.exist')
     cy.get('#nested')
       .findByText('Button Text 2')
@@ -170,8 +167,7 @@ describe('find* dom-testing-library commands', () => {
       expect(err.message).to.contain(errorMessage)
     })
 
-    cy.findByText('Button Text 1', {timeout: 100})
-      .should('not.exist')
+    cy.findByText('Button Text 1', {timeout: 100}).should('not.exist')
   })
 
   it('findByLabelText should forward useful error messages from @testing-library/dom', () => {
@@ -196,11 +192,14 @@ describe('find* dom-testing-library commands', () => {
     cy.window()
       .findByText('Button Text 1')
       .should('exist')
+    cy.location()
+      .findByText('Button Text 1')
+      .should('exist')
   })
 
   it('findByText should show as a parent command if it starts a chain', () => {
     const assertLog = (attrs, log) => {
-      if(log.get('name') === 'findByText') {
+      if (log.get('name') === 'findByText') {
         expect(log.get('type')).to.equal('parent')
         cy.off('log:added', assertLog)
       }
@@ -211,7 +210,7 @@ describe('find* dom-testing-library commands', () => {
 
   it('findByText should show as a child command if it continues a chain', () => {
     const assertLog = (attrs, log) => {
-      if(log.get('name') === 'findByText') {
+      if (log.get('name') === 'findByText') {
         expect(log.get('type')).to.equal('child')
         cy.off('log:added', assertLog)
       }
@@ -219,6 +218,17 @@ describe('find* dom-testing-library commands', () => {
     cy.on('log:added', assertLog)
     cy.get('body').findByText('Button Text 1')
   })
+
+  it('should chain findBy* with subject different of document, element or window', () => {
+    cy.wrap(true)
+      .should('be.true')
+      .findByText('Error message')
+      .findByLabelText(/Required/i)
+      .type('something')
+      .findByText('Submit')
+      .queryByText('Error message')
+      .should('not.be.visible')
+  })
 })
 
 /* global cy */

From a6a5431a016a257f94573ba45f5bfa31c48b4eec Mon Sep 17 00:00:00 2001
From: Jaime Leonardo Suncin Cruz <leosuncin@gmail.com>
Date: Sun, 2 Feb 2020 17:38:08 -0600
Subject: [PATCH 2/2] fix: set previous subject to be optional

This allows to commands to be chained with a previous command
that not returns a document, element or window

FIX testing-library/cypress-testing-library#114
---
 src/index.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/index.js b/src/index.js
index 8b92fbc..26740ed 100644
--- a/src/index.js
+++ b/src/index.js
@@ -44,7 +44,7 @@ const findCommands = findQueryNames.map(queryName => {
 function createCommand(queryName, implementationName) {
   return {
     name: queryName,
-    options: {prevSubject: ['optional', 'document', 'element', 'window']},
+    options: {prevSubject: ['optional']},
     command: (prevSubject, ...args) => {
       const lastArg = args[args.length - 1]
       const defaults = {
