E1867 allow reviewer to say review can be shown to class as an example
Introduction
Expertiza is a web based open source peer reviewing tool developed and maintained by current and past students of North Carolina State University. Peer review is a great way for a student to learn how to approach a project and get ideas for their own projects. Currently, there is no way for a student to view another student's work, unless they are reviewing the other student's work.
Problem Statement
The objective of this project is to:
- Add a feature for students to make their reviews 'public', that is, allow other students to view them.
- Add a feature for TA to select a subset of 'public' reviews, and make those reviews visible as sample reviews of the particular assignment.
- Add a feature for Instructor to select a subset of 'public' reviews and make those reviews visible as sample reviews of any of his/her assignments in the course.
- Create a view where the student can see a list of sample reviews of the assignment and have a detailed view of each.
- Allow the student to toggle the visibility of a review he/she has submitted.
Proposed Flow
During each review, there will be a checkbox that says "I agree to share this review anonymously as an example to the entire class", and students can decide to check this box or not just before they submit reviews. Behind the scenes, we need to set a "status" field to “public”of the Response object.
Project Tasks
The tasks that need to be completed as part of this project can be listed as follows:
- Add checkbox against each assignment in student_tasks/list view to allow student to publish their work
- Add column make_public to table teams for storing user permission (published or not) for each project.
- Add controller for Sample submissions and implement corresponding methods.
- Add views for sample submissions
- Make changes to page student_task/view which would have link to sample submissions for each project.
- Add drop down in assignments/edit/_general view.
- Add controller code for the drop down.
- Add column sample_assignment_id to table assignment.
- Add test scripts for all the functionality.
Current status of Development
In progress
Test Plan
Many of our changes are reflected on views (user interface). So we build tests with rspec/capybara. Following is the list of tests that conduct:
- it "is able to make an assignment public"
- it "is able to view sample submissions page"
- it "should not see current assignment submissions if deadline is not met"
- it "should see current assignment submissions if deadline is met"
- it "should not see instructor selected submissions if instructor has not selected them"
- it "should see instructor selected submissions if instructor has selected them"
def create_assignment_team(assignment_name, parent_id) assignment_team = AssignmentTeam.new assignment_team.name = assignment_name assignment_team.parent_id = parent_id assignment_team.save! end def init_test # create assignment and topic assignment = build(Assignment) course = Course.new course.name = "SampleSubmissionTestCourse" course.save assignment.course_id = course.id assignment.save create_assignemnt_team("ss_assignment_team_1", assignment.id) create_assignemnt_team("ss_assignment_team_2", assignment.id) end def visit_sample_submissions_page visit '/student_task/list' click_on "Example Assignment" click_on "Sample Submissions" end describe "sample submission test" do before(:each) { init_test } it "is able to make an assignment public" do visit '/student_task/list' find(:css, "#makeSubPublic[teamid='6050']").trigger("click") click_button 'OK' expect(page).to have_http_status(200) end it "should not see current assignment submissions if deadline is not met" do # Set deadline after current time. visit_sample_submissions_page expect(page).to have_content "No sample submissions from current assignment made public yet" end it "should see current assignment submissions if deadline is met" do # Set deadline before current time. visit_sample_submissions_page expect(page).to_not have_content "No sample submissions from current assignment made public yet" end it "should not see instructor selected submissions if instructor has not selected them" do visit_sample_submissions_page expect(page).to have_content "No sample submissions from previous assignment made available yet" end it "should see instructor selected submissions if instructor has selected them" do # Instructor makes submission available. visit_sample_submissions_page expect(page).to_not have_content "No sample submissions from previous assignment made available yet" end end
Future Scope
- Publishing sample reviews made by students: This feature will require students to allow their review to be published as a sample review. This can then be made available to future students.
External Links
References
- Link to expertiza website: [4]