Implement DeleteRowObject functionality

dotnet/RarelySimple.AvatarScriptLink.Net.Tests/Decorators/FieldObjectDecoratorTests.cs

@@ -6,6 +6,16 @@
     [TestClass]
     public class FieldObjectDecoratorTests
     {
+        #region Constructor
+
+        [TestMethod]
+        public void TestFieldObjectDecorator_Constructor_NullFieldObject()
+        {
+            Assert.ThrowsException<ArgumentNullException>(() => new FieldObjectDecorator((FieldObject)null));
+        }
+
+        #endregion
+
         [TestMethod]
         public void TestFieldObjectIsNotModified()
         {

dotnet/RarelySimple.AvatarScriptLink.Net.Tests/Decorators/FormObjectDecoratorTests.cs

@@ -8,6 +8,16 @@
 [TestClass]
 public class FormObjectDecoratorTests
 {
+    #region Constructor
+
+    [TestMethod]
+    public void TestFormObjectDecorator_Constructor_NullFormObject()
+    {
+        Assert.ThrowsException<ArgumentNullException>(() => new FormObjectDecorator(null));
+    }
+
+    #endregion
+
     [TestMethod]
     public void TestFormObjectDecorator_ReturnsNoRows()
     {
@@ -1036,6 +1046,67 @@
 
     #endregion
 
+    #region DeleteRowObject
+
+    [TestMethod]
+    public void TestFormObjectDecorator_DeleteRowObject_CurrentRow()
+    {
+        var rowObject = new RowObject()
+        {
+            RowId = "1||1"
+        };
+        var formObject = new FormObject()
+        {
+            FormId = "1",
+            CurrentRow = rowObject
+        };
+        var decorator = new FormObjectDecorator(formObject);
+        decorator.DeleteRowObject("1||1");
+        Assert.AreEqual(RowActions.Delete, decorator.CurrentRow.RowAction);
+    }
+
+    [TestMethod]
+    public void TestFormObjectDecorator_DeleteRowObject_OtherRow()
+    {
+        var row1 = new RowObject() { RowId = "1||1" };
+        var row2 = new RowObject() { RowId = "1||2" };
+        var formObject = new FormObject()
+        {
+            FormId = "1",
+            CurrentRow = row1,
+            MultipleIteration = true,
+            OtherRows = [row2]
+        };
+        var decorator = new FormObjectDecorator(formObject);
+        decorator.DeleteRowObject("1||2");
+        Assert.AreEqual(RowActions.Delete, decorator.OtherRows[0].RowAction);
+    }
+
+    [TestMethod]
+    public void TestFormObjectDecorator_DeleteRowObject_NullRowId()
+    {
+        var formObject = new FormObject()
+        {
+            FormId = "1"
+        };
+        var decorator = new FormObjectDecorator(formObject);
+        Assert.ThrowsException<ArgumentNullException>(() => decorator.DeleteRowObject((string)null));
+    }
+
+    [TestMethod]
+    public void TestFormObjectDecorator_DeleteRowObject_NotFound()
+    {
+        var formObject = new FormObject()
+        {
+            FormId = "1",
+            CurrentRow = new RowObject() { RowId = "1||1" }
+        };
+        var decorator = new FormObjectDecorator(formObject);
+        Assert.ThrowsException<ArgumentException>(() => decorator.DeleteRowObject("1||2"));
+    }
+
+    #endregion
+
     #region Edge Case Tests - Regression & Coverage
 
     [TestMethod]

dotnet/RarelySimple.AvatarScriptLink.Net.Tests/Decorators/OptionObject2015DecoratorTests.cs

@@ -8,6 +8,16 @@ namespace RarelySimple.AvatarScriptLink.Net.Tests.Decorators;
 [TestClass]
 public class OptionObject2015DecoratorTests
 {
+    #region Constructor
+
+    [TestMethod]
+    public void TestOptionObject2015Decorator_Constructor_NullOptionObject()
+    {
+        Assert.ThrowsException<ArgumentNullException>(() => new OptionObject2015Decorator(null));
+    }
+
+    #endregion
+
     [TestMethod]
     public void TestOptionObject2015Decorator_ReturnsNoForms()
     {
@@ -1389,6 +1399,67 @@ public void IsRowMarkedForDeletion_NullRowId()
 
     #endregion
 
+    #region DeleteRowObject
+
+    [TestMethod]
+    public void TestOptionObject2015Decorator_DeleteRowObject_CurrentRow()
+    {
+        var rowObject = new RowObject()
+        {
+            RowId = "456||1"
+        };
+        var formObject = new FormObject()
+        {
+            FormId = "456",
+            CurrentRow = rowObject
+        };
+        var optionObject = new OptionObject2015()
+        {
+            OptionId = "TEST123",
+            Forms = [formObject]
+        };
+        var decorator = new OptionObject2015Decorator(optionObject);
+        decorator.DeleteRowObject("456", "456||1");
+        Assert.AreEqual(RowActions.Delete, decorator.Forms[0].CurrentRow.RowAction);
+    }
+
+    [TestMethod]
+    public void TestOptionObject2015Decorator_DeleteRowObject_Errors()
+    {
+        var optionObject = new OptionObject2015()
+        {
+            OptionId = "TEST123"
+        };
+        var decorator = new OptionObject2015Decorator(optionObject);
+        Assert.ThrowsException<ArgumentNullException>(() => decorator.DeleteRowObject(null, "1||1"));
+        Assert.ThrowsException<ArgumentNullException>(() => decorator.DeleteRowObject("456", null));
+        Assert.ThrowsException<ArgumentException>(() => decorator.DeleteRowObject("999", "1||1"));
+    }
+
+    [TestMethod]
+    public void TestOptionObject2015Decorator_DeleteRowObject_OtherRow()
+    {
+        var row1 = new RowObject() { RowId = "456||1" };
+        var row2 = new RowObject() { RowId = "456||2" };
+        var formObject = new FormObject()
+        {
+            FormId = "456",
+            CurrentRow = row1,
+            MultipleIteration = true,
+            OtherRows = [row2]
+        };
+        var optionObject = new OptionObject2015()
+        {
+            OptionId = "TEST123",
+            Forms = [formObject]
+        };
+        var decorator = new OptionObject2015Decorator(optionObject);
+        decorator.DeleteRowObject("456", "456||2");
+        Assert.AreEqual(RowActions.Delete, decorator.Forms[0].OtherRows[0].RowAction);
+    }
+
+    #endregion
+
     #region IsRowPresent
 
     [TestMethod]
@@ -1641,15 +1712,5 @@ public void TestOptionObject2015Decorator_Constructor_WithEmptyFormsCollection()
         Assert.AreEqual(0, decorator.Forms.Count);
     }
 
-    [TestMethod]
-    public void TestOptionObject2015Decorator_AddRowObject_RegressionNullFormIdHandling()
-    {
-        var formObject = new FormObject() { FormId = "1", MultipleIteration = true };
-        var optionObject = new OptionObject2015() { OptionId = "TEST123", Forms = [formObject] };
-        var decorator = new OptionObject2015Decorator(optionObject);
-        var rowObject = new RowObject();
-        Assert.ThrowsException<ArgumentNullException>(() => decorator.AddRowObject(null, rowObject));
-    }
-
     #endregion
 }

dotnet/RarelySimple.AvatarScriptLink.Net.Tests/Decorators/OptionObject2DecoratorTests.cs

@@ -8,6 +8,16 @@ namespace RarelySimple.AvatarScriptLink.Net.Tests.Decorators;
 [TestClass]
 public class OptionObject2DecoratorTests
 {
+    #region Constructor
+
+    [TestMethod]
+    public void TestOptionObject2Decorator_Constructor_NullOptionObject()
+    {
+        Assert.ThrowsException<ArgumentNullException>(() => new OptionObject2Decorator(null));
+    }
+
+    #endregion
+
     [TestMethod]
     public void TestOptionObject2Decorator_ReturnsNoForms()
     {
@@ -1406,6 +1416,67 @@ public void IsRowMarkedForDeletion_NullRowId()
 
     #endregion
 
+    #region DeleteRowObject
+
+    [TestMethod]
+    public void TestOptionObject2Decorator_DeleteRowObject_CurrentRow()
+    {
+        var rowObject = new RowObject()
+        {
+            RowId = "456||1"
+        };
+        var formObject = new FormObject()
+        {
+            FormId = "456",
+            CurrentRow = rowObject
+        };
+        var optionObject = new OptionObject2()
+        {
+            OptionId = "TEST123",
+            Forms = [formObject]
+        };
+        var decorator = new OptionObject2Decorator(optionObject);
+        decorator.DeleteRowObject("456", "456||1");
+        Assert.AreEqual(RowActions.Delete, decorator.Forms[0].CurrentRow.RowAction);
+    }
+
+    [TestMethod]
+    public void TestOptionObject2Decorator_DeleteRowObject_Errors()
+    {
+        var optionObject = new OptionObject2()
+        {
+            OptionId = "TEST123"
+        };
+        var decorator = new OptionObject2Decorator(optionObject);
+        Assert.ThrowsException<ArgumentNullException>(() => decorator.DeleteRowObject(null, "1||1"));
+        Assert.ThrowsException<ArgumentNullException>(() => decorator.DeleteRowObject("456", null));
+        Assert.ThrowsException<ArgumentException>(() => decorator.DeleteRowObject("999", "1||1"));
+    }
+
+    [TestMethod]
+    public void TestOptionObject2Decorator_DeleteRowObject_OtherRow()
+    {
+        var row1 = new RowObject() { RowId = "456||1" };
+        var row2 = new RowObject() { RowId = "456||2" };
+        var formObject = new FormObject()
+        {
+            FormId = "456",
+            CurrentRow = row1,
+            MultipleIteration = true,
+            OtherRows = [row2]
+        };
+        var optionObject = new OptionObject2()
+        {
+            OptionId = "TEST123",
+            Forms = [formObject]
+        };
+        var decorator = new OptionObject2Decorator(optionObject);
+        decorator.DeleteRowObject("456", "456||2");
+        Assert.AreEqual(RowActions.Delete, decorator.Forms[0].OtherRows[0].RowAction);
+    }
+
+    #endregion
+
     #region IsRowPresent
 
     [TestMethod]
@@ -1658,15 +1729,5 @@ public void TestOptionObject2Decorator_Constructor_WithEmptyFormsCollection()
         Assert.AreEqual(0, decorator.Forms.Count);
     }
 
-    [TestMethod]
-    public void TestOptionObject2Decorator_AddRowObject_RegressionNullFormIdHandling()
-    {
-        var formObject = new FormObject() { FormId = "1", MultipleIteration = true };
-        var optionObject = new OptionObject2() { OptionId = "TEST123", Forms = [formObject] };
-        var decorator = new OptionObject2Decorator(optionObject);
-        var rowObject = new RowObject();
-        Assert.ThrowsException<ArgumentNullException>(() => decorator.AddRowObject(null, rowObject));
-    }
-
     #endregion
 }