Add code coverage for ToolStripPanel_ToolStripPanelRowCollection (#12111

)

related #10453

Proposed changes
Add unit tests for ToolStripPanel_ToolStripPanelRowCollection to test its properties & methods & events

src/System.Windows.Forms/tests/UnitTests/System/Windows/Forms/ToolStripPanel.ToolStripPanelRowCollectionTests.cs

@@ -0,0 +1,333 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+#nullable enable
+
+using static System.Windows.Forms.ToolStripPanel;
+
+namespace System.Windows.Forms.Tests;
+
+public class ToolStripPanel_ToolStripPanelRowCollectionTests : IDisposable
+{
+    private readonly ToolStripPanel _toolStripPanel;
+    private readonly ToolStripPanelRowCollection _toolStripPanelRowCollection;
+
+    public ToolStripPanel_ToolStripPanelRowCollectionTests()
+    {
+        _toolStripPanel = new();
+        _toolStripPanelRowCollection = new(_toolStripPanel);
+    }
+
+    public void Dispose() => _toolStripPanel.Dispose();
+
+    private void DisposeRows(ToolStripPanelRow[]? rows)
+    {
+        if (rows is null)
+            return;
+
+        int length = rows.Length;
+        for (int i = 0; i < length; i++)
+        {
+            rows[i]?.Dispose();
+            rows[i] = null!;
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_ConstructorWithOwner_SetsOwner()
+    {
+        using ToolStripPanel toolStripPanel = _toolStripPanelRowCollection.TestAccessor().Dynamic._owner;
+        toolStripPanel.Should().BeSameAs(_toolStripPanel);
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_ConstructorWithOwnerAndRows_SetsOwnerAndAddsRows()
+    {
+        using ToolStripPanelRow toolStripPanelRow1 = new(_toolStripPanel);
+        ToolStripPanelRow[] toolStripPanelRowArray = { toolStripPanelRow1 };
+        ToolStripPanelRowCollection toolStripPanelRowCollection = new(_toolStripPanel, toolStripPanelRowArray);
+        ToolStripPanel toolStripPanel = toolStripPanelRowCollection.TestAccessor().Dynamic._owner;
+
+        toolStripPanel.Should().BeSameAs(_toolStripPanel);
+        toolStripPanelRowCollection.Count.Should().Be(1);
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_ConstructorWithOwnerAndValue_AddsRows()
+    {
+        var rows = AddRowsToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel, count: 2);
+
+        try
+        {
+            _toolStripPanelRowCollection.Count.Should().Be(2);
+            _toolStripPanelRowCollection[0].Should().Be(rows[0]);
+            _toolStripPanelRowCollection[1].Should().Be(rows[1]);
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_Add_AddsRow()
+    {
+        ToolStripPanelRow row = AddRowToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel);
+        ToolStripPanelRow[] rows = { row };
+
+        try
+        {
+            _toolStripPanelRowCollection.Count.Should().Be(1);
+            _toolStripPanelRowCollection[0].Should().Be(row);
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_Add_NullValue_ThrowsArgumentNullException()
+    {
+        Action action = () => _toolStripPanelRowCollection.Add(value: null!);
+
+        action.Should().Throw<ArgumentNullException>()
+            .WithMessage("Value cannot be null. (Parameter 'value')");
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_AddRange_NullValue_ThrowsArgumentNullException()
+    {
+        Action action = () => _toolStripPanelRowCollection.AddRange(value: (ToolStripPanelRow[])null!);
+
+        action.Should().Throw<ArgumentNullException>()
+            .WithMessage("Value cannot be null. (Parameter 'value')");
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_AddRange_Collection_AddsRows()
+    {
+        var rows = AddRowsToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel, count: 2);
+
+        try
+        {
+            _toolStripPanelRowCollection.AddRange(new ToolStripPanelRowCollection(_toolStripPanel) { rows[0], rows[1] });
+
+            _toolStripPanelRowCollection.Count.Should().Be(4);
+            _toolStripPanelRowCollection[2].Should().Be(rows[0]);
+            _toolStripPanelRowCollection[3].Should().Be(rows[1]);
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_AddRange_Collection_NullValue_ThrowsArgumentNullException()
+    {
+        Action action = () => _toolStripPanelRowCollection.AddRange(value: (ToolStripPanelRowCollection)null!);
+
+        action.Should().Throw<ArgumentNullException>()
+            .WithMessage("Value cannot be null. (Parameter 'value')");
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_Clear_RemovesAllRows()
+    {
+        var rows = AddRowsToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel, count: 2);
+
+        try
+        {
+            _toolStripPanelRowCollection.Count.Should().Be(2);
+
+            _toolStripPanelRowCollection.Clear();
+
+            _toolStripPanelRowCollection.Count.Should().Be(0);
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_Clear_EmptyCollection_DoesNothing()
+    {
+        _toolStripPanelRowCollection.Clear();
+
+        _toolStripPanelRowCollection.Count.Should().Be(0);
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_Remove_RemovesRow()
+    {
+        var row = AddRowToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel);
+        ToolStripPanelRow[] rows = { row };
+
+        try
+        {
+            _toolStripPanelRowCollection.Remove(row);
+
+            _toolStripPanelRowCollection.Count.Should().Be(0);
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_Remove_RowNotInCollection_DoesNothing()
+    {
+        var row1 = AddRowToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel);
+        ToolStripPanelRow row2 = new(_toolStripPanel);
+        ToolStripPanelRow[] rows = { row1, row2 };
+
+        try
+        {
+            _toolStripPanelRowCollection.Remove(row2);
+
+            _toolStripPanelRowCollection.Count.Should().Be(1);
+            _toolStripPanelRowCollection[0].Should().Be(row1);
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_RemoveAt_RemovesRowAtIndex()
+    {
+        var row1 = AddRowToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel);
+        var row2 = AddRowToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel);
+        ToolStripPanelRow[] rows = { row1, row2 };
+
+        try
+        {
+            _toolStripPanelRowCollection.RemoveAt(0);
+
+            _toolStripPanelRowCollection.Count.Should().Be(1);
+            _toolStripPanelRowCollection[0].Should().Be(row2);
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_RemoveAt_IndexOutOfRange_ThrowsArgumentOutOfRangeException()
+    {
+        var row = AddRowToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel);
+        ToolStripPanelRow[] rows = { row };
+
+        try
+        {
+            Action action = () => _toolStripPanelRowCollection.RemoveAt(1);
+
+            action.Should().Throw<ArgumentOutOfRangeException>()
+                .WithMessage("Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index')");
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_RemoveAt_EmptyCollection_ThrowsArgumentOutOfRangeException()
+    {
+        Action action = () => _toolStripPanelRowCollection.RemoveAt(0);
+
+        action.Should().Throw<ArgumentOutOfRangeException>()
+            .WithMessage("Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index')");
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_CopyTo_CopiesRowsToArray()
+    {
+        var rows = AddRowsToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel, count: 2);
+        var array = new ToolStripPanelRow[2];
+
+        try
+        {
+            _toolStripPanelRowCollection.CopyTo(array: array, index: 0);
+
+            array[0].Should().Be(rows[0]);
+            array[1].Should().Be(rows[1]);
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_CopyTo_NullArray_ThrowsArgumentNullException()
+    {
+        Action action = () => _toolStripPanelRowCollection.CopyTo(array: null!, index: 0);
+
+        action.Should().Throw<ArgumentNullException>()
+            .WithMessage("Value cannot be null. (Parameter 'destinationArray')");
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_CopyTo_IndexOutOfRange_ThrowsArgumentException()
+    {
+        var row = AddRowToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel);
+        ToolStripPanelRow[] rows = { row };
+        var array = new ToolStripPanelRow[1];
+
+        try
+        {
+            Action action = () => _toolStripPanelRowCollection.CopyTo(array: array, index: 1);
+
+            action.Should().Throw<ArgumentException>()
+                .WithMessage("Destination array was not long enough. Check the destination index, length, and the array's lower bounds. (Parameter 'destinationArray')");
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    [WinFormsFact]
+    public void ToolStripPanelRowCollection_CopyTo_ArrayTooSmall_ThrowsArgumentException()
+    {
+        var rows = AddRowsToCollection(collection: _toolStripPanelRowCollection, panel: _toolStripPanel, count: 2);
+        var array = new ToolStripPanelRow[1];
+
+        try
+        {
+            Action action = () => _toolStripPanelRowCollection.CopyTo(array: array, index: 0);
+
+            action.Should().Throw<ArgumentException>()
+                .WithMessage("Destination array was not long enough. Check the destination index, length, and the array's lower bounds. (Parameter 'destinationArray')");
+        }
+        finally
+        {
+            DisposeRows(rows);
+        }
+    }
+
+    private ToolStripPanelRow AddRowToCollection(ToolStripPanelRowCollection collection, ToolStripPanel panel)
+    {
+        ToolStripPanelRow row = new(panel);
+        collection.Add(row);
+        return row;
+    }
+
+    private ToolStripPanelRow[] AddRowsToCollection(ToolStripPanelRowCollection collection, ToolStripPanel panel, int count)
+    {
+        var rows = new ToolStripPanelRow[count];
+        for (int i = 0; i < count; i++)
+        {
+            rows[i] = new ToolStripPanelRow(panel);
+            collection.Add(rows[i]);
+        }
+
+        return rows;
+    }
+}