fix: ensure returning false when getter/setter is missing from proper…

…ty and trying to get/set value

src/ObjectTreeWalker/ObjectAccessor.cs

@@ -36,8 +36,15 @@ public ObjectAccessor(Type objectType)
 		_objectType = objectType;
 		foreach (var propertyInfo in objectType.GetProperties(BindingFlags.NonPublic | BindingFlags.Instance | BindingFlags.Public))
 		{
-			_getPropertyMethods.Add(propertyInfo.Name, CreateGetPropertyFunc(propertyInfo));
-			_setPropertyMethods.Add(propertyInfo.Name, CreateSetPropertyFunc(propertyInfo));
+			if (propertyInfo.GetMethod != null)
+			{
+				_getPropertyMethods.Add(propertyInfo.Name, CreateGetPropertyFunc(propertyInfo));
+			}
+
+			if (propertyInfo.SetMethod != null)
+			{
+				_setPropertyMethods.Add(propertyInfo.Name, CreateSetPropertyFunc(propertyInfo));
+			}
 		}
 
 		foreach (var fieldInfo in objectType.GetFields(BindingFlags.NonPublic | BindingFlags.Instance | BindingFlags.Public))

src/ObjectTreeWalker/ObjectTreeWalker.cs

@@ -1,4 +1,5 @@
 using System.Collections.Concurrent;
+using Microsoft.Extensions.ObjectPool;
 
 namespace ObjectTreeWalker
 {
@@ -8,6 +9,8 @@ namespace ObjectTreeWalker
 	public class ObjectMemberIterator
 	{
 		private static readonly ConcurrentDictionary<Type, ObjectAccessor> ObjectAccessorCache = new();
+		private static readonly ObjectPool<Queue<KeyValuePair<string, object>>> TraversalQueuePool =
+			new DefaultObjectPoolProvider().Create<Queue<KeyValuePair<string, object>>>();
 
 		/// <summary>
 		/// Initializes a new instance of the <see cref="ObjectMemberIterator"/> class

tests/ObjectTreeWalker.Tests/ObjectAccessorTests.cs

@@ -13,6 +13,28 @@ internal class PublicFooBar
 			public PrivateFooBar Bar { get; set; } = new();
 		}
 
+		internal class PublicFooBarNoSet
+		{
+			public int Foo { get; } = 123;
+
+			public int Test123 = 555;
+
+			public PrivateFooBar Bar { get; set; } = new();
+		}
+
+		internal class PublicFooBarNoGet
+		{
+			public int Foo
+			{
+				set => _foo = value;
+			}
+
+			public int Test123 = 555;
+			private int _foo;
+
+			public PrivateFooBar Bar { get; set; } = new();
+		}
+
 		internal struct PublicFooBarStruct
 		{
 			public int Foo { get; set; } = 123;
@@ -77,6 +99,13 @@ public void Can_get_public_value_type_property(Type type)
 			Assert.Equal(123, value);
 		}
 
+		[Fact]
+		public void Should_fail_get_public_value_no_get()
+		{
+			var accessor = new ObjectAccessor(typeof(PublicFooBarNoGet));
+			Assert.False(accessor.TryGetValue(new PublicFooBarNoGet(), "Foo", out var value));
+		}
+
 		[Theory]
 		[InlineData(typeof(PublicFooBar))]
 		[InlineData(typeof(PublicFooBarStruct))]
@@ -137,6 +166,15 @@ public void Can_set_public_value_type_property(Type type)
 			Assert.Equal(345, ((dynamic)obj).Foo);
 		}
 
+		[Fact]
+		public void Should_fail_setting_property_without_set()
+		{
+			var accessor = new ObjectAccessor(typeof(PublicFooBarNoSet));
+			var obj = new PublicFooBarNoSet();
+
+			Assert.False(accessor.TrySetValue(obj, "Foo", 345));
+		}
+
 		[Theory]
 		[InlineData(typeof(PublicFooBar))]
 		[InlineData(typeof(PublicFooBarStruct))]