Remove regions (#3366)

Co-authored-by: Amaury Levé <amauryleve@microsoft.com>
microsoft · Feb 15, 2022 · 128056e · 128056e
1 parent 07e31ab
commit 128056e
Show file tree

Hide file tree

Showing 223 changed files with 0 additions and 2,485 deletions.
diff --git a/...osoft.TestPlatform.Extensions.EventLogCollector/CollectorNameValueConfigurationManager.cs b/...osoft.TestPlatform.Extensions.EventLogCollector/CollectorNameValueConfigurationManager.cs
@@ -16,24 +16,15 @@ namespace Microsoft.TestPlatform.Extensions.EventLogCollector;
 /// </summary>
 internal class CollectorNameValueConfigurationManager
 {
-    #region Private constants
     // Configuration XML constants
     private const string SettingNameAttributeName = "name";
 
     private const string SettingValueAttributeName = "value";
 
-    #endregion
-
-    #region Private fields
-
     /// <summary>
     /// The name/value pairs loaded from the configuration XML element
     /// </summary>
 
-    #endregion
-
-    #region Constructor
-
     /// <summary>
     /// Initializes a new instance of the <see cref="CollectorNameValueConfigurationManager"/> class.
     /// Loads the configuration name/value information from the provided XML element into a dictionary
@@ -88,10 +79,6 @@ public CollectorNameValueConfigurationManager(XmlElement configurationElement)
         }
     }
 
-    #endregion
-
-    #region Public properties
-
     internal IDictionary<string, string> NameValuePairs { get; } = new Dictionary<string, string>();
 
     /// <summary>
@@ -114,5 +101,4 @@ public string this[string name]
 
         set => NameValuePairs[name] = value;
     }
-    #endregion
 }
diff --git a/...taCollectors/Microsoft.TestPlatform.Extensions.EventLogCollector/EventLogDataCollector.cs b/...taCollectors/Microsoft.TestPlatform.Extensions.EventLogCollector/EventLogDataCollector.cs
@@ -27,8 +27,6 @@ namespace Microsoft.TestPlatform.Extensions.EventLogCollector;
 [DataCollectorFriendlyName("Event Log")]
 public class EventLogDataCollector : DataCollector
 {
-    #region Constants
-
     /// <summary>
     /// The event log file name.
     /// </summary>
@@ -39,10 +37,6 @@ public class EventLogDataCollector : DataCollector
     /// </summary>
     private const string DefaultUri = @"datacollector://Microsoft/EventLog/2.0";
 
-    #endregion
-
-    #region Private fields
-
     /// <summary>
     /// Event handler delegate for the SessionStart event
     /// </summary>
@@ -98,10 +92,6 @@ public class EventLogDataCollector : DataCollector
     /// </summary>
     private readonly IDictionary<string, IEventLogContainer> _eventLogContainerMap = new Dictionary<string, IEventLogContainer>();
 
-    #endregion
-
-    #region Constructor
-
     /// <summary>
     /// Initializes a new instance of the <see cref="EventLogDataCollector"/> class.
     /// </summary>
@@ -128,10 +118,6 @@ internal EventLogDataCollector(IFileHelper fileHelper)
         _fileHelper = fileHelper;
     }
 
-    #endregion
-
-    #region Internal Fields
-
     internal int MaxEntries { get; private set; }
 
     internal ISet<string> EventSources { get; private set; }
@@ -145,10 +131,6 @@ internal EventLogDataCollector(IFileHelper fileHelper)
     /// </summary>
     internal Dictionary<DataCollectionContext, EventLogSessionContext> ContextMap { get; }
 
-    #endregion
-
-    #region DataCollector Members
-
     /// <summary>
     /// Initializes the data collector
     /// </summary>
@@ -197,10 +179,6 @@ public override void Initialize(
         events.TestCaseEnd += _testCaseEndEventHandler;
     }
 
-    #endregion
-
-    #region Internal
-
     /// <summary>
     /// The write event logs.
     /// </summary>
@@ -297,10 +275,6 @@ internal string WriteEventLogs(List<EventLogEntry> eventLogEntries, int maxLogEn
 
         return eventLogPath;
     }
-    #endregion
-
-    #region IDisposable Members
-
     /// <summary>
     /// Cleans up resources allocated by the data collector
     /// </summary>
@@ -325,8 +299,6 @@ protected override void Dispose(bool disposing)
         RemoveTempEventLogDirs(_eventLogDirectories);
     }
 
-    #endregion
-
     private static ISet<string> ParseCommaSeparatedList(string commaSeparatedList)
     {
         ISet<string> strings = new HashSet<string>();
@@ -339,8 +311,6 @@ private static ISet<string> ParseCommaSeparatedList(string commaSeparatedList)
         return strings;
     }
 
-    #region Event Handlers
-
     private void OnSessionStart(object sender, SessionStartEventArgs e)
     {
         ValidateArg.NotNull(e, "SessionStartEventArgs");
@@ -392,10 +362,6 @@ private void OnTestCaseEnd(object sender, TestCaseEndEventArgs e)
         WriteCollectedEventLogEntries(e.Context, false, TimeSpan.MaxValue, DateTime.UtcNow);
     }
 
-    #endregion
-
-    #region Private methods
-
     private void RemoveTempEventLogDirs(List<string> tempDirs)
     {
         if (tempDirs != null)
@@ -596,5 +562,4 @@ private EventLogSessionContext GetEventLogSessionContext(DataCollectionContext d
         return eventLogSessionContext;
     }
 
-    #endregion
 }
diff --git a/src/DataCollectors/Microsoft.TestPlatform.Extensions.EventLogCollector/EventLogXmlWriter.cs b/src/DataCollectors/Microsoft.TestPlatform.Extensions.EventLogCollector/EventLogXmlWriter.cs
@@ -20,8 +20,6 @@ namespace Microsoft.TestPlatform.Extensions.EventLogCollector;
 /// </summary>
 internal static class EventLogXmlWriter
 {
-    #region Public methods
-
     /// <summary>
     /// The write event log entries to xml file.
     /// </summary>
@@ -83,4 +81,3 @@ public static void WriteEventLogEntriesToXmlFile(string xmlFilePath, List<EventL
         fileHelper.WriteAllTextToFile(xmlFilePath, stringBuilder.ToString());
     }
 }
-#endregion
diff --git a/src/DataCollectors/Microsoft.TestPlatform.Extensions.EventLogCollector/Friends.cs b/src/DataCollectors/Microsoft.TestPlatform.Extensions.EventLogCollector/Friends.cs
@@ -3,8 +3,5 @@
 
 using System.Runtime.CompilerServices;
 
-#region Test Assemblies
-
 [assembly: InternalsVisibleTo("Microsoft.TestPlatform.Extensions.EventLogCollector.UnitTests, PublicKey=002400000480000094000000060200000024000052534131000400000100010007d1fa57c4aed9f0a32e84aa0faefd0de9e8fd6aec8f87fb03766c834c99921eb23be79ad9d5dcc1dd9ad236132102900b723cf980957fc4e177108fc607774f29e8320e92ea05ece4e821c0a5efe8f1645c4c0c93c1ab99285d622caa652c1dfad63d745d6f2de5f17e5eaf0fc4963d261c8a12436518206dc093344d5ad293")]
 [assembly: InternalsVisibleTo("DynamicProxyGenAssembly2, PublicKey=0024000004800000940000000602000000240000525341310004000001000100c547cac37abd99c8db225ef2f6c8a3602f3b3606cc9891605d02baa56104f4cfc0734aa39b93bf7852f7d9266654753cc297e7d2edfe0bac1cdcf9f717241550e0a7b191195b7667bb4f64bcb8e2121380fd1d9d46ad2d92d2d15605093924cceaf74c4861eff62abf69b9291ed0a340e113be11e6a7d3113e92484cf7045cc7")]
-#endregion
diff --git a/src/Microsoft.TestPlatform.Build/Friends.cs b/src/Microsoft.TestPlatform.Build/Friends.cs
@@ -3,8 +3,5 @@
 
 using System.Runtime.CompilerServices;
 
-#region Test Assemblies
-
 [assembly: InternalsVisibleTo("Microsoft.TestPlatform.Build.UnitTests, PublicKey=002400000480000094000000060200000024000052534131000400000100010007d1fa57c4aed9f0a32e84aa0faefd0de9e8fd6aec8f87fb03766c834c99921eb23be79ad9d5dcc1dd9ad236132102900b723cf980957fc4e177108fc607774f29e8320e92ea05ece4e821c0a5efe8f1645c4c0c93c1ab99285d622caa652c1dfad63d745d6f2de5f17e5eaf0fc4963d261c8a12436518206dc093344d5ad293")]
 
-#endregion
diff --git a/src/Microsoft.TestPlatform.Client/DesignMode/DesignModeClient.cs b/src/Microsoft.TestPlatform.Client/DesignMode/DesignModeClient.cs
@@ -561,8 +561,6 @@ private void StopTestSession(StopTestSessionPayload payload, ITestRequestManager
         });
     }
 
-    #region IDisposable Support
-
     private bool _disposedValue; // To detect redundant calls
 
     protected virtual void Dispose(bool disposing)
@@ -584,5 +582,4 @@ public void Dispose()
         // Do not change this code. Put cleanup code in Dispose(bool disposing) above.
         Dispose(true);
     }
-    #endregion
 }
diff --git a/src/Microsoft.TestPlatform.Client/DesignMode/DesignModeTestEventsRegistrar.cs b/src/Microsoft.TestPlatform.Client/DesignMode/DesignModeTestEventsRegistrar.cs
@@ -23,8 +23,6 @@ public DesignModeTestEventsRegistrar(IDesignModeClient designModeClient)
         _designModeClient = designModeClient;
     }
 
-    #region ITestDiscoveryEventsRegistrar
-
     public void RegisterDiscoveryEvents(IDiscoveryRequest discoveryRequest)
     {
         discoveryRequest.OnRawMessageReceived += OnRawMessageReceived;
@@ -35,10 +33,6 @@ public void UnregisterDiscoveryEvents(IDiscoveryRequest discoveryRequest)
         discoveryRequest.OnRawMessageReceived -= OnRawMessageReceived;
     }
 
-    #endregion
-
-    #region ITestRunEventsRegistrar
-
     public void RegisterTestRunEvents(ITestRunRequest testRunRequest)
     {
         testRunRequest.OnRawMessageReceived += OnRawMessageReceived;
@@ -49,8 +43,6 @@ public void UnregisterTestRunEvents(ITestRunRequest testRunRequest)
         testRunRequest.OnRawMessageReceived -= OnRawMessageReceived;
     }
 
-    #endregion
-
     /// <summary>
     /// RawMessage received handler for getting rawmessages directly from the host
     /// </summary>

diff --git a/src/Microsoft.TestPlatform.Client/Discovery/DiscoveryRequest.cs b/src/Microsoft.TestPlatform.Client/Discovery/DiscoveryRequest.cs
@@ -200,8 +200,6 @@ public DiscoveryCriteria DiscoveryCriteria
     /// </summary>
     internal ITestLoggerManager LoggerManager { get; }
 
-    #region ITestDiscoveryEventsHandler2 Methods
-
     /// <inheritdoc/>
     public void HandleDiscoveryComplete(DiscoveryCompleteEventArgs discoveryCompleteEventArgs, IEnumerable<TestCase> lastChunk)
     {
@@ -419,10 +417,6 @@ private string UpdateRawMessageWithTelemetryInfo(DiscoveryCompletePayload discov
         return rawMessage;
     }
 
-    #endregion
-
-    #region IDisposable implementation
-
     /// <summary>
     /// Performs application-defined tasks associated with freeing, releasing, or
     /// resetting unmanaged resources.
@@ -459,10 +453,6 @@ private void Dispose(bool disposing)
         EqtTrace.Info("DiscoveryRequest.Dispose: Completed.");
     }
 
-    #endregion
-
-    #region privates fields
-
     /// <summary>
     /// Request Data
     /// </summary>
@@ -488,5 +478,4 @@ private void Dispose(bool disposing)
     /// </summary>
     private DateTime _discoveryStartTime;
 
-    #endregion
 }
diff --git a/src/Microsoft.TestPlatform.Client/Execution/TestRunRequest.cs b/src/Microsoft.TestPlatform.Client/Execution/TestRunRequest.cs
@@ -94,8 +94,6 @@ internal TestRunRequest(IRequestData requestData, TestRunCriteria testRunCriteri
         _requestData = requestData;
     }
 
-    #region ITestRunRequest
-
     /// <summary>
     /// Execute the test run asynchronously
     /// </summary>
@@ -329,10 +327,6 @@ internal ITestLoggerManager LoggerManager
         get; private set;
     }
 
-    #endregion
-
-    #region IDisposable implementation
-
     // Summary:
     // Performs application-defined tasks associated with freeing, releasing, or
     // resetting unmanaged resources.
@@ -343,8 +337,6 @@ public void Dispose()
         GC.SuppressFinalize(this);
     }
 
-    #endregion
-
     /// <summary>
     /// The criteria/config for this test run request.
     /// </summary>

diff --git a/src/Microsoft.TestPlatform.Client/Friends.cs b/src/Microsoft.TestPlatform.Client/Friends.cs
@@ -3,12 +3,5 @@
 
 using System.Runtime.CompilerServices;
 
-#region Product Assemblies
-
-#endregion
-
-#region Test Assemblies
-
 [assembly: InternalsVisibleTo("Microsoft.TestPlatform.Client.UnitTests, PublicKey=002400000480000094000000060200000024000052534131000400000100010007d1fa57c4aed9f0a32e84aa0faefd0de9e8fd6aec8f87fb03766c834c99921eb23be79ad9d5dcc1dd9ad236132102900b723cf980957fc4e177108fc607774f29e8320e92ea05ece4e821c0a5efe8f1645c4c0c93c1ab99285d622caa652c1dfad63d745d6f2de5f17e5eaf0fc4963d261c8a12436518206dc093344d5ad293")]
 
-#endregion
diff --git a/src/Microsoft.TestPlatform.Common/DataCollection/DataCollectionAttachmentManager.cs b/src/Microsoft.TestPlatform.Common/DataCollection/DataCollectionAttachmentManager.cs
@@ -43,8 +43,6 @@ internal class DataCollectionAttachmentManager : IDataCollectionAttachmentManage
 {
     private readonly object _attachmentTaskLock = new();
 
-    #region Fields
-
     /// <summary>
     /// Default results directory to be used when user didn't specify.
     /// </summary>
@@ -70,10 +68,6 @@ internal class DataCollectionAttachmentManager : IDataCollectionAttachmentManage
     /// </summary>
     private readonly IFileHelper _fileHelper;
 
-    #endregion
-
-    #region Constructor
-
     /// <summary>
     /// Initializes a new instance of the <see cref="DataCollectionAttachmentManager"/> class.
     /// </summary>
@@ -94,10 +88,6 @@ protected DataCollectionAttachmentManager(IFileHelper fileHelper)
         AttachmentSets = new ConcurrentDictionary<DataCollectionContext, ConcurrentDictionary<Uri, AttachmentSet>>();
     }
 
-    #endregion
-
-    #region Properties
-
     /// <summary>
     /// Gets the session output directory.
     /// </summary>
@@ -110,10 +100,6 @@ internal ConcurrentDictionary<DataCollectionContext, ConcurrentDictionary<Uri, A
     {
         get; private set;
     }
-    #endregion
-
-    #region public methods
-
     /// <inheritdoc/>
     public void Initialize(SessionId id, string outputDirectory, IMessageSink messageSink)
     {
@@ -210,10 +196,6 @@ public void Cancel()
         _cancellationTokenSource.Cancel();
     }
 
-    #endregion
-
-    #region private methods
-
     /// <summary>
     /// Sanity checks on CopyRequestData 
     /// </summary>
@@ -378,5 +360,4 @@ private void LogError(string errorMessage, Uri collectorUri, string collectorFri
         _messageSink.SendMessage(args);
     }
 
-    #endregion
 }