[Mailer] Add test to resourceName exception case in Azure Bridge

hafael · hafael · commit d385220fefcb · 2023-12-06T17:11:33.000-03:00
diff --git a/src/Symfony/Component/Mailer/Bridge/Azure/Tests/Transport/AzureApiTransportTest.php b/src/Symfony/Component/Mailer/Bridge/Azure/Tests/Transport/AzureApiTransportTest.php
@@ -117,4 +117,12 @@ public function testTagAndMetadataHeaders()
         $this->assertSame('blue', $payload['headers']['X-Metadata-Color']);
         $this->assertSame('12345', $payload['headers']['X-Metadata-Client-ID']);
     }
+
+    public function testItDoesNotAllowToAddResourceNameWithDot()
+    {
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage('Resource name cannot contain or end with a dot');
+
+        new AzureApiTransport('KEY', 'ACS_RESOURCE_NAME.');
+    }
 }
diff --git a/src/Symfony/Component/Mailer/Bridge/Azure/Transport/AzureTransportFactory.php b/src/Symfony/Component/Mailer/Bridge/Azure/Transport/AzureTransportFactory.php
@@ -22,7 +22,7 @@ public function create(Dsn $dsn): TransportInterface
     {
         $scheme = $dsn->getScheme();
 
-        if (\in_array($scheme, ['azure+api', 'azure'], true)) {
+        if (!in_array($scheme, ['azure+api', 'azure'], true)) {
             throw new UnsupportedSchemeException($dsn, 'azure', $this->getSupportedSchemes());
         }
 

Original file line number	Diff line number	Diff line change
`@@ -117,4 +117,12 @@ public function testTagAndMetadataHeaders()`
`117`	`117`	`$this->assertSame('blue', $payload['headers']['X-Metadata-Color']);`
`118`	`118`	`$this->assertSame('12345', $payload['headers']['X-Metadata-Client-ID']);`
`119`	`119`	`}`
	`120`	`+`
	`121`	`+ public function testItDoesNotAllowToAddResourceNameWithDot()`
	`122`	`+ {`
	`123`	`+ $this->expectException(\Exception::class);`
	`124`	`+ $this->expectExceptionMessage('Resource name cannot contain or end with a dot');`
	`125`	`+`
	`126`	`+ new AzureApiTransport('KEY', 'ACS_RESOURCE_NAME.');`
	`127`	`+ }`
`120`	`128`	`}`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public function create(Dsn $dsn): TransportInterface`
`22`	`22`	`{`
`23`	`23`	`$scheme = $dsn->getScheme();`
`24`	`24`
`25`		`- if (\in_array($scheme, ['azure+api', 'azure'], true)) {`
	`25`	`+ if (!in_array($scheme, ['azure+api', 'azure'], true)) {`
`26`	`26`	`throw new UnsupportedSchemeException($dsn, 'azure', $this->getSupportedSchemes());`
`27`	`27`	`}`
`28`	`28`