[Validator] Fix tests

ogizanagi · ogizanagi · commit 22c58f59ce58 · 2019-07-04T17:27:57.000+02:00
diff --git a/src/Symfony/Component/Validator/ConstraintViolation.php b/src/Symfony/Component/Validator/ConstraintViolation.php
@@ -84,7 +84,7 @@ public function __toString()
             $class .= '.';
         }
 
-        if ('' !== $code = $this->code) {
+        if ('' !== $code = (string) $this->code) {
             $code = ' (code '.$code.')';
         }
 
diff --git a/src/Symfony/Component/Validator/Tests/ConstraintViolationTest.php b/src/Symfony/Component/Validator/Tests/ConstraintViolationTest.php
@@ -109,10 +109,6 @@ public function testToStringOmitsEmptyCodes()
         $this->assertSame($expected, (string) $violation);
     }
 
-    /**
-     * @group legacy
-     * @expectedDeprecation Not using a string as the error code in Symfony\Component\Validator\ConstraintViolation::__construct() is deprecated since Symfony 4.4. A type-hint will be added in 5.0.
-     */
     public function testNonStringCode()
     {
         $violation = new ConstraintViolation(
@@ -126,6 +122,6 @@ public function testNonStringCode()
             42
         );
 
-        self::assertSame(42, $violation->getCode());
+        self::assertSame('42', $violation->getCode());
     }
 }
diff --git a/src/Symfony/Component/Validator/Tests/Violation/ConstraintViolationBuilderTest.php b/src/Symfony/Component/Validator/Tests/Violation/ConstraintViolationBuilderTest.php
@@ -19,18 +19,13 @@
 
 class ConstraintViolationBuilderTest extends TestCase
 {
-    /**
-     * @group legacy
-     * @expectedDeprecation Not using a string as the error code in Symfony\Component\Validator\Violation\ConstraintViolationBuilder::setCode() is deprecated since Symfony 4.4. A type-hint will be added in 5.0.
-     * @expectedDeprecation Not using a string as the error code in Symfony\Component\Validator\ConstraintViolation::__construct() is deprecated since Symfony 4.4. A type-hint will be added in 5.0.
-     */
     public function testNonStringCode()
     {
         $constraintViolationList = new ConstraintViolationList();
         (new ConstraintViolationBuilder($constraintViolationList, new ConstraintA(), 'invalid message', [], null, 'foo', 'baz', new IdentityTranslator()))
             ->setCode(42)
             ->addViolation();
 
-        self::assertSame(42, $constraintViolationList->get(0)->getCode());
+        self::assertSame('42', $constraintViolationList->get(0)->getCode());
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,7 @@ public function __toString()`
`84`	`84`	`$class .= '.';`
`85`	`85`	`}`
`86`	`86`
`87`		`- if ('' !== $code = $this->code) {`
	`87`	`+ if ('' !== $code = (string) $this->code) {`
`88`	`88`	`$code = ' (code '.$code.')';`
`89`	`89`	`}`
`90`	`90`
Original file line number	Diff line number	Diff line change
`@@ -109,10 +109,6 @@ public function testToStringOmitsEmptyCodes()`
`109`	`109`	`$this->assertSame($expected, (string) $violation);`
`110`	`110`	`}`
`111`	`111`
`112`		`- /**`
`113`		`- * @group legacy`
`114`		`- * @expectedDeprecation Not using a string as the error code in Symfony\Component\Validator\ConstraintViolation::__construct() is deprecated since Symfony 4.4. A type-hint will be added in 5.0.`
`115`		`- */`
`116`	`112`	`public function testNonStringCode()`
`117`	`113`	`{`
`118`	`114`	`$violation = new ConstraintViolation(`
`@@ -126,6 +122,6 @@ public function testNonStringCode()`
`126`	`122`	`42`
`127`	`123`	`);`
`128`	`124`
`129`		`- self::assertSame(42, $violation->getCode());`
	`125`	`+ self::assertSame('42', $violation->getCode());`
`130`	`126`	`}`
`131`	`127`	`}`
Original file line number	Diff line number	Diff line change
`@@ -19,18 +19,13 @@`
`19`	`19`
`20`	`20`	`class ConstraintViolationBuilderTest extends TestCase`
`21`	`21`	`{`
`22`		`- /**`
`23`		`- * @group legacy`
`24`		`- * @expectedDeprecation Not using a string as the error code in Symfony\Component\Validator\Violation\ConstraintViolationBuilder::setCode() is deprecated since Symfony 4.4. A type-hint will be added in 5.0.`
`25`		`- * @expectedDeprecation Not using a string as the error code in Symfony\Component\Validator\ConstraintViolation::__construct() is deprecated since Symfony 4.4. A type-hint will be added in 5.0.`
`26`		`- */`
`27`	`22`	`public function testNonStringCode()`
`28`	`23`	`{`
`29`	`24`	`$constraintViolationList = new ConstraintViolationList();`
`30`	`25`	`(new ConstraintViolationBuilder($constraintViolationList, new ConstraintA(), 'invalid message', [], null, 'foo', 'baz', new IdentityTranslator()))`
`31`	`26`	`->setCode(42)`
`32`	`27`	`->addViolation();`
`33`	`28`
`34`		`- self::assertSame(42, $constraintViolationList->get(0)->getCode());`
	`29`	`+ self::assertSame('42', $constraintViolationList->get(0)->getCode());`
`35`	`30`	`}`
`36`	`31`	`}`