Convert to auto-implemented getter-only properties (pt. 19)

AutoFixture · Mar 16, 2016 · 67bc524 · 67bc524
1 parent 5de1a83
commit 67bc524
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 38 deletions.
diff --git a/Src/TestTypeFoundation/AbstractTypeWithConstructorWithMultipleParameters.cs b/Src/TestTypeFoundation/AbstractTypeWithConstructorWithMultipleParameters.cs
@@ -7,25 +7,16 @@ namespace Ploeh.TestTypeFoundation
 {
     public abstract class AbstractTypeWithConstructorWithMultipleParameters<T1, T2>
     {
-        private readonly T1 property1;
-        private readonly T2 property2;
-
         protected AbstractTypeWithConstructorWithMultipleParameters(
             T1 parameter1,
             T2 parameter2)
         {
-            this.property1 = parameter1;
-            this.property2 = parameter2;
+            this.Property1 = parameter1;
+            this.Property2 = parameter2;
         }
 
-        public T1 Property1
-        {
-            get { return property1; }
-        }
+        public T1 Property1 { get; }
 
-        public T2 Property2
-        {
-            get { return property2; }
-        }
+        public T2 Property2 { get; }
     }
 }
diff --git a/Src/TestTypeFoundation/AbstractTypeWithNonDefaultConstructor.cs b/Src/TestTypeFoundation/AbstractTypeWithNonDefaultConstructor.cs
@@ -4,21 +4,16 @@ namespace Ploeh.TestTypeFoundation
 {
     public abstract class AbstractTypeWithNonDefaultConstructor<T>
     {
-        private readonly T property;
-
         protected AbstractTypeWithNonDefaultConstructor(T value)
         {
             if (value == null)
             {
                 throw new ArgumentNullException(nameof(value));
             }
 
-            this.property = value;
+            this.Property = value;
         }
 
-        public T Property
-        {
-            get { return this.property; }
-        }
+        public T Property { get; }
     }
 }
diff --git a/Src/TestTypeFoundation/CompositeType.cs b/Src/TestTypeFoundation/CompositeType.cs
@@ -6,8 +6,6 @@ namespace Ploeh.TestTypeFoundation
 {
     public class CompositeType : AbstractType
     {
-        private readonly IEnumerable<AbstractType> types;
-
         public CompositeType(IEnumerable<AbstractType> types)
             : this(types.ToArray())
         {
@@ -20,12 +18,9 @@ public CompositeType(params AbstractType[] types)
                 throw new ArgumentNullException(nameof(types));
             }
 
-            this.types = types;
+            this.Types = types;
         }
 
-        public IEnumerable<AbstractType> Types
-        {
-            get { return this.types; }
-        }
+        public IEnumerable<AbstractType> Types { get; }
     }
 }
diff --git a/Src/TestTypeFoundation/GenericType.cs b/Src/TestTypeFoundation/GenericType.cs
@@ -4,18 +4,16 @@ namespace Ploeh.TestTypeFoundation
 {
     public class GenericType<T> where T : class
     {
-        private readonly T t;
-
         public GenericType(T t)
         {
             if (t == null)
             {
                 throw new ArgumentNullException(nameof(t));
             }
 
-            this.t = t;
+            this.Value = t;
         }
 
-        T Value { get { return this.t; }}
+        private T Value { get; }
     }
 }
diff --git a/Src/TestTypeFoundation/GuardedConstructorHost.cs b/Src/TestTypeFoundation/GuardedConstructorHost.cs
@@ -4,21 +4,16 @@ namespace Ploeh.TestTypeFoundation
 {
     public class GuardedConstructorHost<T> where T : class
     {
-        private readonly T item;
-
         public GuardedConstructorHost(T item)
         {
             if (item == null)
             {
                 throw new ArgumentNullException(nameof(item));
             }
 
-            this.item = item;
+            this.Item = item;
         }
 
-        public T Item
-        {
-            get { return this.item; }
-        }
+        public T Item { get; }
     }
 }