From b507f9a2626f6930f4b8bd0591f0d1c564e084de Mon Sep 17 00:00:00 2001
From: Andrew Best <andrew.best@readify.net>
Date: Thu, 10 May 2018 09:17:36 +1000
Subject: [PATCH] Adds the facility to use development.json for alternative dev
 settings that wont corrupt source

---
 .gitignore                                    |  1 +
 .../Conventional.Tests.Roslyn.csproj          |  5 +-
 .../DatabaseConventionSpecificationTests.cs   | 79 +++++++++++--------
 Conventional.Tests/DevelopmentSettings.cs     | 31 ++++++++
 Conventional.sln                              |  1 +
 README.md                                     |  4 +
 development.json.example                      |  3 +
 7 files changed, 90 insertions(+), 34 deletions(-)
 create mode 100644 Conventional.Tests/DevelopmentSettings.cs
 create mode 100644 development.json.example

diff --git a/.gitignore b/.gitignore
index 86ce5bf..4e5a17e 100644
--- a/.gitignore
+++ b/.gitignore
@@ -168,3 +168,4 @@ FakesAssemblies/
 # nuget pack output
 *.nupkg
 
+/development.json
diff --git a/Conventional.Tests.Roslyn/Conventional.Tests.Roslyn.csproj b/Conventional.Tests.Roslyn/Conventional.Tests.Roslyn.csproj
index ac738d3..2a7c364 100644
--- a/Conventional.Tests.Roslyn/Conventional.Tests.Roslyn.csproj
+++ b/Conventional.Tests.Roslyn/Conventional.Tests.Roslyn.csproj
@@ -4,7 +4,7 @@
   <PropertyGroup>
     <Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
     <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
-    <ProjectGuid>10DC5795-E692-4B6E-854B-E1E1AA9F1B36</ProjectGuid>
+    <ProjectGuid>{10DC5795-E692-4B6E-854B-E1E1AA9F1B36}</ProjectGuid>
     <OutputType>Library</OutputType>
     <AppDesignerFolder>Properties</AppDesignerFolder>
     <RootNamespace>Conventional.Tests.Roslyn</RootNamespace>
@@ -104,6 +104,9 @@
   <ItemGroup>
     <None Include="packages.config" />
   </ItemGroup>
+  <ItemGroup>
+    <Service Include="{82A7F48D-3B50-4B1E-B82E-3ADA8210C358}" />
+  </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it.
        Other similar extension points exist, see Microsoft.Common.targets.
diff --git a/Conventional.Tests/Conventional/Conventions/Database/DatabaseConventionSpecificationTests.cs b/Conventional.Tests/Conventional/Conventions/Database/DatabaseConventionSpecificationTests.cs
index ff96c2f..dbb5557 100644
--- a/Conventional.Tests/Conventional/Conventions/Database/DatabaseConventionSpecificationTests.cs
+++ b/Conventional.Tests/Conventional/Conventions/Database/DatabaseConventionSpecificationTests.cs
@@ -1,5 +1,7 @@
-using System.Data;
+using System;
+using System.Data;
 using System.Data.SqlClient;
+using System.Diagnostics;
 using System.IO;
 using System.Resources;
 using FluentAssertions;
@@ -9,16 +11,26 @@ namespace Conventional.Tests.Conventional.Conventions.Database
 {
     public class DatabaseConventionSpecificationTests
     {
-#if DEBUG
-        private const string TestDbConnectionString = @"Server=.\SQLEXPRESS;Database=Conventional;Integrated Security=true;";
-#else
-        private const string TestDbConnectionString = @"Server=(local)\SQL2014;Database=Conventional;User ID=sa;Password=Password12!";
-#endif
+        private readonly DevelopmentSettings _settings = DevelopmentSettings.Create();
 
         [SetUp]
         public void Setup()
         {
-            CreateDatabase();
+            try
+            {
+                CreateDatabase();
+            }
+            catch (SqlException e)
+            {
+                Assert.Fail(
+                    "Conventional's test suite requires a default named .\\SQLEXPRESS instance. If you want to use an alternative database instance, create a development.json file in the solution root with your desired connection string.\n See development.json.example in the solution root for an example that uses LocalDB."
+                    + Environment.NewLine
+                    + Environment.NewLine
+                    + $"Exception Message: {e.Message}"
+                    + Environment.NewLine
+                    + Environment.NewLine
+                    + $"Stack Trace: {e.StackTrace}");
+            }
         }
 
         [TearDown]
@@ -33,7 +45,7 @@ public void AllCheckConstraintsMustBeNamedConventionalSpecification_Success()
             ExecuteSqlScriptFromResource("AllCheckConstraintsMustBeNamedConventionalSpecification_Success.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllCheckConstraintsMustBeNamed)
                 .IsSatisfied
                 .Should()
@@ -46,7 +58,7 @@ public void AllCheckConstraintsMustBeNamedConventionalSpecification_Fail()
             ExecuteSqlScriptFromResource("AllCheckConstraintsMustBeNamedConventionalSpecification_Fail.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllCheckConstraintsMustBeNamed);
 
             result.IsSatisfied.Should().BeFalse();
@@ -59,7 +71,7 @@ public void AllDefaultConstraintsMustBeNamedConventionalSpecification_Success()
             ExecuteSqlScriptFromResource("AllDefaultConstraintsMustBeNamedConventionalSpecification_Success.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllDefaultConstraintsMustBeNamed)
                 .IsSatisfied
                 .Should()
@@ -72,7 +84,7 @@ public void AllDefaultConstraintsMustBeNamedConventionalSpecification_Fail()
             ExecuteSqlScriptFromResource("AllDefaultConstraintsMustBeNamedConventionalSpecification_Fail.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllDefaultConstraintsMustBeNamed);
 
             result.IsSatisfied.Should().BeFalse();
@@ -85,7 +97,7 @@ public void AllPrimaryKeyConstraintsMustBeNamedConventionalSpecification_Success
             ExecuteSqlScriptFromResource("AllPrimaryKeyConstraintsMustBeNamedConventionalSpecification_Success.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllPrimaryKeyConstraintsMustBeNamed)
                 .IsSatisfied
                 .Should()
@@ -98,7 +110,7 @@ public void AllPrimaryKeyConstraintsMustBeNamedConventionalSpecification_Fail()
             ExecuteSqlScriptFromResource("AllPrimaryKeyConstraintsMustBeNamedConventionalSpecification_Fail.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllPrimaryKeyConstraintsMustBeNamed);
 
             result.IsSatisfied.Should().BeFalse();
@@ -111,7 +123,7 @@ public void AllReferenceConstraintsMustBeNamedConventionalSpecification_Success(
             ExecuteSqlScriptFromResource("AllReferenceConstraintsMustBeNamedConventionalSpecification_Success.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllReferenceConstraintsMustBeNamed)
                 .IsSatisfied
                 .Should()
@@ -124,7 +136,7 @@ public void AllReferenceConstraintsMustBeNamedConventionalSpecification_Fail()
             ExecuteSqlScriptFromResource("AllReferenceConstraintsMustBeNamedConventionalSpecification_Fail.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllReferenceConstraintsMustBeNamed);
 
             result.IsSatisfied.Should().BeFalse();
@@ -137,7 +149,7 @@ public void AllUniqueConstraintsMustBeNamedConventionalSpecification_Success()
             ExecuteSqlScriptFromResource("AllUniqueConstraintsMustBeNamedConventionalSpecification_Success.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllUniqueConstraintsMustBeNamed)
                 .IsSatisfied
                 .Should()
@@ -150,7 +162,7 @@ public void AllUniqueConstraintsMustBeNamedConventionalSpecification_Fail()
             ExecuteSqlScriptFromResource("AllUniqueConstraintsMustBeNamedConventionalSpecification_Fail.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllUniqueConstraintsMustBeNamed);
 
             result.IsSatisfied.Should().BeFalse();
@@ -163,7 +175,7 @@ public void AllNamedColumnsMustBeNullableConventionSpecification_Success()
             ExecuteSqlScriptFromResource("AllNamedColumnsMustBeNullableConventionalSpecification_Success.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllNamedColumnsMustBeNullable("UpdatedDateTime"))
                 .IsSatisfied
                 .Should()
@@ -176,19 +188,20 @@ public void AllNamedColumnsMustBeNonNullableConventionSpecification_Fails()
             ExecuteSqlScriptFromResource("AllNamedColumnsMustBeNullableConventionalSpecification_Fail.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllNamedColumnsMustBeNullable("UpdatedDateTime"));
 
             result.IsSatisfied.Should().BeFalse();
             result.Failures.Should().HaveCount(1);
         }
 
+        [Test]
         public void AllNamedColumnsMustBeNonNullableConventionSpecification_Success()
         {
             ExecuteSqlScriptFromResource("AllNamedColumnsMustBeNullableConventionalSpecification_Success.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllNamedColumnsMustBeNonNullable("CreatedDateTime"))
                 .IsSatisfied
                 .Should()
@@ -201,7 +214,7 @@ public void AllNamedColumnsMustBeNullableConventionSpecification_Fails()
             ExecuteSqlScriptFromResource("AllNamedColumnsMustBeNullableConventionalSpecification_Fail.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllNamedColumnsMustBeNullable("UpdatedDateTime"));
 
             result.IsSatisfied.Should().BeFalse();
@@ -214,7 +227,7 @@ public void AllIdentityColumnsMustBeNamedTableNameIdConventionSpecification_Succ
             ExecuteSqlScriptFromResource("AllIdentityColumnsMustBeNamedTableNameIdConventionSpecificationSuccess.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllIdentityColumnsMustBeNamedTableNameId)
                 .IsSatisfied
                 .Should()
@@ -227,7 +240,7 @@ public void AllIdentityColumnsMustBeNamedTableNameIdConventionSpecification_Fail
             ExecuteSqlScriptFromResource("AllIdentityColumnsMustBeNamedTableNameIdConventionSpecificationFailure.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllIdentityColumnsMustBeNamedTableNameId);
 
             result.IsSatisfied.Should().BeFalse();
@@ -240,7 +253,7 @@ public void AllTablesMustHaveAClusteredIndex_Success()
             ExecuteSqlScriptFromResource("TablesWithoutClusteredIndexSuccess.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllTablesMustHaveAClusteredIndex)
                 .IsSatisfied
                 .Should()
@@ -253,7 +266,7 @@ public void AllTablesMustHaveAClusteredIndex_Failure()
             ExecuteSqlScriptFromResource("TablesWithoutClusteredIndexFailure.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.AllTablesMustHaveAClusteredIndex);
 
             result.IsSatisfied.Should().BeFalse();
@@ -266,7 +279,7 @@ public void EachRowMustHaveACorrespondingEnum_Success()
             ExecuteSqlScriptFromResource("EachRowMustHaveACorrespondingEnum_Success.sql");
 
             TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.EachRowMustHaveACorrespondingEnum<CloudServiceEnum>("dbo.CloudService", "CloudServiceId"))
                 .IsSatisfied
                 .Should()
@@ -279,14 +292,14 @@ public void EachRowMustHaveACorrespondingEnum_Fail()
             ExecuteSqlScriptFromResource("EachRowMustHaveACorrespondingEnum_Fail.sql");
 
             var result = TheDatabase
-                .WithConnectionString(TestDbConnectionString)
+                .WithConnectionString(_settings.ConnectionString)
                 .MustConformTo(Convention.EachRowMustHaveACorrespondingEnum<CloudServiceEnum>("dbo.CloudService", "CloudServiceId"));
 
             result.IsSatisfied.Should().BeFalse();
             result.Failures.Should().HaveCount(1);
         }
 
-        private static void ExecuteSqlScriptFromResource(string resourceName)
+        private void ExecuteSqlScriptFromResource(string resourceName)
         {
             string script;
 
@@ -301,7 +314,7 @@ private static void ExecuteSqlScriptFromResource(string resourceName)
                 }
             }
 
-            using (IDbConnection dbConnection = new SqlConnection(TestDbConnectionString))
+            using (IDbConnection dbConnection = new SqlConnection(_settings.ConnectionString))
             {
                 dbConnection.Open();
                 var command = dbConnection.CreateCommand();
@@ -310,9 +323,9 @@ private static void ExecuteSqlScriptFromResource(string resourceName)
             }
         }
 
-        private static void CreateDatabase()
+        private void CreateDatabase()
         {
-            var sb = new SqlConnectionStringBuilder(TestDbConnectionString);
+            var sb = new SqlConnectionStringBuilder(_settings.ConnectionString);
             var dbName = sb.InitialCatalog;
             sb.InitialCatalog = "master";
 
@@ -325,9 +338,9 @@ private static void CreateDatabase()
             }
         }
         
-        private static void DropDatabase()
+        private void DropDatabase()
         {
-            var sb = new SqlConnectionStringBuilder(TestDbConnectionString);
+            var sb = new SqlConnectionStringBuilder(_settings.ConnectionString);
             var dbName = sb.InitialCatalog;
             sb.InitialCatalog = "master";
 
diff --git a/Conventional.Tests/DevelopmentSettings.cs b/Conventional.Tests/DevelopmentSettings.cs
new file mode 100644
index 0000000..eec2b60
--- /dev/null
+++ b/Conventional.Tests/DevelopmentSettings.cs
@@ -0,0 +1,31 @@
+using System.IO;
+using Newtonsoft.Json;
+
+namespace Conventional.Tests
+{
+    public class DevelopmentSettings
+    {
+        private DevelopmentSettings()
+        {
+        }
+
+#if DEBUG
+        public string ConnectionString { get; set; } = @"Server=.\SQLEXPRESS;Database=Conventional;Integrated Security=true;";
+#else
+        public string ConnectionString { get; set; } = @"Server=(local)\SQL2014;Database=Conventional;User ID=sa;Password=Password12!";
+#endif
+
+        public static DevelopmentSettings Create()
+        {
+            DevelopmentSettings settings = null;
+
+            var developmentSettingsFile = Path.Combine(KnownPaths.SolutionRoot, "development.json");
+            if (File.Exists(developmentSettingsFile))
+            {
+                settings = JsonConvert.DeserializeObject<DevelopmentSettings>(File.ReadAllText(developmentSettingsFile));
+            }
+
+            return settings ?? new DevelopmentSettings();
+        }
+    }
+}
\ No newline at end of file
diff --git a/Conventional.sln b/Conventional.sln
index 44eb5a7..57c3262 100644
--- a/Conventional.sln
+++ b/Conventional.sln
@@ -13,6 +13,7 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Conventional.Tests.Roslyn",
 EndProject
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Getting Started", "Getting Started", "{C38DA37B-5388-4443-8C6C-617A677D86AD}"
 	ProjectSection(SolutionItems) = preProject
+		development.json.example = development.json.example
 		README.md = README.md
 	EndProjectSection
 EndProject
diff --git a/README.md b/README.md
index adc01e7..fd8909f 100644
--- a/README.md
+++ b/README.md
@@ -24,6 +24,10 @@ To get started with Conventional, please check out [the wiki](https://github.com
 
 Not sure how to get started with Conventional? Check out [the sample repository](https://github.com/andrewabest/Conventional.Samples) which contains a bunch of real-world usage examples
 
+## Contributing
+
+Conventional's test suite requires a default named `.\SQLEXPRESS` instance. If you have another instance you would like to use for development, create a copy of `development.settings.example` in the solution root and rename to `development.settings`, and supply your own connection string.
+
 ## License
 
 Licensed under the terms of the [MS-PL](https://opensource.org/licenses/MS-PL) license
diff --git a/development.json.example b/development.json.example
new file mode 100644
index 0000000..bbd85d7
--- /dev/null
+++ b/development.json.example
@@ -0,0 +1,3 @@
+{
+  "connectionString": "Server=(localdb)\\MSSQLLocalDB;Database=Conventional;Integrated Security=true;"
+}
\ No newline at end of file