Add testing case for gitRepo

iKnowJavaScript · iKnowJavaScript · commit da246d0a495c · 2019-04-12T12:59:22.000+01:00
diff --git a/.gitignore b/.gitignore
@@ -5,6 +5,7 @@ node_modules
 package-lock.json
 *tempCodeRunnerFile.js
 /coverage
+OOP.js
 # 
 # Optional npm cache directory
 .npm
diff --git a/operations/git_api/getRepo.js b/operations/git_api/getRepo.js
@@ -1,3 +1,4 @@
+const fetch = require("node-fetch")
 //Takes a username as an argument and makes a call to GitHub API
 //and returns all the repositoriesowned by the user as an array.
 
@@ -8,7 +9,7 @@ async function showGitHubRepos(handle) {
 
   return repos.map(repo => repo.name);
 }
-//showGitHubRepos("whiteh").then(data => console.log(data));
+showGitHubRepos("whiteh").then(data => console.log(data));
 
 const myrepo = [
   "Ball-Bouncing-and-Collision",
diff --git a/operations/git_api/getRepo.test.js b/operations/git_api/getRepo.test.js
@@ -1,38 +1,15 @@
-let showGitHubRepos = require("./getRepo")
 
-describe("Mock testing showGitHubRepos implementation", () => {
-  it("Test user with repositories", async () => {
-    showGitHubRepos = jest
-      .fn()
-      .mockResolvedValue([
-        "Ball-Bouncing-and-Collision",
-        "css-grid",
-        "CSS-Grid-Calculator",
-        "CSS-Grid-Phone",
-        "css-protips",
-        "Databaseless-API"
-      ]);
 
-    let repos = await showGitHubRepos().then(data => data);
 
-    expect(showGitHubRepos).toHaveBeenCalled();
+describe("Mock testing getRepo implementation", () => {
+  it("Make sure fetchRepo function is called", async () => {
+    let repos = await showGitHubRepos("iKnowJavaScript");
+
     expect(repos).toContain("CSS-Grid-Phone");
   });
   it("Should return 'User has not repository yet' if it return an empty array", async () => {
-    showGitHubRepos = jest.fn().mockResolvedValue([]);
-
-    let repos = await showGitHubRepos().then(data => data);
+    let repos = await showGitHubRepos("rasaki");
 
-    expect(showGitHubRepos).toHaveBeenCalled();
     expect(repos).toEqual([]);
   });
-  // it("Should return 'User not found' if no array was returned", async () => {
-  //   showGitHubRepos = jest.fn().mockRejectedValue(new Error("User Not Found"));
-
-  //   const error = await showGitHubRepos();
-
-  //   expect(showGitHubRepos).toHaveBeenCalled();
-  //   expect(error).toMatch(/Not Found/);
-  // });
-});
-
+});
diff --git a/package.json b/package.json
@@ -19,5 +19,8 @@
   "homepage": "https://github.com/iKnowJavaScript/tdd-with-jest#readme",
   "devDependencies": {
     "jest": "^24.7.1"
+  },
+  "dependencies": {
+    "nodemon": "^1.18.11"
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@ node_modules`
`5`	`5`	`package-lock.json`
`6`	`6`	`*tempCodeRunnerFile.js`
`7`	`7`	`/coverage`
	`8`	`+OOP.js`
`8`	`9`	`#`
`9`	`10`	`# Optional npm cache directory`
`10`	`11`	`.npm`
Original file line number	Diff line number	Diff line change
`@@ -19,5 +19,8 @@`
`19`	`19`	`"homepage": "https://github.com/iKnowJavaScript/tdd-with-jest#readme",`
`20`	`20`	`"devDependencies": {`
`21`	`21`	`"jest": "^24.7.1"`
	`22`	`+ },`
	`23`	`+ "dependencies": {`
	`24`	`+ "nodemon": "^1.18.11"`
`22`	`25`	`}`
`23`	`26`	`}`