Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Unit tests for ProfileFollowing View #1472

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

belalmn-cmu
Copy link

@belalmn-cmu belalmn-cmu commented Nov 13, 2022

Closes #1447

Description

This PR aims to supply the existing (completed) Issue #1447 with some Unit Tests to keep consistent with the project's QA methods. I created a new entry in the test_views.py file, and simulated adding two profiles to a user's following list, and checking if the template includes their usernames as intended. There are also additional misc tests such as checking if the correct template "profile_following.html" is used in the view.

@codeclimate
Copy link

codeclimate bot commented Nov 13, 2022

Code Climate has analyzed commit f4fa2cc and detected 0 issues on this pull request.

View more on Code Climate.

Comment on lines +227 to +230
username="newuser2", email="[email protected]", password="password123"
)
self.user3 = get_user_model().objects.create_user(
username="newuser3", email="[email protected]", password="password123"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should these users have distinct email addresses?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add "Following" tab to the user profile view
2 participants