Save test case on assigning a test data source

Dropping a test data source on test case should update the test case.

Author: Nfactor26

src/Pixel.Automation.Designer.ViewModels/DragDropHandlers/ComponentDragHandler.cs

@@ -10,13 +10,11 @@
 using Pixel.Automation.Core.Components;
 using Pixel.Automation.Core.Components.Prefabs;
 using Pixel.Automation.Core.Components.Sequences;
-using Pixel.Automation.Core.Components.TestCase;
 using Pixel.Automation.Core.Enums;
 using Pixel.Automation.Core.Interfaces;
 using Pixel.Automation.Editor.Core;
 using Pixel.Automation.Editor.Core.Interfaces;
 using Pixel.Automation.Editor.Core.ViewModels;
-using Pixel.Scripting.Editor.Core.Contracts;
 using Serilog;
 using System.IO;
 using System.Windows;

src/Pixel.Automation.TestExplorer.ViewModels/Pixel.Automation.TestExplorer.ViewModels.csproj

@@ -10,6 +10,7 @@
 
   <ItemGroup>
     <PackageReference Include="Caliburn.Micro" Version="4.0.212" />
+    <PackageReference Include="gong-wpf-dragdrop" Version="3.2.0" />
   </ItemGroup>
 
   <ItemGroup>

src/Pixel.Automation.TestExplorer.Views/DragDropHandler/TestDataSourceDropHandler.cs renamed to src/Pixel.Automation.TestExplorer.ViewModels/TestDataSourceDropHandler.cs

@@ -1,6 +1,8 @@
-using GongSolutions.Wpf.DragDrop;
+using Dawn;
+using GongSolutions.Wpf.DragDrop;
 using Pixel.Automation.Core.TestData;
 using Pixel.Automation.TestExplorer.ViewModels;
+using Pixel.Persistence.Services.Client.Interfaces;
 using Serilog;
 using System;
 using System.Windows;
@@ -9,6 +11,14 @@ namespace Pixel.Automation.TestExplorer.Views
 {
     public class TestDataSourceDropHandler : IDropTarget
     {
+        private readonly ILogger logger = Log.ForContext<TestDataSourceDropHandler>();
+        private readonly ITestCaseManager testCaseManager;
+
+        public TestDataSourceDropHandler(ITestCaseManager testCaseManager)
+        {
+            this.testCaseManager = Guard.Argument(testCaseManager, nameof(testCaseManager)).NotNull().Value;
+        }
+
         public void DragOver(IDropInfo dropInfo)
         {
             if (dropInfo.Data != null)
@@ -33,12 +43,13 @@ public void Drop(IDropInfo dropInfo)
                 if (dropInfo.Data is TestDataSource testDataSource)
                 {
                     var testCase = (dropInfo.VisualTarget as FrameworkElement).DataContext as TestCaseViewModel;
-                    testCase.SetTestDataSource(testDataSource.DataSourceId);             
+                    testCase.SetTestDataSource(testDataSource.DataSourceId);
+                    _ = this.testCaseManager.UpdateTestCaseAsync(testCase.TestCase);
                 }
             }
             catch (Exception ex)
             {
-                Log.Error(ex, ex.Message);
+                logger.Error(ex, ex.Message);
             }
         }
     }

src/Pixel.Automation.TestExplorer.ViewModels/TestExplorerViewModel.cs

@@ -1,12 +1,14 @@
 using Caliburn.Micro;
 using Dawn;
+using GongSolutions.Wpf.DragDrop;
 using Pixel.Automation.Core;
 using Pixel.Automation.Core.Components.TestCase;
 using Pixel.Automation.Core.Enums;
 using Pixel.Automation.Core.Interfaces;
 using Pixel.Automation.Core.TestData;
 using Pixel.Automation.Editor.Core.Interfaces;
 using Pixel.Automation.Editor.Notifications;
+using Pixel.Automation.TestExplorer.Views;
 using Pixel.Persistence.Services.Client.Interfaces;
 using Pixel.Scripting.Editor.Core;
 using Pixel.Scripting.Editor.Core.Contracts;
@@ -47,6 +49,8 @@ public class TestExplorerViewModel : Screen, ITestExplorer, IHandle<TestFilterNo
 
         public ITestRunner TestRunner { get; private set; }
 
+        public IDropTarget TestDataSourceDropHandler { get;}
+
         /// <summary>
         /// A collection of TestFixtures belonging to the Automation Project
         /// </summary>
@@ -100,6 +104,7 @@ public TestExplorerViewModel(IEventAggregator eventAggregator, IAutomationProjec
             this.platformProvider = Guard.Argument(platformProvider).NotNull().Value;
             this.applicationSettings = Guard.Argument(applicationSettings).NotNull();
             this.eventAggregator.SubscribeOnPublishedThread(this);
+            this.TestDataSourceDropHandler = new TestDataSourceDropHandler(this.testCaseManager);
 
             CreateDefaultView();
         }

src/Pixel.Automation.TestExplorer.Views/TestExplorerView.xaml

@@ -20,8 +20,6 @@
         <converters:TrimmedTextConverter x:Key="trimmedTextConverter"/>
         <BooleanToVisibilityConverter x:Key="boolToVisConverter"/>
 
-        <local:TestDataSourceDropHandler x:Key="dataSourceDropHandler"/>
-
         <HierarchicalDataTemplate DataType="{x:Type localVM:TestFixtureViewModel}" ItemsSource="{Binding Tests}">
         </HierarchicalDataTemplate>
 
@@ -83,7 +81,7 @@
                     <ControlTemplate TargetType="TreeViewItem">
                         <Border x:Name="TestCase" ToolTip="{Binding Description}" ToolTipService.Placement="Right">
                             <StackPanel Orientation="Vertical" dd:DragDrop.IsDropTarget="True"  
-                                        dd:DragDrop.DropHandler="{StaticResource dataSourceDropHandler}">
+                                        dd:DragDrop.DropHandler="{Binding DataContext.TestDataSourceDropHandler, ElementName=TestExplorerTreeView}">
                                 <StackPanel Orientation="Horizontal" Margin="10 0 0 0" HorizontalAlignment="Left" VerticalAlignment="Center"
                                         Background="{DynamicResource MahApps.Brushes.Control.Background}"
                                         Tag="{Binding DataContext, ElementName=TestExplorerTreeView}">