adds test for #640

pardeike · Dec 19, 2024 · 4852c9c · 4852c9c
1 parent 5be64c9
commit 4852c9c
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 0 deletions.
diff --git a/HarmonyTests/Patching/Arguments.cs b/HarmonyTests/Patching/Arguments.cs
@@ -396,5 +396,23 @@ public void Test_RenamedArguments()
 			var log = RenamedArgumentsPatch.log.Join();
 			Assert.AreEqual("val1, patched, val2, hello", log);
 		}
+
+		[Test]
+		public void Test_NullableResults()
+		{
+			var res1 = new NullableResults().Method();
+			Assert.True(res1.HasValue);
+			Assert.False(res1.Value);
+
+			var harmony = new Harmony("test");
+			var processor = new PatchClassProcessor(harmony, typeof(NullableResultsPatch));
+			var patches = processor.Patch();
+			Assert.NotNull(patches, "patches");
+			Assert.AreEqual(1, patches.Count);
+
+			var res2 = new NullableResults().Method();
+			Assert.True(res2.HasValue);
+			Assert.True(res2.Value);
+		}
 	}
 }
diff --git a/HarmonyTests/Patching/Assets/ArgumentCases.cs b/HarmonyTests/Patching/Assets/ArgumentCases.cs
@@ -144,6 +144,24 @@ public static void Postfix([HarmonyArgument("__instance")] RenamedArguments foo2
 		}
 	}
 
+	public class NullableResults
+	{
+		private string s = "foo";
+
+		[MethodImpl(MethodImplOptions.NoInlining)]
+		public bool? Method() => false;
+	}
+
+	[HarmonyPatch(typeof(NullableResults), nameof(NullableResults.Method))]
+	public static class NullableResultsPatch
+	{
+		public static bool Prefix(ref bool? __result)
+		{
+			__result = true;
+			return false;
+		}
+	}
+
 	public class ArgumentArrayMethods
 	{
 		public struct SomeStruct