make tests final and use assertSame where it is possible

ismail1432 · ismail1432 · commit 234621d56b21 · 2021-08-05T20:49:59.000+01:00
diff --git a/src/Symfony/Component/Notifier/Tests/Event/FailedMessageEventTest.php b/src/Symfony/Component/Notifier/Tests/Event/FailedMessageEventTest.php
@@ -18,7 +18,7 @@
 /**
  * @author Smaïne Milianni <smaine.milianni@gmail.com>
  */
-class FailedMessageEventTest extends TestCase
+final class FailedMessageEventTest extends TestCase
 {
     /**
      * @dataProvider messagesProvider
@@ -33,15 +33,15 @@ public function testConstruct(MessageInterface $message, \Throwable $error, Fail
      */
     public function testGetMessage(MessageInterface $message, \Throwable $error, FailedMessageEvent $event)
     {
-        $this->assertEquals($message, $event->getMessage());
+        $this->assertSame($message, $event->getMessage());
     }
 
     /**
      * @dataProvider messagesProvider
      */
     public function testGetError(MessageInterface $message, \Throwable $error, FailedMessageEvent $event)
     {
-        $this->assertEquals($error, $event->getError());
+        $this->assertSame($error, $event->getError());
     }
 
     public function testFailedMessageEventIsDisptachIfError()
diff --git a/src/Symfony/Component/Notifier/Tests/Event/SentMessageEventTest.php b/src/Symfony/Component/Notifier/Tests/Event/SentMessageEventTest.php
@@ -11,7 +11,7 @@
 /**
  * @author Smaïne Milianni <smaine.milianni@gmail.com>
  */
-class SentMessageEventTest extends TestCase
+final class SentMessageEventTest extends TestCase
 {
     /**
      * @dataProvider messagesProvider
@@ -26,7 +26,7 @@ public function testConstruct(SentMessage $message, SentMessageEvent $event)
      */
     public function testGetMessage(SentMessage $message, SentMessageEvent $event)
     {
-        $this->assertEquals($message, $event->getMessage());
+        $this->assertSame($message, $event->getMessage());
     }
 
     public function messagesProvider(): iterable

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@`
`18`	`18`	`/**`
`19`	`19`	`* @author Smaïne Milianni <smaine.milianni@gmail.com>`
`20`	`20`	`*/`
`21`		`-class FailedMessageEventTest extends TestCase`
	`21`	`+final class FailedMessageEventTest extends TestCase`
`22`	`22`	`{`
`23`	`23`	`/**`
`24`	`24`	`* @dataProvider messagesProvider`
`@@ -33,15 +33,15 @@ public function testConstruct(MessageInterface $message, \Throwable $error, Fail`
`33`	`33`	`*/`
`34`	`34`	`public function testGetMessage(MessageInterface $message, \Throwable $error, FailedMessageEvent $event)`
`35`	`35`	`{`
`36`		`- $this->assertEquals($message, $event->getMessage());`
	`36`	`+ $this->assertSame($message, $event->getMessage());`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`/**`
`40`	`40`	`* @dataProvider messagesProvider`
`41`	`41`	`*/`
`42`	`42`	`public function testGetError(MessageInterface $message, \Throwable $error, FailedMessageEvent $event)`
`43`	`43`	`{`
`44`		`- $this->assertEquals($error, $event->getError());`
	`44`	`+ $this->assertSame($error, $event->getError());`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`public function testFailedMessageEventIsDisptachIfError()`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`/**`
`12`	`12`	`* @author Smaïne Milianni <smaine.milianni@gmail.com>`
`13`	`13`	`*/`
`14`		`-class SentMessageEventTest extends TestCase`
	`14`	`+final class SentMessageEventTest extends TestCase`
`15`	`15`	`{`
`16`	`16`	`/**`
`17`	`17`	`* @dataProvider messagesProvider`
`@@ -26,7 +26,7 @@ public function testConstruct(SentMessage $message, SentMessageEvent $event)`
`26`	`26`	`*/`
`27`	`27`	`public function testGetMessage(SentMessage $message, SentMessageEvent $event)`
`28`	`28`	`{`
`29`		`- $this->assertEquals($message, $event->getMessage());`
	`29`	`+ $this->assertSame($message, $event->getMessage());`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`public function messagesProvider(): iterable`