Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.69%. Comparing base (e8d4cd0) to head (fe3a608).
⚠️ Report is 6 commits behind head on main.

@@            Coverage Diff            @@
##               main   #12066   +/-   ##
=========================================
  Coverage     21.69%   21.69%           
  Complexity     9587     9587           
=========================================
  Files           268      268           
  Lines         36593    36593           
  Branches        475      475           
=========================================
  Hits           7940     7940           
  Misses        28182    28182           
  Partials        471      471           

When a gradeable is created, the dropdown says the lowest privileged access user is instructor. However, it seems that the lowest privileged user is TA. In fact, after some testing, I was unable to make it so that a TA could not have grading access when selecting instructor in the dropdown.

I think the dropdown should start with the lowest privileged access user as TA, or at least automatically change to TA when the instructor clicks the button to open it to them.

Also, you mention that this PR will allow graders to view a submitted assignment in office hours, but wouldn't that only be for the sections they are a part of?

Just to confirm @JManion32 was that behavior not present on main or does my dropdown keep replicating existing buggy behavior?

When a gradeable is created, the dropdown says the lowest privileged access user is instructor. However, it seems that the lowest privileged user is TA. In fact, after some testing, I was unable to make it so that a TA could not have grading access when selecting instructor in the dropdown.
I think the dropdown should start with the lowest privileged access user as TA, or at least automatically change to TA when the instructor clicks the button to open it to them.
Also, you mention that this PR will allow graders to view a submitted assignment in office hours, but wouldn't that only be for the sections they are a part of?

Just to confirm @JManion32 was that behavior not present on main or does my dropdown keep replicating existing buggy behavior?

You're right, this is existing buggy behavior.