Skip to content

Commit 2b81039

Browse files
committed
Add nuget packages config detector to control group.
1 parent 98bbc3e commit 2b81039

File tree

2 files changed

+12
-1
lines changed

2 files changed

+12
-1
lines changed

src/Microsoft.ComponentDetection.Orchestrator/Experiments/Configs/LinuxApplicationLayerExperiment.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ or NpmComponentDetector
2525
or NpmLockfileDetectorBase
2626
or PipReportComponentDetector
2727
or NuGetComponentDetector
28-
or NuGetProjectModelProjectCentricComponentDetector;
28+
or NuGetProjectModelProjectCentricComponentDetector
29+
or NuGetPackagesConfigDetector;
2930

3031
/// <inheritdoc />
3132
public bool IsInExperimentGroup(IComponentDetector componentDetector) =>

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

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,13 @@ public void IsInControlGroup_NuGetProjectModelProjectCentricComponentDetector_Re
5656
this.experiment.IsInControlGroup(nuGetDetector).Should().BeTrue();
5757
}
5858

59+
[TestMethod]
60+
public void IsInControlGroup_NuGetPackagesConfigDetector_ReturnsTrue()
61+
{
62+
var nuGetPackagesConfigDetector = new NuGetPackagesConfigDetector(null, null, null);
63+
this.experiment.IsInControlGroup(nuGetPackagesConfigDetector).Should().BeTrue();
64+
}
65+
5966
[TestMethod]
6067
public void IsInControlGroup_PipReportComponentDetector_ReturnsTrue()
6168
{
@@ -179,5 +186,8 @@ public void ShouldRecord_ControlGroup_AlwaysReturnsTrue()
179186

180187
var nuGetProjectCentricDetector = new NuGetProjectModelProjectCentricComponentDetector(null, null, null, null);
181188
this.experiment.ShouldRecord(nuGetProjectCentricDetector, 0).Should().BeTrue();
189+
190+
var nuGetPackagesConfigDetector = new NuGetPackagesConfigDetector(null, null, null);
191+
this.experiment.ShouldRecord(nuGetPackagesConfigDetector, 0).Should().BeTrue();
182192
}
183193
}

0 commit comments

Comments
 (0)