test: add dotenv configuration to test files and remove unused contributions mock

Author: O2sa

lib/github.test.ts

@@ -1,3 +1,6 @@
+import "dotenv/config";
+
+
 import {describe, expect, it} from "vitest";
 import {parseCountEnv} from "./github";
 

test/github/github-cache.test.ts

@@ -1,3 +1,5 @@
+import "dotenv/config";
+
 import { afterEach, describe, expect, test, vi } from "vitest";
 import {
   createGitHubUserDataFetcher,
@@ -19,10 +21,7 @@ function makeExecutor(
   delayMs = 0,
 ): GitHubFetcherDependencies["executor"] {
   return {
-    async execute<
-      TData,
-      TVariables extends Record<string, unknown>,
-    >(params: {
+    async execute<TData, TVariables extends Record<string, unknown>>(params: {
       operationName: string;
       query: string;
       variables: TVariables;
@@ -174,8 +173,7 @@ function isGitHubUserData(value: unknown): value is GitHubUserData {
     value !== null &&
     "avatarUrl" in value &&
     "repos" in value &&
-    "pullRequests" in value &&
-    "contributions" in value
+    "pullRequests" in value
   );
 }
 
@@ -186,14 +184,11 @@ describe("GitHub user data caching", () => {
   test("cache hit skips GitHub calls", async () => {
     const cachedPayload: GitHubUserData = {
       name: "Cached",
+      location: "any",
+      login: "Cached",
       avatarUrl: "https://example.com/cached.png",
       repos: [],
       pullRequests: [],
-      contributions: {
-        totalCommitContributions: 0,
-        totalPullRequestContributions: 0,
-        totalIssueContributions: 0,
-      },
       issues: [],
       discussions: [],
     };
@@ -266,10 +261,16 @@ describe("GitHub user data caching", () => {
     vi.stubEnv("GITHUB_ISSUE_COUNT", "2");
     vi.stubEnv("GITHUB_DISCUSSION_COUNT", "2");
 
-    const calls: Array<{ operationName: string; variables: Record<string, unknown> }> = [];
+    const calls: Array<{
+      operationName: string;
+      variables: Record<string, unknown>;
+    }> = [];
     const fetcher = createGitHubUserDataFetcher({
       executor: {
-        async execute<TData, TVariables extends Record<string, unknown>>(params: {
+        async execute<
+          TData,
+          TVariables extends Record<string, unknown>,
+        >(params: {
           operationName: string;
           query: string;
           variables: TVariables;
@@ -416,8 +417,13 @@ describe("GitHub user data caching", () => {
     const result = await fetcher("testuser");
 
     expect(result.pullRequests).toHaveLength(2);
-    expect(calls.filter((call) => call.operationName === "FetchUserPullRequests")).toHaveLength(1);
-    expect(calls.find((call) => call.operationName === "FetchUserPullRequests")?.variables.prCount).toBe(2);
+    expect(
+      calls.filter((call) => call.operationName === "FetchUserPullRequests"),
+    ).toHaveLength(1);
+    expect(
+      calls.find((call) => call.operationName === "FetchUserPullRequests")
+        ?.variables.prCount,
+    ).toBe(2);
   });
 
   test("cache write failure does not fail request", async () => {

test/github/github-graphql-client.test.ts

@@ -1,3 +1,5 @@
+import "dotenv/config";
+
 import { describe, expect, test } from "vitest";
 import {
   GitHubApiError,

test/scoring/calculateUserScore.contribution.test.ts

@@ -2,7 +2,6 @@ import { describe, expect, test } from "vitest";
 
 import { calculateUserScore } from "@/lib/score";
 import {
-  makeContributions,
   makeDiscussion,
   makeIssue,
   makePullRequest,
@@ -17,11 +16,6 @@ describe("calculateUserScore - contribution scoring", () => {
       makeUserScoreInput({
         repos: [],
         pullRequests: [],
-        contributions: makeContributions({
-          totalCommitContributions: 10_000,
-          totalPullRequestContributions: 0,
-          totalIssueContributions: 0,
-        }),
       }),
       "octocat",
     );
@@ -34,11 +28,6 @@ describe("calculateUserScore - contribution scoring", () => {
       makeUserScoreInput({
         repos: [],
         pullRequests: [],
-        contributions: makeContributions({
-          totalCommitContributions: 0,
-          totalPullRequestContributions: 10_000,
-          totalIssueContributions: 0,
-        }),
       }),
       "octocat",
     );

test/scoring/calculateUserScore.language.test.ts

@@ -3,7 +3,6 @@ import { describe, expect, test } from "vitest";
 import { calculateUserScore } from "@/lib/score";
 import { getLanguageFactor } from "@/lib/scoring/languageScoring";
 import {
-  makeContributions,
   makePullRequest,
   makeRepo,
   makeRepoLanguages,
@@ -313,7 +312,6 @@ describe("calculateUserScore - language scoring", () => {
             },
           }),
         ],
-        contributions: makeContributions(),
         selectedLanguages: ["TypeScript"],
       }),
       "octocat",

test/scoring/calculateUserScore.repo.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, test } from "vitest";
 
 import { calculateUserScore } from "@/lib/score";
-import { makeContributions, makeRepo, makeUserScoreInput } from "@/test/fixtures/github";
+import {  makeRepo, makeUserScoreInput } from "@/test/fixtures/github";
 import { expectedRepoScore, sumRepoScores } from "@/test/helpers/score";
 
 describe("calculateUserScore - repository scoring", () => {
@@ -10,7 +10,6 @@ describe("calculateUserScore - repository scoring", () => {
       {
         repos: [],
         pullRequests: [],
-        contributions: makeContributions(),
       },
       "octocat",
     );

test/scoring/calculateUserScore.scenario.test.ts

@@ -2,7 +2,6 @@ import { describe, expect, test } from "vitest";
 
 import { calculateUserScore } from "@/lib/score";
 import {
-  makeContributions,
   makeIssue,
   makePullRequest,
   makeRepo,
@@ -95,7 +94,6 @@ describe("calculateUserScore - final score behavior", () => {
       {
         repos: [],
         pullRequests: [],
-        contributions: makeContributions(),
       },
       "ghost",
     );