Fix tests on Windows

Author: thecrypticace

packages/tailwindcss-language-server/src/project-locator.test.ts

@@ -5,6 +5,7 @@ import { URL, fileURLToPath } from 'url'
 import { Settings } from '@tailwindcss/language-service/src/util/state'
 import { createResolver } from './resolver'
 import { css, defineTest, js, json, scss, Storage, TestUtils } from './testing'
+import { normalizePath } from './utils'
 
 let settings: Settings = {
   tailwindCSS: {
@@ -29,12 +30,14 @@ function testFixture(fixture: string, details: any[]) {
 
       let detail = details[i]
 
-      let configPath = path.relative(fixturePath, project.config.path)
+      let configPath = path.posix.relative(normalizePath(fixturePath), project.config.path)
 
       expect(configPath).toEqual(detail?.config)
 
       if (detail?.content) {
-        let expected = detail?.content.map((path) => path.replace('{URL}', fixturePath)).sort()
+        let expected = detail?.content
+          .map((path) => path.replace('{URL}', normalizePath(fixturePath)))
+          .sort()
 
         let actual = project.documentSelector
           .filter((selector) => selector.priority === 1 /** content */)
@@ -45,7 +48,9 @@ function testFixture(fixture: string, details: any[]) {
       }
 
       if (detail?.selectors) {
-        let expected = detail?.selectors.map((path) => path.replace('{URL}', fixturePath)).sort()
+        let expected = detail?.selectors
+          .map((path) => path.replace('{URL}', normalizePath(fixturePath)))
+          .sort()
 
         let actual = project.documentSelector.map((selector) => selector.pattern).sort()
 
@@ -364,7 +369,7 @@ async function prepare({ root }: TestUtils) {
   } as Settings
 
   function adjustPath(filepath: string) {
-    filepath = filepath.replace(root, '{URL}')
+    filepath = filepath.replace(normalizePath(root), '{URL}')
 
     if (filepath.startsWith('{URL}/')) {
       filepath = filepath.slice(5)

packages/tailwindcss-language-server/src/testing/index.ts

@@ -58,7 +58,13 @@ async function setup<T>(config: TestConfig<T>): Promise<TestUtils> {
 
   onTestFinished(async (result) => {
     // Once done, move all the files to a new location
-    await fs.rename(baseDir, doneDir)
+    try {
+      await fs.rename(baseDir, doneDir)
+    } catch {
+      // If it fails it doesn't really matter. It only fails on Windows and then
+      // only randomly so whatever
+      console.error('Failed to move test files to done directory')
+    }
 
     if (result.state === 'fail') return
 

packages/tailwindcss-language-server/tests/document-links/document-links.test.js

@@ -1,6 +1,7 @@
 import { test } from 'vitest'
-import { withFixture } from '../common'
 import * as path from 'path'
+import { URI } from 'vscode-uri'
+import { withFixture } from '../common'
 
 withFixture('basic', (c) => {
   async function testDocumentLinks(name, { text, lang, expected }) {
@@ -19,9 +20,7 @@ withFixture('basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/basic/tailwind.config.js')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/basic/tailwind.config.js')).toString(),
         range: { start: { line: 0, character: 8 }, end: { line: 0, character: 28 } },
       },
     ],
@@ -32,9 +31,7 @@ withFixture('basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/basic/does-not-exist.js')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/basic/does-not-exist.js')).toString(),
         range: { start: { line: 0, character: 8 }, end: { line: 0, character: 27 } },
       },
     ],
@@ -58,9 +55,7 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/v4/basic/tailwind.config.js')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/v4/basic/tailwind.config.js')).toString(),
         range: { start: { line: 0, character: 8 }, end: { line: 0, character: 28 } },
       },
     ],
@@ -71,9 +66,7 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/v4/basic/does-not-exist.js')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/v4/basic/does-not-exist.js')).toString(),
         range: { start: { line: 0, character: 8 }, end: { line: 0, character: 27 } },
       },
     ],
@@ -84,9 +77,7 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/v4/basic/plugin.js')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/v4/basic/plugin.js')).toString(),
         range: { start: { line: 0, character: 8 }, end: { line: 0, character: 19 } },
       },
     ],
@@ -97,9 +88,7 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/v4/basic/does-not-exist.js')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/v4/basic/does-not-exist.js')).toString(),
         range: { start: { line: 0, character: 8 }, end: { line: 0, character: 27 } },
       },
     ],
@@ -110,9 +99,7 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/v4/basic/index.html')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/v4/basic/index.html')).toString(),
         range: { start: { line: 0, character: 8 }, end: { line: 0, character: 20 } },
       },
     ],
@@ -123,9 +110,7 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/v4/basic/does-not-exist.html')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/v4/basic/does-not-exist.html')).toString(),
         range: { start: { line: 0, character: 8 }, end: { line: 0, character: 29 } },
       },
     ],
@@ -136,9 +121,7 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/v4/basic/index.html')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/v4/basic/index.html')).toString(),
         range: { start: { line: 0, character: 12 }, end: { line: 0, character: 24 } },
       },
     ],
@@ -149,9 +132,7 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path
-          .resolve('./tests/fixtures/v4/basic/does-not-exist.html')
-          .replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures/v4/basic/does-not-exist.html')).toString(),
         range: { start: { line: 0, character: 12 }, end: { line: 0, character: 33 } },
       },
     ],
@@ -177,11 +158,11 @@ withFixture('v4/basic', (c) => {
     lang: 'css',
     expected: [
       {
-        target: `file://${path.resolve('./tests/fixtures').replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures')).toString(),
         range: { start: { line: 1, character: 35 }, end: { line: 1, character: 43 } },
       },
       {
-        target: `file://${path.resolve('./tests/fixtures').replace(/@/g, '%40')}`,
+        target: URI.file(path.resolve('./tests/fixtures')).toString(),
         range: { start: { line: 2, character: 33 }, end: { line: 2, character: 41 } },
       },
     ],