Add meta test to find missing [Fact]/[Theory]

dahlbyk · dahlbyk · commit fc78c72fd507 · 2013-10-21T21:32:35.000-05:00
diff --git a/LibGit2Sharp.Tests/BranchFixture.cs b/LibGit2Sharp.Tests/BranchFixture.cs
@@ -450,6 +450,7 @@ public void LookingOutABranchByNameWithBadParamsThrows()
             }
         }
 
+        [Fact]
         public void CanGetInformationFromUnbornBranch()
         {
             string repoPath = InitNewRepository(true);
diff --git a/LibGit2Sharp.Tests/MetaFixture.cs b/LibGit2Sharp.Tests/MetaFixture.cs
@@ -4,12 +4,37 @@
 using System.Linq;
 using System.Reflection;
 using System.Text;
+using LibGit2Sharp.Tests.TestHelpers;
 using Xunit;
+using Xunit.Extensions;
 
 namespace LibGit2Sharp.Tests
 {
     public class MetaFixture
     {
+        [Fact]
+        public void PublicTestMethodsAreFactsOrTheories()
+        {
+            var exceptions = new[]
+            {
+                "LibGit2Sharp.Tests.FilterBranchFixture.Dispose",
+            };
+
+            var fixtures = from t in Assembly.GetAssembly(typeof(MetaFixture)).GetExportedTypes()
+                           where t.IsPublic && !t.IsNested
+                           where t.Namespace != typeof(BaseFixture).Namespace // Exclude helpers
+                           let methods = t.GetMethods(BindingFlags.DeclaredOnly | BindingFlags.Instance | BindingFlags.Public)
+                           from m in methods
+                           where !m.GetCustomAttributes(typeof(FactAttribute), false)
+                                   .Concat(m.GetCustomAttributes(typeof(TheoryAttribute), false))
+                                   .Any()
+                           let name = t.FullName + "." + m.Name
+                           where !exceptions.Contains(name)
+                           select name;
+
+            Assert.Equal("", string.Join(Environment.NewLine, fixtures.ToArray()));
+        }
+
         // Related to https://github.com/libgit2/libgit2sharp/pull/251
         [Fact]
         public void TypesInLibGit2DecoratedWithDebuggerDisplayMustFollowTheStandardImplPattern()

Original file line number	Diff line number	Diff line change
`@@ -450,6 +450,7 @@ public void LookingOutABranchByNameWithBadParamsThrows()`
`450`	`450`	`}`
`451`	`451`	`}`
`452`	`452`
	`453`	`+ [Fact]`
`453`	`454`	`public void CanGetInformationFromUnbornBranch()`
`454`	`455`	`{`
`455`	`456`	`string repoPath = InitNewRepository(true);`