Skip to content

Commit c38178a

Browse files
committed
Fix LRS retrieve profile methods
* Similar fix to previous one for State where the profile itself wasn't attached to the LRS response object in the content * Added unit test assertion to confirm LRS response content is correct kind of requested document
1 parent 9ad2c6d commit c38178a

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

TinCan/RemoteLRS.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -655,6 +655,7 @@ public ActivityProfileLRSResponse RetrieveActivityProfile(String id, Activity ac
655655
return r;
656656
}
657657
r.success = true;
658+
r.content = profile;
658659

659660
return r;
660661
}
@@ -705,6 +706,7 @@ public AgentProfileLRSResponse RetrieveAgentProfile(String id, Agent agent)
705706
return r;
706707
}
707708
r.success = true;
709+
r.content = profile;
708710

709711
return r;
710712
}

TinCanTests/RemoteLRSResourceTest.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,7 @@ public void TestRetrieveState()
229229
{
230230
StateLRSResponse lrsRes = lrs.RetrieveState("test", Support.activity, Support.agent);
231231
Assert.IsTrue(lrsRes.success);
232+
Assert.IsInstanceOf<TinCan.Documents.StateDocument>(lrsRes.content);
232233
}
233234

234235
[Test]
@@ -275,6 +276,7 @@ public void TestRetrieveActivityProfile()
275276
{
276277
ActivityProfileLRSResponse lrsRes = lrs.RetrieveActivityProfile("test", Support.activity);
277278
Assert.IsTrue(lrsRes.success);
279+
Assert.IsInstanceOf<TinCan.Documents.ActivityProfileDocument>(lrsRes.content);
278280
}
279281

280282
[Test]
@@ -312,6 +314,7 @@ public void TestRetrieveAgentProfile()
312314
{
313315
AgentProfileLRSResponse lrsRes = lrs.RetrieveAgentProfile("test", Support.agent);
314316
Assert.IsTrue(lrsRes.success);
317+
Assert.IsInstanceOf<TinCan.Documents.AgentProfileDocument>(lrsRes.content);
315318
}
316319

317320
[Test]

0 commit comments

Comments
 (0)