Enable skipped test

Enable skipped test that has actually been usable since #186 was merged. Remove obsolete code to skip tests when a real CosmosDB instance isn't configured.
martincostello · Jul 1, 2018 · 135aef2 · 135aef2
1 parent c25eef6
commit 135aef2
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 17 deletions.
diff --git a/tests/LondonTravel.Site.Tests/Integration/AccountTests.cs b/tests/LondonTravel.Site.Tests/Integration/AccountTests.cs
@@ -28,12 +28,10 @@ public AccountTests(TestServerFixture fixture)
         {
         }
 
-        [SkippableFact]
+        [Fact]
         public async Task Can_Perform_Operations_On_Users_And_Get_Preferences_From_Api()
         {
             // Arrange
-            Skip.IfNot(CosmosDB.IsConfigured());
-
             var emailAddress = $"some.user.{Guid.NewGuid()}@some.domain.com";
 
             var user = new LondonTravelUser()
@@ -166,6 +164,11 @@ public async Task Can_Perform_Operations_On_Users_And_Get_Preferences_From_Api()
         /// </returns>
         protected IUserStore<LondonTravelUser> GetUserStore()
         {
+            // HACK Force server start-up
+            using (Fixture.CreateDefaultClient())
+            {
+            }
+
             return Fixture.Server.Host.Services.GetService(typeof(IUserStore<LondonTravelUser>)) as IUserStore<LondonTravelUser>;
         }
     }

diff --git a/tests/LondonTravel.Site.Tests/Integration/CosmosDB.cs b/tests/LondonTravel.Site.Tests/Integration/CosmosDB.cs
diff --git a/tests/LondonTravel.Site.Tests/LondonTravel.Site.Tests.csproj b/tests/LondonTravel.Site.Tests/LondonTravel.Site.Tests.csproj
@@ -26,7 +26,6 @@
     <PackageReference Include="Shouldly" Version="3.0.0" />
     <PackageReference Include="xunit" Version="2.3.1" />
     <PackageReference Include="xunit.runner.visualstudio" Version="2.3.1" />
-    <PackageReference Include="Xunit.SkippableFact" Version="1.3.6" />
   </ItemGroup>
   <ItemGroup>
     <Service Include="{82a7f48d-3b50-4b1e-b82e-3ada8210c358}" />