CSC/ECE 517 Fall 2019 - E1968. Fixes for adding members to teams

This wiki page is for the description of changes made under E1968 OSS assignment for Fall 2019, CSC/ECE 517.

Background

This project fixed several bugs related to Assignment teams. Expertiza has Assignment objects, which represent an assignment usually assigned by an in structor for a course. If the instructor allowe teams for the assignment, the participants in that course can create teams with each other. Also, the instructor can create teams or add team membert to a exsiting team. Our project is focused on fixing bugs related to these functionalities.

Problem Description

In this project we fixed three problems, the problems and the way to reproduce them is shown below.
1. Adding a member from the UI (typing the ID into the textbox) raises an error.

   Reproduciton:
   (1)Login as an instructor.
   (2)Go to Assignment tab under Manage.
   (3)Click any assignment for example : OSS project & documentation.
   (4)Select Create Teams.
   (5)Select Add Team Member option for any team. 
   (6)Adding a member from the UI (typing the ID into the textbox) raises an error.

2. TODO
3. TODO

Solutions

1. The reason for the first problem was actually due to a key error. In teams_users_controller.rb line 35, 'assignment_id' was used to find AssignmentParticipant. But in the model of AssignmentParticipant, the foreign key name is acctually 'parent_id'. So, by changing it, problem solved. 2. TODO 4. TODO

Testing Details

RSpec

For each of the problems, we did feature tests with Rspec. They all follows the reproduction steps decribed above.

Feature Testing

• Problem 1: assignment_team_member_spec.rb

Following steps needs to be performed to test this code:
1. Login as instructor.
2. Create a course and an assignment under that course.
2. Add at least two students as participants to the assignment.
3. Create a team for the assignment.
4. Add a participant to the team.
5. Check if the student is a team member of that team.

• Problem 2 TODO
• Problem 3 TODO

Run the test locally: TODO: the snapshot of console result of all test.