diff --git a/utils/sorting.js b/utils/sorting.js index cd158b5..b2f7f83 100644 --- a/utils/sorting.js +++ b/utils/sorting.js @@ -5,20 +5,6 @@ * @property {"desc"} DESC */ -/** - * Sorts repos by the number of open PRs - * @param {UiRepo[]} repos - * @param {SortDirection} sortDirection - * @returns {UiRepo[]} - */ -export const sortByOpenPrs = (repos, sortDirection) => { - if (!sortDirection) return repos; - if (sortDirection === "asc") { - return repos.sort((a, b) => a.openPrsCount - b.openPrsCount); - } - return repos.sort((a, b) => b.openPrsCount - a.openPrsCount); -}; - /** * Sorts repos by the a numeric value * @param {UiRepo[]} repos diff --git a/utils/sorting.test.js b/utils/sorting.test.js index 78993c2..c3ae686 100644 --- a/utils/sorting.test.js +++ b/utils/sorting.test.js @@ -1,52 +1,6 @@ import { describe, it } from "node:test"; import expect from "node:assert"; -import { sortByOpenPrs, sortByNumericValue, sortByType } from "./sorting.js"; - -describe("sortByOpenPrs", () => { - it("returns the original array if sortDirection is not provided", () => { - const repos = [ - { name: "Repo 1", openPrsCount: 5 }, - { name: "Repo 2", openPrsCount: 3 }, - { name: "Repo 3", openPrsCount: 7 }, - ]; - - const result = sortByOpenPrs(repos); - - expect.deepEqual(result, repos); - }); - - it("sorts the array in ascending order by openPrsCount if sortDirection is 'asc'", () => { - const repos = [ - { name: "Repo 1", openPrsCount: 5 }, - { name: "Repo 2", openPrsCount: 3 }, - { name: "Repo 3", openPrsCount: 7 }, - ]; - - const result = sortByOpenPrs(repos, "asc"); - - expect.deepEqual(result, [ - { name: "Repo 2", openPrsCount: 3 }, - { name: "Repo 1", openPrsCount: 5 }, - { name: "Repo 3", openPrsCount: 7 }, - ]); - }); - - it("sorts the array in descending order by openPrsCount if sortDirection is 'desc'", () => { - const repos = [ - { name: "Repo 1", openPrsCount: 5 }, - { name: "Repo 2", openPrsCount: 3 }, - { name: "Repo 3", openPrsCount: 7 }, - ]; - - const result = sortByOpenPrs(repos, "desc"); - - expect.deepEqual(result, [ - { name: "Repo 3", openPrsCount: 7 }, - { name: "Repo 1", openPrsCount: 5 }, - { name: "Repo 2", openPrsCount: 3 }, - ]); - }); -}); +import { sortByNumericValue, sortByType } from "./sorting.js"; describe("sortByNumericValue", () => { it("returns the original array if sortDirection is not provided", () => {