Fixed ambiguous code when a nested type and a property-like method with overloads have the same name

Signed-off-by: Dimitar Dobrev <dpldobrev@protonmail.com>

src/Generator/Passes/GetterSetterToPropertyPass.cs

@@ -38,7 +38,10 @@ namespace CppSharp.Passes
                 foreach (Method getter in
                     from getter in getters
                     where getter.IsGenerated &&
-                          getter.SynthKind != FunctionSynthKind.ComplementOperator
+                          getter.SynthKind != FunctionSynthKind.ComplementOperator &&
+                          ((Class) getter.Namespace).Methods.All(
+                              m => m == getter || !m.IsGenerated || m.Name != getter.Name ||
+                                   m.Parameters.Count(p => p.Kind == ParameterKind.Regular) == 0)
                     select getter)
                 {
                     // Make it a read-only property

tests/CSharp/CSharp.Tests.cs

@@ -1265,6 +1265,6 @@ public unsafe class CSharpTests : GeneratorTestFixture
 
     private class OverrideVirtualTemplate : VirtualTemplate<int>
     {
-        public override int Function => 10;
+        public override int Function() => 10;
     }
 }

tests/Common/Common.Tests.cs

@@ -480,7 +480,8 @@ public class CommonTests : GeneratorTestFixture
     public void TestProperties()
     {
         // Test field property
-        var prop = new TestProperties();
+        using (var prop = new TestProperties())
+        {
             Assert.That(prop.Field, Is.EqualTo(0));
             prop.Field = 10;
             Assert.That(prop.Field, Is.EqualTo(10));
@@ -499,6 +500,10 @@ public class CommonTests : GeneratorTestFixture
 
             prop.VirtualSetterReturnsBoolean = 45;
             Assert.That(prop.VirtualSetterReturnsBoolean, Is.EqualTo(45));
+
+            Assert.That(prop.nestedEnum(), Is.EqualTo(5));
+            Assert.That(prop.nestedEnum(55), Is.EqualTo(55));
+        }
     }
 
     [Test]

tests/Common/Common.cpp

@@ -586,6 +586,16 @@ bool TestProperties::setVirtualSetterReturnsBoolean(int value)
     return changed;
 }
 
+int TestProperties::nestedEnum()
+{
+    return 5;
+}
+
+int TestProperties::nestedEnum(int i)
+{
+    return i;
+}
+
 HasOverridenSetter::HasOverridenSetter()
 {
 }

tests/Common/Common.h

@@ -578,6 +578,12 @@ DLL_API int Function()
 struct DLL_API TestProperties
 {
 public:
+    enum class NestedEnum
+    {
+        Value1,
+        Value2
+    };
+
     TestProperties();
     int Field;
 
@@ -600,6 +606,9 @@ public:
 
     virtual int virtualSetterReturnsBoolean();
     virtual bool setVirtualSetterReturnsBoolean(int value);
+
+    int nestedEnum();
+    int nestedEnum(int i);
 private:
     int FieldValue;
     double _refToPrimitiveInSetter;