Enabled and fixed issues for the rule SA1413. (#2013)

Author: MursalovAltun

lib/PuppeteerSharp.ruleset

@@ -23,7 +23,7 @@
     <Rule Id="SA1513" Action="None" /> <!-- Error SA1513: Closing brace should be followed by blank line (SA1513)-->
     <Rule Id="SA1124" Action="Error" /> <!-- Error SA1124: Don't use regions-->
     <Rule Id="SA1009" Action="Error" /> <!-- Error SA1009: Closing parenthesis should not be preceded by a space. (SA1009)-->
-    <Rule Id="SA1413" Action="None" /> <!-- Error SA1413: Use trailing comma in multi-line initializers (SA1413)-->
+    <Rule Id="SA1413" Action="Error" /> <!-- Error SA1413: Use trailing comma in multi-line initializers (SA1413)-->
     <Rule Id="SA1602" Action="None" /> <!-- Error SA1602: Enumeration items should be documented (SA1602)-->
     <Rule Id="SA1005" Action="Error" /> <!-- Error SA1005: Single line comment should begin with a space. (SA1005)-->
     <Rule Id="SA1203" Action="Error" /> <!-- Error SA1203: Constant fields should appear before non-constant fields (SA1203)-->

lib/PuppeteerSharp/BoundingBox.cs

@@ -59,7 +59,7 @@ internal Clip ToClip()
                 X = X,
                 Y = Y,
                 Width = Width,
-                Height = Height
+                Height = Height,
             };
         }
 

lib/PuppeteerSharp/Browser.cs

@@ -407,7 +407,7 @@ internal async Task<Page> CreatePageInContextAsync(string contextId)
         {
             var createTargetRequest = new TargetCreateTargetRequest
             {
-                Url = "about:blank"
+                Url = "about:blank",
             };
 
             if (contextId != null)
@@ -426,7 +426,7 @@ internal async Task DisposeContextAsync(string contextId)
         {
             await Connection.SendAsync("Target.disposeBrowserContext", new TargetDisposeBrowserContextRequest
             {
-                BrowserContextId = contextId
+                BrowserContextId = contextId,
             }).ConfigureAwait(false);
             _contexts.TryRemove(contextId, out var _);
         }
@@ -555,7 +555,7 @@ internal static async Task<Browser> CreateAsync(
 
             await connection.SendAsync("Target.setDiscoverTargets", new TargetSetDiscoverTargetsRequest
             {
-                Discover = true
+                Discover = true,
             }).ConfigureAwait(false);
 
             return browser;

lib/PuppeteerSharp/BrowserContext.cs

@@ -125,7 +125,7 @@ public Task OverridePermissionsAsync(string origin, IEnumerable<OverridePermissi
             {
                 Origin = origin,
                 BrowserContextId = _id,
-                Permissions = permissions.ToArray()
+                Permissions = permissions.ToArray(),
             });
 
         /// <summary>
@@ -135,7 +135,7 @@ public Task OverridePermissionsAsync(string origin, IEnumerable<OverridePermissi
         public Task ClearPermissionOverridesAsync()
             => _connection.SendAsync("Browser.resetPermissions", new BrowserResetPermissionsRequest
             {
-                BrowserContextId = _id
+                BrowserContextId = _id,
             });
 
         internal void OnTargetCreated(Browser browser, TargetChangedArgs args) => TargetCreated?.Invoke(browser, args);

lib/PuppeteerSharp/BrowserFetcher.cs

@@ -239,7 +239,7 @@ public RevisionInfo RevisionInfo(string revision)
                 FolderPath = GetFolderPath(revision),
                 Url = GetDownloadURL(Product, Platform, DownloadHost, revision),
                 Revision = revision,
-                Platform = Platform
+                Platform = Platform,
             };
             result.ExecutablePath = GetExecutablePath(Product, Platform, revision, result.FolderPath);
             result.Local = new DirectoryInfo(result.FolderPath).Exists;
@@ -369,7 +369,7 @@ private Task InstallDMGAsync(string dmgPath, string folderPath)
 
                 using var process = new Process
                 {
-                    EnableRaisingEvents = true
+                    EnableRaisingEvents = true,
                 };
 
                 process.StartInfo.FileName = "hdiutil";

lib/PuppeteerSharp/CDPSession.cs

@@ -139,7 +139,7 @@ public async Task<JObject> SendAsync(string method, object args = null, bool wai
                 callback = new MessageTask
                 {
                     TaskWrapper = new TaskCompletionSource<JObject>(TaskCreationOptions.RunContinuationsAsynchronously),
-                    Method = method
+                    Method = method,
                 };
                 _callbacks[id] = callback;
             }
@@ -173,7 +173,7 @@ public Task DetachAsync()
 
             return Connection.SendAsync("Target.detachFromTarget", new TargetDetachFromTargetRequest
             {
-                SessionId = Id
+                SessionId = Id,
             });
         }
 
@@ -193,7 +193,7 @@ internal void OnMessage(ConnectionResponse obj)
                 MessageReceived?.Invoke(this, new MessageEventArgs
                 {
                     MessageID = method,
-                    MessageData = obj.Params
+                    MessageData = obj.Params,
                 });
             }
         }

lib/PuppeteerSharp/ChromiumLauncher.cs

@@ -116,7 +116,7 @@ internal static string[] GetDefaultArgs(LaunchOptions options)
                 {
                     "--headless",
                     "--hide-scrollbars",
-                    "--mute-audio"
+                    "--mute-audio",
                 });
             }
 

lib/PuppeteerSharp/Connection.cs

@@ -125,7 +125,7 @@ internal async Task<JObject> SendAsync(string method, object args = null, bool w
                 callback = new MessageTask
                 {
                     TaskWrapper = new TaskCompletionSource<JObject>(TaskCreationOptions.RunContinuationsAsynchronously),
-                    Method = method
+                    Method = method,
                 };
                 _callbacks[id] = callback;
             }
@@ -145,7 +145,7 @@ internal async Task<CDPSession> CreateSessionAsync(TargetInfo targetInfo)
             var sessionId = (await SendAsync<TargetAttachToTargetResponse>("Target.attachToTarget", new TargetAttachToTargetRequest
             {
                 TargetId = targetInfo.TargetId,
-                Flatten = true
+                Flatten = true,
             }).ConfigureAwait(false)).SessionId;
             return await GetSessionAsync(sessionId).ConfigureAwait(false);
         }
@@ -271,7 +271,7 @@ private void ProcessIncomingMessage(ConnectionResponse obj)
                 MessageReceived?.Invoke(this, new MessageEventArgs
                 {
                     MessageID = method,
-                    MessageData = obj.Params
+                    MessageData = obj.Params,
                 });
             }
         }

lib/PuppeteerSharp/DOMWorldType.cs

@@ -11,6 +11,6 @@ internal enum DOMWorldType
         [EnumMember(Value = "isolated")]
         Isolated,
         [EnumMember(Value = "default")]
-        Default
+        Default,
     }
 }

lib/PuppeteerSharp/Dialog.cs

@@ -63,7 +63,7 @@ public Task Accept(string promptText = "")
             => _client.SendAsync("Page.handleJavaScriptDialog", new PageHandleJavaScriptDialogRequest
             {
                 Accept = true,
-                PromptText = promptText
+                PromptText = promptText,
             });
 
         /// <summary>
@@ -73,7 +73,7 @@ public Task Accept(string promptText = "")
         public Task Dismiss()
             => _client.SendAsync("Page.handleJavaScriptDialog", new PageHandleJavaScriptDialogRequest
             {
-                Accept = false
+                Accept = false,
             });
     }
 }