diff --git a/package-lock.json b/package-lock.json index 3d9c3a36a..f44c954c6 100644 --- a/package-lock.json +++ b/package-lock.json @@ -21,7 +21,7 @@ "@eslint/markdown": "^7.5.1", "@types/node": "^25.5.0", "@vitest/coverage-v8": "^4.0.18", - "@vitest/eslint-plugin": "^1.6.9", + "@vitest/eslint-plugin": "^1.6.12", "eslint": "^10.0.3", "eslint-config-prettier": "^10.1.8", "eslint-plugin-package-json": "^0.91.0", @@ -1836,9 +1836,9 @@ } }, "node_modules/@vitest/eslint-plugin": { - "version": "1.6.9", - "resolved": "https://registry.npmjs.org/@vitest/eslint-plugin/-/eslint-plugin-1.6.9.tgz", - "integrity": "sha512-9WfPx1OwJ19QLCSRLkqVO7//1WcWnK3fE/3fJhKMAmDe8+9G4rB47xCNIIeCq3FdEzkIoLTfDlwDlPBaUTMhow==", + "version": "1.6.12", + "resolved": "https://registry.npmjs.org/@vitest/eslint-plugin/-/eslint-plugin-1.6.12.tgz", + "integrity": "sha512-4kI47BJNFE+EQ5bmPbHzBF+ibNzx2Fj0Jo9xhWsTPxMddlHwIWl6YAxagefh461hrwx/W0QwBZpxGS404kBXyg==", "dev": true, "license": "MIT", "dependencies": { diff --git a/package.json b/package.json index 8e63c1afe..22524c563 100644 --- a/package.json +++ b/package.json @@ -55,7 +55,7 @@ "@eslint/markdown": "^7.5.1", "@types/node": "^25.5.0", "@vitest/coverage-v8": "^4.0.18", - "@vitest/eslint-plugin": "^1.6.9", + "@vitest/eslint-plugin": "^1.6.12", "eslint": "^10.0.3", "eslint-config-prettier": "^10.1.8", "eslint-plugin-package-json": "^0.91.0", diff --git a/tests/issue-management.test.ts b/tests/issue-management.test.ts index e0223aaf7..59aeea9d2 100644 --- a/tests/issue-management.test.ts +++ b/tests/issue-management.test.ts @@ -54,7 +54,7 @@ describe("[env variable mock]", () => { test("getIssue fails gracefully on connection issues", async () => { expect.assertions(1); - await expect(getIssue()).rejects.toThrowError(IssueListError); + await expect(getIssue()).rejects.toThrow(IssueListError); }); test("getIssue fails gracefully on nonexistent repo", async () => { @@ -65,7 +65,7 @@ describe("[env variable mock]", () => { .query({ creator: "github-actions[bot]", labels: "wpvc" }) .reply(404); - await expect(getIssue()).rejects.toThrowError(IssueListError); + await expect(getIssue()).rejects.toThrow(IssueListError); }); test("commentOnIssue works correctly", async () => { @@ -85,7 +85,7 @@ describe("[env variable mock]", () => { test("commentOnIssue fails gracefully on connection issues", async () => { expect.assertions(1); - await expect(commentOnIssue(123, "ISSUE_BODY")).rejects.toThrowError( + await expect(commentOnIssue(123, "ISSUE_BODY")).rejects.toThrow( IssueCommentError, ); }); @@ -101,7 +101,7 @@ describe("[env variable mock]", () => { }) .reply(404); - await expect(commentOnIssue(123, issueBody)).rejects.toThrowError( + await expect(commentOnIssue(123, issueBody)).rejects.toThrow( IssueCommentError, ); expect(scope.isDone()).toBe(true); @@ -122,7 +122,7 @@ describe("[env variable mock]", () => { test("closeIssue fails gracefully on connection issues", async () => { expect.assertions(1); - await expect(closeIssue(123)).rejects.toThrowError(IssueUpdateError); + await expect(closeIssue(123)).rejects.toThrow(IssueUpdateError); }); test("closeIssue fails gracefully on nonexistent repo", async () => { @@ -134,7 +134,7 @@ describe("[env variable mock]", () => { }) .reply(404); - await expect(closeIssue(123)).rejects.toThrowError(IssueUpdateError); + await expect(closeIssue(123)).rejects.toThrow(IssueUpdateError); expect(scope.isDone()).toBe(true); }); @@ -181,9 +181,9 @@ describe("[env variable mock]", () => { test("createIssue fails gracefully on connection issues", async () => { expect.assertions(1); - await expect( - createIssue("ISSUE_TITLE", "ISSUE_BODY", []), - ).rejects.toThrowError(IssueCreationError); + await expect(createIssue("ISSUE_TITLE", "ISSUE_BODY", [])).rejects.toThrow( + IssueCreationError, + ); }); test("createIssue fails gracefully on nonexistent repo", async () => { @@ -191,9 +191,9 @@ describe("[env variable mock]", () => { nock("https://api.github.com").post("/repos/OWNER/REPO/issues").reply(404); - await expect( - createIssue("ISSUE_TITLE", "ISSUE_BODY", []), - ).rejects.toThrowError(IssueCreationError); + await expect(createIssue("ISSUE_TITLE", "ISSUE_BODY", [])).rejects.toThrow( + IssueCreationError, + ); }); test("updateIssue works correctly with an up-to-date-issue", async () => { @@ -254,9 +254,7 @@ describe("[env variable mock]", () => { .patch("/repos/OWNER/REPO/issues/123", { body, title }) .reply(200); - await expect(updateIssue(123, title, body)).rejects.toThrowError( - GetIssueError, - ); + await expect(updateIssue(123, title, body)).rejects.toThrow(GetIssueError); }); test("updateIssue fails gracefully on connection issues on updating the existing issue", async () => { @@ -269,7 +267,7 @@ describe("[env variable mock]", () => { .get("/repos/OWNER/REPO/issues/123") .reply(200, { body, title: "WRONG_TITLE" }); - await expect(updateIssue(123, title, body)).rejects.toThrowError( + await expect(updateIssue(123, title, body)).rejects.toThrow( IssueUpdateError, ); expect(scope.isDone()).toBe(true); @@ -284,8 +282,8 @@ describe("[env variable mock]", () => { .patch("/repos/OWNER/REPO/issues/123") .reply(404); - await expect( - updateIssue(123, "ISSUE_TITLE", "ISSUE_BODY"), - ).rejects.toThrowError(GetIssueError); + await expect(updateIssue(123, "ISSUE_TITLE", "ISSUE_BODY")).rejects.toThrow( + GetIssueError, + ); }); }); diff --git a/tests/tested-version.test.ts b/tests/tested-version.test.ts index 4e40d21e4..e13d3456f 100644 --- a/tests/tested-version.test.ts +++ b/tests/tested-version.test.ts @@ -63,9 +63,7 @@ describe("[env variable mock]", () => { readme: ["path/to/readme.txt"], }; - await expect(testedVersion(config)).rejects.toThrowError( - InvalidReadmeError, - ); + await expect(testedVersion(config)).rejects.toThrow(InvalidReadmeError); }); test("testedVersion fails gracefully on no readme", async () => { @@ -82,9 +80,7 @@ describe("[env variable mock]", () => { .get(`/repos/OWNER/REPO/contents/${encodeURIComponent(readmePath)}`) .reply(404); - await expect(testedVersion(config)).rejects.toThrowError( - InvalidReadmeError, - ); + await expect(testedVersion(config)).rejects.toThrow(InvalidReadmeError); }); test("testedVersion fails gracefully on invalid response", async () => { @@ -101,9 +97,7 @@ describe("[env variable mock]", () => { .get(`/repos/OWNER/REPO/contents/${encodeURIComponent(readmePath)}`) .reply(200); - await expect(testedVersion(config)).rejects.toThrowError( - InvalidReadmeError, - ); + await expect(testedVersion(config)).rejects.toThrow(InvalidReadmeError); }); test("testedVersion fails gracefully on invalid response 2", async () => { @@ -122,9 +116,7 @@ describe("[env variable mock]", () => { content: "OOPS", }); - await expect(testedVersion(config)).rejects.toThrowError( - InvalidReadmeError, - ); + await expect(testedVersion(config)).rejects.toThrow(InvalidReadmeError); }); test.each([ @@ -150,9 +142,7 @@ describe("[env variable mock]", () => { content: Buffer.from(readme).toString("base64"), }); - await expect(testedVersion(config)).rejects.toThrowError( - InvalidReadmeError, - ); + await expect(testedVersion(config)).rejects.toThrow(InvalidReadmeError); }); test("testedVersion works correctly with one invalid and one valid readme", async () => { diff --git a/tests/wordpress-versions.test.ts b/tests/wordpress-versions.test.ts index 7e71bdb73..f0d76a47b 100644 --- a/tests/wordpress-versions.test.ts +++ b/tests/wordpress-versions.test.ts @@ -51,7 +51,7 @@ test("wordpressVersions works correctly when stable, RC, and beta versions are a test("wordpressVersions fails gracefully on connection issues", async () => { expect.assertions(1); - await expect(wordpressVersions()).rejects.toThrowError(LatestVersionError); + await expect(wordpressVersions()).rejects.toThrow(LatestVersionError); }); test("wordpressVersions fails gracefully on connection issues 2", async () => { @@ -61,7 +61,7 @@ test("wordpressVersions fails gracefully on connection issues 2", async () => { .get("/core/version-check/1.7/?channel=beta") .reply(404); - await expect(wordpressVersions()).rejects.toThrowError(LatestVersionError); + await expect(wordpressVersions()).rejects.toThrow(LatestVersionError); }); test("wordpressVersions fails gracefully on invalid response", async () => { @@ -71,7 +71,7 @@ test("wordpressVersions fails gracefully on invalid response", async () => { .get("/core/version-check/1.7/?channel=beta") .reply(200); - await expect(wordpressVersions()).rejects.toThrowError(LatestVersionError); + await expect(wordpressVersions()).rejects.toThrow(LatestVersionError); }); test("wordpressVersions fails gracefully on invalid response 2", async () => { @@ -83,7 +83,7 @@ test("wordpressVersions fails gracefully on invalid response 2", async () => { translations: [], }); - await expect(wordpressVersions()).rejects.toThrowError(LatestVersionError); + await expect(wordpressVersions()).rejects.toThrow(LatestVersionError); }); test("wordpressVersions fails gracefully on invalid response 3", async () => { @@ -96,7 +96,7 @@ test("wordpressVersions fails gracefully on invalid response 3", async () => { translations: [], }); - await expect(wordpressVersions()).rejects.toThrowError(LatestVersionError); + await expect(wordpressVersions()).rejects.toThrow(LatestVersionError); }); test("wordpressVersions fails gracefully on invalid response 4", async () => { @@ -113,7 +113,7 @@ test("wordpressVersions fails gracefully on invalid response 4", async () => { translations: [], }); - await expect(wordpressVersions()).rejects.toThrowError(LatestVersionError); + await expect(wordpressVersions()).rejects.toThrow(LatestVersionError); }); test("wordpressVersions fails gracefully on invalid response 5", async () => { @@ -130,5 +130,5 @@ test("wordpressVersions fails gracefully on invalid response 5", async () => { translations: [], }); - await expect(wordpressVersions()).rejects.toThrowError(LatestVersionError); + await expect(wordpressVersions()).rejects.toThrow(LatestVersionError); }); diff --git a/tests/wpvc-config.test.ts b/tests/wpvc-config.test.ts index d9b7046eb..6069804f1 100644 --- a/tests/wpvc-config.test.ts +++ b/tests/wpvc-config.test.ts @@ -84,7 +84,7 @@ describe("Mocked env variables", () => { test("getWPVCConfig fails gracefully on connection issues", async () => { expect.assertions(1); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig returns defaults on no config", async () => { @@ -115,7 +115,7 @@ describe("Mocked env variables", () => { .get("/repos/OWNER/REPO/contents/.wordpress-version-checker.json") .reply(200); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig fails gracefully on invalid response 2", async () => { @@ -127,7 +127,7 @@ describe("Mocked env variables", () => { content: "OOPS", }); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig fails gracefully on invalid config", async () => { @@ -139,7 +139,7 @@ describe("Mocked env variables", () => { content: Buffer.from(JSON.stringify(false)).toString("base64"), }); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig fails gracefully on invalid config 2", async () => { @@ -155,7 +155,7 @@ describe("Mocked env variables", () => { content: Buffer.from(JSON.stringify(config)).toString("base64"), }); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig fails gracefully on invalid config 3", async () => { @@ -171,7 +171,7 @@ describe("Mocked env variables", () => { content: Buffer.from(JSON.stringify(config)).toString("base64"), }); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig fails gracefully on invalid config 4", async () => { @@ -187,7 +187,7 @@ describe("Mocked env variables", () => { content: Buffer.from(JSON.stringify(config)).toString("base64"), }); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig fails gracefully on invalid config 5", async () => { @@ -203,7 +203,7 @@ describe("Mocked env variables", () => { content: Buffer.from(JSON.stringify(config)).toString("base64"), }); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig fails gracefully on invalid config 6", async () => { @@ -219,7 +219,7 @@ describe("Mocked env variables", () => { content: Buffer.from(JSON.stringify(config)).toString("base64"), }); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); test("getWPVCConfig fails gracefully on invalid config 7", async () => { @@ -235,6 +235,6 @@ describe("Mocked env variables", () => { content: Buffer.from(JSON.stringify(config)).toString("base64"), }); - await expect(getWPVCConfig()).rejects.toThrowError(ConfigError); + await expect(getWPVCConfig()).rejects.toThrow(ConfigError); }); });