Added access modifiers

Author: Ja bist du narrisch

src/Migrator.Tests/MigrationTestCase.cs

@@ -22,7 +22,7 @@ namespace Migrator.Tests
     /// </summary>
     public abstract class MigrationsTestCase
     {
-        Migrator _migrator;
+        private Migrator _migrator;
 
         protected abstract TransformationProvider TransformationProvider { get; }
         protected abstract string ConnectionString { get; }

src/Migrator.Tests/MigrationTypeComparerTest.cs

@@ -21,7 +21,7 @@ namespace Migrator.Tests
     [TestFixture]
     public class MigrationTypeComparerTest
     {
-        readonly Type[] _types = {
+        private readonly Type[] _types = {
                                      typeof (Migration1),
                                      typeof (Migration2),
                                      typeof (Migration3)

src/Migrator.Tests/MigratorTest.cs

@@ -50,7 +50,7 @@ private void SetUpCurrentVersion(long version, bool assertRollbackIsCalled)
         SetUpCurrentVersion(version, assertRollbackIsCalled, true);
     }
 
-    void SetUpCurrentVersion(long version, bool assertRollbackIsCalled, bool includeBad)
+    private void SetUpCurrentVersion(long version, bool assertRollbackIsCalled, bool includeBad)
     {
         var providerMock = new DynamicMock(typeof(ITransformationProvider));
 

src/Migrator.Tests/Providers/Base/TransformationProviderBase.cs

@@ -454,7 +454,7 @@ public void RemoveIndex()
     }
 
 
-    int[] GetVals(IDataReader reader)
+    private int[] GetVals(IDataReader reader)
     {
         var vals = new int[2];
         Assert.That(reader.Read(), Is.True);
@@ -465,7 +465,7 @@ int[] GetVals(IDataReader reader)
         return vals;
     }
 
-    string[] GetStringVals(IDataReader reader)
+    private string[] GetStringVals(IDataReader reader)
     {
         var vals = new string[2];
         Assert.That(reader.Read(), Is.True);

src/Migrator.Tests/Providers/SQLServerCe/SqlServerCeTransformationProviderTest.cs

@@ -43,7 +43,7 @@ public void SetUp()
 
         #endregion
 
-        void EnsureDatabase(string constr)
+        private void EnsureDatabase(string constr)
         {
             var connection = new SqlCeConnection(constr);
             if (!File.Exists(connection.Database))

src/Migrator/Compile/ScriptEngine.cs

@@ -9,8 +9,8 @@ namespace Migrator.Compile
 {
     public class ScriptEngine
     {
-        readonly string _codeType = "csharp";
-        readonly CodeDomProvider _provider;
+        private readonly string _codeType = "csharp";
+        private readonly CodeDomProvider _provider;
         public readonly string[] extraReferencedAssemblies;
 
         public ScriptEngine() : this(null, null)
@@ -41,7 +41,7 @@ public Assembly Compile(string directory)
             return Compile(files);
         }
 
-        string[] GetFilesRecursive(string directory)
+        private string[] GetFilesRecursive(string directory)
         {
             var files = GetFilesRecursive(new DirectoryInfo(directory));
             var fileNames = new string[files.Length];
@@ -52,7 +52,7 @@ string[] GetFilesRecursive(string directory)
             return fileNames;
         }
 
-        FileInfo[] GetFilesRecursive(DirectoryInfo d)
+        private FileInfo[] GetFilesRecursive(DirectoryInfo d)
         {
             var files = new List<FileInfo>();
             files.AddRange(d.GetFiles(String.Format("*.{0}", _provider.FileExtension)));
@@ -83,7 +83,7 @@ public Assembly Compile(params string[] files)
             return compileResult.CompiledAssembly;
         }
 
-        CompilerParameters SetupCompilerParams()
+        private CompilerParameters SetupCompilerParams()
         {
             var migrationFrameworkPath = FrameworkAssemblyPath();
             var parms = new CompilerParameters();
@@ -107,7 +107,7 @@ CompilerParameters SetupCompilerParams()
             return parms;
         }
 
-        static string FrameworkAssemblyPath()
+        private static string FrameworkAssemblyPath()
         {
             var path = typeof(MigrationAttribute).Module.FullyQualifiedName;
             Console.Out.WriteLine("Framework DLL: " + path);

src/Migrator/Framework/JoiningTableTransformationProviderExtensions.cs

@@ -24,7 +24,7 @@ public static ITransformationProvider AddManyToManyJoiningTable(this ITransforma
             return AddManyToManyJoiningTable(database, schema, lhsTableName, lhsKey, rhsTableName, rhsKey, joiningTable);
         }
 
-        static string GetNameOfJoiningTable(string lhsTableName, string rhsTableName)
+        private static string GetNameOfJoiningTable(string lhsTableName, string rhsTableName)
         {
             return (Inflector.Singularize(lhsTableName) ?? lhsTableName) + (Inflector.Pluralize(rhsTableName) ?? rhsTableName);
         }
@@ -58,7 +58,7 @@ public static ITransformationProvider AddManyToManyJoiningTable(this ITransforma
             return database;
         }
 
-        static string ShortenKeyNameToBeSuitableForOracle(string pkName)
+        private static string ShortenKeyNameToBeSuitableForOracle(string pkName)
         {
             return TransformationProviderUtility.AdjustNameToSize(pkName, TransformationProviderUtility.MaxLengthForForeignKeyInOracle, false);
         }

src/Migrator/Framework/Loggers/Logger.cs

@@ -21,8 +21,8 @@ namespace Migrator.Framework.Loggers
     /// </summary>
     public class Logger : IAttachableLogger
     {
-        readonly bool _trace;
-        readonly List<ILogWriter> _writers = new List<ILogWriter>();
+        private readonly bool _trace;
+        private readonly List<ILogWriter> _writers = new List<ILogWriter>();
 
         public Logger(bool trace)
         {
@@ -120,7 +120,7 @@ public void Started(long currentVersion, long finalVersion)
             WriteLine("Current version : {0}.  Target version : {1}", currentVersion, finalVersion);
         }
 
-        void LogExceptionDetails(Exception ex)
+        private void LogExceptionDetails(Exception ex)
         {
             WriteLine("{0}", ex.Message);
             WriteLine("{0}", ex.StackTrace);
@@ -138,15 +138,15 @@ public void Finished(long originalVersion, long currentVersion)
             WriteLine("Migrated to version {0}", currentVersion);
         }
 
-        void Write(string message, params object[] args)
+        private void Write(string message, params object[] args)
         {
             foreach (var writer in _writers)
             {
                 writer.Write(message, args);
             }
         }
 
-        void WriteLine(string message, params object[] args)
+        private void WriteLine(string message, params object[] args)
         {
             foreach (var writer in _writers)
             {
@@ -159,7 +159,7 @@ public static ILogger ConsoleLogger()
             return new Logger(false, new ConsoleWriter());
         }
 
-        string LatestVersion(List<long> versions)
+        private string LatestVersion(List<long> versions)
         {
             if (versions.Count > 0)
             {

src/Migrator/Framework/Loggers/SqlScriptFileLogger.cs

@@ -6,8 +6,8 @@ namespace Migrator.Framework.Loggers
 {
     public class SqlScriptFileLogger : ILogger, IDisposable
     {
-        readonly ILogger _innerLogger;
-        TextWriter _streamWriter;
+        private readonly ILogger _innerLogger;
+        private TextWriter _streamWriter;
 
         public SqlScriptFileLogger(ILogger logger, TextWriter streamWriter)
         {

src/Migrator/Framework/SchemaBuilder/AddColumnExpression.cs

@@ -15,8 +15,8 @@ namespace Migrator.Framework.SchemaBuilder
 {
     public class AddColumnExpression : ISchemaBuilderExpression
     {
-        readonly IFluentColumn _column;
-        readonly string _toTable;
+        private readonly IFluentColumn _column;
+        private readonly string _toTable;
 
         public AddColumnExpression(string toTable, IFluentColumn column)
         {