Migrate chai and sinon to vitest

.eslintrc

@@ -194,8 +194,11 @@
     {
       // Specific rules for test files
       files: [ "packages/**/test/**/*-test.js" ],
+      plugins: [ "eslint-plugin-promise" ],
       rules: {
-        "import/no-extraneous-dependencies": "off"
+        "import/no-extraneous-dependencies": "off",
+        "promise/prefer-await-to-then": 2,
+        "promise/prefer-await-to-callbacks": 2
       }
     },
     {

package.json

@@ -14,21 +14,19 @@
     "@types/request": "^2.48.13",
     "@typescript-eslint/eslint-plugin": "^5.62.0",
     "@typescript-eslint/parser": "^5.62.0",
-    "chai": "^4.0.0",
+    "@vitest/coverage-v8": "^4.1.10",
     "coveralls": "^3.0.9",
     "eslint": "^7.0.0",
     "eslint-plugin-import": "^2.22.0",
+    "eslint-plugin-promise": "^7.3.0",
     "lerna": "^4.0.0",
     "manual-git-changelog": "^1.0.1",
-    "mocha": "^8.0.0",
     "node-gyp": "^11.0.0",
-    "nyc": "^15.0.0",
     "pre-commit": "^1.1.3",
     "rdf-object": "^1.14.0",
-    "sinon": "^1.17.4",
-    "sinon-chai": "^2.14.0",
     "supertest": "^6.0.0",
-    "typescript": "^5.9.3"
+    "typescript": "^5.9.3",
+    "vitest": "^4.1.10"
   },
   "pre-commit": [
     "lint",
@@ -37,12 +35,11 @@
   "scripts": {
     "test-changed": "lerna run test --since HEAD",
     "lint-changed": "lerna run lint --since HEAD",
-    "mocha": "mocha \"packages/*/test/**/*-test.js\" --recursive --require ./test/test-setup --timeout 500",
-    "test": "nyc npm run mocha",
-    "test-ci": "nyc --reporter=lcov npm run mocha",
+    "test": "vitest run --coverage",
+    "test-ci": "vitest run --coverage",
     "typecheck": "tsc -p tsconfig.json --noEmit",
     "build": "tsc -p tsconfig.json",
-    "lint": "eslint packages/*/bin/* packages/*/lib packages/*/test --ext .js,.ts",
+    "lint": "eslint packages/*/bin/* packages/*/lib packages/*/test test --ext .js,.ts",
     "clean": "rm -rf ./node_modules && rm -rf ./packages/*/node_modules",
     "dedupe": "npx yarn-deduplicate yarn.lock --scopes",
     "publish": "lerna publish",

packages/core/lib/CliRunner.ts

@@ -16,9 +16,9 @@ import type { WorkerConfig } from './types';
 type Writable = { write(chunk: string): void };
 
 // Run function for starting the server from the command line
-export function runCli(moduleRootPath: string): void {
+export function runCli(moduleRootPath: string): Promise<void> {
   let argv = process.argv.slice(2);
-  runCustom(argv, process.stdin, process.stdout, process.stderr, null, { mainModulePath: moduleRootPath });
+  return runCustom(argv, process.stdin, process.stdout, process.stderr, null, { mainModulePath: moduleRootPath });
 }
 
 // Generic run function for starting the server from a given config
@@ -29,7 +29,7 @@ export function runCustom(
   stderr: Writable,
   componentConfigUri: string | null,
   properties: Record<string, any>,
-): void {
+): Promise<void> {
   if (args.length < 1 || args.length > 4 || /^--?h(elp)?$/.test(args[0])) {
     stdout.write('usage: server config.json [port [workers [componentConfigUri]]]\n');
     return process.exit(1);
@@ -39,7 +39,7 @@ export function runCustom(
       cliWorkers = parseInt(args[2], 10),
       configUri = args[3] || componentConfigUri || 'urn:ldf-server:my';
 
-  ComponentsManager.build<LinkedDataFragmentsServerWorker>({
+  return ComponentsManager.build<LinkedDataFragmentsServerWorker>({
     ...properties,
     configLoader: (registry: ConfigRegistry) => registry.register(args[0]),
   } as IComponentsManagerBuilderOptions<LinkedDataFragmentsServerWorker>)

packages/core/lib/UrlData.ts

@@ -23,7 +23,7 @@ export class UrlData {
     options = options || {};
     this.baseURL = (options.baseURL || '/').replace(/\/?$/, '/');
     this.baseURLRoot = this.baseURL.match(/^(?:https?:\/\/[^\/]+)?/)![0];
-    this.baseURLPath = this.baseURL.substr(this.baseURLRoot.length);
+    this.baseURLPath = this.baseURL.slice(this.baseURLRoot.length);
     this.blankNodePath = this.baseURLRoot ? '/.well-known/genid/' : '';
     this.blankNodePrefix = this.blankNodePath ? this.baseURLRoot + this.blankNodePath : 'genid:';
     this.blankNodePrefixLength = this.blankNodePrefix.length;

packages/core/lib/datasources/Datasource.ts

@@ -151,11 +151,11 @@ export class Datasource extends EventEmitter {
     // Translate blank nodes IRIs in the query to blank nodes
     let blankNodePrefix = this.urlData.blankNodePrefix, blankNodePrefixLength = this.urlData.blankNodePrefixLength;
     if (query.subject && query.subject.termType === 'NamedNode' && query.subject.value.indexOf(blankNodePrefix) === 0)
-      query.subject = this.dataFactory.blankNode(query.subject.value.substr(blankNodePrefixLength));
+      query.subject = this.dataFactory.blankNode(query.subject.value.slice(blankNodePrefixLength));
     if (query.object && query.object.termType === 'NamedNode'  && query.object.value.indexOf(blankNodePrefix) === 0)
-      query.object  = this.dataFactory.blankNode(query.object.value.substr(blankNodePrefixLength));
+      query.object  = this.dataFactory.blankNode(query.object.value.slice(blankNodePrefixLength));
     if (query.graph && query.graph.termType === 'NamedNode'   && query.graph.value.indexOf(blankNodePrefix) === 0)
-      query.graph   = this.dataFactory.blankNode(query.graph.value.substr(blankNodePrefixLength));
+      query.graph   = this.dataFactory.blankNode(query.graph.value.slice(blankNodePrefixLength));
 
     // Force the default graph if QPF support is disable
     if (!this._supportsQuads)
@@ -214,7 +214,7 @@ export class Datasource extends EventEmitter {
       break;
     // Read a file from the local filesystem
     case 'file':
-      stream = fs.createReadStream(url.substr(protocolMatch[0].length), { encoding: 'utf8' });
+      stream = fs.createReadStream(url.slice(protocolMatch[0].length), { encoding: 'utf8' });
       break;
     default:
       stream = new EventEmitter();

packages/core/lib/routers/DatasourceRouter.ts

@@ -17,7 +17,7 @@ export class DatasourceRouter {
   extractQueryParams(request: RouterRequest, query: Query): void {
     (query.features || (query.features = {})).datasource = true;
     let path = request.url && request.url.pathname || '/';
-    query.datasource = path.substr(this._baseLength);
+    query.datasource = path.slice(this._baseLength);
   }
 }
 

packages/core/package.json

@@ -24,9 +24,6 @@
     "test": "nyc mocha",
     "lint": "eslint bin/* lib test"
   },
-  "engines": {
-    "node": ">=10.0"
-  },
   "dependencies": {
     "asynciterator": "^3.0.0",
     "componentsjs": "^4.0.4",