[Notifier] fix PR checks

symfony · May 10, 2023 · 9622d22 · 9622d22
1 parent 85520f2
commit 9622d22
Show file tree

Hide file tree

Showing 6 changed files with 42 additions and 38 deletions.
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