CodeCoverage: Tests

src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsMissingFileIdTestFixture.cs

@@ -111,14 +111,6 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage
 			Assert.AreEqual(expectedName, name);
 		}
 		
-		[Test, Ignore("Replaced by test below")]
-		public void SequencePointsCount_NUnitNotEqualAssertFailMethod_ReturnsAllSequencePoints()
-		{
-			int sequencePointCount = FirstModuleFirstMethod.SequencePoints.Count;
-			int expectedSequencePointCount = 3;
-			Assert.AreEqual(expectedSequencePointCount, sequencePointCount);
-		}
-
 		/// <summary> No method.FileID => No sequence points!
 		/// SD.CodeCoverage DOES NOT RETURN SequencePoints
 		/// for assemblies without debug info,

src/AddIns/Analysis/CodeCoverage/Test/Coverage/CodeCoverageResultsTestFixture.cs

@@ -36,7 +36,7 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage
 				"\t\t\t<FullName>C:\\Projects\\Test\\Foo.Tests\\bin\\Foo.Tests.DLL</FullName>\r\n" +
 				"\t\t\t<ModuleName>Foo.Tests</ModuleName>\r\n" +
 				"\t\t\t<Files>\r\n" +
-				"\t\t\t\t<File uid=\"1\" fullPath=\"c:\\Projects\\Foo\\FooTestFixture.cs\" />\r\n" +
+				"\t\t\t\t<File uid=\"2\" fullPath=\"c:\\Projects\\Foo\\FooTestFixture.cs\" />\r\n" +
 				"\t\t\t</Files>\r\n" +
 				"\t\t\t<Classes>\r\n" +
 				"\t\t\t\t<Class>\r\n" +
@@ -45,11 +45,11 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage
 				"\t\t\t\t\t\t<Method visited=\"true\" cyclomaticComplexity=\"1\" sequenceCoverage=\"100\" branchCoverage=\"100\" isConstructor=\"false\" isStatic=\"false\" isGetter=\"false\" isSetter=\"false\">\r\n" +
 				"\t\t\t\t\t\t\t<MetadataToken>100663297</MetadataToken>\r\n" +
 				"\t\t\t\t\t\t\t<Name>System.Void Foo.Tests.FooTestFixture::SimpleTest()</Name>\r\n" +
-				"\t\t\t\t\t\t\t<FileRef uid=\"1\" />\r\n" +
+				"\t\t\t\t\t\t\t<FileRef uid=\"2\" />\r\n" +
 				"\t\t\t\t\t\t\t<SequencePoints>\r\n" +
-				"\t\t\t\t\t\t\t\t<SequencePoint vc=\"1\" sl=\"20\" sc=\"3\" el=\"20\" ec=\"4\" />\r\n" +
-				"\t\t\t\t\t\t\t\t<SequencePoint vc=\"1\" sl=\"21\" sc=\"13\" el=\"21\" ec=\"32\" />\r\n" +
-				"\t\t\t\t\t\t\t\t<SequencePoint vc=\"0\" sl=\"24\" sc=\"3\" el=\"24\" ec=\"4\" />\r\n" +
+				"\t\t\t\t\t\t\t\t<SequencePoint vc=\"1\" sl=\"20\" sc=\"3\" el=\"20\" ec=\"4\" fileid=\"2\" />\r\n" +
+				"\t\t\t\t\t\t\t\t<SequencePoint vc=\"1\" sl=\"21\" sc=\"13\" el=\"21\" ec=\"32\" fileid=\"2\" />\r\n" +
+				"\t\t\t\t\t\t\t\t<SequencePoint vc=\"0\" sl=\"24\" sc=\"3\" el=\"24\" ec=\"4\" fileid=\"2\" />\r\n" +
 				"\t\t\t\t\t\t\t</SequencePoints>\r\n" +
 				"\t\t\t\t\t\t</Method>\r\n" +
 				"\t\t\t\t\t</Methods>\r\n" +
@@ -123,10 +123,14 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage
 			Assert.AreEqual(3, count);
 		}
 		
-		[Test, Ignore("SequencePoint.Length is not 1 anymore")]
+		[Test]
 		public void SequencePoint_FirstSequencePoint_HasExpectedPropertyValues()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
+
 			CodeCoverageSequencePoint point = base.CreateSequencePoint();
+			// ??? if in this.SetUpFixture fileId is set to "1" then filename from another test fixture is returned ???
+			// ??? "c:\test\MyTests\Class1.cs" 
 			point.Document = @"c:\Projects\Foo\FooTestFixture.cs";
 			point.VisitCount = 1;
 			point.Column = 3;
@@ -166,16 +170,18 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage
 			Assert.AreEqual(1, count);
 		}
 		
-		[Test, Ignore("SequencePoint.Length is not 1 anymore")]
+		[Test]
 		public void GetVisitedCodeLength_FirstMethod_ReturnsSummedLengthOfVisitedSequencePoints()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			int length = FirstModuleFirstMethod.GetVisitedCodeLength();
 			Assert.AreEqual(2, length);
 		}
 		
-		[Test, Ignore("SequencePoint.Length is not 1 anymore")]
+		[Test]
 		public void GetUnvisitedCodeLength_FirstMethod_ReturnsSummedLengthOfUnvisitedSequencePoints()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			int length = FirstModuleFirstMethod.GetUnvisitedCodeLength();
 			Assert.AreEqual(1, length);
 		}

src/AddIns/Analysis/CodeCoverage/Test/Coverage/ModuleVisitedSequencePointsTestFixture.cs

@@ -110,33 +110,37 @@ namespace ICSharpCode.CodeCoverage.Tests.Coverage
 			get { return SecondModule; }
 		}
 		
-		[Test, Ignore("SequencePoint.Length is not 1 anymore")]
+		[Test]
 		public void ModuleGetVisitedCodeLength_FooModule_ReturnsTotalLengthOfAllVisitedMethodSequencePoints()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			int length = FooModule.GetVisitedCodeLength();
 			int expectedLength = 4;
 			Assert.AreEqual(expectedLength, length);
 		}
 		
-		[Test, Ignore("SequencePoint.Length is not 1 anymore")]
+		[Test]
 		public void ModuleGetUnvisitedCodeLength_FooModule_ReturnsTotalLengthOfAllNonVisitedMethodSequencePoints()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			int length = FooModule.GetUnvisitedCodeLength();
 			int expectedLength = 2;
 			Assert.AreEqual(expectedLength, length);
 		}
 		
-		[Test, Ignore("SequencePoint.Length is not 1 anymore")]
+		[Test]
 		public void ModuleGetVisitedCodeLength_BarModule_ReturnsTotalLengthOfAllVisitedMethodSequencePoints()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			int length = BarModule.GetVisitedCodeLength();
 			int expectedLength = 2;
 			Assert.AreEqual(expectedLength, length);
 		}
 		
-		[Test, Ignore("SequencePoint.Length is not 1 anymore")]
+		[Test]
 		public void ModuleGetUnvisitedCodeLength_BarModule_ReturnsTotalLengthOfAllNonVisitedMethodSequencePoints()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			int length = BarModule.GetUnvisitedCodeLength();
 			int expectedLength = 1;
 			Assert.AreEqual(expectedLength, length);

src/AddIns/Analysis/CodeCoverage/Test/Gui/PropertiesInCodeCoverageTreeView.cs

@@ -112,9 +112,9 @@ namespace ICSharpCode.CodeCoverage.Tests.Gui
 		}
 		
 		[Test]
-		[Ignore("CodeCoverageImageListIndex.Property is 8, ImageIndex is 9")]
 		public void PropertyTreeNodeImageIndex_CountPropertyTreeNode_ImageIndexIsProperty()
 		{
+			Assert.Inconclusive("??? CodeCoverageImageListIndex.Property is 8, ImageIndex is 9 ???");
 			Assert.AreEqual(CodeCoverageImageListIndex.Property, (CodeCoverageImageListIndex)countPropertyTreeNode.ImageIndex);
 		}
 
@@ -125,24 +125,24 @@ namespace ICSharpCode.CodeCoverage.Tests.Gui
 		}
 		
 		[Test]
-		[Ignore("Visited length not implemented with OpenCover")]
 		public void PropertyTreeNodeVisitedCodeLength_CountPropertyTreeNode_ReturnsThree()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			Assert.AreEqual(3, countPropertyTreeNode.VisitedCodeLength);
 		}
 		
 		[Test]
-		[Ignore("Visited length not implemented with OpenCover")]
 		public void VisitedCodeLength_PropertyTreeNode_ReturnsThree()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			int count = countPropertyTreeNode.VisitedCodeLength;
 			Assert.AreEqual(3, count);
 		}
 		
 		[Test]
-		[Ignore("Unvisited length not implemented with OpenCover")]
 		public void UnvisitedCodeLength_PropertyTreeNode_ReturnsThree()
 		{
+			Assert.Inconclusive("SequencePoint.Length is computed from text-source not present in this test");
 			int count = countPropertyTreeNode.UnvisitedCodeLength;
 			Assert.AreEqual(1, count);
 		}