Use expression bodied properties (#653)

+semver:patch

Author: hazzik

src/FluentNHibernate.Specs.ExternalFixtures/Entity.cs

@@ -15,10 +15,7 @@ public class Entity
 
     public enum TestEnum {}
 
-    public EntityChild ReadOnlyChild
-    {
-        get { return readOnlyChild; }
-    }
+    public EntityChild ReadOnlyChild => readOnlyChild;
 }
 
 public class EntityChild

src/FluentNHibernate.Specs/Automapping/Fixtures/ReadOnlyEnumerableEntity.cs

@@ -9,8 +9,5 @@ class ReadOnlyEnumerableEntity
 
     public int Id { get; set; }
     public IEnumerable<EntityChild> AutoPropertyCollection { get; private set; }
-    public IEnumerable<EntityChild> BackingFieldCollection
-    {
-        get { return backingFieldCollection; }
-    }
+    public IEnumerable<EntityChild> BackingFieldCollection => backingFieldCollection;
 }

src/FluentNHibernate.Specs/Automapping/Fixtures/UserTypes.cs

@@ -102,15 +102,9 @@ public SqlType[] SqlTypes
         get { return new[] { new SqlType(DbType.String) }; }
     }
 
-    public Type ReturnedType
-    {
-        get { return typeof(CustomUserType); }
-    }
+    public Type ReturnedType => typeof(CustomUserType);
 
-    public bool IsMutable
-    {
-        get { return true; }
-    }
+    public bool IsMutable => true;
 }
 
 public struct UserValueType : IUserType
@@ -169,6 +163,6 @@ public void NullSafeSet(DbCommand cmd, object value, int index, ISessionImplemen
     }
 
     public SqlType[] SqlTypes { get; private set; }
-    public Type ReturnedType { get { return typeof(UserValueType); } }
+    public Type ReturnedType => typeof(UserValueType);
     public bool IsMutable { get; private set; }
 }

src/FluentNHibernate.Specs/FluentInterface/ClassMapSpecs/ClassMapSpecs.Id.cs

@@ -30,7 +30,7 @@ public class when_class_map_is_told_to_map_an_id_without_a_property_or_column :
 
     static ClassMapping mapping;
 
-    static IdMapping Id { get { return mapping.Id as IdMapping; }}
+    static IdMapping Id => mapping.Id as IdMapping;
 }
 
 public class when_class_map_has_a_composite_id_with_a_key_reference_with_multiple_columns : ProviderSpec

src/FluentNHibernate.Testing/AutoMapping/TestFixtures.cs

@@ -43,13 +43,7 @@ public class ExampleCustomColumn
     {
         public int Id { get; set; }
         public int ExampleCustomColumnId { get; set; }
-        public int CustomColumn
-        {
-            get
-            {
-                return 12;
-            }
-        }
+        public int CustomColumn => 12;
     }
 
     public class ExampleInheritedClass : ExampleClass
@@ -306,26 +300,17 @@ public SqlType[] SqlTypes
             get { return new[] {new SqlType(DbType.String)}; }
         }
 
-        public Type ReturnedType
-        {
-            get { return typeof(Custom); }
-        }
+        public Type ReturnedType => typeof(Custom);
 
-        public bool IsMutable
-        {
-            get { return true; }
-        }
+        public bool IsMutable => true;
     }
 }
 
 namespace FluentNHibernate.Automapping.TestFixtures.CustomCompositeTypes
 {
     public class DoubleStringType : ICompositeUserType
     {
-        public System.Type ReturnedClass
-        {
-            get { return typeof(string[]); }
-        }
+        public System.Type ReturnedClass => typeof(string[]);
 
         public new bool Equals(object x, object y)
         {
@@ -357,10 +342,7 @@ public Object DeepCopy(Object x)
             return result;
         }
 
-        public bool IsMutable
-        {
-            get { return true; }
-        }
+        public bool IsMutable => true;
 
         public object NullSafeGet(DbDataReader dr, string[] names, ISessionImplementor session, object owner)
         {
@@ -431,13 +413,7 @@ public class SuperType
     public class ExampleCustomColumn : SuperType
     {
         public int ExampleCustomColumnId { get; set; }
-        public int CustomColumn
-        {
-            get
-            {
-                return 12;
-            }
-        }
+        public int CustomColumn => 12;
     }
 
     public class ExampleInheritedClass : ExampleClass

src/FluentNHibernate.Testing/Cfg/Db/CacheSettingsBuilderTester.cs

@@ -111,9 +111,6 @@ private void HasProperty(string key, string value)
 
     private class CacheSettingsBuilderDouble : CacheSettingsBuilder
     {
-        public IDictionary<string, string> Properties
-        {
-            get { return Create(); }
-        }
+        public IDictionary<string, string> Properties => Create();
     }
 }

src/FluentNHibernate.Testing/Cfg/Db/ConnectionStringBuilderTester.cs

@@ -41,9 +41,6 @@ public void ConnectionStringSetFromConnectionStrings()
 
     private class ConnectionStringBuilderDouble : ConnectionStringBuilder
     {
-        public string ConnectionString
-        {
-            get { return Create(); }
-        }
+        public string ConnectionString => Create();
     }
 }

src/FluentNHibernate.Testing/ConventionsTests/RunnableConventionsTests.cs

@@ -692,18 +692,11 @@ public void NullSafeSet(DbCommand cmd, object value, int index, ISessionImplemen
             throw new NotImplementedException();
         }
 
-        public SqlType[] SqlTypes
-        {
-            get { return null; }
-        }
-        public Type ReturnedType
-        {
-            get { return typeof(OtherObject); }
-        }
-        public bool IsMutable
-        {
-            get { return false; }
-        }
+        public SqlType[] SqlTypes => null;
+
+        public Type ReturnedType => typeof(OtherObject);
+
+        public bool IsMutable => false;
     }
 
     #endregion

src/FluentNHibernate.Testing/DomainModel/Mapping/ComponentPropertyMapTester.cs

@@ -257,20 +257,11 @@ public void NullSafeSet(DbCommand cmd, object value, int index, ISessionImplemen
             throw new NotImplementedException();
         }
 
-        public SqlType[] SqlTypes
-        {
-            get { return null; }
-        }
+        public SqlType[] SqlTypes => null;
 
-        public Type ReturnedType
-        {
-            get { return null; }
-        }
+        public Type ReturnedType => null;
 
-        public bool IsMutable
-        {
-            get { return false; }
-        }
+        public bool IsMutable => false;
     }
     #endregion
 }

src/FluentNHibernate.Testing/DomainModel/Mapping/OneToManyTester.cs

@@ -43,8 +43,7 @@ public class OneToManyTarget
     public virtual IList<ChildObject> GetOtherChildren() { return otherChildren; }
 
     private IList<ChildObject> listToArrayChild = new List<ChildObject>();
-    public virtual ChildObject[] ListToArrayChild { get { return listToArrayChild.ToArray(); } }
-
+    public virtual ChildObject[] ListToArrayChild => listToArrayChild.ToArray();
 }
 
 public class ValueObject