Re-order LRS.VoidStatement, Add Unit Test

Author: brianjmiller

TinCan/ILRS.cs

@@ -25,12 +25,12 @@ public interface ILRS
         AboutLRSResponse About();
 
         StatementLRSResponse SaveStatement(Statement statement);
+        StatementLRSResponse VoidStatement(Guid id, Agent agent);
         StatementsResultLRSResponse SaveStatements(List<Statement> statements);
         StatementLRSResponse RetrieveStatement(Guid id);
         StatementLRSResponse RetrieveVoidedStatement(Guid id);
         StatementsResultLRSResponse QueryStatements(StatementsQuery query);
         StatementsResultLRSResponse MoreStatements(StatementsResult result);
-        StatementLRSResponse VoidStatement(Guid id, Agent agent);
 
         ProfileKeysLRSResponse RetrieveStateIds(Activity activity, Agent agent, Nullable<Guid> registration = null);
         StateLRSResponse RetrieveState(String id, Activity activity, Agent agent, Nullable<Guid> registration = null);

TinCan/RemoteLRS.cs

@@ -425,6 +425,22 @@ public StatementLRSResponse SaveStatement(Statement statement)
 
             return r;
         }
+        public StatementLRSResponse VoidStatement(Guid id, Agent agent)
+        {
+            var voidStatement = new Statement
+            {
+                actor = agent,
+                verb = new Verb
+                {
+                    id = new Uri("http://adlnet.gov/expapi/verbs/voided"),
+                    display = new LanguageMap()
+                },
+                target = new StatementRef { id = id }
+            };
+            voidStatement.verb.display.Add("en-US", "voided");
+
+            return SaveStatement(voidStatement);
+        }
         public StatementsResultLRSResponse SaveStatements(List<Statement> statements)
         {
             var r = new StatementsResultLRSResponse();
@@ -524,23 +540,6 @@ public StatementsResultLRSResponse MoreStatements(StatementsResult result)
 
             return r;
         }
-        
-        public StatementLRSResponse VoidStatement(Guid id, Agent agent)
-        {
-            var voidStatement = new Statement
-            {
-                actor = agent,
-                verb = new Verb
-                {
-                    id = new Uri("http://adlnet.gov/expapi/verbs/voided"),
-                    display = new LanguageMap()
-                },
-                target = new StatementRef { id = id }
-            };
-            voidStatement.verb.display.Add("en-US", "voided");
-
-            return SaveStatement(voidStatement);
-        }
 
         // TODO: since param
         public ProfileKeysLRSResponse RetrieveStateIds(Activity activity, Agent agent, Nullable<Guid> registration = null)

TinCanTests/RemoteLRSResourceTest.cs

@@ -122,6 +122,17 @@ public void TestSaveStatementSubStatement()
             Assert.AreEqual(statement, lrsRes.content);
         }
 
+        [Test]
+        public void TestVoidStatement()
+        {
+            Guid toVoid = Guid.NewGuid();
+            StatementLRSResponse lrsRes = lrs.VoidStatement(toVoid, Support.agent);
+
+            Assert.IsTrue(lrsRes.success, "LRS response successful");
+            Assert.AreEqual(new Uri("http://adlnet.gov/expapi/verbs/voided"), lrsRes.content.verb.id, "voiding statement uses voided verb");
+            Assert.AreEqual(toVoid, ((StatementRef) lrsRes.content.target).id, "voiding statement target correct id");
+        }
+
         [Test]
         public void TestSaveStatements()
         {