Merge pull request #11 from alhardy/dev

1.2.0 Release
AppMetrics · Jun 7, 2017 · 4392838 · 4392838
2 parents b3be969 + 29dba68
commit 4392838
Show file tree

Hide file tree

Showing 34 changed files with 1,314 additions and 671 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -4,12 +4,14 @@ notifications:
       - al_hardy@live.com.au
 
 language: csharp
-# dotnet: 1.0.0-preview2-003131
+dotnet: 1.0.4
 os:
-  - osx
+#  - osx
   - linux
+before_script:
+  - chmod a+x ./build.sh
 script:
-  - ./build_travis.sh
+  - ./build.sh
 
 # .NET CLI require Ubuntu 14.04
 sudo: required
@@ -24,11 +26,11 @@ addons:
     - libunwind8
 
 # .NET CLI require OSX 10.10
-osx_image: xcode7.1
-before_install:  
-  - if test "$TRAVIS_OS_NAME" == "osx"; then brew update; brew install openssl; ln -s /usr/local/opt/openssl/lib/libcrypto.1.0.0.dylib /usr/local/lib/; ln -s /usr/local/opt/openssl/lib/libssl.1.0.0.dylib /usr/local/lib/; fi
+# osx_image: xcode7.1
+# before_install:  
+#  - if test "$TRAVIS_OS_NAME" == "osx"; then brew update; brew install openssl; ln -s /usr/local/opt/openssl/lib/libcrypto.1.0.0.dylib /usr/local/lib/; ln -s /usr/local/opt/openssl/lib/libssl.1.0.0.dylib /usr/local/lib/; fi
 
-mono:  
+mono: 
   - 4.6.0
 
 cache:

diff --git a/AppMetrics.Extensions.InfluxDB.sln b/AppMetrics.Extensions.InfluxDB.sln
@@ -1,7 +1,7 @@
 
 Microsoft Visual Studio Solution File, Format Version 12.00
 # Visual Studio 15
-VisualStudioVersion = 15.0.26228.12
+VisualStudioVersion = 15.0.26403.7
 MinimumVisualStudioVersion = 10.0.40219.1
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "src", "src", "{2D805782-756E-4C98-B22E-F502BEE95318}"
 EndProject
@@ -48,6 +48,15 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "sandbox", "sandbox", "{B0E3
 EndProject
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "App.Metrics.InfluxDB.Sandbox", "sandbox\App.Metrics.InfluxDB.Sandbox\App.Metrics.InfluxDB.Sandbox.csproj", "{4D15C49F-F4DC-4631-B8FA-1D307C461227}"
 EndProject
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "App.Metrics.Formatters.InfluxDB", "src\App.Metrics.Formatters.InfluxDB\App.Metrics.Formatters.InfluxDB.csproj", "{5ACC61B5-8285-4798-A215-B7677A59EA22}"
+EndProject
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "App.Metrics.Formatting.InfluxDB", "src\App.Metrics.Formatting.InfluxDB\App.Metrics.Formatting.InfluxDB.csproj", "{A1659468-1D7F-4DAB-8487-4E14DC9D92BB}"
+EndProject
+Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Build", "Build", "{28CE67A4-2662-43AA-B42E-B3C8F33DF31B}"
+	ProjectSection(SolutionItems) = preProject
+		build\common.props = build\common.props
+	EndProjectSection
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -66,6 +75,14 @@ Global
 		{4D15C49F-F4DC-4631-B8FA-1D307C461227}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{4D15C49F-F4DC-4631-B8FA-1D307C461227}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{4D15C49F-F4DC-4631-B8FA-1D307C461227}.Release|Any CPU.Build.0 = Release|Any CPU
+		{5ACC61B5-8285-4798-A215-B7677A59EA22}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{5ACC61B5-8285-4798-A215-B7677A59EA22}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{5ACC61B5-8285-4798-A215-B7677A59EA22}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{5ACC61B5-8285-4798-A215-B7677A59EA22}.Release|Any CPU.Build.0 = Release|Any CPU
+		{A1659468-1D7F-4DAB-8487-4E14DC9D92BB}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{A1659468-1D7F-4DAB-8487-4E14DC9D92BB}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{A1659468-1D7F-4DAB-8487-4E14DC9D92BB}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{A1659468-1D7F-4DAB-8487-4E14DC9D92BB}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE
@@ -78,5 +95,8 @@ Global
 		{5BC3B2AF-AFC7-4C92-BD1E-D347E83B2497} = {31A4DDB1-952E-4EED-96EF-29C669279A86}
 		{44E50731-4AE5-456F-A84C-CE3C4C977097} = {5BC3B2AF-AFC7-4C92-BD1E-D347E83B2497}
 		{4D15C49F-F4DC-4631-B8FA-1D307C461227} = {B0E330F2-942D-47DC-940D-692E01459A06}
+		{5ACC61B5-8285-4798-A215-B7677A59EA22} = {2D805782-756E-4C98-B22E-F502BEE95318}
+		{A1659468-1D7F-4DAB-8487-4E14DC9D92BB} = {2D805782-756E-4C98-B22E-F502BEE95318}
+		{28CE67A4-2662-43AA-B42E-B3C8F33DF31B} = {31A4DDB1-952E-4EED-96EF-29C669279A86}
 	EndGlobalSection
 EndGlobal
diff --git a/README.md b/README.md
@@ -1,40 +1,46 @@
-# App Metrics InfluxDB Extensions
+# App Metrics InfluxDB Extensions <img src="http://app-metrics.io/logo.png" alt="App Metrics" width="50px"/> 
+[![Official Site](https://img.shields.io/badge/site-appmetrics-blue.svg?style=flat-square)](http://app-metrics.io/reporting/influxdb.html) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg?style=flat-square)](https://opensource.org/licenses/Apache-2.0)
 
-[![Official Site](https://img.shields.io/badge/site-appmetrics-blue.svg)](https://alhardy.github.io/app-metrics-docs/getting-started/intro.html) [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) [![Coverage Status](https://coveralls.io/repos/github/alhardy/AppMetrics.Extensions.InfluxDB/badge.svg?branch=master)](https://coveralls.io/github/alhardy/AppMetrics.Extensions.InfluxDB?branch=master)
+## What is it?
 
-|AppVeyor|Travis|
-|:--------:|:--------:|
-|[![Build status](https://ci.appveyor.com/api/projects/status/cdda6bfu3bput7h3?svg=true)](https://ci.appveyor.com/project/alhardy/appmetrics-extensions-influxdb/branch/master)|[![Build status](https://travis-ci.org/alhardy/AppMetrics.Extensions.InfluxDB.svg?branch=master)](https://travis-ci.org/alhardy/AppMetrics.Extensions.InfluxDB?branch=master)|
+The repo contains InfluxDB extension packages to [App Metrics](https://github.com/alhardy/AppMetrics).
 
-|Package|Dev Release|Pre-Release|Latest Release|
-|------|:--------:|:--------:|:--------:|
-|App.Metrics.Extensions.Reporting.InfluxDB|[![MyGet Status](https://img.shields.io/myget/alhardy/v/App.Metrics.Extensions.Reporting.InfluxDB.svg)](https://www.myget.org/feed/alhardy/package/nuget/App.Metrics.Extensions.Reporting.InfluxDB)|[![NuGet Status](https://img.shields.io/nuget/vpre/App.Metrics.Extensions.Reporting.InfluxDB.svg)](https://www.nuget.org/packages/App.Metrics.Extensions.Reporting.InfluxDB/)|[![NuGet Status](https://img.shields.io/nuget/v/App.Metrics.Extensions.Reporting.InfluxDB.svg)](https://www.nuget.org/packages/App.Metrics.Extensions.Reporting.InfluxDB/)
+## Latest Builds, Packages & Repo Stats
 
-## What is it?
+|Branch|AppVeyor|Travis|Coverage|
+|------|:--------:|:--------:|:--------:|
+|dev|[![AppVeyor](https://img.shields.io/appveyor/ci/alhardy/appmetrics-extensions-influxdb/dev.svg?style=flat-square&label=appveyor%20build)](https://ci.appveyor.com/project/alhardy/appmetrics-extensions-influxdb/branch/dev)|[![Travis](https://img.shields.io/travis/alhardy/AppMetrics.Extensions.InfluxDB/dev.svg?style=flat-square&label=travis%20build)](https://travis-ci.org/alhardy/AppMetrics.Extensions.InfluxDB)|[![Coveralls](https://img.shields.io/coveralls/alhardy/AppMetrics.Extensions.InfluxDB/dev.svg?style=flat-square)](https://coveralls.io/github/alhardy/AppMetrics.Extensions.InfluxDB?branch=dev)
+|master|[![AppVeyor](https://img.shields.io/appveyor/ci/alhardy/appmetrics-extensions-influxdb/master.svg?style=flat-square&label=appveyor%20build)](https://ci.appveyor.com/project/alhardy/appmetrics-extensions-influxdb/branch/master)| [![Travis](https://img.shields.io/travis/alhardy/AppMetrics.Extensions.InfluxDB/master.svg?style=flat-square&label=travis%20build)](https://travis-ci.org/alhardy/AppMetrics.Extensions.InfluxDB)| [![Coveralls](https://img.shields.io/coveralls/alhardy/AppMetrics.Extensions.InfluxDB/master.svg?style=flat-square)](https://coveralls.io/github/alhardy/AppMetrics.Extensions.InfluxDB?branch=master)|
 
-The repo contains InfluxDB extension packages to [App Metrics](https://github.com/alhardy/AppMetrics).
+|Package|Dev Release|PreRelease|Latest Release|
+|------|:--------:|:--------:|:--------:|
+|App.Metrics.Extensions.Reporting.InfluxDB|[![MyGet Status](https://img.shields.io/myget/alhardy/v/App.Metrics.Extensions.Reporting.InfluxDB.svg?style=flat-square)](https://www.myget.org/feed/alhardy/package/nuget/App.Metrics.Extensions.Reporting.InfluxDB)|[![NuGet Status](https://img.shields.io/nuget/vpre/App.Metrics.Extensions.Reporting.InfluxDB.svg?style=flat-square)](https://www.nuget.org/packages/App.Metrics.Extensions.Reporting.InfluxDB/)|[![NuGet Status](https://img.shields.io/nuget/v/App.Metrics.Extensions.Reporting.InfluxDB.svg?style=flat-square)](https://www.nuget.org/packages/App.Metrics.Extensions.Reporting.InfluxDB/)
+|App.Metrics.Formatters.InfluxDB|[![MyGet Status](https://img.shields.io/myget/alhardy/v/App.Metrics.Formatters.InfluxDB.svg?style=flat-square)](https://www.myget.org/feed/alhardy/package/nuget/App.Metrics.Formatters.InfluxDB)|[![NuGet Status](https://img.shields.io/nuget/vpre/App.Metrics.Formatters.InfluxDB.svg?style=flat-square)](https://www.nuget.org/packages/App.Metrics.Formatters.InfluxDB/)|[![NuGet Status](https://img.shields.io/nuget/v/App.Metrics.Formatters.InfluxDB.svg?style=flat-square)](https://www.nuget.org/packages/App.Metrics.Formatters.InfluxDB/)
+|App.Metrics.Formatting.InfluxDB|[![MyGet Status](https://img.shields.io/myget/alhardy/v/App.Metrics.Formatting.InfluxDB.svg?style=flat-square)](https://www.myget.org/feed/alhardy/package/nuget/App.Metrics.Formatting.InfluxDB)|[![NuGet Status](https://img.shields.io/nuget/vpre/App.Metrics.Formatting.InfluxDB.svg?style=flat-square)](https://www.nuget.org/packages/App.Metrics.Formatting.InfluxDB/)|[![NuGet Status](https://img.shields.io/nuget/v/App.Metrics.Formatting.InfluxDB.svg?style=flat-square)](https://www.nuget.org/packages/App.Metrics.Formatting.InfluxDB/)
 
 #### Grafana/InfluxDB Web Monitoring
 
 ![Grafana/InfluxDB Generic Web Dashboard Demo](https://github.com/alhardy/AppMetrics.DocFx/blob/master/images/generic_grafana_dashboard_demo.gif)
 
-> Grab the dashboard [here](https://github.com/alhardy/AppMetrics.Extensions.InfluxDB/blob/dev/visualization/grafana-dashbaords/App.Metrics.Sandbox-InfluxDB-GenericWeb.json)
+> Grab the dashboard [here](https://grafana.com/dashboards/2125)
 
 #### Grafana/InfluxDB OAuth2 Client Monitoring on a Web API
 
 ![Grafana/InfluxDB Generic OAuth2 Web Dashboard Demo](https://github.com/alhardy/AppMetrics.DocFx/blob/master/images/generic_grafana_oauth2_dashboard_demo.gif)
 
-> Grab the dashboard [here](https://github.com/alhardy/AppMetrics.Extensions.InfluxDB/blob/dev/visualization/grafana-dashbaords/App.Metrics.Sandbox-InfluxDB-GenericWebOAuth2.json)
+> Grab the dashboard [here](https://grafana.com/dashboards/2137)
 
 ### Grafana/InfluxDB Web Application Setup
 
 - Download and install [InfluxDB](https://docs.influxdata.com/influxdb/v1.2/introduction/installation/). *Runs well on Windows using* `Bash on Windows on Ubuntu`
 - Create a new [InfluxDB Database](https://docs.influxdata.com/influxdb/v1.2/introduction/getting_started/). *Keep note of this for configuring the InfluxDB reporter in your web application and configuring the InfluxDB Datasource in Grafana*
-- Download and install [Grafana](https://grafana.com/grafana/download), then create a new [InfluxDB Datasource](http://docs.grafana.org/features/datasources/influxdb/)  pointing the the Database just created and import App.Metrics [web dashboard](https://github.com/alhardy/AppMetrics.Extensions.InfluxDB/blob/dev/visualization/grafana-dashbaords/App.Metrics.Sandbox-InfluxDB-GenericWeb.json)
+- Download and install [Grafana](https://grafana.com/grafana/download), then create a new [InfluxDB Datasource](http://docs.grafana.org/features/datasources/influxdb/) pointing the the Database just created and [import](http://docs.grafana.org/reference/export_import/#importing-a-dashboard) App.Metrics [web dashboard](https://grafana.com/dashboards/2125)
 - Drop in the `App.Metrics.Extensions.Mvc` and `App.Metrics.Extensions.Reporting.InfluxDB` nuget packages into your web application. 
-- Add [App.Metrics configuration](https://alhardy.github.io/app-metrics-docs/getting-started/fundamentals/middleware-configuration.html) to the `Startup.cs` of your web application, including the [InfluxDB reporter configuration](https://alhardy.github.io/app-metrics-docs/getting-started/reporting/index.html#influxdb-reporter). *You might want to check out the [Sandbox](https://github.com/alhardy/AppMetrics/tree/1.0.0/sandbox/App.Metrics.Sandbox) or [Sample](https://github.com/alhardy/AppMetrics.Samples) projects if you get stuck*
+- Add [App.Metrics configuration](https://alhardy.github.io/app-metrics-docs/getting-started/fundamentals/middleware-configuration.html) to the `Startup.cs` of your web application, including the [InfluxDB reporter configuration](http://app-metrics.io/reporting/influxdb.html). *You might want to check out the [Sandbox](https://github.com/alhardy/AppMetrics.Extensions.InfluxDB/tree/master/sandbox/App.Metrics.InfluxDB.Sandbox) project if you get stuck*
 - Run your app and Grafana at visit `http://localhost:3000`
 
+**There is also a more detailed step-by-step guide [here](https://al-hardy.blog/2017/04/28/asp-net-core-monitoring-with-influxdb-grafana/)**
+
 ## How to build
 
 [AppVeyor](https://ci.appveyor.com/project/alhardy/appmetrics-extensions-influxdb/branch/master) and [Travis CI](https://travis-ci.org/alhardy/AppMetrics.Extensions.InfluxDB) builds are triggered on commits and PRs to `dev` and `master` branches.

diff --git a/appveyor.yml b/appveyor.yml
@@ -3,15 +3,15 @@ image: Visual Studio 2017
 environment:
   COVERALLS_REPO_TOKEN:
     secure: jE+uYnSDVrBExOFkaPPHpJeKfHxVuS+lYvS+8BlcKBTSioz25h+rBq3RxMGl9JBd
-  PreReleaseSuffix: stable
+  PreReleaseSuffix: alpha
   DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true
   DOTNET_CLI_TELEMETRY_OPTOUT: true
   GitUser:
     secure: zAdAOUUof3XGDsdOBRYg7J7wZS44iL4VjI/MVGw+JnU=
   GitOwner:
     secure: n6W5JZ7Q/xfZC7b2k3+ORA==
   GitPassword:
-    secure: G1CZ0EzJTYjH60jzYYaitkieeslue7VdeeVW4vYS+msXQ9gSCf7KstBKEbXdI+W2
+    secure: fHangGPAk14u3V1eP3kg5EdekQoKdOsKC0F/A1mvh9IODHCZ92FGjfWb0U2aQ8bu
 build_script:
 - ps: .\build.ps1 -Target AppVeyor
 test: off

diff --git a/build.cake b/build.cake
@@ -9,6 +9,7 @@
 #tool "nuget:?package=JetBrains.ReSharper.CommandLineTools"
 #tool "nuget:?package=coveralls.io"
 #tool "nuget:?package=gitreleasemanager"
+#tool "nuget:?package=ReportGenerator"
 
 //////////////////////////////////////////////////////////////////////
 // ARGUMENTS
@@ -20,24 +21,24 @@ var coverWith					= HasArgument("CoverWith") ? Argument<string>("CoverWith") :
                                   EnvironmentVariable("CoverWith") != null ? EnvironmentVariable("CoverWith") : "OpenCover"; // None, DotCover, OpenCover
 var skipReSharperCodeInspect    = Argument("SkipCodeInspect", false) || !IsRunningOnWindows();
 var preReleaseSuffix            = HasArgument("PreReleaseSuffix") ? Argument<string>("PreReleaseSuffix") :
-	                              (AppVeyor.IsRunningOnAppVeyor && EnvironmentVariable("PreReleaseSuffix") == null) ? null :
+	                              (AppVeyor.IsRunningOnAppVeyor && EnvironmentVariable("PreReleaseSuffix") == null || AppVeyor.Environment.Repository.Tag.IsTag) ? null :
                                   EnvironmentVariable("PreReleaseSuffix") != null ? EnvironmentVariable("PreReleaseSuffix") : "ci";
 var buildNumber                 = HasArgument("BuildNumber") ? Argument<int>("BuildNumber") :
                                   AppVeyor.IsRunningOnAppVeyor ? AppVeyor.Environment.Build.Number :
                                   TravisCI.IsRunningOnTravisCI ? TravisCI.Environment.Build.BuildNumber :
                                   EnvironmentVariable("BuildNumber") != null ? int.Parse(EnvironmentVariable("BuildNumber")) : 0;
 var gitUser						= HasArgument("GitUser") ? Argument<string>("GitUser") : EnvironmentVariable("GitUser");
 var gitPassword					= HasArgument("GitPassword") ? Argument<string>("GitPassword") : EnvironmentVariable("GitPassword");
+var skipHtmlCoverageReport		= Argument("SkipHtmlCoverageReport", true) || !IsRunningOnWindows();
 
 //////////////////////////////////////////////////////////////////////
 // DEFINE FILES & DIRECTORIES
 //////////////////////////////////////////////////////////////////////
-var packDirs                    = new [] { Directory("./src/App.Metrics.Extensions.Reporting.InfluxDB") };
+var packDirs                    = new [] { Directory("./src/App.Metrics.Extensions.Reporting.InfluxDB"), Directory("./src/App.Metrics.Formatters.InfluxDB"), Directory("./src/App.Metrics.Formatting.InfluxDB") };
 var artifactsDir                = (DirectoryPath) Directory("./artifacts");
 var testResultsDir              = (DirectoryPath) artifactsDir.Combine("test-results");
 var coverageResultsDir          = (DirectoryPath) artifactsDir.Combine("coverage");
 var reSharperReportsDir         = (DirectoryPath) artifactsDir.Combine("resharper-reports");
-var testCoverageOutputFilePath  = coverageResultsDir.CombineWithFilePath("coverage.xml");
 var testOCoverageOutputFilePath = coverageResultsDir.CombineWithFilePath("openCoverCoverage.xml");
 var htmlCoverageReport			= coverageResultsDir.FullPath + "/coverage.html";
 var mergedCoverageSnapshots		= coverageResultsDir.FullPath + "/coverage.dcvr";
@@ -58,6 +59,17 @@ var openCoverExcludeFile        = "*/*Designer.cs;*/*.g.cs;*/*.g.i.cs";
 var coverIncludeFilter			= "+:App.Metrics*";
 var coverExcludeFilter			= "-:*.Facts";
 var excludeFromCoverage			= "*.AppMetricsExcludeFromCodeCoverage*";
+string versionSuffix			= null;
+
+if (!string.IsNullOrEmpty(preReleaseSuffix))
+{
+	versionSuffix = preReleaseSuffix + "-" + buildNumber.ToString("D4");
+}
+ else if (AppVeyor.IsRunningOnAppVeyor && !AppVeyor.Environment.Repository.Tag.IsTag)
+ {
+ 	versionSuffix = buildNumber.ToString("D4");
+ }
+
 
 //////////////////////////////////////////////////////////////////////
 // TASKS
@@ -115,7 +127,14 @@ Task("Build")
     .IsDependentOn("Restore")
     .Does(() =>
 {	
-	var settings = new DotNetCoreBuildSettings  { Configuration = configuration };
+	var settings = new DotNetCoreBuildSettings  { Configuration = configuration, VersionSuffix = versionSuffix };
+
+	Context.Information("Building using preReleaseSuffix: " + preReleaseSuffix);
+	Context.Information("Building using versionSuffix: " + versionSuffix);
+
+	// Workaround to fixing pre-release version package references - https://github.com/NuGet/Home/issues/4337
+	settings.ArgumentCustomization = args=>args.Append("/t:Restore /p:RestoreSources=https://api.nuget.org/v3/index.json;https://www.myget.org/F/alhardy/api/v3/index.json;");
+
 
 	if (IsRunningOnWindows())
 	{
@@ -164,13 +183,6 @@ Task("Pack")
 	}
 
 	Context.Information("Packing using preReleaseSuffix: " + preReleaseSuffix);
-
-    string versionSuffix = null;
-    if (!string.IsNullOrEmpty(preReleaseSuffix))
-    {
-        versionSuffix = preReleaseSuffix + "-" + buildNumber.ToString("D4");
-    }
-
 	Context.Information("Packing using versionSuffix: " + versionSuffix);
 
     var settings = new DotNetCorePackSettings
@@ -231,7 +243,7 @@ Task("RunTests")
 });
 
 Task("HtmlCoverageReport")    
-    .WithCriteria(() => FileExists(testCoverageOutputFilePath) && coverWith != "None" && IsRunningOnWindows())    
+    .WithCriteria(() => IsRunningOnWindows() && FileExists(testOCoverageOutputFilePath) && coverWith != "None" && !skipHtmlCoverageReport)    
     .IsDependentOn("RunTests")
     .Does(() => 
 {
@@ -246,7 +258,7 @@ Task("HtmlCoverageReport")
 	}
 	else if (coverWith == "OpenCover")
 	{
-		ReportGenerator(testCoverageOutputFilePath, coverageResultsDir);
+		ReportGenerator(testOCoverageOutputFilePath, coverageResultsDir);
 	}
 });
 
@@ -394,19 +406,19 @@ Task("PublishCoverage")
 //////////////////////////////////////////////////////////////////////
 
 Task("Default")	
-    	.IsDependentOn("Build")
+    .IsDependentOn("Build")
 	.IsDependentOn("PublishTestResults")	
-    	.IsDependentOn("Pack")
+    .IsDependentOn("Pack")
 	.IsDependentOn("HtmlCoverageReport")
 	.IsDependentOn("RunInspectCode");	
 
 Task("AppVeyor")
-    	.IsDependentOn("Build")
+    .IsDependentOn("Build")
 	.IsDependentOn("PublishTestResults")	
-    	.IsDependentOn("Pack")
+    .IsDependentOn("Pack")
 	.IsDependentOn("HtmlCoverageReport")
 	.IsDependentOn("RunInspectCode")	
-    	.IsDependentOn("PublishCoverage")
+    .IsDependentOn("PublishCoverage")
 	.IsDependentOn("ReleaseNotes");
 
 //////////////////////////////////////////////////////////////////////