aws-samples / enable-lti

https://aws-samples.github.io/enable-lti/
Apache License 2.0
10 stars 2 forks source link

Refactor to improve readability and unit test coverage #20

Closed avikaza closed 10 months ago

avikaza commented 11 months ago

Description of changes: These are updates from our AWS T&C fork of this library. We improved code readability, unit-test coverage and added ability to submit score without the need to have a JWT token as part of the request. Learning tool developers must make sure that the accounts that can call this endpoint are controlled.

Testing: Unit tests pass with a unit-test coverage of 87%.
Integration tests passed. Deployed to Test Canvas instance and tested DeepLinking and Launch flows.
( https://educatetest.instructure.com/login/canvas | devkey name: eLTIOSSAvinashKey | client Id: 96950000000000236 )

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.