feat: fix tests and move to all native test

Author: spence-s

.github/workflows/node.js.yml

@@ -19,9 +19,9 @@ jobs:
     name: Node ${{ matrix.node-version }}
     steps:
       - uses: actions/checkout@v4
-      - name: Use Node.js ${{ matrix.node-version }}
+      - name: Setup Node
         uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
       - run: npm install
-      - run: npm run test
+      - run: npm test

.gitignore

@@ -1,6 +1,5 @@
 node_modules
 dist
 *.vsix
-.tsimp
 
 coverage

.travis.yml

@@ -16,6 +16,12 @@
 			"rules": {
 				"@typescript-eslint/consistent-type-definitions": ["error", "interface"]
 			}
+		},
+		{
+			"files": "test/**/*.ts",
+			"rules": {
+				"@typescript-eslint/no-floating-promises": "off"
+			}
 		}
 	]
 }

.xo-config.json

@@ -43,8 +43,8 @@
 		"lint": "xo && npm run lint:md",
 		"lint:md": "prettier --parser markdown '**/*.md' --check",
 		"prepare": "husky install",
-		"test:": "rimraf .tsimp && NODE_NO_WARNINGS=1 node --import tsimp/import --test test/native/**/*.ts",
-		"test:coverage": "rimraf .tsimp && NODE_NO_WARNINGS=1 c8 node --import tsimp/import --test test/native/**/*.ts",
+		"test": "NODE_NO_WARNINGS=1 node --require tsx/cjs --test test/index.ts",
+		"test:coverage": "NODE_NO_WARNINGS=1 c8 node --require tsx/cjs --test test/index.ts",
 		"vscode:prepublish": "npm run check && npm run build"
 	},
 	"contributes": {
@@ -212,7 +212,7 @@
 		"prettier": "^3.1.1",
 		"prettier-plugin-packagejson": "^2.4.9",
 		"rimraf": "^5.0.5",
-		"tsimp": "^2.0.10",
+		"tsx": "^4.7.0",
 		"typescript": "^5.3.3",
 		"webfont": "^6.0.2",
 		"xo": "^0.56.0"

package-lock.json

@@ -1,15 +1,7 @@
-/* eslint-disable @typescript-eslint/no-floating-promises */
-import {test, describe} from 'node:test';
-import assert from 'node:assert';
-import Server from '../server/server.js';
+/* eslint-disable import/no-unassigned-import */
+// since globs are not fully supported in node v18 and v20 we import the files manually here
 
-describe('Server', () => {
-	test('Server is a function', (t) => {
-		assert.strictEqual(typeof Server, 'function');
-	});
-
-	test('Server can instantiate', (t) => {
-		const server = new Server({isTest: true});
-		assert.strictEqual(typeof server, 'object');
-	});
-});
+// TODO: remove this file once node v21 is LTS
+import './server.js';
+import './lsp/document-sync.js';
+import './lsp/initialization.js';

package.json

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-floating-promises */
 import {test, describe, mock, type Mock} from 'node:test';
 import assert from 'node:assert';
 import {TextDocument} from 'vscode-languageserver-textdocument';

test/index.ts

@@ -1,16 +1,15 @@
-/* eslint-disable @typescript-eslint/no-floating-promises */
 import {describe, test} from 'node:test';
 import assert from 'node:assert';
 import Server from '../../server/server.js';
 
-const server = new Server({isTest: true});
+describe('Server.handleInitialize', async () => {
+	const server = new Server({isTest: true});
 
-describe('Server.handleInitialize', () => {
-	test('Server.handleInitialize is a function', (t) => {
+	await test('Server.handleInitialize is a function', (t) => {
 		assert.equal(typeof server.handleInitialize, 'function');
 	});
 
-	test('InitializeResult matches snapshot', async (t) => {
+	await test('InitializeResult matches snapshot', async (t) => {
 		const result = await server.handleInitialize();
 		assert.deepEqual(result, {
 			capabilities: {

test/lsp/document-sync.ts

@@ -0,0 +1,14 @@
+import {test, describe} from 'node:test';
+import assert from 'node:assert';
+import Server from '../server/server.js';
+
+describe('Server', () => {
+	test('Server is a function', (t) => {
+		assert.strictEqual(typeof Server, 'function');
+	});
+
+	test('Server can instantiate', (t) => {
+		const server = new Server({isTest: true});
+		assert.strictEqual(typeof server, 'object');
+	});
+});