From 598ddc17fa2f766c4b81a3c24d7fd40edf18b877 Mon Sep 17 00:00:00 2001 From: Reinier Criel Date: Sat, 25 Oct 2025 14:14:36 -0700 Subject: [PATCH] Fix linting issue --- .../src/registryProxy/parsePackageFromUrl.js | 2 -- .../src/shell-integration/helpers.spec.js | 18 ------------------ 2 files changed, 20 deletions(-) diff --git a/packages/safe-chain/src/registryProxy/parsePackageFromUrl.js b/packages/safe-chain/src/registryProxy/parsePackageFromUrl.js index 4061578..4b440af 100644 --- a/packages/safe-chain/src/registryProxy/parsePackageFromUrl.js +++ b/packages/safe-chain/src/registryProxy/parsePackageFromUrl.js @@ -1,5 +1,3 @@ -import { parse } from "semver"; - export const knownJsRegistries = ["registry.npmjs.org","registry.yarnpkg.com"]; export const knownPipRegistries = ["files.pythonhosted.org", "pypi.org", "pypi.python.org", "pythonhosted.org"]; diff --git a/packages/safe-chain/src/shell-integration/helpers.spec.js b/packages/safe-chain/src/shell-integration/helpers.spec.js index d7013db..4f18c36 100644 --- a/packages/safe-chain/src/shell-integration/helpers.spec.js +++ b/packages/safe-chain/src/shell-integration/helpers.spec.js @@ -181,22 +181,4 @@ describe("removeLinesMatchingPatternTests", () => { const resultLines = result.split("\n"); assert.strictEqual(resultLines.length, 5, "Should have exactly 5 lines"); }); - - it("should include pip in knownAikidoTools and in the package manager list", async () => { - // Import helpers after setting up the mock - const { knownAikidoTools, getPackageManagerList } = await import("./helpers.js"); - - // Verify pip tool - const hasPip = knownAikidoTools.some( - (t) => t.tool === "pip" && t.aikidoCommand === "aikido-pip" - ); - assert.ok(hasPip, "knownAikidoTools should include pip"); - - // Verify pip appears in the human-readable list - const list = getPackageManagerList(); - assert.ok( - /(^|[,\s])pip(,|\s| and)/.test(list) && /commands$/.test(list), - `getPackageManagerList should include 'pip' (actual: ${list})` - ); - }); });