Ensure access modifier of partial properties is inspected and used correctly

Author: thomasclaudiushuber

src/MvvmGen.SourceGenerators.Tests/ViewModelGeneratorTests/PropertyAttributeTests.cs

@@ -60,6 +60,8 @@ public string FirstName
         [InlineData("public")]
         [InlineData("private")]
         [InlineData("protected")]
+        [InlineData("protected internal")]
+        [InlineData("internal")]
         [Theory]
         public void GeneratePartialProperty(string accessModifier)
         {

src/MvvmGen.SourceGenerators.Tests/ViewModelGeneratorTests/ViewModelGenerateInterfaceAttributeTests.cs

@@ -26,6 +26,8 @@ public partial class EmployeeViewModel
 
     [Property] public partial string MiddleName {{ get; set; }}
 
+    [Property] internal partial string InternalNotForInterface {{ get; set; }}
+
     public string LastName {{ get; set; }}
 
     public string FullName => FirstName + "" "" + LastName;
@@ -75,6 +77,21 @@ public partial string MiddleName
                 }}
             }}
         }}
+
+        private string _internalNotForInterface;
+
+        internal partial string InternalNotForInterface
+        {{
+            get => _internalNotForInterface;
+            set
+            {{
+                if (_internalNotForInterface != value)
+                {{
+                    _internalNotForInterface = value;
+                    OnPropertyChanged(""InternalNotForInterface"");
+                }}
+            }}
+        }}
     }}
 
     public interface IEmployeeViewModel : System.ComponentModel.INotifyPropertyChanged

src/MvvmGen.SourceGenerators/Inspectors/ViewModelGenerateInterfaceAttributeInspector.cs

@@ -47,9 +47,12 @@ internal static class ViewModelGenerateInterfaceAttributeInspector
             {
                 foreach (var propertyToGenerate in propertiesToGenerate)
                 {
-                    properties ??= new();
-                    properties.Add(new InterfaceProperty(propertyToGenerate.PropertyName,
-                        propertyToGenerate.PropertyType, propertyToGenerate.IsReadOnly));
+                    if (propertyToGenerate.AccessModifier == "public") // partial properties can have other access modifiers than public, for example internal
+                    {
+                        properties ??= new();
+                        properties.Add(new InterfaceProperty(propertyToGenerate.PropertyName,
+                            propertyToGenerate.PropertyType, propertyToGenerate.IsReadOnly));
+                    }
                 }
             }
 

src/MvvmGen.SourceGenerators/Inspectors/ViewModelMemberInspector.cs

@@ -216,9 +216,18 @@ private static void FindPropertiesToGenerate(ISymbol symbol, List<PropertyToGene
                 }
 
                 var generateBackingField = symbol.Kind == SymbolKind.Property;
+                var accessModifier = "public";
+                if(symbol.Kind== SymbolKind.Property)
+                {
+                    accessModifier = propertySymbol!.DeclaredAccessibility switch
+                    {
+                        Accessibility.ProtectedOrInternal => "protected internal",
+                        _ => propertySymbol!.DeclaredAccessibility.ToString().ToLower()
+                    };
+                }
 
                 propertiesToGenerate.Add(new PropertyToGenerate(propertyName, propertyType, fieldName,
-                    generateBackingField, isReadOnly: false, accessModifier: (symbol.Kind == SymbolKind.Field ? "public" : propertySymbol!.DeclaredAccessibility.ToString().ToLower()))
+                    generateBackingField, isReadOnly: false, accessModifier)
                 {
                     EventsToPublish = eventsToPublish,
                     MethodsToCall = methodsToCall