Canvas Integration Tests (at long last)

[bsharplydian]

Overview

Added further Canvas integration tests, with several dependencies. Resolves #65. Resolves #649.

Details

• wrote tests for each of the CanvasIntegrationImpl functions.
• coverage: 100% Class, 96% Method, 90% Line, and 60% Branch.
• added default values for RubricConfigDao
• thoroughly investigated the possibility of getting course information at runtime instead of using hardcoded values (I think it's possible and I opened an issue for it)

Testing

• Added/updated unit tests
• Tested edge cases
• Manual testing (if needed)

Future Work

• Find a way to get the course number dynamically, as described in Get Course Number Dynamically from Canvas Tests #636.
• Use the default Rubric Config values for MySQL to make setting up the autograder environment easier, as described in Automatically configure Rubric Config in MySQL #650.

[mewilker]

I didn't see the tests running in the actions, were you planning to do that in this PR? Also, if it does run in the actions, will it print student data when something goes wrong? We need to make sure to sensor that.

[mewilker]

I believe these tests are good. We still need to integrate this into the pipeline, as well as address the comments mentioned above, but I know @bsharplydian is itching to be done with this work.

[ZakkeryDaRebel]

In addition to checking to see if netids is not null, should we also check to make sure it has a list of greater than 1 student? This way we are ensuring that it is grabbing multiple net IDs?

[ZakkeryDaRebel]

It all looks good, I just had 1 comment on one of the tests to ensure it is returning multiple like the test states. But it isn't all that required. Other than that, everything looked good.