All test classes have to inherit from TestBaseClass

Otherwise the IsInUnitTests flag won't be set and we'll get into trouble.

Author: shana

src/UnitTests/GitHub.App/Caches/CredentialCacheTests.cs

@@ -8,7 +8,7 @@
 
 public class CredentialCacheTests : TestBaseClass
 {
-    public class TheGetObjectMethod
+    public class TheGetObjectMethod : TestBaseClass
     {
         [Fact]
         public async Task RetrievesValueWithAlternateKeys()
@@ -65,7 +65,7 @@ await Assert.ThrowsAsync<ObjectDisposedException>(
         }
     }
 
-    public class TheInsertObjectMethod
+    public class TheInsertObjectMethod : TestBaseClass
     {
         [Fact]
         public async Task StoresCredentialForKeyAndGitKey()
@@ -110,7 +110,7 @@ await Assert.ThrowsAsync<ObjectDisposedException>(
         }
     }
 
-    public class TheInsertMethod
+    public class TheInsertMethod : TestBaseClass
     {
         [Fact]
         public async Task ThrowsInvalidOperationException()
@@ -123,7 +123,7 @@ await Assert.ThrowsAsync<InvalidOperationException>(
         }
     }
 
-    public class TheGetMethod
+    public class TheGetMethod : TestBaseClass
     {
         [Fact]
         public async Task RetrievesPasswordAsUnicodeBytes()
@@ -167,7 +167,7 @@ await Assert.ThrowsAsync<ObjectDisposedException>(
         }
     }
 
-    public class TheInvalidateMethod
+    public class TheInvalidateMethod : TestBaseClass
     {
         [Fact]
         public async Task InvalidatesTheCredential()
@@ -206,7 +206,7 @@ await Assert.ThrowsAsync<ObjectDisposedException>(
         }
     }
 
-    public class TheInvalidateObjectMethod
+    public class TheInvalidateObjectMethod : TestBaseClass
     {
         [Fact]
         public async Task InvalidatesTheCredential()
@@ -252,7 +252,7 @@ await Assert.ThrowsAsync<KeyNotFoundException>(
         }
     }
 
-    public class TheFlushMethod
+    public class TheFlushMethod : TestBaseClass
     {
         [Fact]
         public async Task ThrowsObjectDisposedExceptionWhenDisposed()
@@ -267,7 +267,7 @@ public async Task ThrowsObjectDisposedExceptionWhenDisposed()
         }
     }
 
-    public class TheDisposeMethod
+    public class TheDisposeMethod : TestBaseClass
     {
         [Fact]
         public void SignalsShutdown()
@@ -281,7 +281,7 @@ public void SignalsShutdown()
         }
     }
 
-    public class MethodsNotImplementedOnPurpose
+    public class MethodsNotImplementedOnPurpose : TestBaseClass
     {
         [Fact]
         public void ThrowNotImplementedException()

src/UnitTests/GitHub.Extensions/UriExtensionTests.cs

@@ -4,7 +4,7 @@
 
 public class UriExtensionTests
 {
-    public class TheAppendMethod
+    public class TheAppendMethod : TestBaseClass
     {
         [Theory]
         [InlineData("https://github.com/foo/bar", "graphs", "https://github.com/foo/bar/graphs")]

src/UnitTests/GitHub.Primitives/UriStringTests.cs

@@ -5,7 +5,7 @@
 
 public class UriStringTests
 {
-    public class TheConstructor
+    public class TheConstructor : TestBaseClass
     {
         [Theory]
         [InlineData("http://192.168.1.3/foo/bar.git", "192.168.1.3", "foo", "bar")]
@@ -107,7 +107,7 @@ public void EnsuresArgumentNotNull()
         }
     }
 
-    public class TheNameWithOwnerProperty
+    public class TheNameWithOwnerProperty : TestBaseClass
     {
         [Theory]
         [InlineData("http://192.168.1.3/foo/bar.git", "foo/bar")]
@@ -125,7 +125,7 @@ public void DependsOnOwnerAndRepoNameNotBeingNull(string url, string expectedNam
         }
     }
 
-    public class TheCombineMethod
+    public class TheCombineMethod : TestBaseClass
     {
         [Theory]
         [InlineData("http://example.com", "foo/bar", @"http://example.com/foo/bar")]
@@ -146,7 +146,7 @@ public void ComparesHostInsensitively(string uriString, string path, string expe
         }
     }
 
-    public class TheIsValidUriProperty
+    public class TheIsValidUriProperty : TestBaseClass
     {
         [Theory]
         [InlineData("http://example.com/", true)]
@@ -159,7 +159,7 @@ public void ReturnWhetherTheUriIsParseableByUri(string uriString, bool expected)
         }
     }
 
-    public class TheToRepositoryUrlMethod
+    public class TheToRepositoryUrlMethod : TestBaseClass
     {
         [Theory]
         [InlineData("file:///C:/dev/exp/foo", "file:///C:/dev/exp/foo")]
@@ -200,7 +200,7 @@ public void ShouldNeverThrow(string url, string expected)
         }
     }
 
-    public class TheAdditionOperator
+    public class TheAdditionOperator : TestBaseClass
     {
         [Theory]
         [InlineData("http://example.com", "foo/bar", @"http://example.com/foo/bar")]
@@ -223,7 +223,7 @@ public void CombinesPaths(string uriString, string addition, string expected)
         }
     }
 
-    public class ImplicitConversionToString
+    public class ImplicitConversionToString : TestBaseClass
     {
         [Fact]
         public void ConvertsBackToString()
@@ -243,7 +243,7 @@ public void ConvertsNullToNull()
         }
     }
 
-    public class ImplicitConversionFromString
+    public class ImplicitConversionFromString : TestBaseClass
     {
         [Fact]
         public void ConvertsToCloneUri()
@@ -260,7 +260,7 @@ public void ConvertsNullToNull()
         }
     }
 
-    public class TheIsHypertextTransferProtocolProperty
+    public class TheIsHypertextTransferProtocolProperty : TestBaseClass
     {
         [Theory]
         [InlineData("http://example.com", true)]
@@ -277,7 +277,7 @@ public void IsTrueOnlyForHttpAndHttps(string url, bool expected)
         }
     }
 
-    public class TheEqualsMethod
+    public class TheEqualsMethod : TestBaseClass
     {
         [Theory]
         [InlineData("https://github.com/foo/bar", "https://github.com/foo/bar", true)]

src/UnitTests/GitHub.VisualStudio/TeamExplorer/Home/GraphsNavigationItemTests.cs

@@ -7,7 +7,7 @@
 
 public class GraphsNavigationItemTests
 {
-    public class TheExecuteMethod
+    public class TheExecuteMethod : TestBaseClass
     {
         [Theory]
         [InlineData("https://github.com/foo/bar.git", "https://github.com/foo/bar/graphs")]