Fix analyzer issue with properties with dot or space in name (#79)

Author: olsh

src/ReSharper.Structured.Logging/Serilog/Parsing/MessageTemplateParser.cs

@@ -217,7 +217,7 @@ static bool IsValidInPropertyTag(char c)
                 c == ':';
         }
 
-        static bool IsValidInPropertyName(char c) => char.IsLetterOrDigit(c) || c == '_';
+        static bool IsValidInPropertyName(char c) => char.IsLetterOrDigit(c) || c == '_' || c == '.' || c == ' ';
 
         static bool TryGetDestructuringHint(char c, out Destructuring destructuring)
         {

test/data/Analyzers/PropertiesNamingAnalyzer/SerilogInvalidNamedPropertyWithDot.cs

@@ -0,0 +1,12 @@
+using Serilog;
+
+namespace ConsoleApp
+{
+    public static class Program
+    {
+        public static void Main()
+        {
+            Log.Logger.Information("{My.Property}", 1);
+        }
+    }
+}

test/data/Analyzers/PropertiesNamingAnalyzer/SerilogInvalidNamedPropertyWithDot.cs.gold

@@ -0,0 +1,15 @@
+using Serilog;
+
+namespace ConsoleApp
+{
+    public static class Program
+    {
+        public static void Main()
+        {
+            Log.Logger.Information("|{My.Property}|(0)", 1);
+        }
+    }
+}
+
+---------------------------------------------------------
+(0): ReSharper Warning: Property name 'My.Property' does not match naming rules. Suggested name is 'MyProperty'.

test/data/Analyzers/PropertiesNamingAnalyzer/SerilogInvalidNamedPropertyWithSpace.cs

@@ -0,0 +1,12 @@
+using Serilog;
+
+namespace ConsoleApp
+{
+    public static class Program
+    {
+        public static void Main()
+        {
+            Log.Logger.Information("{My Property}", 1);
+        }
+    }
+}

test/data/Analyzers/PropertiesNamingAnalyzer/SerilogInvalidNamedPropertyWithSpace.cs.gold

@@ -0,0 +1,15 @@
+using Serilog;
+
+namespace ConsoleApp
+{
+    public static class Program
+    {
+        public static void Main()
+        {
+            Log.Logger.Information("|{My Property}|(0)", 1);
+        }
+    }
+}
+
+---------------------------------------------------------
+(0): ReSharper Warning: Property name 'My Property' does not match naming rules. Suggested name is 'MyProperty'.

test/src/Analyzer/PropertiesNamingAnalyzerTests.cs

@@ -13,5 +13,9 @@ public class PropertiesNamingAnalyzerTests : MessageTemplateAnalyzerTestBase
         [Test] public void TestSerilogValidDestructuredNamedProperty() => DoNamedTest2();
 
         [Test] public void TestSerilogContextInvalidNamedProperty() => DoNamedTest2();
+
+        [Test] public void TestSerilogInvalidNamedPropertyWithDot() => DoNamedTest2();
+
+        [Test] public void TestSerilogInvalidNamedPropertyWithSpace() => DoNamedTest2();
     }
 }