CSC/ECE 517 Spring 2023 - E2327. Fix Back link on New Late Policy page

Problem Description

Project Overview

Expertiza was created as an online platform to assist in peer reviewing and submission of academic work, which can include articles, codes, and websites. With this application, instructors are given the ability to add assignments, grade them, and allocate students into teams based on their preferred topics. The Expertiza platform provides a 'Late Policy' feature that can be applied to any assignment, so that points are automatically deducted if a student submits or reviews projects or assignments late after the due date has passed.The Late Policy feature is implemented by applying the policy to an assignment through the Due Dates tab during creation or editing of an assignment. The goal of this project is to address certain comments and issues associated with the solutions to the problems related to "Back" link on the "New Late Policy" page by making appropriate modifications and adding comprehensive test cases that cover all possible scenarios.

Project Scope

Aim of this Project

Te aim of this project is listed as follows:

1. Improving code structure and documentation via refactoring
2. Adding code comments for parts of code refactored or improved
3. Improving code readability
4. Writing tests for any changes made
5. Removing redundant or unused code

Problems Resolved in the Current Implementation

Work Flow Diagram

Issues

The following problems have been covered and corrected in the current implementation:

Issue 1: Removing Error Message

Under the “Due Date” tab when click on the "New late policy" link an error message shows up.

Issue 2: Fixing Back Link

When creating a late policy, the “back” link does not take the user back to editing the assignment.

Issue 3: Fixing Back Link upon creation

After creating a late policy, the “back” link does not take the user back to editing the assignment.

Resolution

The following are the screenshots after the issues were resolved within the current implementation:

• Issue 1 - Resolution where the error message doesn't show up.

• Issue 2 - Displaying back link takes back to late policies

• Issue 3 - After creation of late policy, back link takes back to late policies

Design Principle

The aim of our project is to improve upon the existing implementation by addressing the main issue which is fixing the backlinks and the error message displayed in Late Due Policy, which is explained at detail in the coming sections.

We checked review comments provided for the last implementation and improved upon it. Hence we will not be updating existing design patterns, and instead will be introducing refactoring within the existing code, by identifying DRY codes wherever possible and apply KISS principle if necessary.
Refactoring is the process of improving the structure, design, and/or implementation of an existing codebase without changing its external behavior. It involves making small, incremental changes to the code to make it more maintainable, easier to understand, and more efficient.

Solution and Files Refactored

Files Refactored or Modified

• assignment_form.rb
• assignments_controller.rb
• late_policy_spec.rb
• late_policy_features_spec.rb

Changes made

We made the following refactorings within the existing solution to improve code readability and maintain sound workflow.

The code in assignments_controller.rb that is displaying an error in New late policy & breaking the backlinks was fixed. The error was due to the parameter assignment_id not being present in session parameter list. It was added and made necessary changes wherever the assignment_id was being rendered. This fix is present within the current implementation.


For the scope of our project, we identified several code areas where the code can be DRIED or KISS principles can be applied.

• We extracted the method that is responsible for maintaining a check on duplicate assignment name and directory path for a given course ID and applied DRY - KISS principles. To ensure that the create method is readable and responsible for carrying out only a single responsibility of creating the assignment, we have separated the checking responsibilities from the code:

• To improve the correctness of the code and avoid any illegal modifications of error messages, we have change the access modifier for the attribute error messages:

• To improve the code readability, we have refactored the local variable names used within the assignment form as follows:

Testing

RSpec Unit Tests

We plan to provide RSpec Unit Tests within the Late Policy Spec page on top of the existing tests in the form of simple unit tests to ensure that our changes and modifications do not break the working logic of the existing implementation.

Tests

Sr No	Test Description
1	Test for flash error message on new late policy page
2	Test for back button interaction on new late policy page
3	Test for back button interaction on new late policy page while creating

• To test for flash error message on new late policy page, we have written this test. It creates a new copy of the old_assignment with a different name, directory_path, and copy_flag attributes. The test ensures that the new_assignment_id is not nil, which indicates that a new copy of the old_assignment was created successfully.

  # Test for flash error message on new late policy page
  it 'Flash Error Message on New Late Policy Page', js: true do
    assignment = Assignment.where(name: 'assignment for late policy test').first
    create(:topic, assignment_id: assignment.id)
    visit "/assignments/#{assignment.id}/edit"
    click_link 'Due Dates'
    click_link 'New Late Policy'
    fails if
      expect(flash[:error]).to be('Failed to save the assignment: ')
  end

• To test for back button interaction on new late policy page, this test has been written. It verifies that the "Back" button on the new late policy page correctly takes the user back to the edit page of the Assignment. The test uses expect method to assert that the page is routed to the correct URL after clicking the "Back" button.

# 
it 'Back Button Interation on New Late Policy Page', js: true do
  assignment = Assignment.where(name: 'assignment for late policy test').first
  create(:topic, assignment_id: assignment.id)
  visit "/assignments/#{assignment.id}/edit"
  click_link 'Due Dates'
  click_button 'New Late Policy'
  click_button 'Back'
  expect(page).to route_to("/assignments/#{assignment.id}/edit")
end

• This test was written for back button interaction on new late policy page while creating. This test verifies that the "Back" button on the new late policy page correctly takes the user back to the edit page of the Assignment while creating a late policy. The test uses RSpec's expect method to assert that the page is routed to the correct URL after clicking the "Back" button.

it 'Back Button Interation on New Late Policy Page while creating', js: true do
  assignment = Assignment.where(name: 'assignment for late policy test').first
  create(:topic, assignment_id: assignment.id)
  visit "/assignments/#{assignment.id}/edit"
  click_link 'Due Dates'
  click_button 'New Late Policy'
  fill_in "policy_name", with: 'Test Late Policy'
  fill_in "penalty_per_unit", with: '15'
  fill_in "max_penalty", with: '20'
  click_button 'Create'
  visit "/late_policies"
  click_button 'Back'
  expect(page).to route_to("/assignments/#{assignment.id}/edit")
end

Manual Testing

When logging in as an instructor, use the following credentials:

username: administrator_star
password: password

Please follow these steps:
1. Edit an assignment by clicking on edit logo under the “Action” column.
2. Under the “Due Date” tab click on the "New late policy" link.
3. In “New late policy” fill in the required details.
4. Clicks “Create” to save the policy, to go to page which shows all late policies.
5. Clicks “Back” to redirect the instructor back to the “Due Date” tab of the assignment which was being edited.
Or
4. Clicking “Back”, should redirect the instructor back to the “Due Date” tab of the assignment which was being edited.

Team

Mentor

Divyang Doshi(ddoshi2@ncsu.edu)

Team Members

Amisha Bipin Waghela (awaghel@ncsu.edu)
Sasank Marabattula (smaraba@ncsu.edu)
Srilekha Gudipati (sngudipa@ncsu.edu)

Reference

Github PR for E2327

Expertiza_wiki

Expertiza Documentation

Expertiza Github