fix for coord adding nonexistent user

[joshualiao]

In model.py, added a check to see if the user is a coordinator in the course.

Refactored the coordinator add when the student is not in the course. It now has the following functionality:

• If the student is a non-existent user and the course is not restricted, create a new user and enroll.
• If the student does exist, ensure they aren't associated with the course and the course is not restricted, then enroll them.

[cypress]

8 failed tests on run #271 ↗︎

8

70

0

0

0

Details:

(in process) fix for coord adding nonexistent user
Project: csm_web	Commit: b8c5721320
Status: Failed	Duration: 02:06 💡
Started: Nov 28, 2023 4:38 AM	Ended: Nov 28, 2023 4:40 AM

  cypress/e2e/section/coordinator-section.cy.ts • 4 failed tests • Tests on Firefox 106

View Output

Test		Artifacts
modifying students > valid operations > should allow adding new student to section		Screenshots
modifying students > valid operations > should allow adding new student associated with a new user		Screenshots
modifying students > invalid operations > should retry when section is at capacity		Screenshots
modifying students > invalid operations > should retry with multiple kinds of errors		Screenshots

  cypress/e2e/section/coordinator-section.cy.ts • 4 failed tests • Tests on Chrome 106

View Output Video

Test		Artifacts
modifying students > valid operations > should allow adding new student to section		Screenshots Video
modifying students > valid operations > should allow adding new student associated with a new user		Screenshots Video
modifying students > invalid operations > should retry when section is at capacity		Screenshots Video
modifying students > invalid operations > should retry with multiple kinds of errors		Screenshots Video

Review all test suite changes for PR #456 ↗︎

[smartspot2]

What's the status on this PR? Some of the more recent changes in master should fix some of the Cypress issues I think you were having.

Additionally, there are some syntax errors when the current code is run (check the changes in views/section.py), which should be fixed before this is merged.