From f67aaec566583a1fbf60376ee6569e904d96f212 Mon Sep 17 00:00:00 2001
From: idubnori <i.dub.nori@gmail.com>
Date: Fri, 6 Dec 2019 10:42:49 +0900
Subject: [PATCH] Fix - Not applied patch if path contains Uppercase (#330)

* Add JsonPatchTests

* Upgrade JsonPatch and Json.net, if netstandard2.0 or netcoreapp2.1
---
 src/KubernetesClient/KubernetesClient.csproj  |  6 ++++--
 .../KubernetesClient.Tests/JsonPatchTests.cs  | 19 +++++++++++++++++++
 2 files changed, 23 insertions(+), 2 deletions(-)
 create mode 100644 tests/KubernetesClient.Tests/JsonPatchTests.cs

diff --git a/src/KubernetesClient/KubernetesClient.csproj b/src/KubernetesClient/KubernetesClient.csproj
index 6129070..9e1bae0 100644
--- a/src/KubernetesClient/KubernetesClient.csproj
+++ b/src/KubernetesClient/KubernetesClient.csproj
@@ -27,13 +27,15 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.AspNetCore.JsonPatch" Version="1.1.2" />
+    <PackageReference Include="Microsoft.AspNetCore.JsonPatch" Version="1.1.2" Condition="'$(TargetFramework)' != 'netstandard2.0' and '$(TargetFramework)' != 'netcoreapp2.1'" />
+    <PackageReference Include="Microsoft.AspNetCore.JsonPatch" Version="3.0.0" Condition="'$(TargetFramework)' == 'netstandard2.0' or '$(TargetFramework)' == 'netcoreapp2.1'" />
     <PackageReference Include="Nerdbank.GitVersioning" Version="2.3.138" PrivateAssets="all" />
     <PackageReference Include="Portable.BouncyCastle" Version="1.8.1.3" />
     <PackageReference Include="Microsoft.AspNetCore.WebUtilities" Version="1.1.2" />
     <PackageReference Include="Microsoft.Rest.ClientRuntime" Version="2.3.10" />
     <PackageReference Include="MSBuild.Sdk.Extras" Version="1.6.68" PrivateAssets="all" Condition="'$(MSBuildRuntimeType)' != 'Core'" />
-    <PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
+    <PackageReference Include="Newtonsoft.Json" Version="9.0.1" Condition="'$(TargetFramework)' != 'netstandard2.0' and '$(TargetFramework)' != 'netcoreapp2.1'" />
+    <PackageReference Include="Newtonsoft.Json" Version="12.0.2" Condition="'$(TargetFramework)' == 'netstandard2.0' or '$(TargetFramework)' == 'netcoreapp2.1'" />
     <PackageReference Include="YamlDotNet" Version="6.0.0" />
     <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0-beta2-18618-05" PrivateAssets="All" />
   </ItemGroup>
diff --git a/tests/KubernetesClient.Tests/JsonPatchTests.cs b/tests/KubernetesClient.Tests/JsonPatchTests.cs
new file mode 100644
index 0000000..1b8c3f1
--- /dev/null
+++ b/tests/KubernetesClient.Tests/JsonPatchTests.cs
@@ -0,0 +1,19 @@
+using System.Linq;
+using k8s.Models;
+using Microsoft.AspNetCore.JsonPatch;
+using Xunit;
+
+namespace k8s.Tests
+{
+    public class JsonPatchTests
+    {
+        [Fact]
+        public void PathContainsUpperCase()
+        {
+            var patch = new JsonPatchDocument<V1HorizontalPodAutoscaler>();
+            patch.Replace(h => h.Spec.MinReplicas, 1);
+
+            Assert.Equal("/spec/minReplicas", patch.Operations.Single().path);
+        }
+    }
+}