Skip to content

Commit 1a3d532

Browse files
Parser changes for test results (#4974)
* Parser changes for test results * Adding featureflagservice * Updating the version * updating packages * adding psmodule * using tokens * adding parameters * Added System.IdentityModel.Tokens.Jwt package --------- Co-authored-by: Denis Nikulin (Akvelon Inc) <denis.nikulin@akvelon.com>
1 parent 5b51e45 commit 1a3d532

File tree

4 files changed

+15
-6
lines changed

4 files changed

+15
-6
lines changed

src/Agent.Sdk/Agent.Sdk.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,6 @@
1919
<PackageReference Include="System.Text.RegularExpressions" Version="4.3.1" />
2020
<PackageReference Include="System.Private.Uri" Version="4.3.2" />
2121
<PackageReference Include="System.Formats.Asn1" Version="8.0.1" />
22+
<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="7.3.1" />
2223
</ItemGroup>
2324
</Project>

src/Agent.Worker/TestResults/Parser.cs

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,8 @@ protected override ITestResultParser GetTestResultParser(IExecutionContext execu
7474
var traceListener = new CommandTraceListener(executionContext);
7575
var featureFlagService = executionContext.GetHostContext().GetService<IFeatureFlagService>();
7676
var enableJunitAttachments = featureFlagService.GetFeatureFlagState(TestResultsConstants.JUnitTestCaseAttachmentsEnabled, TestResultsConstants.TCMServiceInstanceGuid);
77-
return new JUnitResultParser(traceListener, false, enableJunitAttachments);
77+
bool enableCustomTestFields = featureFlagService.GetFeatureFlagState(TestResultsConstants.CustomTestFieldsInPTRInputFilesEnabled, TestResultsConstants.TCMServiceInstanceGuid);
78+
return new JUnitResultParser(traceListener, false, enableJunitAttachments,enableCustomTestFields);
7879
}
7980
}
8081

@@ -86,7 +87,9 @@ public class XUnitParser : Parser, IParser
8687
protected override ITestResultParser GetTestResultParser(IExecutionContext executionContext)
8788
{
8889
var traceListener = new CommandTraceListener(executionContext);
89-
return new XUnitResultParser(traceListener);
90+
var featureFlagService = executionContext.GetHostContext().GetService<IFeatureFlagService>();
91+
bool enableCustomTestFields = featureFlagService.GetFeatureFlagState(TestResultsConstants.CustomTestFieldsInPTRInputFilesEnabled, TestResultsConstants.TCMServiceInstanceGuid);
92+
return new XUnitResultParser(traceListener, setNameAsDisplayName: false, isTestCaseParallelReportingEnabled: false, enableCustomTestFields);
9093
}
9194

9295
}
@@ -101,7 +104,8 @@ protected override ITestResultParser GetTestResultParser(IExecutionContext execu
101104
var traceListener = new CommandTraceListener(executionContext);
102105
var featureFlagService = executionContext.GetHostContext().GetService<IFeatureFlagService>();
103106
var enableXUnitHeirarchicalParsing = featureFlagService.GetFeatureFlagState(TestResultsConstants.EnableXUnitHeirarchicalParsing, TestResultsConstants.TFSServiceInstanceGuid);
104-
return new TrxResultParser(traceListener, enableXUnitHeirarchicalParsing);
107+
bool enableCustomTestFields = featureFlagService.GetFeatureFlagState(TestResultsConstants.CustomTestFieldsInPTRInputFilesEnabled, TestResultsConstants.TCMServiceInstanceGuid);
108+
return new TrxResultParser(traceListener, enableXUnitHeirarchicalParsing, enableCustomTestFields);
105109
}
106110

107111
}
@@ -114,7 +118,9 @@ public class NUnitParser : Parser, IParser
114118
protected override ITestResultParser GetTestResultParser(IExecutionContext executionContext)
115119
{
116120
var traceListener = new CommandTraceListener(executionContext);
117-
return new NUnitResultParser(traceListener);
121+
var featureFlagService = executionContext.GetHostContext().GetService<IFeatureFlagService>();
122+
bool enableCustomTestFields = featureFlagService.GetFeatureFlagState(TestResultsConstants.CustomTestFieldsInPTRInputFilesEnabled, TestResultsConstants.TCMServiceInstanceGuid);
123+
return new NUnitResultParser(traceListener, isTestCaseParallelReportingEnabled: false, enableCustomTestFields);
118124
}
119125

120126
}
@@ -143,4 +149,4 @@ protected override ITestResultParser GetTestResultParser(IExecutionContext execu
143149
return new ContainerStructureTestResultParser(traceListener);
144150
}
145151
}
146-
}
152+
}

src/Agent.Worker/TestResults/Utils/TestResultsConstants.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,5 +26,7 @@ public class TestResultsConstants
2626
public static readonly string JUnitTestCaseAttachmentsEnabled = "TestManagement.Server.JUnitTestCaseAttachmentsEnabled";
2727

2828
public static readonly string EnableAzureTestPlanTaskFeatureFlag = "TestManagement.Server.EnableAzureTestPlanTaskFlow";
29+
30+
public static readonly string CustomTestFieldsInPTRInputFilesEnabled = "TestManagement.Server.CustomTestFieldsInPTRInputFilesEnabled";
2931
}
3032
}

src/Common.props

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
<OSPlatform>OS_UNKNOWN</OSPlatform>
1212
<OSArchitecture>ARCH_UNKNOWN</OSArchitecture>
1313
<DebugConstant></DebugConstant>
14-
<VssApiVersion>0.5.238-private</VssApiVersion>
14+
<VssApiVersion>0.5.245-private</VssApiVersion>
1515
<CodeAnalysis>$(CodeAnalysis)</CodeAnalysis>
1616
<InvariantGlobalization>false</InvariantGlobalization>
1717
<EnforceCodeStyleInBuild>false</EnforceCodeStyleInBuild>

0 commit comments

Comments
 (0)