Merge pull request #88 from farhan7reza7/test-correcter

updated source codes and tests for releasing v1.1.1
farhan7reza7 · Feb 1, 2024 · fdefcdb · fdefcdb
2 parents 5278c20 + 6b0f6d7
commit fdefcdb
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 25 deletions.
diff --git a/src/follow-back.js b/src/follow-back.js
@@ -1,6 +1,5 @@
 // Importing required modules
 const axios = require('axios');
-//require('dotenv').config();
 
 /**
  * Module for managing followers and followings on GitHub.
@@ -249,9 +248,7 @@ function followBack(yourUsername, yourToken) {
           console.error(`Failed to unfollow: ${username}`);
         }
       } else {
-        console.log(
-          `Sorry, ${username} is not in not-following-back users`,
-        );
+        console.log(`Sorry, ${username} is not in not-following-back users`);
       }
     },
 
@@ -279,7 +276,7 @@ function followBack(yourUsername, yourToken) {
           console.error(`Failed to unfollow: ${user}`);
         }
       }
-      console.log("Finished not following back users!");
+      console.log('Finished not following back users!');
     },
   };
 }

diff --git a/test/follow-back.test.js b/test/follow-back.test.js
@@ -1,9 +1,6 @@
 const followBack = require('../src/follow-back'); // Import the module to be tested
-//require('dotenv').config();
-//const { TOKEN: token, USER: user} = process.env;
-
-const token = process.env.TOKEN;
-const user = process.env.USER;
+require('dotenv').config();
+const { TOKEN: token, USER: user } = process.env;
 
 describe('Follow Back Module', () => {
   // Mocking axios calls
@@ -52,73 +49,79 @@ describe('Follow Back Module', () => {
   describe('isFollower', () => {
     // Test case to check if a user is a follower
     it('should return true if the user is a follower', async () => {
-      const result = await followBack(token, user).isFollower('farhan7reza7');
+      const result = await followBack(user, token).isFollower('farhan7reza7');
       expect(result).toBe('Yes, farhan7reza7 follows you!');
     });
 
     // Test case to check if a user is not a follower
     it('should return false if the user is not a follower', async () => {
-      const result = await followBack(token, user).isFollower('diff-ymd-package');
+      const result = await followBack(user, token).isFollower(
+        'diff-ymd-package',
+      );
       expect(result).toBe('No, diff-ymd-package does not follow you!');
     });
   });
 
   describe('isFollowing', () => {
     // Test case to check if a user is followed
     it('should return true if the user is followed', async () => {
-      const result = await followBack(token, user).isFollowing('farhan7reza7');
+      const result = await followBack(user, token).isFollowing('farhan7reza7');
       expect(result).toBe('Yes, you follow farhan7reza7!');
     });
 
     // Test case to check if a user is not followed
     it('should return false if the user is not followed', async () => {
-      const result = await followBack(token, user).isFollowing('anaseem80');
+      const result = await followBack(user, token).isFollowing('anaseem80');
       expect(result).toBe('No, you do not follow anaseem80!');
     });
   });
 
   // Test case for the total number of followers
   describe('totalFollowers', () => {
     it('should return the total number of followers', async () => {
-      const result = await followBack(token, user).totalFollowers();
+      const result = await followBack(user, token).totalFollowers();
       expect(result).toBe(3);
     });
   });
 
   // Test case for the total number of followings
   describe('totalFollowings', () => {
     it('should return the total number of followings', async () => {
-      const result = await followBack(token, user).totalFollowings();
+      const result = await followBack(user, token).totalFollowings();
       expect(result).toBe(3);
     });
   });
 
   // Test case for users who are not following back
   describe('whoNotFollowingBack', () => {
     it('should return users who are not following back', async () => {
-      const result = await followBack(token, user).whoNotFollowingBack();
+      const result = await followBack(user, token).whoNotFollowingBack();
       expect(result).toEqual(['diff-ymd-package', 'Open-Sourced-Org']);
     });
   });
 
   // Test case for users who are following back
   describe('whoFollowingBack', () => {
     it('should return users who are following back', async () => {
-      const result = await followBack(token, user).whoFollowingBack();
+      const result = await followBack(user, token).whoFollowingBack();
       expect(result).toEqual(['farhan7reza7']);
     });
   });
 
   // Test case to check if a user is following back
   describe('isFollowingBack', () => {
     it('should return true if the user is following back', async () => {
-      const result = await followBack(token, user).isFollowingBack('farhan7reza7');
+      const result = await followBack(user, token).isFollowingBack(
+        'farhan7reza7',
+      );
       expect(result).toBe('Yes, farhan7reza7 following back!');
     });
 
     // Test case to check if a user is not following back
     it('should return false if the user is not following back', async () => {
-      const result = await followBack(token, user).isFollowingBack('diff-ymd-package');
+      const result = await followBack(user, token).isFollowingBack(
+        'diff-ymd-package',
+      );
       expect(result).toBe('No, diff-ymd-package does not following back!');
     });
   });
@@ -128,7 +131,9 @@ describe('Follow Back Module', () => {
     it('should unfollow a user who is not following back', async () => {
       axios.delete.mockResolvedValueOnce({ status: 204 }); // Mock the successful deletion
 
-      await followBack(token, user).unfollowNotFollowingBack('diff-ymd-package');
+      await followBack(user, token).unfollowNotFollowingBack(
+        'diff-ymd-package',
+      );
       expect(axios.delete).toHaveBeenCalledTimes(0); // Assuming there are one user to unfollow
     });
   });
@@ -138,10 +143,8 @@ describe('Follow Back Module', () => {
     it('should unfollow all users who are not following back', async () => {
       axios.delete.mockResolvedValueOnce({ status: 204 }); // Mock the successful deletion
 
-      await followBack(token, user).unfollowAllNotFollowingBack();
+      await followBack(user, token).unfollowAllNotFollowingBack();
       expect(axios.delete).toHaveBeenCalledTimes(0); // Assuming there are one user to unfollow
     });
-  });  
+  });
 });
-
-