Merge pull request #153 from LowlyDBA/development

Development

.gitattributes

@@ -1,5 +1,6 @@
 # linguist overrides
 *.sql linguist-language=TSQL
+*.sql diff
 
 # line endings for code coverage files
 appveyor/sqlcover/*       text eol=lf

.gitignore

@@ -1,3 +1,4 @@
 .markdownlint.json
 node_modules/*
 package-lock.json
+debug.log

appveyor/build_tsqlt_tests.ps1

@@ -9,6 +9,8 @@ param(
     $Color = "Green"
     )
 
+$ErrorActionPreference = "Stop"
+
 Write-Host "Building tSQLt Tests..." -ForegroundColor $Color
 
 If ($IsAzureSQL) {

appveyor/generate_combined_script.ps1

@@ -5,4 +5,4 @@ if (Test-Path $File) {
     Remove-Item $File
 }
 
-Get-Item $Filter | Get-Content | Out-File $File
+Get-Item $Filter | Get-Content | Out-File $File -Encoding utf8

appveyor/install_dependencies.ps1

@@ -7,13 +7,17 @@ Write-Host "Installing dependencies..." -ForegroundColor $Color
 
 # TSQLLinter
 # Try/Catch to stop appveyor unnecessary errors
-Try { npm install tsqllint -g | Out-Null }
-Catch { }
+$result = npm list -g --depth=0
+If (-Not ($result -Match "tsqllint")) {
+    npm install tsqllint -g | Out-Null 
+}
 
 # SQLServer Module
 if (!(Get-Module -ListAvailable -Name SqlServer)) {
     Install-Module SqlServer -Force -AllowClobber
 }
 
 # DbaTools Module
-Install-Module DbaTools -Force -AllowClobber
+if (!(Get-Module -ListAvailable -Name DbaTools)) {
+    Install-Module DbaTools -Force -AllowClobber
+}

appveyor/run_tsqllint.ps1

@@ -4,5 +4,7 @@ param(
     $Color = "Green"
     )
 
+$ErrorActionPreference = "Stop"
+
 Write-Host "Running TSQLLint with config $Config..." -ForegroundColor $Color
 tsqllint -c $Config *.sql

appveyor/sqlcover/Run_SQLCover.ps1

@@ -52,7 +52,7 @@ If ($IsCoverStarted) {
         $tmpFile = Join-Path $env:TEMP "Coverage.html"
         Set-Content -Path $tmpFile -Value $coverageResults.Html2() -Force
         Invoke-Item $tmpFile
-        Start-Sleep -Seconds 1
+        Start-Sleep -Seconds 3
         Remove-Item $tmpFile
     }
 }

appveyor/sqlcover/[dbo].[sp_doc]

@@ -4,6 +4,7 @@ CREATE PROCEDURE [dbo].[sp_doc]
 	,@ExtendedPropertyName SYSNAME = 'Description'
 	,@LimitStoredProcLength BIT = 1
 	,@Emojis BIT = 0
+	,@Verbose BIT = 1
 	/* Parameters defined here for testing only */
 	,@SqlMajorVersion TINYINT = 0
 	,@SqlMinorVersion SMALLINT = 0
@@ -78,6 +79,11 @@ BEGIN
 	IF (@DatabaseName IS NULL)
 		BEGIN
 			SET @DatabaseName = DB_NAME();
+			IF (@Verbose = 1)
+				BEGIN;
+					SET @Msg = 'No database provided, assuming current database.';
+					RAISERROR(@Msg, 10, 1) WITH NOWAIT;
+				END;
 		END
 	ELSE IF (DB_ID(@DatabaseName) IS NULL)
 		BEGIN;