additional test coverage

Author: jsmnhou

canvasapi/course.py

@@ -459,7 +459,7 @@ def create_lti_resource_link(self, url, title=None, custom=None, **kwargs):
         """
 
         if not url:
-            raise RequiredFieldMissing("The 'url' paramter is required.")
+            raise RequiredFieldMissing("url is required as a str.")
 
         kwargs["url"] = url
         if title:

tests/fixtures/lti_resource_link.json

@@ -22,7 +22,8 @@
 		"endpoint": "courses/1/lti_resource_links/45",
 		"data": {
 			"id": 45,
-            "title": "Test LTI Resource Link"
+            "title": "Test LTI Resource Link",
+			"url": "https://example.com/lti/launch/content_item/123"
 		},
 		"status_code": 200
 	},

tests/test_course.py

@@ -1894,14 +1894,22 @@ def test_resolve_path_null(self, m):
     # create_lti_resource_link()
     def test_create_lti_resource_link(self, m):
         register_uris({"lti_resource_link": ["create_lti_resource_link"]}, m)
+        custom_dict = {"hello": "world"}
+
         evnt = self.course.create_lti_resource_link(
             url="https://example.com/lti/launch/content_item/123",
             title="Test LTI Resource Link",
+            custom=custom_dict,
         )
         self.assertIsInstance(evnt, LTIResourceLink)
+
         self.assertEqual(evnt.title, "Test LTI Resource Link")
         self.assertEqual(evnt.url, "https://example.com/lti/launch/content_item/123")
 
+    def test_create_lti_resource_link_fail(self, m):
+        with self.assertRaises(RequiredFieldMissing):
+            self.course.create_lti_resource_link({})
+
     # get_lti_resource_links()
     def test_get_lti_resource_links(self, m):
         register_uris({"lti_resource_link": ["list_lti_resource_links"]}, m)
@@ -1916,6 +1924,7 @@ def test_get_lti_resource_link(self, m):
         register_uris({"lti_resource_link": ["get_lti_resource_link"]}, m)
 
         lti_resource_link_by_id = self.course.get_lti_resource_link(45)
+
         self.assertIsInstance(lti_resource_link_by_id, LTIResourceLink)
         self.assertEqual(lti_resource_link_by_id.title, "Test LTI Resource Link")
         lti_resource_link_by_obj = self.course.get_lti_resource_link(

tests/test_lti_resource_link.py

@@ -0,0 +1,31 @@
+import unittest
+
+import requests_mock
+
+from canvasapi import Canvas
+from canvasapi.exceptions import RequiredFieldMissing
+from tests import settings
+from tests.util import register_uris
+
+
+@requests_mock.Mocker()
+class TestLTIResourceLink(unittest.TestCase):
+    def setUp(self):
+        self.canvas = Canvas(settings.BASE_URL, settings.API_KEY)
+
+        with requests_mock.Mocker() as m:
+            register_uris(
+                {
+                    "course": ["get_by_id"],
+                    "lti_resource_link": ["get_lti_resource_link"],
+                },
+                m,
+            )
+
+            self.course = self.canvas.get_course(1)
+            self.resource_link = self.course.get_lti_resource_link(45)
+
+    # __str__()
+    def test__str__(self, m):
+        string = str(self.resource_link)
+        self.assertIsInstance(string, str)