fix(ApiExplorer): SubstitutedType have invalid property setter

The PropertySetter of the SubstitutedType have 0 parameters: they had the same signature of the Getter.

resolves: #1104

Author: AndreaCuneo

src/Common/src/Common.OData.ApiExplorer/OData/DefaultModelTypeBuilder.cs

@@ -417,7 +417,11 @@ private static PropertyBuilder AddProperty(
         var field = addTo.DefineField( "_" + name, shouldBeAdded, FieldAttributes.Private );
         var propertyBuilder = addTo.DefineProperty( name, PropertyAttributes.HasDefault, shouldBeAdded, null );
         var getter = addTo.DefineMethod( "get_" + name, propertyMethodAttributes, shouldBeAdded, Type.EmptyTypes );
-        var setter = addTo.DefineMethod( "set_" + name, propertyMethodAttributes, shouldBeAdded, Type.EmptyTypes );
+
+        /* returnType is 'null' instead of type(void) as per docs
+        * see: https://learn.microsoft.com/en-us/dotnet/api/system.reflection.emit.propertybuilder?view=net-9.0
+        */
+        var setter = addTo.DefineMethod( "set_" + name, propertyMethodAttributes, null, [shouldBeAdded] );
         var il = getter.GetILGenerator();
 
         il.Emit( OpCodes.Ldarg_0 );

src/Common/test/Common.OData.ApiExplorer.Tests/OData/DefaultModelTypeBuilderTest.cs

@@ -409,6 +409,41 @@ public void substitute_should_resolve_types_that_reference_a_model_that_match_th
         substitutionType.Should().NotBeOfType<TypeBuilder>();
     }
 
+    [Fact]
+    public void substituted_type_should_have_valid_runtime_properties__issue1104()
+    {
+        // arrange
+        var modelBuilder = new ODataConventionModelBuilder();
+
+        var address = modelBuilder.EntitySet<Address>( nameof( Address ) ).EntityType;
+        address.Ignore( x => x.City ); // force substitution
+        var addressType = typeof( Address );
+
+        var context = NewContext( modelBuilder.GetEdmModel() );
+
+        // act
+        var substitutedType = addressType.SubstituteIfNecessary( context );
+
+        // assert
+        substitutedType.Should().NotBe( addressType );
+#if NET452
+        substitutedType.GetRuntimeProperties().Should().HaveCount( 5 );
+        foreach ( var substitutedProperty in substitutedType.GetRuntimeProperties() )
+        {
+            substitutedProperty.Should().NotBeNull();
+            substitutedProperty.GetSetMethod( true ).Should().NotBeNull()
+                .And.Match( p => p.ReturnType == typeof( void ) )
+                .And.Match( p => p.GetParameters().Length == 1 );
+        }
+#else
+        substitutedType.GetRuntimeProperties().Should().HaveCount( 5 )
+            .And.AllSatisfy( prop => prop.GetSetMethod( true ).Should()
+                .NotBeNull()
+                .And.ReturnVoid()
+                .And.Match( setter => setter.GetParameters().Length == 1 ) );
+#endif
+    }
+
     public static IEnumerable<object[]> SubstitutionNotRequiredData
     {
         get