style(*): Remove trailing comma for 7.2

Author: julienloizelet

src/AbstractBouncer.php

@@ -48,7 +48,7 @@ abstract class AbstractBouncer
     public function __construct(
         array $configs,
         LapiRemediation $remediationEngine,
-        ?LoggerInterface $logger = null,
+        ?LoggerInterface $logger = null
     ) {
         // @codeCoverageIgnoreStart
         if (!$logger) {
@@ -258,7 +258,7 @@ public function pruneCache(): bool
     public function pushUsageMetrics(
         string $bouncerName,
         string $bouncerVersion,
-        string $bouncerType = LapiConstants::METRICS_TYPE,
+        string $bouncerType = LapiConstants::METRICS_TYPE
     ): array {
         try {
             return $this->remediationEngine->pushUsageMetrics($bouncerName, $bouncerVersion, $bouncerType);
@@ -424,7 +424,7 @@ protected function getBanHtml(): string
     protected function getCaptchaHtml(
         bool $error,
         string $captchaImageSrc,
-        string $captchaResolutionFormUrl,
+        string $captchaResolutionFormUrl
     ): string {
         $template = new Template('captcha.html.twig');
 

src/Helper.php

@@ -34,7 +34,7 @@ private function buildRawBodyFromSuperglobals(
         $stream,
         array $serverData = [], // $_SERVER
         array $postData = [], // $_POST
-        array $filesData = [], // $_FILES
+        array $filesData = [] // $_FILES
     ): string {
         $contentType = $serverData['CONTENT_TYPE'] ?? '';
         // The threshold is the maximum body size converted in bytes + 1
@@ -53,7 +53,7 @@ private function appendFileData(
         string $fileKey,
         string $boundary,
         int $threshold,
-        int &$currentSize,
+        int &$currentSize
     ): string {
         $fileName = is_array($fileArray['name']) ? $fileArray['name'][$index] : $fileArray['name'];
         $fileTmpName = is_array($fileArray['tmp_name']) ? $fileArray['tmp_name'][$index] : $fileArray['tmp_name'];
@@ -119,7 +119,7 @@ private function getMultipartRawBody(
         string $contentType,
         int $threshold,
         array $postData,
-        array $filesData,
+        array $filesData
     ): string {
         try {
             $boundary = $this->extractBoundary($contentType);

tests/Integration/AbstractBouncerTest.php

@@ -1127,7 +1127,6 @@ public function testResetMetrics()
             'The origin count for clean should be 2'
         );
 
-
         // Test: reset metrics
         $result = $bouncer->resetUsageMetrics();
         $originCountItem = $cache->getItem(AbstractCache::ORIGINS_COUNT)->get();

tests/Integration/WatcherClient.php

@@ -50,7 +50,7 @@ public function __construct(array $configs)
     private function manageRequest(
         string $method,
         string $endpoint,
-        array $parameters = [],
+        array $parameters = []
     ): array {
         $this->logger->debug('', [
             'type' => 'WATCHER_CLIENT_REQUEST',
@@ -158,7 +158,7 @@ public function addDecision(
         string $dateTimeDurationString,
         string $value,
         string $type,
-        string $scope = Constants::SCOPE_IP,
+        string $scope = Constants::SCOPE_IP
     ) {
         $stopAt = (clone $now)->modify($dateTimeDurationString)->format('Y-m-d\TH:i:s.000\Z');
         $startAt = $now->format('Y-m-d\TH:i:s.000\Z');