CSC/ECE 517 Fall 2016/OSS E1637

E1637: Refactor penalty_helper.rb and late_policies_controller.rb

Expertiza

Expertiza is an educational web application built on Ruby on Rails framework. It provides functionality for the instructor to create assignments for a specific course. The students can sign up for a list of topics created by the instructor. Expertiza allows students to form teams and work together. Students can provide feedback to assignments and projects submitted by other students. The students can also provide reviews about each individual member in their team as well. The numerous functionalities provided by expertiza makes the process of assignment submission, feedback and management much simpler and for the students and the instructors.

Motivation

This project deals with LatePoliciesController and PenaltyHelper files. It focuses on calculating a penalty which would be deducted from the total score obtained by the student if the student submits the assignment or review past the specified deadline. The goal of our project is to make the task of calculating and deducting penalties function perfectly and also refactor the code to make it more intuitive and modular.

Files Modified

The files modified for the penalty functionality to work are as follows,

Controllers

• late_policies_controller.rb
• grades_controller.rb

Helpers

• penalty_helper.rb

Views

• _participant.html.erb
• _participant_charts.html.erb
• _participant_title.html.erb

LatePolicyController

This controller contains the functions to create, update and delete a late policy. The instructor can create a late policy for any assignment, where in the instructor can specify the points per unit and the maximum penalty that can be applied for a particular assignment. The create and the update functions in this module was very long and it had to be refactored so that the code was modular.

GradesController

In grades controller, the total penalty was not getting deducted from the total score due to the mismatch of conditions. The conditions were modified in this controller for the penalty to be deducted from the total score obtained.

PenaltyHelper

This helper file has functions to calculate submission, review and metareview penalties. This function was not returning the values of the calculated penalty as required and modifications were made so that the correct penalty values are returned.

Views

In the current implementation, the penalty deducted is not displayed when a student views scores. The views were modified so that the deducted penalties and total score are displayed appropriately.

List of Tasks

• T1: Modification to the PenaltyHelper so that correct penalty value is calculated and returned
• T2: Refactor the create function in LatePoliciesController to make the code more modular
• T3: Refactor the update function in LatePoliciesController to make the code more modular
• T4: Modify the GradesController to deduct the total penalty from the total score
• T5: Modify the ViewScores view to display the penalties deducted

Testing

UI Testing

Testing LatePoliciesController

The create and the update code were refactored and modularized in the LatePoliciesController. The following steps can be followed from the Web UI to ensure that the refactoring did not affect the functionality of LatePoliciesController.

1. Login as instructor
2. Create an assignment in course
3. In the due dates tab in create assignment check the "Apply Late Policy" box
4. Select "New Late Policy" link
5. Fill in the form with necessary details and submit
6. The newly created late policy will be displayed in the list of all late policies
7. Select the "Edit" option for the created late policy
8. Change the required fields and save the changes
9. The updated late policy is displayed in the list of late policies
10. Edit the assignment/create a new assignment
11. In the due dates tab, the newly created late policy will be displayed in the dropdown

Testing Penalty Calculation and Deduction

Changes were made in the Penalty Helper file and Grades Controller to calculate the correct penalty value and deduct it from the total scores obtained. Changes were made in so that the View Scores option would display the penalties in addition to the total score. The following steps can be followed to ensure that the penalties get deducted appropriately from the total score and is visible to the student.

1. Login as instructor
2. Create an assignment in course along with a late policy
3. Create a team of 2-3 students for the created assignment
4. Login as a student
5. Make a submission for the assignment past the mentioned deadline
6. Go to view scores and the deducted penalty will be displayed