CSC/ECE 517 Fall 2017/E17A4 Allow calibration to be part of an assignment Team34

Problem Statement

Currently, the instructor is able to calibrate the students’ peer review abilities on Expertiza. But s/he needs to create a separate assignment just for calibration purpose. Ideally, calibration should become one part of the normal assignment. The instructor could turn on this feature if necessary.

Currently, the functionality of calibration works well. In assignments/edit page, when instructor checks “Calibration for training?” and save the assignment. There will be an extra tab appear named “Calibration”. At this time, the instructor needs to add several sample assignments (right now instructor needs to impersonate several students to submit sample assignments) and do expert review beforehand. After that instructor could set the due dates to review. Then, students could start working on calibration. During the calibration period, students will be pre-assign several sample assignments. And after students finish one peer review. There will be a link named “Show calibration results” to show the calibration report.

Requirement Specification

1. If we want to integration calibration feature into normal assignment. We need to add a new kind of due date, that is calibration due date.

• Write a migration to add a new record in deadline_types table.

2. When instructor checks “Calibration for training?” and save the assignment. In “Due dates” tab, a row should appear to deal with calibration due date. The other content of this row is just the same as other due dates.

3. In student_task/list page, current stage column should display calibration when assignment is in calibration period.

4. In student_review/list page, you may notice that there are several restrictions to choose other students’ work.

• Eg. “Note: You can't have more than 2 outstanding reviews. You must complete one of your outstanding reviews before selecting another. ” or “Note: You can't do more than 4 reviews according to assignment policy.”

• Change the logic in this view and exclude calibration reviews when counting the review number. One easy to achieve the goal is to filter the reviews with timestamp earlier than submission due date, which means reviews are done in calibration period.

• Change the wording and numbering on this view. Calibrated review 1,2,3 to represent review done in calibration period; Review 1, 2, 3 to represent reviews done in normal review period.

5. Write feature tests to verify your modifications

• Create calibration_spec.rb file in spec/features folder

Test Plan

1. When it is in assignments#edit page:

• To check it has a checkbox with title "Calibration for training?" in assignments#edit page

2. When it is in assignments#edit page and clicking "Calibration for training?" checkbox and clicking "save" button:

• To check it displays a new tab named "Calibration" and adds a calibration due date in "Due dates" tab

• To check it allows instructors to change and save date & time and permissions of calibration due date

3.When current assignment is calibration stage:

• To check it shows current stage of the assignment to be "Calibration" on student_task#view page when current assignment is in calibration stage

• To check it shows "Calibration review 1,2,3..." instead of "Review 1,2,3..." on student_review#list page

• To check it allows students to do calibration review and the date can be saved successfully

• To check the student is able to compare the results of expert review by clicking "show calibration results" link

4. When current assignment if in review_stage:

• To check it excludes calibration reviews from outstanding review restriction and total review restriction

• To check it shows "Review 1,2,3..." instead of "Calibration review 1,2,3..." on student_review#list page