fix: force forward slash to backslash on Windows in spec search input (…

…#23776)
cypress-io · Sep 12, 2022 · 7c6c231 · 7c6c231 · cypress-bot · Sep 12, 2022
1 parent 043ab9a
commit 7c6c231
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/packages/app/cypress/e2e/specs_list_e2e.cy.ts b/packages/app/cypress/e2e/specs_list_e2e.cy.ts
@@ -237,6 +237,14 @@ describe('App: Spec List (E2E)', () => {
         cy.get('button').contains('23 Matches')
       })
 
+      it('normalizes directory path separators for Windows', function () {
+        // On Windows, when a user types `e2e/accounts`, it should match `e2e\accounts`
+        clearSearchAndType('e2e/accounts')
+        cy.findAllByTestId('spec-item').should('have.length', 2)
+
+        cy.findByText('No specs matched your search:').should('not.be.visible')
+      })
+
       // TODO: fix flaky test https://github.com/cypress-io/cypress/issues/23305
       it.skip('saves the filter when navigating to a spec and back', function () {
         const targetSpecFile = 'accounts_list.spec.js'

diff --git a/packages/app/src/specs/spec-utils.ts b/packages/app/src/specs/spec-utils.ts
@@ -125,8 +125,10 @@ function getHighlightIndexes <T extends FoundSpec> (node: SpecTreeNode<T>) {
 }
 
 export function fuzzySortSpecs <T extends FuzzyFoundSpec> (specs: T[], searchValue: string) {
+  const normalizedSearchValue = getPlatform() === 'win32' ? searchValue.replaceAll('/', '\\') : searchValue
+
   const fuzzySortResult = fuzzySort
-  .go(searchValue, specs, { keys: ['relative', 'baseName'], allowTypo: false, threshold: -3000 })
+  .go(normalizedSearchValue, specs, { keys: ['relative', 'baseName'], allowTypo: false, threshold: -3000 })
   .map((result) => {
     const [relative, baseName] = result