diff --git a/src/DemaConsulting.TestResults/IO/JUnitSerializer.cs b/src/DemaConsulting.TestResults/IO/JUnitSerializer.cs index 99ddfe9..e64356a 100644 --- a/src/DemaConsulting.TestResults/IO/JUnitSerializer.cs +++ b/src/DemaConsulting.TestResults/IO/JUnitSerializer.cs @@ -66,11 +66,7 @@ public static string Serialize(TestResults results) new XAttribute("name", results.Name)); // Add test suites for each class - foreach (var suiteGroup in testSuites) - { - var testSuite = CreateTestSuiteElement(suiteGroup); - root.Add(testSuite); - } + root.Add(testSuites.Select(CreateTestSuiteElement)); // Write the XML text var doc = new XDocument(root); @@ -98,11 +94,7 @@ private static XElement CreateTestSuiteElement(IGrouping sui new XAttribute("time", suiteTests.Sum(t => t.Duration.TotalSeconds).ToString(TimeFormatString, CultureInfo.InvariantCulture))); // Add test cases - foreach (var test in suiteTests) - { - var testCase = CreateTestCaseElement(test); - testSuite.Add(testCase); - } + testSuite.Add(suiteTests.Select(CreateTestCaseElement)); return testSuite; } @@ -239,12 +231,7 @@ public static TestResults Deserialize(string junitContents) private static void ParseTestSuite(XElement testSuiteElement, TestResults results) { var testCaseElements = testSuiteElement.Elements("testcase"); - - foreach (var testCaseElement in testCaseElements) - { - var testResult = ParseTestCase(testCaseElement); - results.Results.Add(testResult); - } + results.Results.AddRange(testCaseElements.Select(ParseTestCase)); } /// diff --git a/src/DemaConsulting.TestResults/IO/TrxSerializer.cs b/src/DemaConsulting.TestResults/IO/TrxSerializer.cs index 6c232bd..67209e0 100644 --- a/src/DemaConsulting.TestResults/IO/TrxSerializer.cs +++ b/src/DemaConsulting.TestResults/IO/TrxSerializer.cs @@ -127,13 +127,7 @@ private static XElement CreateRootElement(TestResults results) private static XElement CreateResultsElement(List testResults) { var resultsElement = new XElement(TrxNamespace + "Results"); - - foreach (var test in testResults) - { - var resultElement = CreateUnitTestResultElement(test); - resultsElement.Add(resultElement); - } - + resultsElement.Add(testResults.Select(CreateUnitTestResultElement)); return resultsElement; } @@ -352,11 +346,7 @@ private static void ParseTestResults(XDocument doc, XmlNamespaceManager nsMgr, T "/trx:TestRun/trx:Results/trx:UnitTestResult", nsMgr); - foreach (var resultElement in resultElements) - { - var testResult = ParseTestResult(doc, nsMgr, resultElement); - results.Results.Add(testResult); - } + results.Results.AddRange(resultElements.Select(e => ParseTestResult(doc, nsMgr, e))); } ///