diff --git a/src/AddIns/Analysis/CodeCoverage/Project/Src/CodeCoverageResults.cs b/src/AddIns/Analysis/CodeCoverage/Project/Src/CodeCoverageResults.cs index f6fb756fec..c2a29a7b07 100644 --- a/src/AddIns/Analysis/CodeCoverage/Project/Src/CodeCoverageResults.cs +++ b/src/AddIns/Analysis/CodeCoverage/Project/Src/CodeCoverageResults.cs @@ -77,15 +77,17 @@ namespace ICSharpCode.CodeCoverage CodeCoverageModule AddModule(XElement reader, string className) { string assemblyName = GetAssemblyName(reader); + CodeCoverageModule module = null; foreach (CodeCoverageModule existingModule in modules) { if (existingModule.Name == assemblyName) { - return existingModule; + module = existingModule; + break; } } - - CodeCoverageModule module = new CodeCoverageModule(assemblyName); - modules.Add(module); - + if (module == null) { + module = new CodeCoverageModule(assemblyName); + modules.Add(module); + } var methods = reader .Elements("Classes") .Elements("Class") diff --git a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsMissingFileIdTestFixture.cs b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsMissingFileIdTestFixture.cs index db71bf28f7..15d07856c1 100644 --- a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsMissingFileIdTestFixture.cs +++ b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsMissingFileIdTestFixture.cs @@ -20,26 +20,51 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage [SetUp] public void SetUpFixture() { - string xml = "\r\n" + - "\r\n" + - "\r\n" + - "\r\n" + - "\r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - "\r\n" + - "\r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - "\r\n" + - ""; + string xml = "\r\n" + + "\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\MyTests\\bin\\nunit.framework.DLL\r\n" + + "\t\t\tnunit.framework\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tNUnit.Framework.NotEqualAsserter\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Boolean NUnit.Framework.NotEqualAsserter::Fail()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\MyTests\\bin\\MyTests.DLL\r\n" + + "\t\t\tMyTests.Tests\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tMyTests.Tests.MyClass\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void MyTests.Tests.MyClass::.ctor()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\r\n" + + ""; base.CreateCodeCoverageResults(xml); } diff --git a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsTestFixture.cs b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsTestFixture.cs index 197a442c0d..9b1372ff23 100644 --- a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsTestFixture.cs +++ b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsTestFixture.cs @@ -14,18 +14,35 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage [TestFixtureSetUp] public void SetUpFixture() { - string xml = - "\r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - ""; + string xml = + "\r\n" + + "\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\Foo.Tests\\bin\\Foo.Tests.DLL\r\n" + + "\t\t\tFoo.Tests\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.FooTestFixture\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.FooTestFixture::SimpleTest()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\r\n" + + ""; base.CreateCodeCoverageResults(xml); } @@ -56,7 +73,7 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage public void MethodName_ModuleHasOneMethod_ReturnsExpectedMethodName() { string name = FirstModuleFirstMethod.Name; - string expectedName = "SimpleTest"; + string expectedName = "System.Void Foo.Tests.FooTestFixture::.SimpleTest()"; Assert.AreEqual(expectedName, name); } diff --git a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsWithEmptyMethodElementTestFixture.cs b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsWithEmptyMethodElementTestFixture.cs index 22dd4644dc..bdddc8bef2 100644 --- a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsWithEmptyMethodElementTestFixture.cs +++ b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsWithEmptyMethodElementTestFixture.cs @@ -14,14 +14,30 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage [TestFixtureSetUp] public void SetUpFixture() { - string xml = - "\r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - ""; + string xml = + "\r\n" + + "\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\Foo.Tests\\bin\\Foo.Tests.DLL\r\n" + + "\t\t\tFoo.Tests\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.FooTestFixture\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.FooTestFixture::.SimpleTest()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\r\n" + + ""; base.CreateCodeCoverageResults(xml); } diff --git a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/GetSequencePointsForFileNameTestFixture.cs b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/GetSequencePointsForFileNameTestFixture.cs index 98d4f2cbf4..db41022a4e 100644 --- a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/GetSequencePointsForFileNameTestFixture.cs +++ b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/GetSequencePointsForFileNameTestFixture.cs @@ -15,34 +15,82 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage [SetUp] public void SetUpFixture() { - string xml = - "\r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - " \r\n" + - ""; + string xml = + "\r\n" + + "\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\Foo.Tests\\bin\\Foo.Tests.DLL\r\n" + + "\t\t\tFoo.Tests\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.FooTestFixture\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.FooTestFixture::FooTest()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.SimpleTestFixture2\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.SimpleTestFixture2::SimpleTest2()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.SimpleTestFixture3\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.SimpleTestFixture3::SimpleTest3()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\Foo.Tests\\bin\\Bar.Tests.DLL\r\n" + + "\t\t\tBar.Tests\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.BarTestFixture\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.BarTestFixture::.SimpleTest2()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\r\n" + + ""; base.CreateCodeCoverageResults(xml); } diff --git a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/InvalidSequencePointDataTestFixture.cs b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/InvalidSequencePointDataTestFixture.cs index 44e26fb473..33bc3eb9cf 100644 --- a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/InvalidSequencePointDataTestFixture.cs +++ b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/InvalidSequencePointDataTestFixture.cs @@ -15,22 +15,49 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage [TestFixtureSetUp] public void SetUpFixture() { - string xml = - "\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\r\n" + - "\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\r\n" + - "\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\r\n" + - "\t\r\n" + - ""; + string xml = + "\r\n" + + "\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\Foo.Tests\\bin\\Foo.Tests.DLL\r\n" + + "\t\t\tFoo.Tests\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.FooTestFixture\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.FooTestFixture::InvalidVisitCount()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.FooTestFixture::InvalidLine()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.FooTestFixture::InvalidColumn()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\r\n" + + ""; base.CreateCodeCoverageResults(xml); } diff --git a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/ModuleVisitedSequencePointsTestFixture.cs b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/ModuleVisitedSequencePointsTestFixture.cs index bc624a4656..e373ca280c 100644 --- a/src/AddIns/Analysis/CodeCoverage/Test/Coverage/ModuleVisitedSequencePointsTestFixture.cs +++ b/src/AddIns/Analysis/CodeCoverage/Test/Coverage/ModuleVisitedSequencePointsTestFixture.cs @@ -14,35 +14,75 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage [TestFixtureSetUp] public void SetUpFixture() { - string xml = - "\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\r\n" + - "\t\r\n" + - "\t\r\n" + - "\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\t\r\n" + - "\t\t\r\n" + - "\t\r\n" + - ""; + string xml = + "\r\n" + + "\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\Foo.Tests\\bin\\Foo.Tests.DLL\r\n" + + "\t\t\tFoo.Tests\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.FooTestFixture1\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.FooTestFixture::SimpleTest1()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.FooTestFixture2\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.FooTestFixture2::SimpleTest2()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\t\r\n" + + "\t\t\tC:\\Projects\\Test\\Foo.Tests\\bin\\Bar.Tests.DLL\r\n" + + "\t\t\tBar.Tests\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\t\tFoo.Tests.FooTestFixture3\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t100663297\r\n" + + "\t\t\t\t\t\t\tSystem.Void Foo.Tests.BarTestFixture::.SimpleTest3()\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\t\r\n" + + "\t\t\t\t\t\r\n" + + "\t\t\t\t\r\n" + + "\t\t\t\r\n" + + "\t\t\r\n" + + "\t\r\n" + + ""; base.CreateCodeCoverageResults(xml); }