fix: display the correct expected length in the Command Log when using 'not.have.length' (#33064)

ryanthemanuel · web-flow · commit 6e4b7800b6dd · 2025-12-05T11:09:50.000-06:00
diff --git a/cli/CHANGELOG.md b/cli/CHANGELOG.md
@@ -12,6 +12,7 @@ _Released 12/16/2025 (PENDING)_
 
 - Fixed an issue where a EPIPE error shows up after CTRL+C is done in terminal. Fixes [#30659](https://github.com/cypress-io/cypress/issues/30659). Addressed in [#32873](https://github.com/cypress-io/cypress/pull/32873).
 - Fixed an issue where the browser would freeze when Cypress intercepts a synchronous XHR request and a `routeHandler` is used. Fixes [#32874](https://github.com/cypress-io/cypress/issues/32874). Addressed in [#32925](https://github.com/cypress-io/cypress/pull/32925).
+- Fixed an issue where the error message for `not.have.length` was not correctly displaying the expected length in the Command Log. Addressed in [#18927](https://github.com/cypress-io/cypress/issues/18927).
 
 ## 15.7.1
 
diff --git a/packages/driver/cypress/e2e/commands/assertions.cy.js b/packages/driver/cypress/e2e/commands/assertions.cy.js
@@ -1571,6 +1571,34 @@ describe('src/cy/commands/assertions', () => {
         cy.get('body').should('have.length', 2)
       })
 
+      it('formats _obj with cypress for `not.have.length`', (done) => {
+        cy.on('log:added', (attrs, log) => {
+          if (attrs.name === 'assert') {
+            cy.removeAllListeners('log:added')
+
+            expect(log.get('message')).to.eq('expected **<button>** to not have a length of **2**')
+
+            done()
+          }
+        })
+
+        cy.get('button:first').should('not.have.length', 2)
+      })
+
+      it('formats error _obj with cypress for `not.have.length`', (done) => {
+        cy.on('log:added', (attrs, log) => {
+          if (attrs.name === 'assert') {
+            cy.removeAllListeners('log:added')
+
+            expect(log.get('_error').message).to.eq('expected \'<body>\' to not have a length of 1')
+
+            done()
+          }
+        })
+
+        cy.get('body').should('not.have.length', 1)
+      })
+
       it('does not touch non DOM objects', () => {
         cy.noop([1, 2, 3]).should('have.length', 3)
       })
diff --git a/packages/driver/src/cy/chai.ts b/packages/driver/src/cy/chai.ts
@@ -344,7 +344,7 @@ chai.use((chai, u) => {
             return this.assert(
               obj.length === length,
               `expected '${node}' to have a length of \#{exp} but got \#{act}`,
-              `expected '${node}' to not have a length of \#{act}`,
+              `expected '${node}' to not have a length of \#{exp}`,
               length,
               obj.length,
             )

Original file line number	Diff line number	Diff line change
`@@ -344,7 +344,7 @@ chai.use((chai, u) => {`
`344`	`344`	`return this.assert(`
`345`	`345`	`obj.length === length,`
`346`	`346`	`expected '${node}' to have a length of \#{exp} but got \#{act}`,
`347`		- `expected '${node}' to not have a length of \#{act}`,
	`347`	+ `expected '${node}' to not have a length of \#{exp}`,
`348`	`348`	`length,`
`349`	`349`	`obj.length,`
`350`	`350`	`)`