Merge pull request #17 from TerribleDev/nullchk

null checks

appveyor.yml

@@ -23,6 +23,6 @@ test_script:
 deploy:
 - provider: NuGet
   api_key:
-    secure: bGn7M6dHOJ3QjwYIv7e34tcY/n9cCUZmL1MnM6jRfmnJOOfwlrS+cdRj2n8Wf31n
+    secure: //tKHlb2yqAtpxnR6p9IAtXwQNaq8UYYyIFSD0QVF3XnEasIxG2gTWdmWuG87fUX
   on:
     appveyor_repo_tag: true

src/UriBuilder.Fluent.UnitTests/ThrowsTests.cs

@@ -20,6 +20,13 @@ namespace FluentUriBuilder.Tests
             Assert.Throws<ArgumentNullException>(() => tstObj.WithParameter(parameterDictionary: null));
             Assert.Throws<ArgumentNullException>(() => tstObj.WithFragment(fragmentDictionary: null));
             Assert.Throws<ArgumentOutOfRangeException>(() => tstObj.WithPort(-1));
+            UriBuilder nullPtr = null;
+            Assert.Throws<ArgumentNullException>(()=>nullPtr.WithFragment("yo", "dawg"));
+            Assert.Throws<ArgumentNullException>(()=>nullPtr.WithParameter("yo", "dawg"));
+            Assert.Throws<ArgumentNullException>(()=>nullPtr.WithPathSegment("yo"));
+            Assert.Throws<ArgumentNullException>(()=>nullPtr.WithScheme("yo"));
+            Assert.Throws<ArgumentNullException>(()=>nullPtr.WithFragment("yo"));
+            Assert.Throws<ArgumentNullException>(()=>nullPtr.WithoutDefaultPort());
         }
     }
 }

src/UriBuilder.Fluent.UnitTests/UriBuilder.Fluent.UnitTests.csproj

@@ -1,6 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
-    <TargetFrameworks>netcoreapp1.1;net461</TargetFrameworks>
+    <TargetFrameworks>netcoreapp2.0;net461</TargetFrameworks>
   </PropertyGroup>
   <ItemGroup>
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.0.0" />

src/UriBuilder.Fluent/TerribleDevUriExtensions.cs

@@ -26,6 +26,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithParameter(this UriBuilder bld, IDictionary<string, string> parameterDictionary)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if(parameterDictionary == null) throw new ArgumentNullException(nameof(parameterDictionary));
             foreach(var item in parameterDictionary)
             {
@@ -43,6 +47,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithParameter(this UriBuilder bld, string key, IEnumerable<object> valuesEnum)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if(string.IsNullOrWhiteSpace(key))
             {
                 throw new ArgumentNullException(nameof(key));
@@ -74,6 +82,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithFragment(this UriBuilder bld, IDictionary<string, string> fragmentDictionary)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if(fragmentDictionary == null) throw new ArgumentNullException(nameof(fragmentDictionary));
             foreach(var item in fragmentDictionary)
             {
@@ -91,6 +103,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithFragment(this UriBuilder bld, string key, IEnumerable<object> valuesEnum)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if(string.IsNullOrWhiteSpace(key))
             {
                 throw new ArgumentNullException(nameof(key));
@@ -113,6 +129,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithPort(this UriBuilder bld, int port)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if(port < 1) throw new ArgumentOutOfRangeException(nameof(port));
             bld.Port = port;
             return bld;
@@ -125,6 +145,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithoutDefaultPort(this UriBuilder bld)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if (bld.Uri.IsDefaultPort) bld.Port = -1;
             return bld;
         }
@@ -138,6 +162,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithPathSegment(this UriBuilder bld, string pathSegment)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if(string.IsNullOrWhiteSpace(pathSegment))
             {
                 throw new ArgumentNullException(nameof(pathSegment));
@@ -156,6 +184,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithScheme(this UriBuilder bld, string scheme)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if(string.IsNullOrWhiteSpace(scheme)) throw new ArgumentNullException(nameof(scheme));
             bld.Scheme = scheme;
             return bld;
@@ -170,6 +202,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder WithHost(this UriBuilder bld, string host)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             if(string.IsNullOrWhiteSpace(host)) throw new ArgumentNullException(nameof(host));
             bld.Host = host;
             return bld;
@@ -185,6 +221,10 @@ namespace System
         /// <returns></returns>
         public static UriBuilder UseHttps(this UriBuilder bld, bool predicate = true)
         {
+            if(bld == null)
+            {
+                throw new ArgumentNullException(nameof(bld));
+            }
             bld.Scheme = predicate ? "https" : "http";
             return bld;
         }

src/UriBuilder.Fluent/UriBuilder.Fluent.csproj

@@ -2,7 +2,7 @@
 
   <PropertyGroup>
     <Authors>Tommy Parnell</Authors>
-    <TargetFrameworks>netstandard1.1;net40;net45</TargetFrameworks>
+    <TargetFrameworks>netstandard1.1;net40;net45;netstandard2.0</TargetFrameworks>
     <AssemblyName>UriBuilder.Fluent</AssemblyName>
     <PackageId>UriBuilder.Fluent</PackageId>
     <PackageTags>Url building;Uri;Uri building;fluent;extension</PackageTags>