minor #15767 [Validator] mark test as legacy (TomasVotruba)

fabpot · fabpot · commit a9555fb98f0b · 2015-09-12T12:03:37.000+02:00
This PR was merged into the 2.8 branch. Discussion ---------- [Validator] mark test as legacy | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | yes | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Ref #15708 (comment) Failing test are hhvm binaries: https://travis-ci.org/symfony/symfony/jobs/79951993#L530 Commits ------- cf18a99 [Validator] mark test as legacy
diff --git a/src/Symfony/Component/Form/Tests/Extension/Validator/EventListener/ValidationListenerTest.php b/src/Symfony/Component/Form/Tests/Extension/Validator/EventListener/ValidationListenerTest.php
@@ -189,6 +189,9 @@ public function testValidatorInterfaceSinceSymfony25()
         $this->assertAttributeSame($validator, 'validator', $listener);
     }
 
+    /**
+     * @group legacy
+     */
     public function testValidatorInterfaceUntilSymfony24()
     {
         // Mock of ValidatorInterface until apiVersion 2.4
@@ -199,6 +202,7 @@ public function testValidatorInterfaceUntilSymfony24()
     }
 
     /**
+     * @group legacy
      * @expectedException \InvalidArgumentException
      */
     public function testInvalidValidatorInterface()

Original file line number	Diff line number	Diff line change
`@@ -189,6 +189,9 @@ public function testValidatorInterfaceSinceSymfony25()`
`189`	`189`	`$this->assertAttributeSame($validator, 'validator', $listener);`
`190`	`190`	`}`
`191`	`191`
	`192`	`+ /**`
	`193`	`+ * @group legacy`
	`194`	`+ */`
`192`	`195`	`public function testValidatorInterfaceUntilSymfony24()`
`193`	`196`	`{`
`194`	`197`	`// Mock of ValidatorInterface until apiVersion 2.4`
`@@ -199,6 +202,7 @@ public function testValidatorInterfaceUntilSymfony24()`
`199`	`202`	`}`
`200`	`203`
`201`	`204`	`/**`
	`205`	`+ * @group legacy`
`202`	`206`	`* @expectedException \InvalidArgumentException`
`203`	`207`	`*/`
`204`	`208`	`public function testInvalidValidatorInterface()`