#164798023 Implement follow and unfollow feature #58
Conversation
…into ft-follow-unfollow-user-#164798023
…into ft-follow-unfollow-user-#164798023
- create follow and unfollow functions(CRUD) - write mock test using chai and mocha [Finishes #164798023]
…into feature/164798023-follow-unfollow-user
[Finishes #164798023]
- remove dead codes from follow controller
| @@ -0,0 +1,165 @@ | |||
| import chai, { expect } from 'chai'; | |||
There was a problem hiding this comment.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
| @@ -0,0 +1,118 @@ | |||
| import model from '../models'; | |||
There was a problem hiding this comment.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
| @@ -0,0 +1,18 @@ | |||
| import express from 'express'; | |||
There was a problem hiding this comment.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
…into feature/164798023-follow-unfollow-user
| .filter( | ||
| followed => | ||
| followed.follower_id === followerId && | ||
| followed.followee_id === followeeId |
There was a problem hiding this comment.
Use a more meaningful variable name in the map function. Also what is the meaning of the variable userFollowing? @amaechi-chuks
There was a problem hiding this comment.
The variable userFollowing is a validation that checks for follower and followee in the database.
There was a problem hiding this comment.
The variable userFollowing is a validation that checks for follower and followee in the database.
Can you describe what the variable is going to contain? Is it an object or array? And what will be its content(s)? @amaechi-chuks
…into feature/164798023-follow-unfollow-user
- use more descriptive variable name - update test cases - update create controller functions(callback and redirect) [Finishes #164798023]
…into feature/164798023-follow-unfollow-user
…into feature/164798023-follow-unfollow-user
| done(); | ||
| }); | ||
| }); | ||
| it('It should return a 200 status code if user unfollow successfully', done => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| done(); | ||
| }); | ||
| }); | ||
| it('lgoin a user', done => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| }); | ||
| }); | ||
| describe('TEST USER UNFOLLOW', () => { | ||
| it('Create a follower', done => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| done(); | ||
| }); | ||
| }); | ||
| it('It should return a 406 if user try to follow him/herself', done => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| done(); | ||
| }); | ||
| }); | ||
| it('It should return a 403 if user try to follow him/herself', done => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| }); | ||
| }); | ||
|
|
||
| it('It should return a 201 if a user is been followed successfully', done => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| done(); | ||
| }); | ||
| }); | ||
| it('lgoin a user', done => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| let token2; | ||
| let token3; | ||
| describe('TEST USER FOLLOW', () => { | ||
| it('Create a follower', done => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| chai.use(chaiHttp); | ||
| let token1; | ||
| let token2; | ||
| let token3; |
There was a problem hiding this comment.
'token3' is assigned a value but never used no-unused-vars
| * @param {followerId} person following the user | ||
| * @returns {object} response and user profile data | ||
| */ | ||
| async followUser(req, res) { |
There was a problem hiding this comment.
Parsing error: Unexpected token followUser
There was a problem hiding this comment.
Have you seen this? @amaechi-chuks
Description
This PR keep tabs of users following each other and by extension the articles they author.
Type of change
How Has This Been Tested?
http://localhost:6000/api/v1/follow/:followeeIdand received a responseSuccessfully followed userfrom the serverhttp://localhost:6000/api/v1/follow/unFolloweeIdand received a responsesuccessfully unfollowed a userChecklist:
chai and mochaPT-ID
164798023
Screenshots
follow a user
unfollow a user
Background
N/A