Move Location of Experiments Call to CD Internal

src/Microsoft.ComponentDetection.Orchestrator/Experiments/DetectorExperiments.cs

@@ -7,6 +7,11 @@ namespace Microsoft.ComponentDetection.Orchestrator.Experiments;
 /// </summary>
 public static class DetectorExperiments
 {
+    /// <summary>
+    /// Check to automatically proccess experiments.
+    /// </summary>
+    public static bool AutomaticallyProcessExperiments { get; set; }
+
     /// <summary>
     /// Manually enables detector experiments.
     /// </summary>

src/Microsoft.ComponentDetection.Orchestrator/Experiments/ExperimentService.cs

@@ -116,6 +116,11 @@ public async Task FinishAsync()
             return;
         }
 
+        if (!DetectorExperiments.AutomaticallyProcessExperiments)
+        {
+            return;
+        }
+
         foreach (var (config, experiment) in this.experiments)
         {
             var controlComponents = experiment.ControlGroupComponents;

src/Microsoft.ComponentDetection.Orchestrator/Services/DetectorProcessingService.cs

@@ -86,9 +86,9 @@ public async Task<DetectorProcessingResult> ProcessDetectorsAsync(IDetectionArgu
                     record.DetectedComponentCount = detectedComponents.Count();
                     var dependencyGraphs = componentRecorder.GetDependencyGraphsByLocation().Values;
                     record.ExplicitlyReferencedComponentCount = dependencyGraphs.Select(dependencyGraph =>
-                        {
-                            return dependencyGraph.GetAllExplicitlyReferencedComponents();
-                        })
+                    {
+                        return dependencyGraph.GetAllExplicitlyReferencedComponents();
+                    })
                         .SelectMany(x => x)
                         .Distinct()
                         .Count();
@@ -122,6 +122,7 @@ public async Task<DetectorProcessingResult> ProcessDetectorsAsync(IDetectionArgu
             }).ToList();
 
         var results = await Task.WhenAll(scanTasks);
+        DetectorExperiments.AutomaticallyProcessExperiments = false;
         await this.experimentService.FinishAsync();
 
         var detectorProcessingResult = this.ConvertDetectorResultsIntoResult(results, exitCode);

test/Microsoft.ComponentDetection.Orchestrator.Tests/Experiments/ExperimentServiceTests.cs

@@ -101,6 +101,7 @@ public async Task RecordDetectorRun_FiltersExperimentsAsync()
             this.loggerMock.Object);
 
         service.RecordDetectorRun(this.detectorMock.Object, this.componentRecorder, this.detectionArgsMock.Object);
+        DetectorExperiments.AutomaticallyProcessExperiments = true;
         await service.FinishAsync();
 
         filterConfigMock.Verify(x => x.ShouldRecord(this.detectorMock.Object, components.Count), Times.Once());
@@ -151,13 +152,37 @@ public async Task FinishAsync_ProcessesExperimentsAsync()
             this.loggerMock.Object);
         service.RecordDetectorRun(this.detectorMock.Object, this.componentRecorder, this.detectionArgsMock.Object);
 
+        DetectorExperiments.AutomaticallyProcessExperiments = true;
+
         await service.FinishAsync();
 
         this.experimentProcessorMock.Verify(
             x => x.ProcessExperimentAsync(this.experimentConfigMock.Object, It.IsAny<ExperimentDiff>()),
             Times.Once());
     }
 
+    [TestMethod]
+    public async Task FinishAsync_DoNotAutomaticallyProcess_ProcessesExperimentsAsync()
+    {
+        var components = ExperimentTestUtils.CreateRandomComponents();
+        this.SetupGraphMock(components);
+
+        var service = new ExperimentService(
+            new[] { this.experimentConfigMock.Object },
+            new[] { this.experimentProcessorMock.Object },
+            this.graphTranslationServiceMock.Object,
+            this.loggerMock.Object);
+        service.RecordDetectorRun(this.detectorMock.Object, this.componentRecorder, this.detectionArgsMock.Object);
+
+        DetectorExperiments.AutomaticallyProcessExperiments = false;
+
+        await service.FinishAsync();
+
+        this.experimentProcessorMock.Verify(
+            x => x.ProcessExperimentAsync(this.experimentConfigMock.Object, It.IsAny<ExperimentDiff>()),
+            Times.Never);
+    }
+
     [TestMethod]
     public async Task FinishAsync_SwallowsExceptionsAsync()
     {

test/Microsoft.ComponentDetection.Orchestrator.Tests/Services/DetectorProcessingServiceTests.cs

@@ -520,6 +520,8 @@ public async Task ProcessDetectorsAsync_FinishesExperimentsAsync()
 
         await this.serviceUnderTest.ProcessDetectorsAsync(DefaultArgs, this.detectorsToUse, new DetectorRestrictions());
 
+        DetectorExperiments.AutomaticallyProcessExperiments = true;
+
         this.experimentServiceMock.Verify(x => x.FinishAsync(), Times.Once());
     }