first bug fix: SerializationRecord.TypeNameMatches should throw ArgumentNullException for null Type argument

Author: adamsitnik

src/libraries/System.Formats.Nrbf/src/System/Formats/Nrbf/SerializationRecord.cs

@@ -50,7 +50,20 @@ internal SerializationRecord() // others can't derive from this type
     /// </remarks>
     /// <param name="type">The type to compare against.</param>
     /// <returns><see langword="true" /> if the serialized type name matches the provided type; otherwise, <see langword="false" />.</returns>
-    public bool TypeNameMatches(Type type) => Matches(type, TypeName);
+    /// <exception cref="ArgumentNullException"><paramref name="type" /> is <see langword="null" />.</exception>
+    public bool TypeNameMatches(Type type)
+    {
+#if NET
+        ArgumentNullException.ThrowIfNull(type);
+#else
+        if (type is null)
+        {
+            throw new ArgumentNullException(nameof(type));
+        }
+#endif
+
+        return Matches(type, TypeName);
+    }
 
     private static bool Matches(Type type, TypeName typeName)
     {

src/libraries/System.Formats.Nrbf/tests/TypeMatchTests.cs

@@ -73,6 +73,16 @@ public void CanRecognizeGenericSystemTypes()
         Verify(new Dictionary<string, List<ValueTuple<int, short>>>());
     }
 
+    [Fact]
+    public void ThrowsForNullType()
+    {
+        List<int> input = new List<int>();
+
+        SerializationRecord record = NrbfDecoder.Decode(Serialize(input));
+
+        Assert.Throws<ArgumentNullException>(() => record.TypeNameMatches(type: null));
+    }
+
     [Fact]
     public void TakesGenericTypeDefinitionIntoAccount()
     {