Revert ""I have changed access modifier from Internal to Public . These classes are being refered outside. (#742)" (#747)

This reverts commit a54eff11b6.

src/Microsoft.SqlTools.ServiceLayer/BatchParser/BatchParserAction.cs

@@ -5,7 +5,7 @@
 
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser
 {
-    public enum BatchParserAction
+    internal enum BatchParserAction
     {
         Continue = 0,
         Abort = 1

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/Batch.cs

@@ -17,7 +17,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Single batch of SQL command
     /// </summary>
-    public class Batch
+    internal class Batch
     {
         #region Private methods
 

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/BatchErrorEventArgs.cs

@@ -11,7 +11,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Error totalAffectedRows for a Batch
     /// </summary>
-    public class BatchErrorEventArgs : EventArgs
+    internal class BatchErrorEventArgs : EventArgs
     {
         #region Private Fields
         private string message = string.Empty;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/BatchMessageEventArgs.cs

@@ -11,25 +11,25 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Event args for notification about non-error message
     /// </summary>
-    public class BatchMessageEventArgs : EventArgs
+    internal class BatchMessageEventArgs : EventArgs
     {
         private readonly string message = string.Empty;
         private readonly string detailedMessage = string.Empty;
         private readonly SqlError error;
-
+                
-        public BatchMessageEventArgs()
+        private BatchMessageEventArgs()
         {
         }
 
-        public BatchMessageEventArgs(string msg)
+        internal BatchMessageEventArgs(string msg)
             : this(string.Empty, msg)
         {            
         }
 
-        public BatchMessageEventArgs(string detailedMsg, string msg) : this(detailedMsg, msg, null)
+        internal BatchMessageEventArgs(string detailedMsg, string msg) : this(detailedMsg, msg, null)
         {
         }
-        public BatchMessageEventArgs(string detailedMsg, string msg, SqlError error)
+        internal BatchMessageEventArgs(string detailedMsg, string msg, SqlError error)
         {
             message = msg;
             detailedMessage = detailedMsg;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/BatchParser.cs

@@ -10,7 +10,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Class that parses queries into batches
     /// </summary>
-    public class BatchParser : 
+    internal class BatchParser : 
         ICommandHandler, 
         IVariableResolver
     {

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/BatchParserExecutionErrorEventArgs.cs

@@ -8,7 +8,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Class associated with batch parser execution errors
     /// </summary>
-    public class BatchParserExecutionErrorEventArgs : BatchErrorEventArgs
+    internal class BatchParserExecutionErrorEventArgs : BatchErrorEventArgs
     {
         private readonly ScriptMessageType messageType;
 

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/BatchParserExecutionFinishedEventArgs.cs

@@ -10,13 +10,13 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Class associated with batch parser execution finished event
     /// </summary>
-    public class BatchParserExecutionFinishedEventArgs : EventArgs
+    internal class BatchParserExecutionFinishedEventArgs : EventArgs
     {
         
         private readonly Batch batch = null;
         private readonly ScriptExecutionResult result;
 
-        public BatchParserExecutionFinishedEventArgs() 
+        private BatchParserExecutionFinishedEventArgs() 
         {
         }
 

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/BatchParserExecutionStartEventArgs.cs

@@ -11,7 +11,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Class associated with batch parser execution start event
     /// </summary>
-    public class BatchParserExecutionStartEventArgs : EventArgs
+    internal class BatchParserExecutionStartEventArgs : EventArgs
     {
         
         private readonly Batch batch = null;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/BatchParserSqlCmd.cs

@@ -14,7 +14,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Class for handling SQL CMD by Batch Parser
     /// </summary>
-    public class BatchParserSqlCmd : BatchParser
+    internal class BatchParserSqlCmd : BatchParser
     {
         /// <summary>
         /// The internal variables that can be used in SqlCommand substitution.

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/BatchResultSetEventArgs.cs

@@ -11,17 +11,17 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
     /// <summary>
     /// Class associated with setting batch results
     /// </summary>
-    public class BatchResultSetEventArgs : EventArgs
+    internal class BatchResultSetEventArgs : EventArgs
     {
         
         private readonly IDataReader dataReader = null;
         private readonly ShowPlanType expectedShowPlan = ShowPlanType.None;
-
+        
         /// <summary>
         /// Default constructor
         /// </summary>
         /// <param name="dataReader"></param>
-        public BatchResultSetEventArgs(IDataReader dataReader, ShowPlanType expectedShowPlan)
+        internal BatchResultSetEventArgs(IDataReader dataReader, ShowPlanType expectedShowPlan)
         {
             this.dataReader = dataReader;
             this.expectedShowPlan = expectedShowPlan;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/ScriptExecutionResult.cs

@@ -8,7 +8,7 @@ using System;
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
 {
     [Flags]
-    public enum ScriptExecutionResult
+    internal enum ScriptExecutionResult
     {
         Success = 0x1,
         Failure = 0x2,  

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/ScriptMessageType.cs

@@ -5,7 +5,7 @@
 
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
 {
-    public enum ScriptMessageType
+    internal enum ScriptMessageType
     {
         FatalError,
         Error,

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/ShowPlanType.cs

@@ -6,7 +6,7 @@
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
 {
     [System.Flags]
-    public enum ShowPlanType
+    internal enum ShowPlanType
     {
         None = 0x0,
         ActualExecutionShowPlan = 0x1,

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ExecutionEngineCode/TextSpan.cs

@@ -5,7 +5,7 @@
 
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser.ExecutionEngineCode
 {
-    public struct TextSpan
+    internal struct TextSpan
     {        
         public int iEndIndex;
         public int iEndLine;        

src/Microsoft.SqlTools.ServiceLayer/BatchParser/ICommandHandler.cs

@@ -7,7 +7,7 @@ using System.IO;
 
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser
 {
-    public interface ICommandHandler
+    internal interface ICommandHandler
     {
         BatchParserAction Go(TextBlock batch, int repeatCount);
         BatchParserAction OnError(Token token, OnErrorAction action);

src/Microsoft.SqlTools.ServiceLayer/BatchParser/IVariableResolver.cs

@@ -5,7 +5,7 @@
 
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser
 {
-    public interface IVariableResolver
+    internal interface IVariableResolver
     {
         string GetVariable(PositionStruct pos, string name);
         void SetVariable(PositionStruct pos, string name, string value);

src/Microsoft.SqlTools.ServiceLayer/BatchParser/LineInfo.cs

@@ -11,7 +11,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser
     /// This class gives information about lines being parsed by
     /// the Batch Parser
     /// </summary>
-    public class LineInfo
+    class LineInfo
     {
         private IEnumerable<Token> tokens;
         private IEnumerable<VariableReference> variableRefs;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/OnErrorAction.cs

@@ -5,7 +5,7 @@
 
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser
 {
-    public enum OnErrorAction
+    internal enum OnErrorAction
     {
         Ignore = 0,
         Exit = 1,

src/Microsoft.SqlTools.ServiceLayer/BatchParser/Parser.cs

@@ -16,7 +16,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser
     /// <summary>
     /// The Parser class on which the Batch Parser is based on
     /// </summary>
-    public sealed class Parser : IDisposable
+    internal sealed class Parser : IDisposable
     {
         private readonly ICommandHandler commandHandler;
         private Lexer lexer;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/PositionStruct.cs

@@ -8,7 +8,7 @@ using System;
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser
 {
     [Serializable]
-    public struct PositionStruct
+    internal struct PositionStruct
     {
         private readonly int line;
         private readonly int column;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/TextBlock.cs

@@ -8,7 +8,7 @@ using System.Collections.Generic;
 
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser
 {
-    public class TextBlock
+    internal class TextBlock
     {
         private readonly Parser parser;
         private readonly IEnumerable<Token> tokens;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/Token.cs

@@ -5,12 +5,12 @@
 
 namespace Microsoft.SqlTools.ServiceLayer.BatchParser
 {
-    public sealed class Token
+    internal sealed class Token
     {
         /// <summary>
         /// Token class used by the lexer in Batch Parser
         /// </summary>
-        public Token(LexerTokenType tokenType, PositionStruct begin, PositionStruct end, string text, string filename)
+        internal Token(LexerTokenType tokenType, PositionStruct begin, PositionStruct end, string text, string filename)
         {
             TokenType = tokenType;
             Begin = begin;

src/Microsoft.SqlTools.ServiceLayer/BatchParser/VariableReference.cs

@@ -8,7 +8,7 @@ namespace Microsoft.SqlTools.ServiceLayer.BatchParser
     /// <summary>
     /// Class for reference of variables used by the lexer
     /// </summary>
-    public sealed class VariableReference
+    internal sealed class VariableReference
     {
         /// <summary>
         /// Constructor method for VariableReference class