Merge pull request #11 from cypress-io/wrap-2

feat: wrap returned elements in jquery, close #2

Author: bahmutov

README.md

@@ -29,7 +29,7 @@ See [cypress/integration/spec.js](cypress/integration/spec.js)
 
 ## Roadmap
 
-- [ ] wrap returned DOM nodes in jQuery [#2](https://github.com/cypress-io/cypress-xpath/issues/2)
+- [x] wrap returned DOM nodes in jQuery [#2](https://github.com/cypress-io/cypress-xpath/issues/2)
 - [ ] retry the assertion that follows [#3](https://github.com/cypress-io/cypress-xpath/issues/3)
 - [ ] add TypeScript definitions [#4](https://github.com/cypress-io/cypress-xpath/issues/4)
 - [ ] search from the previous subject element [#5](https://github.com/cypress-io/cypress-xpath/issues/5)

cypress/integration/spec.js

@@ -10,6 +10,13 @@ describe('cypress-xpath', () => {
     cy.xpath('//h1').should('have.length', 1)
   })
 
+  it('returns jQuery wrapped elements', () => {
+    cy.visit('cypress/integration/index.html')
+    cy.xpath('//h1').then((el$) => {
+      expect(el$).to.have.property('jquery')
+    })
+  })
+
   it('gets h1 text', () => {
     cy.visit('cypress/integration/index.html')
     cy.xpath('//h1/text()').its('0.textContent').should('equal', 'cypress-xpath')

src/index.js

@@ -65,7 +65,7 @@ const xpath = (selector) => {
 
   // TODO set found elements on the command log?
 
-  return nodes
+  return Cypress.$(nodes)
 }
 
 Cypress.Commands.add('xpath', xpath)