Apply patches

anuraghazra#1122 (comment)
rickstaa · Feb 22, 2022 · 2b1aba3 · 2b1aba3
1 parent 0436356
commit 2b1aba3
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 11 deletions.
diff --git a/api/index.js b/api/index.js
@@ -48,9 +48,9 @@ module.exports = async (req, res) => {
   try {
     const stats = await fetchStats(
       username,
+      parseArray(role),
       parseBoolean(count_private),
       parseBoolean(include_all_commits),
-      parseArray(role),
     );
 
     const cacheSeconds = clampValue(

diff --git a/api/top-langs.js b/api/top-langs.js
@@ -45,9 +45,8 @@ module.exports = async (req, res) => {
   try {
     const topLangs = await fetchTopLanguages(
       username,
-      parseArray(exclude_repo),
       parseArray(role),
-      parseArray(hide),
+      parseArray(exclude_repo),
     );
 
     const cacheSeconds = clampValue(

diff --git a/src/fetchers/stats-fetcher.js b/src/fetchers/stats-fetcher.js
@@ -89,9 +89,9 @@ const totalCommitsFetcher = async (username) => {
 
 async function fetchStats(
   username,
+  ownerAffiliations,
   count_private = false,
   include_all_commits = false,
-  ownerAffiliations,
 ) {
   if (!username) throw Error("Invalid username");
 

diff --git a/src/fetchers/top-languages-fetcher.js b/src/fetchers/top-languages-fetcher.js
@@ -34,7 +34,7 @@ const fetcher = (variables, token) => {
   );
 };
 
-async function fetchTopLanguages(username, exclude_repo = [], ownerAffiliations) {
+async function fetchTopLanguages(username, ownerAffiliations, exclude_repo = []) {
   if (!username) throw Error("Invalid username");
 
   // Set default value for ownerAffiliations in GraphQL query won't work because

diff --git a/tests/fetchStats.test.js b/tests/fetchStats.test.js
@@ -52,7 +52,7 @@ describe("Test fetchStats", () => {
   it("should fetch correct stats", async () => {
     mock.onPost("https://api.github.com/graphql").reply(200, data);
 
-    let stats = await fetchStats("anuraghazra");
+    let stats = await fetchStats("anuraghazra", []);
     const rank = calculateRank({
       totalCommits: 100,
       totalRepos: 5,
@@ -77,15 +77,15 @@ describe("Test fetchStats", () => {
   it("should throw error", async () => {
     mock.onPost("https://api.github.com/graphql").reply(200, error);
 
-    await expect(fetchStats("anuraghazra")).rejects.toThrow(
+    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 () => {
     mock.onPost("https://api.github.com/graphql").reply(200, data);
 
-    let stats = await fetchStats("anuraghazra", true);
+    let stats = await fetchStats("anuraghazra", [], true);
     const rank = calculateRank({
       totalCommits: 150,
       totalRepos: 5,
@@ -113,7 +113,7 @@ describe("Test fetchStats", () => {
       .onGet("https://api.github.com/search/commits?q=author:anuraghazra")
       .reply(200, { total_count: 1000 });
 
-    let stats = await fetchStats("anuraghazra", true, true);
+    let stats = await fetchStats("anuraghazra", [], true, true);
     const rank = calculateRank({
       totalCommits: 1050,
       totalRepos: 5,

diff --git a/tests/fetchTopLanguages.test.js b/tests/fetchTopLanguages.test.js
@@ -63,7 +63,7 @@ describe("FetchTopLanguages", () => {
   it("should fetch correct language data", async () => {
     mock.onPost("https://api.github.com/graphql").reply(200, data_langs);
 
-    let repo = await fetchTopLanguages("anuraghazra");
+    let repo = await fetchTopLanguages("anuraghazra", []);
     expect(repo).toStrictEqual({
       HTML: {
         color: "#0f0",
@@ -99,7 +99,7 @@ describe("FetchTopLanguages", () => {
   it("should throw error", async () => {
     mock.onPost("https://api.github.com/graphql").reply(200, error);
 
-    await expect(fetchTopLanguages("anuraghazra")).rejects.toThrow(
+    await expect(fetchTopLanguages("anuraghazra", [])).rejects.toThrow(
       "Could not resolve to a User with the login of 'noname'.",
     );
   });