CSC/ECE 517 Fall 2016/OSS E1637

From Expertiza_Wiki
Jump to navigation Jump to search

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

The following steps can be followed from the Web UI to ensure that the penalty is deducted and the correct scores are updated according to the specified late policy.

  1. Login as instructor
  2. Create an assignment in course
  3. Set the deadline for the assignment
  4. Add a late policy
  5. Create a team of 2-3 students
  6. Login as a student to submit the assignment and the reviews
  7. Make sure to add certain late submission
  8. Check the scores to make sure that the penalties are deducted from the total score
Retrieved from "https://wiki.expertiza.ncsu.edu/index.php?title=CSC/ECE_517_Fall_2016/OSS_E1637&oldid=103999"