User:Kvysyar/sandbox

This wiki documentation is for the open source refactoring project E1601: Refactor methods related to submitted_hyperlinks for the course ECE/CSC 517.

Peer Review Information
We suggest the reviewers to use the below students and instructor for the purpose of reviewing. These students belong to the same team and work on an assignment given by the below instructor: Please note: Please do not delete the above users or their team. If you wish to do so, please add them back so as to aid other reviewers. Also note that the option to submit hyperlinks in the UI will only be displayed to the enrolled students only before the assignment deadline.
 * Instructor login: username -> instructor6, password -> password
 * Student login: username -> student5884,  password -> password
 * Student login: username -> student6420, password -> password

Expertiza Introduction
Expertiza is an open source web application which enhances learning through peer reviews. The students can form teams and work on assignments given by course instructors. The peers can review the submissions and give feedback. The students also get to review the reviewers!

Problem statement
Before: When student submitted files and hyperlinks, that content was submitted on behalf of a team, not individuals (even there is only one person in the team). Some of the previous contributors worked on a project which associated all the submitted content to teams: there were submitted_hyperlinks and dirctory_num fields in teams table. All the hyperlinks should ideally be recorded in one submitted_hyperlinks field. All the submitted files were uploaded to the file space and in the “course_folder/assignment_folder/team_folder” path.

What was wrong with that: There are multiple methods in assignment_participant.rb which still look like the author submit the hyperlinks on behalf of him/herself. They should be moved to assignment_teams.rb.

What refactoring we have done to improve:
 * Removed “hyperlinks_array” method and “hyperlinks” method from assignment_participant.rb. Created “hyperlinks” method in assignment_team.rb.
 * Removed “submit_hyperlink” and “remove_hyperlink” from assignment_pariticpant.rb and created equivalent methods in assignment_team.rb.


 * Removed “has_submissions?” from assginment_participant.rb and created equivalent method in assignment_team.rb.
 * All the calls to the above methods were refactored to call the appropriate method's functions.
 * Wrote tests for submitting and removing submitted hyperlinks and files.

Key changes
Files with major modifications: Four methods "hyperlinks”, “has_submissions?”, “submit_hyperlink” and “remove_hyperlink” were moved from the assignment_participant.rb model to assignment_team.rb, and the "hyperlinks_array" method removed.
 * 1) The model: assignment_participant.rb
 * 2) The model: assignment_team.rb

1.a) The below hyperlinks_array and hyperlinks method are removed from the assignment_participant.rb: b) They are both merged into a single method in assignment_team.rb under the hyperlinks mehtod:

2. a) Earlier, the following method submit_hyperlink was in assignment_participant.rb: b) It is now moved to assignment_team.rb:

3) Similarly the methods "has_submissions?", "remove_hyperlink" were moved.

4) Most importantly, at all the places in the Expertiza code, where there were calls to the above methods in assignment_participant.rb, which are now moved to assignment_team.rb, were refactored accordingly to call them appropriately.

RSpec
There were no existing tests for the hyperlinks related methods. We used RSpec to write test cases using TTD approach. The assignment_team_spec.rb in the spec folder will have these tests. All the tests can be executed by rspec spec command, or can also be executed individually using the command "rspec spec/models/assignment_team_spec.rb".

UI Testing
Following steps needs to be performed to test this code from UI:
 * 1) Login as a student.
 * 2) In an assignment, that is still due, add one or more hyperlinks.
 * 3) Login as another student who is a teammate of the first student working on the same assignment.
 * 4) You should be able to see the newly uploaded link(s) by the other team member.
 * 5) Try adding a link that is not a valid http or https link. It will give an error message.
 * 6) Try submitting the same link once again. This will give an error message too.