[Notifier] fix PR checks

PEtanguy · PEtanguy · commit 9a467e7af0b1 · 2023-05-10T18:29:35.000+01:00
diff --git a/src/Symfony/Component/Notifier/Bridge/Brevo/BrevoTransport.php b/src/Symfony/Component/Notifier/Bridge/Brevo/BrevoTransport.php
@@ -57,7 +57,7 @@ protected function doSend(MessageInterface $message): SentMessage
 
         $sender = $message->getFrom() ?: $this->sender;
 
-        $response = $this->client->request('POST', 'https://' . $this->getEndpoint() . '/v3/transactionalSMS/sms', [
+        $response = $this->client->request('POST', 'https://'. $this->getEndpoint(). '/v3/transactionalSMS/sms', [
             'json' => [
                 'sender' => $sender,
                 'recipient' => $message->getPhone(),
@@ -77,12 +77,12 @@ protected function doSend(MessageInterface $message): SentMessage
         if (201 !== $statusCode) {
             $error = $response->toArray(false);
 
-            throw new TransportException('Unable to send the SMS: ' . $error['message'], $response);
+            throw new TransportException('Unable to send the SMS: '. $error['message'], $response);
         }
 
         $success = $response->toArray(false);
 
-        $sentMessage = new SentMessage($message, (string)$this);
+        $sentMessage = new SentMessage($message, (string) $this);
         $sentMessage->setMessageId($success['messageId']);
 
         return $sentMessage;
diff --git a/src/Symfony/Component/Notifier/Bridge/Brevo/LICENSE b/src/Symfony/Component/Notifier/Bridge/Brevo/LICENSE
@@ -1,4 +1,4 @@
-Copyright (c) 2022-present Fabien Potencier
+Copyright (c) 2023-present Fabien Potencier
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
diff --git a/src/Symfony/Component/Notifier/Bridge/Brevo/Tests/BrevoTransportTest.php b/src/Symfony/Component/Notifier/Bridge/Brevo/Tests/BrevoTransportTest.php
@@ -54,7 +54,7 @@ public function testSendWithErrorResponseThrowsTransportException()
             ->method('getContent')
             ->willReturn(json_encode(['code' => 400, 'message' => 'bad request']));
 
-        $client = new MockHttpClient(static fn(): ResponseInterface => $response);
+        $client = new MockHttpClient(static fn (): ResponseInterface => $response);
 
         $transport = self::createTransport($client);
 
diff --git a/src/Symfony/Component/Notifier/Bridge/Brevo/composer.json b/src/Symfony/Component/Notifier/Bridge/Brevo/composer.json
@@ -1,36 +1,34 @@
 {
-  "name": "symfony/brevo-notifier",
-  "type": "symfony-notifier-bridge",
-  "description": "Symfony Brevo Notifier Bridge",
-  "keywords": [
-    "brevo",
-    "notifier"
-  ],
-  "homepage": "https://symfony.com",
-  "license": "MIT",
-  "authors": [
-    {
-      "name": "Pierre Tanguy",
-      "homepage": "https://github.com/petanguy"
+    "name": "symfony/brevo-notifier",
+    "type": "symfony-notifier-bridge",
+    "description": "Symfony Brevo Notifier Bridge",
+    "keywords": ["brevo", "notifier"],
+    "homepage": "https://symfony.com",
+    "license": "MIT",
+    "authors": [
+        {
+            "name": "Pierre Tanguy",
+            "homepage": "https://github.com/petanguy"
+        },
+        {
+            "name": "Symfony Community",
+            "homepage": "https://symfony.com/contributors"
+        }
+    ],
+    "require": {
+        "php": ">=8.1",
+        "symfony/http-client": "^5.4|^6.0",
+        "symfony/notifier": "^6.3"
     },
-    {
-      "name": "Symfony Community",
-      "homepage": "https://symfony.com/contributors"
-    }
-  ],
-  "require": {
-    "php": ">=8.1",
-    "symfony/http-client": "^5.4|^6.0",
-    "symfony/notifier": "^6.3"
-  },
-  "require-dev": {
-    "symfony/event-dispatcher": "^5.4|^6.0"
-  },
-  "autoload": {
-    "psr-4": {"Symfony\\Component\\Notifier\\Bridge\\Brevo\\": ""},
-    "exclude-from-classmap": [
-      "/Tests/"
-    ]
-  },
-  "minimum-stability": "dev"
+    "require-dev": {
+        "symfony/event-dispatcher": "^5.4|^6.0"
+    },
+    "autoload": {
+        "psr-4": {"Symfony\\Component\\Notifier\\Bridge\\Brevo\\": ""},
+        "exclude-from-classmap": [
+            "/Tests/"
+        ]
+    },
+    "minimum-stability": "dev"
 }
+
diff --git a/src/Symfony/Component/Notifier/Bridge/Sendinblue/Tests/SendinblueTransportFactoryTest.php b/src/Symfony/Component/Notifier/Bridge/Sendinblue/Tests/SendinblueTransportFactoryTest.php
@@ -14,6 +14,9 @@
 use Symfony\Component\Notifier\Bridge\Sendinblue\SendinblueTransportFactory;
 use Symfony\Component\Notifier\Test\TransportFactoryTestCase;
 
+/**
+ * @group legacy
+ */
 final class SendinblueTransportFactoryTest extends TransportFactoryTestCase
 {
     public function createFactory(): SendinblueTransportFactory
diff --git a/src/Symfony/Component/Notifier/Bridge/Sendinblue/Tests/SendinblueTransportTest.php b/src/Symfony/Component/Notifier/Bridge/Sendinblue/Tests/SendinblueTransportTest.php
@@ -21,6 +21,9 @@
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 use Symfony\Contracts\HttpClient\ResponseInterface;
 
+/**
+ * @group legacy
+ */
 final class SendinblueTransportTest extends TransportTestCase
 {
     public static function createTransport(HttpClientInterface $client = null): SendinblueTransport

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-Copyright (c) 2022-present Fabien Potencier`
	`1`	`+Copyright (c) 2023-present Fabien Potencier`
`2`	`2`
`3`	`3`	`Permission is hereby granted, free of charge, to any person obtaining a copy`
`4`	`4`	`of this software and associated documentation files (the "Software"), to deal`
Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,9 @@`
`14`	`14`	`use Symfony\Component\Notifier\Bridge\Sendinblue\SendinblueTransportFactory;`
`15`	`15`	`use Symfony\Component\Notifier\Test\TransportFactoryTestCase;`
`16`	`16`
	`17`	`+/**`
	`18`	`+ * @group legacy`
	`19`	`+ */`
`17`	`20`	`final class SendinblueTransportFactoryTest extends TransportFactoryTestCase`
`18`	`21`	`{`
`19`	`22`	`public function createFactory(): SendinblueTransportFactory`
Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,9 @@`
`21`	`21`	`use Symfony\Contracts\HttpClient\HttpClientInterface;`
`22`	`22`	`use Symfony\Contracts\HttpClient\ResponseInterface;`
`23`	`23`
	`24`	`+/**`
	`25`	`+ * @group legacy`
	`26`	`+ */`
`24`	`27`	`final class SendinblueTransportTest extends TransportTestCase`
`25`	`28`	`{`
`26`	`29`	`public static function createTransport(HttpClientInterface $client = null): SendinblueTransport`