Browse Source

Fixed tests

CPK 4 years ago
parent
commit
02318037e6

+ 4 - 4
PixiEditorTests/ModelsTests/ControllersTests/BitmapManagerTests.cs

@@ -13,8 +13,8 @@ namespace PixiEditorTests.ModelsTests.ControllersTests
         public void TestThatBitmapManagerSetsCorrectTool()
         public void TestThatBitmapManagerSetsCorrectTool()
         {
         {
             BitmapManager bitmapManager = new BitmapManager();
             BitmapManager bitmapManager = new BitmapManager();
-            bitmapManager.SetActiveTool(new MockedSinglePixelPen());
-            Assert.Equal(ToolType.Pen, bitmapManager.SelectedTool.ToolType);
+            bitmapManager.SetActiveTool(new MockedSinglePixelPenTool());
+            Assert.Equal(typeof(MockedSinglePixelPenTool), bitmapManager.SelectedTool.GetType());
         }
         }
 
 
         [Fact]
         [Fact]
@@ -63,7 +63,7 @@ namespace PixiEditorTests.ModelsTests.ControllersTests
         [Fact]
         [Fact]
         public void TestThatIsOperationToolWorks()
         public void TestThatIsOperationToolWorks()
         {
         {
-            MockedSinglePixelPen singlePixelPen = new MockedSinglePixelPen();
+            MockedSinglePixelPenTool singlePixelPen = new MockedSinglePixelPenTool();
             Assert.True(BitmapManager.IsOperationTool(singlePixelPen));
             Assert.True(BitmapManager.IsOperationTool(singlePixelPen));
         }
         }
 
 
@@ -81,7 +81,7 @@ namespace PixiEditorTests.ModelsTests.ControllersTests
             bitmapManager.ActiveDocument = bitmapManager.Documents[0];
             bitmapManager.ActiveDocument = bitmapManager.Documents[0];
 
 
             bitmapManager.ActiveDocument.AddNewLayer("Layer");
             bitmapManager.ActiveDocument.AddNewLayer("Layer");
-            bitmapManager.SetActiveTool(new MockedSinglePixelPen());
+            bitmapManager.SetActiveTool(new MockedSinglePixelPenTool());
             bitmapManager.PrimaryColor = Colors.Green;
             bitmapManager.PrimaryColor = Colors.Green;
 
 
             bitmapManager.MouseController.StartRecordingMouseMovementChanges(true);
             bitmapManager.MouseController.StartRecordingMouseMovementChanges(true);

+ 1 - 1
PixiEditorTests/ModelsTests/ControllersTests/BitmapOperationsUtilityTests.cs

@@ -41,7 +41,7 @@ namespace PixiEditorTests.ModelsTests.ControllersTests
 
 
             List<Coordinates> mouseMove = new List<Coordinates>(new[] { new Coordinates(0, 0) });
             List<Coordinates> mouseMove = new List<Coordinates>(new[] { new Coordinates(0, 0) });
 
 
-            util.ExecuteTool(new Coordinates(0, 0), mouseMove, new MockedSinglePixelPen());
+            util.ExecuteTool(new Coordinates(0, 0), mouseMove, new MockedSinglePixelPenTool());
             Assert.Equal(manager.ActiveLayer.GetPixel(0, 0), Colors.Black);
             Assert.Equal(manager.ActiveLayer.GetPixel(0, 0), Colors.Black);
         }
         }
     }
     }

+ 3 - 5
PixiEditorTests/ModelsTests/ControllersTests/MockedSinglePixelPen.cs → PixiEditorTests/ModelsTests/ControllersTests/MockedSinglePixelPenTool.cs

@@ -1,15 +1,13 @@
-using System.Windows.Media;
-using PixiEditor.Models.DataHolders;
+using PixiEditor.Models.DataHolders;
 using PixiEditor.Models.Layers;
 using PixiEditor.Models.Layers;
 using PixiEditor.Models.Position;
 using PixiEditor.Models.Position;
 using PixiEditor.Models.Tools;
 using PixiEditor.Models.Tools;
+using System.Windows.Media;
 
 
 namespace PixiEditorTests.ModelsTests.ControllersTests
 namespace PixiEditorTests.ModelsTests.ControllersTests
 {
 {
-    public class MockedSinglePixelPen : BitmapOperationTool
+    public class MockedSinglePixelPenTool : BitmapOperationTool
     {
     {
-        public override ToolType ToolType { get; } = ToolType.Pen;
-
         public override LayerChange[] Use(Layer layer, Coordinates[] mouseMove, Color color)
         public override LayerChange[] Use(Layer layer, Coordinates[] mouseMove, Color color)
         {
         {
             return Only(BitmapPixelChanges.FromSingleColoredArray(new[] { mouseMove[0] }, color), 0);
             return Only(BitmapPixelChanges.FromSingleColoredArray(new[] { mouseMove[0] }, color), 0);

+ 0 - 2
PixiEditorTests/ModelsTests/ControllersTests/TestReadonlyTool.cs

@@ -13,8 +13,6 @@ namespace PixiEditorTests.ModelsTests.ControllersTests
 
 
         public Action ToolAction { get; set; }
         public Action ToolAction { get; set; }
 
 
-        public override ToolType ToolType => ToolType.Select;
-
         public override void Use(Coordinates[] pixels)
         public override void Use(Coordinates[] pixels)
         {
         {
             ToolAction();
             ToolAction();

+ 4 - 3
PixiEditorTests/ViewModelsTests/ViewModelMainTests.cs

@@ -6,6 +6,7 @@ using PixiEditor.Models.DataHolders;
 using PixiEditor.Models.IO;
 using PixiEditor.Models.IO;
 using PixiEditor.Models.Position;
 using PixiEditor.Models.Position;
 using PixiEditor.Models.Tools;
 using PixiEditor.Models.Tools;
+using PixiEditor.Models.Tools.Tools;
 using PixiEditor.ViewModels;
 using PixiEditor.ViewModels;
 using Xunit;
 using Xunit;
 
 
@@ -72,11 +73,11 @@ namespace PixiEditorTests.ViewModelsTests
         {
         {
             ViewModelMain viewModel = new ViewModelMain();
             ViewModelMain viewModel = new ViewModelMain();
 
 
-            Assert.Equal(ToolType.Move, viewModel.BitmapManager.SelectedTool.ToolType);
+            Assert.Equal(typeof(MoveTool), viewModel.BitmapManager.SelectedTool.GetType());
 
 
-            viewModel.ToolsSubViewModel.SelectToolCommand.Execute(ToolType.Line);
+            viewModel.ToolsSubViewModel.SelectToolCommand.Execute(typeof(LineTool));
 
 
-            Assert.Equal(ToolType.Line, viewModel.BitmapManager.SelectedTool.ToolType);
+            Assert.Equal(typeof(LineTool), viewModel.BitmapManager.SelectedTool.GetType());
         }
         }
 
 
         [StaFact]
         [StaFact]