@flint.fyi/rule-tester 0.20.0 → 0.20.1

package/dist/index.mjs

@@ -2,7 +2,7 @@ import assert from "node:assert/strict";
 import { isDeepStrictEqual } from "node:util";
 import { CachedFactory } from "cached-factory";
 import { resolve } from "pathe";
-import { applyChangesToText, createDiskBackedLinterHost, createEphemeralLinterHost, createVFSLinterHost, formatReport, getPositionOfColumnAndLine, isSuggestionForFiles, parseOptions, processRuleReport, withRepositoryRoot } from "@flint.fyi/core";
+import { applyChangesToText, createDiskBackedLinterHost, createEphemeralLinterHost, createVFSLinterHost, formatReport, getPositionOfColumnAndLine, isFileSystemCaseSensitive, isSuggestionForFiles, parseOptions, processRuleReport, withRepositoryRoot } from "@flint.fyi/core";
 import { isTruthy, normalizePath, nullThrows, pathKey } from "@flint.fyi/utils";
 //#region src/createOutput.ts
 function createOutput(reports, testCaseNormalized) {
@@ -115,7 +115,7 @@ function resolveReportedSuggestions(reports, testCaseNormalized) {
 function resolveReportedSuggestionForFiles(suggestionReported, suggestionExpected) {
 	if (!suggestionExpected) return {};
 	if (!isTestSuggestionForFiles(suggestionExpected)) throw new Error("This test case describes a suggestion to its own file, but the rule is reporting changes across files.");
-	assert.deepStrictEqual(Object.keys(suggestionReported.files).sort(), Object.keys(suggestionExpected.files).sort(), "Reported suggestion target paths must exactly match expected target paths.");
+	assert.deepStrictEqual(Object.keys(suggestionReported.files).toSorted(), Object.keys(suggestionExpected.files).toSorted(), "Reported suggestion target paths must exactly match expected target paths.");
 	return Object.fromEntries(Object.entries(suggestionExpected.files).map(([filePath, suggestionCasesExpected]) => {
 		const changes = suggestionReported.files[filePath];
 		assert.ok(changes, `Expected reported suggestion "${suggestionReported.id}" to provide changes for target "${filePath}".`);
@@ -163,7 +163,7 @@ async function runTestCaseRule(fileFactories, linterHost, { options, rule }, { c
 		await ruleRuntime.teardown?.();
 	}
 	return {
-		languageReports: collectLanguageReports ? rule.language.getLanguageReports?.(file) ?? [] : [],
+		languageReports: collectLanguageReports ? rule.language.getLanguageReports?.(file, linterHost) ?? [] : [],
 		reports: reports.toSorted((a, b) => a.range.begin.raw - b.range.begin.raw || a.range.end.raw - b.range.end.raw)
 	};
 }
@@ -175,17 +175,23 @@ var RuleTester = class {
 	#testerOptions;
 	constructor({ assertNoLanguageReports = true, defaults = {}, describe, diskBackedFSRoot, it, only, scope = globalThis, skip } = {}) {
 		const virtualRoot = diskBackedFSRoot == null ? void 0 : resolve(process.cwd(), diskBackedFSRoot, "_flint-rule-tester-virtual");
-		let baseHost = virtualRoot != null ? createEphemeralLinterHost(withRepositoryRoot(createDiskBackedLinterHost(virtualRoot), virtualRoot)) : void 0;
+		let baseHost = virtualRoot == null ? void 0 : createEphemeralLinterHost(withRepositoryRoot(createDiskBackedLinterHost(virtualRoot), virtualRoot));
 		const { files: defaultFiles = {} } = defaults;
 		if (Object.keys(defaultFiles).length) {
-			const vfs = createVFSLinterHost(baseHost == null ? { cwd: process.cwd() } : { baseHost });
+			const vfs = createVFSLinterHost(baseHost == null ? {
+				caseSensitive: isFileSystemCaseSensitive(),
+				cwd: process.cwd()
+			} : { baseHost });
 			for (const [name, content] of Object.entries(defaultFiles)) {
 				const filePath = resolve(vfs.getCurrentDirectory(), name);
 				vfs.vfsUpsertFile(filePath, content);
 			}
 			baseHost = vfs;
 		}
-		this.#linterHost = createVFSLinterHost(baseHost == null ? { cwd: process.cwd() } : { baseHost });
+		this.#linterHost = createVFSLinterHost(baseHost == null ? {
+			caseSensitive: isFileSystemCaseSensitive(),
+			cwd: process.cwd()
+		} : { baseHost });
 		this.#fileFactories = new CachedFactory((language) => language.createFileFactory(this.#linterHost));
 		it = defaultTo(it, scope, "it");
 		if (!skip && "skip" in it && typeof it.skip === "function") skip = it.skip;
@@ -232,10 +238,7 @@ var RuleTester = class {
 	}
 	#itTestCase(testCase, seenTestCases, setup) {
 		let test = testCase.only ? this.#testerOptions.only : this.#testerOptions.it;
-		if (testCase.skip) {
-			if ("skip" in test && typeof test.skip === "function") test = test.skip;
-			else test = this.#testerOptions.skip;
-		}
+		if (testCase.skip) test = "skip" in test && typeof test.skip === "function" ? test.skip : this.#testerOptions.skip;
 		test(testCase.name ?? ("files" in testCase ? JSON.stringify({
 			[testCase.fileName]: testCase.code,
 			...testCase.files

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flint.fyi/rule-tester",
-  "version": "0.20.0",
+  "version": "0.20.1",
   "description": "[Experimental] Rule unit tests for Flint.",
   "homepage": "https://flint.fyi",
   "repository": {
@@ -16,14 +16,15 @@
   "sideEffects": false,
   "type": "module",
   "exports": {
-    ".": "./dist/index.mjs"
+    ".": "./dist/index.mjs",
+    "./package.json": "./package.json"
   },
   "files": [
     "dist/"
   ],
   "dependencies": {
-    "@flint.fyi/core": "^0.27.0",
-    "@flint.fyi/utils": "^0.16.0",
+    "@flint.fyi/core": "^0.28.0",
+    "@flint.fyi/utils": "^0.16.1",
     "cached-factory": "^0.3.0",
     "pathe": "^2.0.3"
   },