mirror of
https://github.com/anuraghazra/github-readme-stats.git
synced 2024-12-27 06:25:47 +08:00
60fae292a3
* feat: enable multi-page stars' fetching for private vercel instances This commit enables multi-page stars' support from fetching on private Vercel instances. This feature can be disabled on the public Vercel instance by adding the `FETCH_SINGLE_PAGE_STARS=true` as an env variable in the public Vercel instance. This variable will not be present when people deploy their own Vercel instance, causing the code to fetch multiple star pages. * fix: improve stats multi-page fetching behavoir This commit makes sure that the GraphQL api is only called one time per 100 repositories. The old method added one unnecesairy GraphQL call. * docs: update documentation * style: improve code syntax Co-authored-by: Matteo Pierro <pierromatteo@gmail.com> * lol happy new year * docs: remove rate limit documentation for now Remove the `FETCH_SINGLE_PAGE_STARS` from documentation for now since it might confuse people. * fix: fix error in automatic merge * feat: make sure env variable is read Co-authored-by: Matteo Pierro <pierromatteo@gmail.com> Co-authored-by: Anurag <hazru.anurag@gmail.com>
315 lines
7.5 KiB
JavaScript
315 lines
7.5 KiB
JavaScript
import "@testing-library/jest-dom";
|
|
import axios from "axios";
|
|
import MockAdapter from "axios-mock-adapter";
|
|
import { calculateRank } from "../src/calculateRank.js";
|
|
import { fetchStats } from "../src/fetchers/stats-fetcher.js";
|
|
|
|
// Test parameters.
|
|
const data_stats = {
|
|
data: {
|
|
user: {
|
|
name: "Anurag Hazra",
|
|
repositoriesContributedTo: { totalCount: 61 },
|
|
contributionsCollection: {
|
|
totalCommitContributions: 100,
|
|
restrictedContributionsCount: 50,
|
|
},
|
|
pullRequests: { totalCount: 300 },
|
|
openIssues: { totalCount: 100 },
|
|
closedIssues: { totalCount: 100 },
|
|
followers: { totalCount: 100 },
|
|
repositories: {
|
|
totalCount: 5,
|
|
nodes: [
|
|
{ name: "test-repo-1", stargazers: { totalCount: 100 } },
|
|
{ name: "test-repo-2", stargazers: { totalCount: 100 } },
|
|
{ name: "test-repo-3", stargazers: { totalCount: 100 } },
|
|
],
|
|
pageInfo: {
|
|
hasNextPage: true,
|
|
cursor: "cursor",
|
|
},
|
|
},
|
|
},
|
|
},
|
|
};
|
|
|
|
const data_repo = {
|
|
data: {
|
|
user: {
|
|
repositories: {
|
|
nodes: [
|
|
{ name: "test-repo-4", stargazers: { totalCount: 50 } },
|
|
{ name: "test-repo-5", stargazers: { totalCount: 50 } },
|
|
],
|
|
pageInfo: {
|
|
hasNextPage: false,
|
|
cursor: "cursor",
|
|
},
|
|
},
|
|
},
|
|
},
|
|
};
|
|
|
|
const data_repo_zero_stars = {
|
|
data: {
|
|
user: {
|
|
repositories: {
|
|
nodes: [
|
|
{ name: "test-repo-1", stargazers: { totalCount: 100 } },
|
|
{ name: "test-repo-2", stargazers: { totalCount: 100 } },
|
|
{ name: "test-repo-3", stargazers: { totalCount: 100 } },
|
|
{ name: "test-repo-4", stargazers: { totalCount: 0 } },
|
|
{ name: "test-repo-5", stargazers: { totalCount: 0 } },
|
|
],
|
|
pageInfo: {
|
|
hasNextPage: true,
|
|
cursor: "cursor",
|
|
},
|
|
},
|
|
},
|
|
},
|
|
};
|
|
|
|
const error = {
|
|
errors: [
|
|
{
|
|
type: "NOT_FOUND",
|
|
path: ["user"],
|
|
locations: [],
|
|
message: "Could not resolve to a User with the login of 'noname'.",
|
|
},
|
|
],
|
|
};
|
|
|
|
const mock = new MockAdapter(axios);
|
|
|
|
beforeEach(() => {
|
|
process.env.FETCH_MULTI_PAGE_STARS = "false"; // Set to `false` to fetch only one page of stars.
|
|
mock
|
|
.onPost("https://api.github.com/graphql")
|
|
.replyOnce(200, data_stats)
|
|
.onPost("https://api.github.com/graphql")
|
|
.replyOnce(200, data_repo);
|
|
});
|
|
|
|
afterEach(() => {
|
|
mock.reset();
|
|
});
|
|
|
|
describe("Test fetchStats", () => {
|
|
it("should fetch correct stats", async () => {
|
|
let stats = await fetchStats("anuraghazra");
|
|
const rank = calculateRank({
|
|
totalCommits: 100,
|
|
totalRepos: 5,
|
|
followers: 100,
|
|
contributions: 61,
|
|
stargazers: 300,
|
|
prs: 300,
|
|
issues: 200,
|
|
});
|
|
|
|
expect(stats).toStrictEqual({
|
|
contributedTo: 61,
|
|
name: "Anurag Hazra",
|
|
totalCommits: 100,
|
|
totalIssues: 200,
|
|
totalPRs: 300,
|
|
totalStars: 300,
|
|
rank,
|
|
});
|
|
});
|
|
|
|
it("should stop fetching when there are repos with zero stars", async () => {
|
|
mock.reset();
|
|
mock
|
|
.onPost("https://api.github.com/graphql")
|
|
.replyOnce(200, data_stats)
|
|
.onPost("https://api.github.com/graphql")
|
|
.replyOnce(200, data_repo_zero_stars);
|
|
|
|
let stats = await fetchStats("anuraghazra");
|
|
const rank = calculateRank({
|
|
totalCommits: 100,
|
|
totalRepos: 5,
|
|
followers: 100,
|
|
contributions: 61,
|
|
stargazers: 300,
|
|
prs: 300,
|
|
issues: 200,
|
|
});
|
|
|
|
expect(stats).toStrictEqual({
|
|
contributedTo: 61,
|
|
name: "Anurag Hazra",
|
|
totalCommits: 100,
|
|
totalIssues: 200,
|
|
totalPRs: 300,
|
|
totalStars: 300,
|
|
rank,
|
|
});
|
|
});
|
|
|
|
it("should throw error", async () => {
|
|
mock.reset();
|
|
mock.onPost("https://api.github.com/graphql").reply(200, error);
|
|
|
|
await expect(fetchStats("anuraghazra")).rejects.toThrow(
|
|
"Could not resolve to a User with the login of 'noname'.",
|
|
);
|
|
});
|
|
|
|
it("should fetch and add private contributions", async () => {
|
|
let stats = await fetchStats("anuraghazra", true);
|
|
const rank = calculateRank({
|
|
totalCommits: 150,
|
|
totalRepos: 5,
|
|
followers: 100,
|
|
contributions: 61,
|
|
stargazers: 300,
|
|
prs: 300,
|
|
issues: 200,
|
|
});
|
|
|
|
expect(stats).toStrictEqual({
|
|
contributedTo: 61,
|
|
name: "Anurag Hazra",
|
|
totalCommits: 150,
|
|
totalIssues: 200,
|
|
totalPRs: 300,
|
|
totalStars: 300,
|
|
rank,
|
|
});
|
|
});
|
|
|
|
it("should fetch total commits", async () => {
|
|
mock
|
|
.onGet("https://api.github.com/search/commits?q=author:anuraghazra")
|
|
.reply(200, { total_count: 1000 });
|
|
|
|
let stats = await fetchStats("anuraghazra", true, true);
|
|
const rank = calculateRank({
|
|
totalCommits: 1050,
|
|
totalRepos: 5,
|
|
followers: 100,
|
|
contributions: 61,
|
|
stargazers: 300,
|
|
prs: 300,
|
|
issues: 200,
|
|
});
|
|
|
|
expect(stats).toStrictEqual({
|
|
contributedTo: 61,
|
|
name: "Anurag Hazra",
|
|
totalCommits: 1050,
|
|
totalIssues: 200,
|
|
totalPRs: 300,
|
|
totalStars: 300,
|
|
rank,
|
|
});
|
|
});
|
|
|
|
it("should exclude stars of the `test-repo-1` repository", async () => {
|
|
mock
|
|
.onGet("https://api.github.com/search/commits?q=author:anuraghazra")
|
|
.reply(200, { total_count: 1000 });
|
|
|
|
let stats = await fetchStats("anuraghazra", true, true, ["test-repo-1"]);
|
|
const rank = calculateRank({
|
|
totalCommits: 1050,
|
|
totalRepos: 5,
|
|
followers: 100,
|
|
contributions: 61,
|
|
stargazers: 200,
|
|
prs: 300,
|
|
issues: 200,
|
|
});
|
|
|
|
expect(stats).toStrictEqual({
|
|
contributedTo: 61,
|
|
name: "Anurag Hazra",
|
|
totalCommits: 1050,
|
|
totalIssues: 200,
|
|
totalPRs: 300,
|
|
totalStars: 200,
|
|
rank,
|
|
});
|
|
});
|
|
|
|
it("should fetch two pages of stars if 'FETCH_MULTI_PAGE_STARS' env variable is set to `true`", async () => {
|
|
process.env.FETCH_MULTI_PAGE_STARS = true;
|
|
|
|
let stats = await fetchStats("anuraghazra");
|
|
const rank = calculateRank({
|
|
totalCommits: 100,
|
|
totalRepos: 5,
|
|
followers: 100,
|
|
contributions: 61,
|
|
stargazers: 400,
|
|
prs: 300,
|
|
issues: 200,
|
|
});
|
|
|
|
expect(stats).toStrictEqual({
|
|
contributedTo: 61,
|
|
name: "Anurag Hazra",
|
|
totalCommits: 100,
|
|
totalIssues: 200,
|
|
totalPRs: 300,
|
|
totalStars: 400,
|
|
rank,
|
|
});
|
|
});
|
|
|
|
it("should fetch one page of stars if 'FETCH_MULTI_PAGE_STARS' env variable is set to `false`", async () => {
|
|
process.env.FETCH_MULTI_PAGE_STARS = "false";
|
|
|
|
let stats = await fetchStats("anuraghazra");
|
|
const rank = calculateRank({
|
|
totalCommits: 100,
|
|
totalRepos: 5,
|
|
followers: 100,
|
|
contributions: 61,
|
|
stargazers: 300,
|
|
prs: 300,
|
|
issues: 200,
|
|
});
|
|
|
|
expect(stats).toStrictEqual({
|
|
contributedTo: 61,
|
|
name: "Anurag Hazra",
|
|
totalCommits: 100,
|
|
totalIssues: 200,
|
|
totalPRs: 300,
|
|
totalStars: 300,
|
|
rank,
|
|
});
|
|
});
|
|
|
|
it("should fetch one page of stars if 'FETCH_MULTI_PAGE_STARS' env variable is not set", async () => {
|
|
process.env.FETCH_MULTI_PAGE_STARS = undefined;
|
|
|
|
let stats = await fetchStats("anuraghazra");
|
|
const rank = calculateRank({
|
|
totalCommits: 100,
|
|
totalRepos: 5,
|
|
followers: 100,
|
|
contributions: 61,
|
|
stargazers: 300,
|
|
prs: 300,
|
|
issues: 200,
|
|
});
|
|
|
|
expect(stats).toStrictEqual({
|
|
contributedTo: 61,
|
|
name: "Anurag Hazra",
|
|
totalCommits: 100,
|
|
totalIssues: 200,
|
|
totalPRs: 300,
|
|
totalStars: 300,
|
|
rank,
|
|
});
|
|
});
|
|
});
|