Fixed incorrect placment of XML comments on properties in classes

Author: bastianeicher

src/CodeGeneration/CSharpClass.cs

@@ -72,8 +72,8 @@ protected override IEnumerable<MemberDeclarationSyntax> GetMemberDeclarations()
             if (BaseClass != null && BaseClass.Parameters.Count != 0)
                 yield return BaseClass.ToDeclarationSyntax(Identifier.Name);
 
-            foreach (var member in base.GetMemberDeclarations())
-                yield return member.WithModifiers(TokenList(Token(SyntaxKind.PublicKeyword)));
+            foreach (var member in Properties.Select(property => property.ToSyntax(makePublic: true)))
+                yield return member;
         }
     }
 }

src/CodeGeneration/CSharpProperty.cs

@@ -80,12 +80,18 @@ internal IEnumerable<string> GetNamespaces()
         /// <summary>
         /// Returns a Roslyn syntax for the property.
         /// </summary>
-        internal PropertyDeclarationSyntax ToSyntax()
+        /// <param name="makePublic">Controls whether to make the property public or not.</param>
+        internal PropertyDeclarationSyntax ToSyntax(bool makePublic = false)
         {
             var propertyDeclaration =
-                PropertyDeclaration(Type.ToSyntax(), Identifier(Name))
-                   .WithAttributeLists(List(Attributes.Select(x => x.ToSyntax())))
-                   .WithDocumentation(Summary);
+                PropertyDeclaration(Type.ToSyntax(), Identifier(Name));
+
+            if (makePublic)
+                propertyDeclaration = propertyDeclaration.AddModifiers(Token(SyntaxKind.PublicKeyword));
+
+            propertyDeclaration = propertyDeclaration
+                                 .WithAttributeLists(List(Attributes.Select(x => x.ToSyntax())))
+                                 .WithDocumentation(Summary);
 
             return (GetterExpression == null)
                 ? propertyDeclaration.WithAccessorList(AccessorList(List(GetAccessors())))

src/UnitTests/CSharpClassFacts.cs

@@ -37,7 +37,6 @@ public void GeneratesCorrectCode()
                     new CSharpProperty(myInterface, "MyProperty")
                     {
                         Summary = "My property",
-                        Attributes = {dummyAttribute},
                         GetterExpression = new CSharpConstructor(otherClass)
                         {
                             Parameters =
@@ -69,7 +68,6 @@ public MyClass(IEndpoint referrer): base(referrer, relativeUri: ""./sample"")
         /// <summary>
         /// My property
         /// </summary>
-        [Dummy(""myValue"", Extra = ""extra"")]
         public MyInterface<MyModel> MyProperty => new OtherClass<MyModel>(this, arg2: ""value"");
     }
 }");