Merge pull request #79 from jakednoble/master

fixed saveActivty bug, added corresponding test to all save document reqs

Author: brianjmiller

test/remote_lrs_test.py

@@ -485,10 +485,19 @@ def test_save_state(self):
             id="test",
             content=bytearray("Test value", encoding="utf-8")
         )
-        response = self.lrs.save_state(doc)
+        saveResp = self.lrs.save_state(doc)
 
+        self.assertIsInstance(saveResp, LRSResponse)
+        self.assertTrue(saveResp.success)
+
+        response = self.lrs.retrieve_state(
+            activity=self.activity,
+            agent=self.agent,
+            state_id="test"
+        )
         self.assertIsInstance(response, LRSResponse)
         self.assertTrue(response.success)
+        self._vars_verifier(response.content, doc)
 
     def test_delete_state(self):
         doc = StateDocument(
@@ -525,10 +534,19 @@ def test_save_activity_profile(self):
             id="test",
             content=bytearray("Test value", encoding="utf-8")
         )
-        response = self.lrs.save_activity_profile(doc)
+        saveResp = self.lrs.save_activity_profile(doc)
+
+        self.assertIsInstance(saveResp, LRSResponse)
+        self.assertTrue(saveResp.success)
 
+        response = self.lrs.retrieve_activity_profile(
+            activity=self.activity,
+            profile_id="test"
+        )
         self.assertIsInstance(response, LRSResponse)
         self.assertTrue(response.success)
+        self._vars_verifier(response.content, doc)
+
 
     def test_delete_activity_profile(self):
         doc = ActivityProfileDocument(activity=self.activity, id="test")
@@ -553,9 +571,18 @@ def test_save_agent_profile(self):
             id="test",
             content=bytearray("Test value", encoding="utf-8")
         )
-        response = self.lrs.save_agent_profile(doc)
+        saveResp = self.lrs.save_agent_profile(doc)
+        self.assertIsInstance(saveResp, LRSResponse)
+        self.assertTrue(saveResp.success)
+
+        response = self.lrs.retrieve_agent_profile(
+            agent=self.agent,
+            profile_id="test"
+        )
         self.assertIsInstance(response, LRSResponse)
         self.assertTrue(response.success)
+        self._vars_verifier(response.content, doc)
+
 
     def test_delete_agent_profile(self):
         doc = AgentProfileDocument(agent=self.agent, id="test")

tincan/remote_lrs.py

@@ -654,7 +654,7 @@ def save_activity_profile(self, profile):
         request = HTTPRequest(
             method="PUT",
             resource="activities/profile",
-            content=profile.content_type
+            content=profile.content
         )
         if profile.content_type is not None:
             request.headers["Content-Type"] = profile.content_type