2019-12-07T04:43:51Z

Sdinaka: /* Code Snippet */

==Expertiza Background ==
Expertiza is an educational web application created and maintained by the joint efforts of the students and the faculty at NCSU. It’s an open source project developed on Ruby on Rails platform and the code is available on Github. It allows students to review each other’s work and improve their work upon this feedback. Students and Instructors (including TA's) use this application though their credentials. Instructors can add new projects, assignments etc as well edit the previous ones and at a later stage can see the student submission and grade them. They can even put a time limit (deadline) for submitting the assignment. Students can form teams for the projects, submit their work through handles (Wiki page, link to a video etc). Students can even review their teammates and other peers, as well give them any suggestions if required

== Problem Statement ==
Expertiza system requires sending emails to students and instructors on occurrence of various events or to facilitate actions further.

Email notifications are sent out when the following events occur:
#The author of a submission gets notified via email when a reviewer has reviewed their work and the feedback is available to view.
#The reviewer receives an email when work has been submitted/resubmitted by an author whose work needs to reviewed.
#An account holder is emailed when a new account is created.
#A confirmation email is sent to a user of the system when they reset their Expertiza account password

In the current scenario, it has been identified that not all emailing functionalities are working as desired.
Also, presently, there is little evidence to claim that the existing implementation is covering all cases and handling all necessary conditions.

Our project is to build robust test suite for mailer functionality that is currently implemented in Expertiza. Developing this would ensure that all the mailing features are backed up with a stronger set of test features.

== Approach Chosen and Why?==
In this project we have first identified all the set of scenarios where the user of the system will get an email. Each of the four scenarios are identified as mentioned above and has been subdivided into a number of ways in which it can be realized. This will ensure that each of the mailers will be robust to any kind of unexpected changes and that the functionality will not be broken when deployed in production. We have chosen to bring about this robustness of test by writing unit test and functional tests using RSpec and Capybara respectively. Unit Tests will ensure that the particular functionality is being tested in a isolated setup which would even include the edge cases whereas the functional tests will ensure that this mailing feature is in resonance with the other features and works coherently.

== Implementation ==

=== Creating a new user Account===

There are various ways to create a new user account in Expertiza. Irrespective of the user creation method, the newly added user should get notified via email when his account is created in the system. This email states the username and a randomized password to allow the user to login for the first time.
Below given are the ways to create new user to system.

1. By filling a user creation form

An instructor can add a new user to the system by filling up a user creation form

2. By importing a CSV file

The instructor can add new users to the system by importing a CSV file with user information

[[File:Fc-3.PNG|center|border]]

3. By importing a CSV file to add participants to an assignment

Instructor can add participants to an assignment by importing the users from a CSV. If the user listed in the CSV does not already exist in the system, then the a new user account is created and the user is added to the assignment.

[[File:Fc-1.PNG|center|border]]

4. User requests an account

A user can request an account by filling up a form with his details. These details get saved in the requested users table. Also, the super administrator will see these details in his pending requests section. The superadmin can approve/reject such requests.
If the superadmin approves a request, a new user account is created in the system and the user receives an account creation email notification

[[File:Fc-2.PNG|center|border]]

==== Cases Tested ====

The following features were checked as part of the email functionality when creating a new user
# Mail delivery count increased by 1 after the mail triggered by the scenario is sent 
# Sender email-id is verified. The sender mail-id is verified to be 'expertiza.development@gmail.com' 
# Receiver email id is checked to be equal to 'expertiza.development@gmail.com' . Since the default recipient is 'expertiza.development@gmail.com' in test environment. 
# The subject is checked to be equal to 'Your Expertiza account and password has been created' 

We considered the following cases while testing specific to the scenario where user requests an account: 
#No email should be sent if new account requested with invalid email address 
#Super admin should receive email about new account request with a valid email address 
#Email notification should be sent to requester when super admin approves account creation request 
#No email notification should be sent to requester when super admin rejects account creation request 
 

==== Outcome ====
We used Capybara to model the scenario as follows: 
#Create user through instructor filling up the form: 

Admin user is created using factory class. 
1.visit expertiza 

2.login_as 'admin_user' 

3.visit list of users 

4.click New User link 

5.fill user_name with 'teststudent' 

6.fill user_fullname with 'test student' 

7.fill user_email with 'test@student.com' 

8.check checkbox to send email to user on review 

9.click 'Create' button. 

 
ii) create user through instructor filling up the form: 
super admin is created using factory class. 
1.visit expertiza 
2.click Request Account link 
3.select instructor from dropdown 
4.fill user name with 'requester1' 
5.fill user fullname with 'requester1, requester1' 
6.fill user email with 'test.com' 
7.fill user_email with 'test@student.com' 
8.select 'North Carolina State University' from user institution 
9.click 'Create' button. 
10.visit list of pending requests from users 
11.Requested user's status is first 'Under Review' 
12.Set status as Approved or Rejected 
13.click on 'Submit' button 

==== Code Snippet ====
[[File:New_user.JPG|center|border]]
[[File:New_user_2.JPG|center|border]]
[[File:New_user_3.JPG|center|border]]
[[File:New_user_assignment.JPG|center|border]]
[[File:New_user_import_file.JPG|center|border]]

=== Resetting Password ===
When the user tries to login, and the password entered is wrong, the user is redirected to reset password page.
[[File:Forgotpwd.PNG|center|border]]

Here the email-id entered is searched through the database,
If such user account is found: url for reset is generated, with random token set as password is sent to the user.

Flowchart:

[[File:Flow.PNG|center|border]]

==== Cases Tested ====
*User enters email id which is not associated with any account 
**No e-mail should be sent
*User enters email id that's present in the user database 
**The following features were checked inside 'it blocks' as part of verifying email functionality 
#Mail delivery count increased by 1 after the mail triggered by the scenario is sent 
#The sender mail-id is verified to be 'expertiza.development@gmail.com' 
#Receiver email id is checked to be equal to 'expertiza.development@gmail.com' . Since all mails triggered have a record in 'expertiza.development@gmail.com' when executed in test environment. 
#The subject is checked to be equal to 'Expertiza password reset' 
 

Files associated with this functionality:
# spec/controllers/PasswordRetrievalController
# app/controllers/PasswordRetrievalController
# app/mailers/mailer.rb
# app/helper/mailer_helper.rb

==== Outcome ====

We used Capybara to model the scenario with following steps: 
1. Visit expertiza home page 
2. Click 'Forgot password?' link 
3. After rediection to password retrieval page 
4. Fill in user email field with 'expertiza@mailinator.com' 
5. Click on 'Request password' button 
 

==== Code Snippet ====
[[File:Reset_password.JPG|center|border]]

=== Authors get mail on receiving review ===

Whenever someone reviews other student or team’s work, then that student or all the members of that team should get notified through the email stating that new peer review is available for your work.
Even when the reviewer updates the reviews, then also that student or all the team members should get notified that updated comments are present on their submitted work.

[[File:Fc-5.PNG|center|border]]

Files associated with this functionality:
#app/controllers/response_controller.rb
#app/models/review_response_map.rb

Following parameters need to be tested:
*Whenever reviewer reviews some team’s work, in that case all the team members should receive an email. So, number of emails delivered should be number of participants assigned to that work.

==== Cases Tested ====
==== Outcome ====
==== Code Snippet ====

=== Reviewer gets mail when authors resubmit work ===
This will be mainly the case when there will be more than one round of the submission. After the first submission, some users will be assigned to review the submitted work. Now, in second round of the submission, whenever author resubmits or edit their work, then in that case all the reviewers assigned to that author should get email notification stating new submission is available for them to review.

Files associated with this functionality:
#app/controllers/submitted_content_controller.rb
#app/helpers/mailer_helper.rb

==== Cases Tested ====
==== Outcome ====
==== Code Snippet ====

== Fixing out of scope bugs ==
=== Account Creation ===
When student accounts are created by importing a CSV file on the Users page, they receive e-mails with their user-ID and password.
However, if an account is created by adding them as participants to an assignment when they don't already have an account, e-mail notification is not being sent.

==== What we fixed ====
The issue here was that an already existing user in the system will not get added as a participant on the assignment.

We fixed the issue by creating a new user if one doesn't exist and we enabling the email functionality by adding the .deliver_now method

This method invokes the mailer to deliver an email immediately.

Now that we have enabled the email functionality irrespective of where a new user is created from, we don't need to explicitly call the mailer from each location where new user is created. The email body was inconsistent across different scenarios. We had fixed this as well.

==== Outcome ====
If the user does not exist in the system, new user account (if there is sufficient information about the user to create a new Expertiza account) is created

If the user already exists as a participant on the assignment, do nothing Add the user to the assignment as a participant; otherwise.

This way, the user(whether newly created or already an Expertiza user) gets added on the assignment as a participant.

==== Code Snippet ====
[[File:Email_fix.JPG]]

[[File:BadgerAddParticipant2.PNG]]

=== Author Review ===
After the reviewer submits the review, the author should get an email notification about this. Also, the instructor should receive an email about the same. This function is not currently working in the beta version. Also, if at all the author resubmits the work, the reviewer as well as the instructor should get an email about the change that was done. This is not currently functional in expertiza beta. As discussed with our mentor, we have successfully fixed that issue and now in both of the cases as discussed above, the intended users( author in first scenario and reviewer and instructor in second one) are receiving the email notification.
==== What we fixed====
# Author receiving email
#
==== Outcome ====

==== Code Snippet ====

[[File:Reviewer_fix.JPG|center|border]]

[[File:Reviewer_fix_2.JPG|center|border]]

[[File:Reviewer_fix_3.JPG|center|border]]

[[File:Reviewer_fix_4.JPG|center|border]]

== Our Team ==
#Mohit Jain(mentor)
#Vishva Shah (vpshah@ncsu.edu)
#Urvish Vasani (uvasani@ncsu.edu)
#Tanvi Pandit(tmpandi2@ncsu.edu)
#Swathi Dinakaran(sdinaka@ncsu.edu)

== References ==

Demo link

Expertiza on GitHub [https://github.com/expertiza/expertiza click here]

To visit our forked repo, [https://github.com/vishva1001/expertiza click here]

View our pull request [https://github.com/expertiza/expertiza/pull/1632 here]

2019-12-07T04:39:50Z

2019-12-07T04:31:26Z

Sdinaka: /* Code Snippet */

==Expertiza Background ==
Expertiza is an educational web application created and maintained by the joint efforts of the students and the faculty at NCSU. It’s an open source project developed on Ruby on Rails platform and the code is available on Github. It allows students to review each other’s work and improve their work upon this feedback. Students and Instructors (including TA's) use this application though their credentials. Instructors can add new projects, assignments etc as well edit the previous ones and at a later stage can see the student submission and grade them. They can even put a time limit (deadline) for submitting the assignment. Students can form teams for the projects, submit their work through handles (Wiki page, link to a video etc). Students can even review their teammates and other peers, as well give them any suggestions if required

== Problem Statement ==
Expertiza system requires sending emails to students and instructors on occurrence of various events or to facilitate actions further.

Email notifications are sent out when the following events occur:
#The author of a submission gets notified via email when a reviewer has reviewed their work and the feedback is available to view.
#The reviewer receives an email when work has been submitted/resubmitted by an author whose work needs to reviewed.
#An account holder is emailed when a new account is created.
#A confirmation email is sent to a user of the system when they reset their Expertiza account password

In the current scenario, it has been identified that not all emailing functionalities are working as desired.
Also, presently, there is little evidence to claim that the existing implementation is covering all cases and handling all necessary conditions.

Our project is to build robust test suite for mailer functionality that is currently implemented in Expertiza. Developing this would ensure that all the mailing features are backed up with a stronger set of test features.

== Approach Chosen and Why?==
In this project we have first identified all the set of scenarios where the user of the system will get an email. Each of the four scenarios are identified as mentioned above and has been subdivided into a number of ways in which it can be realized. This will ensure that each of the mailers will be robust to any kind of unexpected changes and that the functionality will not be broken when deployed in production. We have chosen to bring about this robustness of test by writing unit test and functional tests using RSpec and Capybara respectively. Unit Tests will ensure that the particular functionality is being tested in a isolated setup which would even include the edge cases whereas the functional tests will ensure that this mailing feature is in resonance with the other features and works coherently.

== Implementation ==

=== Creating a new user Account===

There are various ways to create a new user account in Expertiza. Irrespective of the user creation method, the newly added user should get notified via email when his account is created in the system. This email states the username and a randomized password to allow the user to login for the first time.
==== Cases Tested ====

Below given are the ways to create new user to system.

1. By filling a user creation form

An instructor can add a new user to the system by filling up a user creation form

2. By importing a CSV file

The instructor can add new users to the system by importing a CSV file with user information

[[File:Fc-3.PNG|center|border]]

The following features were checked as part of user creating user in case (i) and case (ii) mentioned above 
# Mail delivery count increased by 1 after the mail triggered by the scenario is sent 
# Sender email-id is verified. The sender mail-id is verified to be 'expertiza.development@gmail.com' 
# Receiver email id is checked to be equal to 'expertiza.development@gmail.com' . Since all mails triggered have a record in 'expertiza.development@gmail.com' when executed in test environment. 
# The subject is checked to be equal to 'Your Expertiza account and password has been created' 

3. By importing a CSV file to add participants to an assignment

Instructor can add participants to an assignment by importing the users from a CSV. If the user listed in the CSV does not already exist in the system, then the a new user account is created and the user is added to the assignment.

[[File:Fc-1.PNG|center|border]]

4. User requests an account

A user can request an account by filling up a form with his details. These details get saved in the requested users table. Also, the super administrator will see these details in his pending requests section. The superadmin can approve/reject such requests.
If the superadmin approves a request, a new user account is created in the system and the user receives an account creation email notification

[[File:Fc-2.PNG|center|border]]

We considered the following cases as it blocks while testing specific to scenario ii: 
#no email should be sent if new account requested with invalid email address 
# super admin should receive email about new account request with a valid email address 
# email notification should be sent to requester when super admin approves account creation request 
# email notification should be sent to requester when super admin rejects account creation request 
 

==== Outcome ====
We used Capybara to model the scenario as follows: 
#Create user through instructor filling up the form: 

Admin user is created using factory class. 
1.visit expertiza 

2.login_as 'admin_user' 

3.visit list of users 

4.click New User link 

5.fill user_name with 'teststudent' 

6.fill user_fullname with 'test student' 

7.fill user_email with 'test@student.com' 

8.check checkbox to send email to user on review 

9.click 'Create' button. 

 
ii) create user through instructor filling up the form: 
super admin is created using factory class. 
1.visit expertiza 
2.click Request Account link 
3.select instructor from dropdown 
4.fill user name with 'requester1' 
5.fill user fullname with 'requester1, requester1' 
6.fill user email with 'test.com' 
7.fill user_email with 'test@student.com' 
8.select 'North Carolina State University' from user institution 
9.click 'Create' button. 
10.visit list of pending requests from users 
11.Requested user's status is first 'Under Review' 
12.Set status as Approved or Rejected 
13.click on 'Submit' button 

==== Code Snippet ====

=== Resetting Password ===
When the user tries to login, and the password entered is wrong, the user is redirected to reset password page.
[[File:Forgotpwd.PNG|center|border]]

Here the email-id entered is searched through the database,
If such user account is found: url for reset is generated, with random token set as password is sent to the user.

Flowchart:

[[File:Flow.PNG|center|border]]

==== Cases Tested ====
*User enters email id which is not associated with any account 
**No e-mail should be sent
*User enters email id that's present in the user database 
**The following features were checked inside 'it blocks' as part of verifying email functionality 
#Mail delivery count increased by 1 after the mail triggered by the scenario is sent 
#The sender mail-id is verified to be 'expertiza.development@gmail.com' 
#Receiver email id is checked to be equal to 'expertiza.development@gmail.com' . Since all mails triggered have a record in 'expertiza.development@gmail.com' when executed in test environment. 
#The subject is checked to be equal to 'Expertiza password reset' 
 

Files associated with this functionality:
# spec/controllers/PasswordRetrievalController
# app/controllers/PasswordRetrievalController
# app/mailers/mailer.rb
# app/helper/mailer_helper.rb

==== Outcome ====

We used Capybara to model the scenario with following steps: 
1. Visit expertiza home page 
2. Click 'Forgot password?' link 
3. After rediection to password retrieval page 
4. Fill in user email field with 'expertiza@mailinator.com' 
5. Click on 'Request password' button 
 

==== Code Snippet ====
[[File:Reset_password.JPG|center|border]]

=== Authors get mail on receiving review ===

Whenever someone reviews other student or team’s work, then that student or all the members of that team should get notified through the email stating that new peer review is available for your work.
Even when the reviewer updates the reviews, then also that student or all the team members should get notified that updated comments are present on their submitted work.

[[File:Fc-5.PNG|center|border]]

Files associated with this functionality:
#app/controllers/response_controller.rb
#app/models/review_response_map.rb

Following parameters need to be tested:
*Whenever reviewer reviews some team’s work, in that case all the team members should receive an email. So, number of emails delivered should be number of participants assigned to that work.

==== Cases Tested ====
==== Outcome ====
==== Code Snippet ====

=== Reviewer gets mail when authors resubmit work ===
This will be mainly the case when there will be more than one round of the submission. After the first submission, some users will be assigned to review the submitted work. Now, in second round of the submission, whenever author resubmits or edit their work, then in that case all the reviewers assigned to that author should get email notification stating new submission is available for them to review.

Files associated with this functionality:
#app/controllers/submitted_content_controller.rb
#app/helpers/mailer_helper.rb

==== Cases Tested ====
==== Outcome ====
==== Code Snippet ====

== Fixing out of scope bugs ==
=== Account Creation ===
When student accounts are created by importing a CSV file on the Users page, they receive e-mails with their user-ID and password.
However, if an account is created by adding them as participants to an assignment when they don't already have an account, e-mail notification is not being sent.

==== What we fixed ====
The issue here was that an already existing user in the system will not get added as a participant on the assignment.

We fixed the issue by creating a new user if one doesn't exist and we enabling the email functionality by adding the .deliver_now method

This method invokes the mailer to deliver an email immediately.

Now that we have enabled the email functionality irrespective of where a new user is created from, we don't need to explicitly call the mailer from each location where new user is created. The email body was inconsistent across different scenarios. We had fixed this as well.

==== Outcome ====
If the user does not exist in the system, new user account (if there is sufficient information about the user to create a new Expertiza account) is created

If the user already exists as a participant on the assignment, do nothing Add the user to the assignment as a participant; otherwise.

This way, the user(whether newly created or already an Expertiza user) gets added on the assignment as a participant.

==== Code Snippet ====
[[File:Email_fix.JPG]]

[[File:BadgerAddParticipant2.PNG]]

=== Author Review ===
After the reviewer submits the review, the author should get an email notification about this. Also, the instructor should receive an email about the same. This function is not currently working in the beta version. Also, if at all the author resubmits the work, the reviewer as well as the instructor should get an email about the change that was done. This is not currently functional in expertiza beta. As discussed with our mentor, we have successfully fixed that issue and now in both of the cases as discussed above, the intended users( author in first scenario and reviewer and instructor in second one) are receiving the email notification.
==== What we fixed====
# Author receiving email
#
==== Outcome ====

==== Code Snippet ====
Reviewer_fix.JPG

== Our Team ==
#Mohit Jain(mentor)
#Vishva Shah (vpshah@ncsu.edu)
#Urvish Vasani (uvasani@ncsu.edu)
#Tanvi Pandit(tmpandi2@ncsu.edu)
#Swathi Dinakaran(sdinaka@ncsu.edu)

== References ==

Demo link

Expertiza on GitHub [https://github.com/expertiza/expertiza click here]

To visit our forked repo, [https://github.com/vishva1001/expertiza click here]

View our pull request [https://github.com/expertiza/expertiza/pull/1632 here]

2019-12-07T03:41:28Z

Sdinaka:

CSC/ECE 517 Fall 2019 - E1995. Tests for email functionality

2019-12-07T03:20:23Z

Sdinaka: /* Code Snippet */

==Expertiza Background ==
Expertiza is an educational web application created and maintained by the joint efforts of the students and the faculty at NCSU. It’s an open source project developed on Ruby on Rails platform and the code is available on Github. It allows students to review each other’s work and improve their work upon this feedback. Students and Instructors (including TA's) use this application though their credentials. Instructors can add new projects, assignments etc as well edit the previous ones and at a later stage can see the student submission and grade them. They can even put a time limit (deadline) for submitting the assignment. Students can form teams for the projects, submit their work through handles (Wiki page, link to a video etc). Students can even review their teammates and other peers, as well give them any suggestions if required

== Problem Statement ==
Expertiza system requires sending emails to students and instructors on occurrence of various events or to facilitate actions further.

Email notifications are sent out when the following events occur:
#The author of a submission gets notified via email when a reviewer has reviewed their work and the feedback is available to view.
#The reviewer receives an email when work has been submitted/resubmitted by an author whose work needs to reviewed.
#An account holder is emailed when a new account is created.
#A confirmation email is sent to a user of the system when they reset their Expertiza account password

In the current scenario, it has been identified that not all emailing functionalities are working as desired.
Also, presently, there is little evidence to claim that the existing implementation is covering all cases and handling all necessary conditions.

Our project is to build robust test suite for mailer functionality that is currently implemented in Expertiza. Developing this would ensure that all the mailing features are backed up with a stronger set of test features.

== Approach Chosen and Why?==
In this project we have first identified all the set of scenarios where the user of the system will get an email. Each of the four scenarios are identified as mentioned above and has been subdivided into a number of ways in which it can be realized. This will ensure that each of the mailers will be robust to any kind of unexpected changes and that the functionality will not be broken when deployed in production. We have chosen to bring about this robustness of test by writing unit test and functional tests using RSpec and Capybara respectively. Unit Tests will ensure that the particular functionality is being tested in a isolated setup which would even include the edge cases whereas the functional tests will ensure that this mailing feature is in resonance with the other features and works coherently.

== Implementation ==

=== Authors get mail on receiving review ===

Whenever someone reviews other student or team’s work, then that student or all the members of that team should get notified through the email stating that new peer review is available for your work.
Even when the reviewer updates the reviews, then also that student or all the team members should get notified that updated comments are present on their submitted work.

[[File:Fc-5.PNG|center|border]]

Files associated with this functionality:
#app/controllers/response_controller.rb
#app/models/review_response_map.rb

Following parameters need to be tested:
*Whenever reviewer reviews some team’s work, in that case all the team members should receive an email. So, number of emails delivered should be number of participants assigned to that work.

==== Cases Tested ====
==== Outcome ====
==== Code Snippet ====

=== Reviewer gets mail when authors resubmit work ===
This will be mainly the case when there will be more than one round of the submission. After the first submission, some users will be assigned to review the submitted work. Now, in second round of the submission, whenever author resubmits or edit their work, then in that case all the reviewers assigned to that author should get email notification stating new submission is available for them to review.

Files associated with this functionality:
#app/controllers/submitted_content_controller.rb
#app/helpers/mailer_helper.rb

==== Cases Tested ====
==== Outcome ====
==== Code Snippet ====

=== Creating a new user Account===

There are various ways to create a new user account in Expertiza. Irrespective of the user creation method, the newly added user should get notified via email when his account is created in the system. This email states the username and a randomized password to allow the user to login for the first time.
==== Cases Tested ====

Below given are the ways to create new user to system.

1. By filling a user creation form
An instructor can add a new user to the system by filling up a user creation form

2. By importing a CSV file
The instructor can add new users to the system by importing a CSV file with user information

[[File:Fc-3.PNG|center|border]]

The following features were checked as part of user creating user in case (i) and case (ii) mentioned above 
# Mail delivery count increased by 1 after the mail triggered by the scenario is sent 
# Sender email-id is verified. The sender mail-id is verified to be 'expertiza.development@gmail.com' 
# Receiver email id is checked to be equal to 'expertiza.development@gmail.com' . Since all mails triggered have a record in 'expertiza.development@gmail.com' when executed in test environment. 
# The subject is checked to be equal to 'Your Expertiza account and password has been created' 

3. By importing a CSV file to add participants to an assignment
Instructor can add participants to an assignment by importing the users from a CSV. If the user listed in the CSV does not already exist in the system, then the a new user account is created and the user is added to the assignment.

[[File:Fc-1.PNG|center|border]]

4. User requests an account

A user can request an account by filling up a form with his details. These details get saved in the requested users table. Also, the super administrator will see these details in his pending requests section. The superadmin can approve/reject such requests.
If the superadmin approves a request, a new user account is created in the system and the user receives an account creation email notification

[[File:Fc-2.PNG|center|border]]

We considered the following cases as it blocks while testing specific to scenario ii: 
#no email should be sent if new account requested with invalid email address 
# super admin should receive email about new account request with a valid email address 
# email notification should be sent to requester when super admin approves account creation request 
# email notification should be sent to requester when super admin rejects account creation request 
 

==== Outcome ====
We used Capybara to model the scenario as follows: 
#Create user through instructor filling up the form: 

Admin user is created using factory class. 
1.visit expertiza 

2.login_as 'admin_user' 

3.visit list of users 

4.click New User link 

5.fill user_name with 'teststudent' 

6.fill user_fullname with 'test student' 

7.fill user_email with 'test@student.com' 

8.check checkbox to send email to user on review 

9.click 'Create' button. 

 
ii) create user through instructor filling up the form: 
super admin is created using factory class. 
1.visit expertiza 
2.click Request Account link 
3.select instructor from dropdown 
4.fill user name with 'requester1' 
5.fill user fullname with 'requester1, requester1' 
6.fill user email with 'test.com' 
7.fill user_email with 'test@student.com' 
8.select 'North Carolina State University' from user institution 
9.click 'Create' button. 
10.visit list of pending requests from users 
11.Requested user's status is first 'Under Review' 
12.Set status as Approved or Rejected 
13.click on 'Submit' button 

==== Code Snippet ====

=== Resetting Password ===
When the user tries to login, and the password entered is wrong, the user is redirected to reset password page.
[[File:Forgotpwd.PNG|center|border]]

Here the email-id entered is searched through the database,
If such user account is found: url for reset is generated, with random token set as password is sent to the user.

Flowchart:

[[File:Flow.PNG|center|border]]

==== Cases Tested ====
*User enters email id which is not associated with any account 
**No e-mail should be sent
*User enters email id that's present in the user database 
**The following features were checked inside 'it blocks' as part of verifying email functionality 
#Mail delivery count increased by 1 after the mail triggered by the scenario is sent 
#The sender mail-id is verified to be 'expertiza.development@gmail.com' 
#Receiver email id is checked to be equal to 'expertiza.development@gmail.com' . Since all mails triggered have a record in 'expertiza.development@gmail.com' when executed in test environment. 
#The subject is checked to be equal to 'Expertiza password reset' 
 

Files associated with this functionality:
# spec/controllers/PasswordRetrievalController
# app/controllers/PasswordRetrievalController
# app/mailers/mailer.rb
# app/helper/mailer_helper.rb

==== Outcome ====

We used Capybara to model the scenario with following steps: 
1. Visit expertiza home page 
2. Click 'Forgot password?' link 
3. After rediection to password retrieval page 
4. Fill in user email field with 'expertiza@mailinator.com' 
5. Click on 'Request password' button 
 

==== Code Snippet ====

== Fixing out of scope bugs ==
=== Account Creation ===
When student accounts are created by importing a CSV file on the Users page, they receive e-mails with their user-ID and password.
However, if an account is created by adding them as participants to an assignment when they don't already have an account, e-mail notification is not being sent.

==== What we fixed ====
The issue here was that an already existing user in the system will not get added as a participant on the assignment.

We fixed the issue by creating a new user if one doesn't exist and we enabling the email functionality by adding the .deliver_now method

This method invokes the mailer to deliver an email immediately.

Now that we have enabled the email functionality irrespective of where a new user is created from, we don't need to explicitly call the mailer from each location where new user is created. The email body was inconsistent across different scenarios. We had fixed this as well.

==== Outcome ====
If the user does not exist in the system, new user account (if there is sufficient information about the user to create a new Expertiza account) is created

If the user already exists as a participant on the assignment, do nothing Add the user to the assignment as a participant; otherwise.

This way, the user(whether newly created or already an Expertiza user) gets added on the assignment as a participant.

==== Code Snippet ====
[[File:Email_fix.JPG]]

[[File:BadgerAddParticipant2.PNG]]

=== Author Review ===
==== What we fixed====
==== Outcome ====
==== Code Snippet ====

== Developed By ==
#Mohit Jain(mentor)
#Vishva Shah (vpshah@ncsu.edu)
#Urvish Vasani (uvasani@ncsu.edu)
#Tanvi Pandit(tmpandi2@ncsu.edu)
#Swathi Dinakaran(sdinaka@ncsu.edu)

== References ==

Demo link

Expertiza on GitHub [https://github.com/expertiza/expertiza click here]

To visit our forked repo, [https://github.com/vishva1001/expertiza click here]

View our pull request [https://github.com/expertiza/expertiza/pull/1632 here]