Merge pull request #100 from nblumhardt/cs-accessors-2

Accessors such as `a.B` should be case-sensitive

Author: nblumhardt

src/Serilog.Expressions/Expressions/Compilation/Linq/LinqExpressionCompiler.cs

@@ -186,9 +186,8 @@ static ExpressionBody CompileLogical(Func<ExpressionBody, ExpressionBody, Expres
     protected override ExpressionBody Transform(AccessorExpression spx)
     {
         var receiver = Transform(spx.Receiver);
-        return LX.Call(TryGetStructurePropertyValueMethod, LX.Constant(StringComparison.OrdinalIgnoreCase), receiver, LX.Constant(spx.MemberName, typeof(string)));
+        return LX.Call(TryGetStructurePropertyValueMethod, LX.Constant(StringComparison.Ordinal), receiver, LX.Constant(spx.MemberName, typeof(string)));
     }
-
     protected override ExpressionBody Transform(ConstantExpression cx)
     {
         return LX.Constant(cx.Constant);

test/Serilog.Expressions.Tests/Cases/expression-evaluation-cases.asv

@@ -24,6 +24,12 @@ false                                ⇶ false
 {a: 1, ..{b: 2}, b: undefined()}     ⇶ {a: 1}
 {a: 1, ..{a: undefined()}}           ⇶ {a: 1}
 {..{a: 1}, ..{b: 2, c: 3}}           ⇶ {a: 1, b: 2, c: 3}
+{a: 1}['a']                          ⇶ 1
+{a: 1}['A']                          ⇶ undefined()
+{a: 1}.a                             ⇶ 1
+{a: 1}.A                             ⇶ undefined()
+ElementAt({a: 1}, 'A')               ⇶ undefined()
+ElementAt({a: 1}, 'A') ci            ⇶ 1
 
 // Strings
 ''                                   ⇶ ''