Merge pull request #257 from pascalberger/feature/improve-testing

Allow to pass test management client factory

Author: pascalberger

src/Cake.AzureDevOps/AzureDevOpsAliases.Pipelines.cs

@@ -47,7 +47,7 @@ public static AzureDevOpsBuild AzureDevOpsBuild(
             context.NotNull(nameof(context));
             settings.NotNull(nameof(settings));
 
-            var build = new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory());
+            var build = new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory(), new TestManagementClientFactory());
 
             if (build.HasBuildLoaded)
             {
@@ -204,7 +204,7 @@ public static bool AzureDevOpsBuildIsFailing(
             settings.NotNull(nameof(settings));
 
             return
-                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory())
+                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory(), new TestManagementClientFactory())
                     .IsBuildFailing();
         }
 
@@ -252,7 +252,7 @@ public static IEnumerable<AzureDevOpsChange> AzureDevOpsBuildChanges(
             settings.NotNull(nameof(settings));
 
             return
-                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory())
+                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory(), new TestManagementClientFactory())
                     .GetChanges();
         }
 
@@ -300,7 +300,7 @@ public static IEnumerable<AzureDevOpsTimelineRecord> AzureDevOpsBuildTimelineRec
             settings.NotNull(nameof(settings));
 
             return
-                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory())
+                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory(), new TestManagementClientFactory())
                     .GetTimelineRecords();
         }
 
@@ -348,7 +348,7 @@ public static IEnumerable<AzureDevOpsBuildArtifact> AzureDevOpsBuildArtifacts(
             settings.NotNull(nameof(settings));
 
             return
-                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory())
+                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory(), new TestManagementClientFactory())
                     .GetArtifacts();
         }
 
@@ -396,7 +396,7 @@ public static IEnumerable<AzureDevOpsTestRun> AzureDevOpsBuildTestRuns(
             settings.NotNull(nameof(settings));
 
             return
-                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory())
+                new AzureDevOpsBuild(context.Log, settings, new BuildClientFactory(), new TestManagementClientFactory())
                     .GetTestRuns();
         }
     }

src/Cake.AzureDevOps/Pipelines/AzureDevOpsBuild.cs

@@ -29,7 +29,7 @@ public sealed class AzureDevOpsBuild
         /// <exception cref="AzureDevOpsBuildNotFoundException">If <see cref="AzureDevOpsBuildSettings.ThrowExceptionIfBuildCouldNotBeFound"/>
         /// is set to <c>true</c> and no build could be found.</exception>
         public AzureDevOpsBuild(ICakeLog log, AzureDevOpsBuildSettings settings)
-            : this(log, settings, new BuildClientFactory())
+            : this(log, settings, new BuildClientFactory(), new TestManagementClientFactory())
         {
         }
 
@@ -58,18 +58,24 @@ internal AzureDevOpsBuild(ICakeLog log, AzureDevOpsBuildsSettings settings, Buil
         /// </summary>
         /// <param name="log">The Cake log context.</param>
         /// <param name="settings">Settings for accessing AzureDevOps.</param>
-        /// <param name="buildClientFactory">A factory to communicate with Build client.</param>
+        /// <param name="buildClientFactory">A factory to communicate with build client.</param>
+        /// <param name="testManagementClientFactory">A factory to communicate with test management client.</param>
         /// <exception cref="AzureDevOpsBuildNotFoundException">If <see cref="AzureDevOpsBuildSettings.ThrowExceptionIfBuildCouldNotBeFound"/>
         /// is set to <c>true</c> and no build could be found.</exception>
-        internal AzureDevOpsBuild(ICakeLog log, AzureDevOpsBuildSettings settings, IBuildClientFactory buildClientFactory)
+        internal AzureDevOpsBuild(
+            ICakeLog log,
+            AzureDevOpsBuildSettings settings,
+            IBuildClientFactory buildClientFactory,
+            ITestManagementClientFactory testManagementClientFactory)
         {
             log.NotNull(nameof(log));
             settings.NotNull(nameof(settings));
             buildClientFactory.NotNull(nameof(buildClientFactory));
+            testManagementClientFactory.NotNull(nameof(testManagementClientFactory));
 
             this.log = log;
             this.buildClientFactory = buildClientFactory;
-            this.testClientFactory = new TestManagementClientFactory();
+            this.testClientFactory = testManagementClientFactory;
             this.credentials = settings.Credentials;
             this.CollectionUrl = settings.CollectionUrl;
             this.throwExceptionIfBuildCouldNotBeFound = settings.ThrowExceptionIfBuildCouldNotBeFound;