Adds code coverage for public members of ChangeToolStripParentVerb.

Related dotnet#10773

- Adds code coverage for public members of `ChangeToolStripParentVerb`.

- None

- No

- Minimal

- Unit tests

- 10.0.100-preview.3.25201.16

Author: Ricardo Bossan (BEYONDSOFT CONSULTING INC) (from Dev Box)

src/System.Windows.Forms.Design/tests/UnitTests/System/Windows/Forms/Design/ChangeToolStripParentVerbTests.cs

@@ -9,65 +9,65 @@ namespace System.Windows.Forms.Design.Tests;
 
 public class ChangeToolStripParentVerbTests : IDisposable
 {
+    private readonly Mock<IDesignerHost> _designerHostMock = new();
+    private readonly Mock<IServiceProvider> _serviceProviderMock = new();
+    private readonly Mock<ISite> _siteMock = new();
+    private readonly Mock<IComponentChangeService> _componentChangeServiceMock = new()  ;
     private readonly ToolStripDesigner _designer = new();
-    private readonly ToolStrip _toolStrip;
-
-    public ChangeToolStripParentVerbTests()
-    {
-        _toolStrip = MockToolStrip();
-        _designer.Initialize(_toolStrip);
-    }
+    private ToolStrip _toolStrip = new();
 
     public void Dispose()
     {
         _toolStrip?.Dispose();
         _designer.Dispose();
     }
 
-    private ToolStrip MockToolStrip()
+    private ToolStrip MockMinimalControl()
     {
-        Mock<ISite> mockSite = new();
         Mock<ISelectionService> mockSelectionService = new();
-        mockSite.Setup(s => s.GetService(typeof(ISelectionService))).Returns(mockSelectionService.Object);
-
-        Mock<IDesignerHost> mockDesignerHost = new();
-        mockSite.Setup(s => s.GetService(typeof(IDesignerHost))).Returns(mockDesignerHost.Object);
-
-        Mock<IComponentChangeService> mockComponentChangeService = new();
-        mockDesignerHost
+        _siteMock
+            .Setup(s => s.GetService(typeof(ISelectionService)))
+            .Returns(mockSelectionService.Object);
+        _siteMock
+            .Setup(s => s.GetService(typeof(IDesignerHost)))
+            .Returns(_designerHostMock.Object);
+        _serviceProviderMock
+            .Setup(s => s.GetService(typeof(IDesignerHost)))
+            .Returns(_designerHostMock.Object);
+        _serviceProviderMock
+            .Setup(s => s.GetService(typeof(IComponentChangeService)))
+            .Returns(new Mock<IComponentChangeService>().Object);
+        _designerHostMock
             .Setup(dh => dh.GetService(typeof(IComponentChangeService)))
-            .Returns(mockComponentChangeService.Object);
+            .Returns(_componentChangeServiceMock.Object);
+        _designerHostMock
+            .Setup(dh => dh.GetDesigner(It.IsAny<IComponent>()))
+            .Returns(new Mock<ParentControlDesigner>().Object);
 
-        ToolStrip toolStrip = new() { Site = mockSite.Object };
+        ToolStrip toolStrip = new() { Site = _siteMock.Object };
 
         return toolStrip;
     }
 
     [Fact]
-    public void Constructor_WithNullDesigner_Throws()
+    public void ChangeParent_WithoutDesignerActionUIService_DoesNotChangeParent()
     {
-        Action action = () => new ChangeToolStripParentVerb(null);
-
-        action.Should().Throw<Exception>();
-    }
+        Control? oldParent = _toolStrip.Parent;
+        _toolStrip = MockMinimalControl();
+        _designer.Initialize(_toolStrip);
 
-    [Fact]
-    public void ChangeParent_DeepestPossibleTest()
-    {
-        var changeToolStripParentVerb = new ChangeToolStripParentVerb(_designer);
+        ChangeToolStripParentVerb changeToolStripParentVerb = new(_designer);
 
         changeToolStripParentVerb.ChangeParent();
-
-        throw new NotImplementedException();
+        Control? newParent = _toolStrip.Parent;
+        newParent.Should().Be(oldParent);
     }
 
     [Fact]
-    public void ChangeParent_WithNullRootDesigner_DoesNotChangesParent()
+    public void Constructor_WithNullDesigner_Throws()
     {
-        var changeToolStripParentVerb = new ChangeToolStripParentVerb(_designer);
-
-        changeToolStripParentVerb.ChangeParent();
+        Action action = () => new ChangeToolStripParentVerb(null);
 
-        throw new NotImplementedException();
+        action.Should().Throw<Exception>();
     }
 }