Merge pull request #904 from airbreather/fix-656

Add a regression test for #656

+semver:fix

YamlDotNet.Test/Serialization/SerializationTests.cs

@@ -2483,6 +2483,16 @@ public void NamingConventionAppliedToEnumWhenDeserializing()
             Assert.Equal(expected, actual);
         }
 
+        [Fact]
+        [Trait("motive", "issue #656")]
+        public void NestedDictionaryTypes_ShouldRoundtrip()
+        {
+            var serializer = new SerializerBuilder().EnsureRoundtrip().Build();
+            var yaml = serializer.Serialize(new HasNestedDictionary { Lookups = { [1] = new HasNestedDictionary.Payload { I = 1 } } }, typeof(HasNestedDictionary));
+            var dct = new DeserializerBuilder().Build().Deserialize<HasNestedDictionary>(yaml);
+            Assert.Contains(new KeyValuePair<int, HasNestedDictionary.Payload>(1, new HasNestedDictionary.Payload { I = 1 }), dct.Lookups);
+        }
+
         public class TestState
         {
             public int OnSerializedCallCount { get; set; }
@@ -2574,5 +2584,15 @@ public void WriteYaml(IEmitter emitter, object value, Type type)
                 emitter.Emit(new Scalar(((NonSerializable)value).Text));
             }
         }
+
+        public sealed class HasNestedDictionary
+        {
+            public Dictionary<int, Payload> Lookups { get; set; } = new Dictionary<int, Payload>();
+
+            public struct Payload
+            {
+                public int I { get; set; }
+            }
+        }
     }
 }