Merge branch '5.1' into 5.2

* 5.1:
  Use ::class keyword when possible

Author: fabpot

Tests/Metadata/InMemoryMetadataStoreTest.php

@@ -77,7 +77,7 @@ public function testGetMetadata()
 
     public function testGetMetadataWithUnknownType()
     {
-        $this->expectException('Symfony\Component\Workflow\Exception\InvalidArgumentException');
+        $this->expectException(\Symfony\Component\Workflow\Exception\InvalidArgumentException::class);
         $this->expectExceptionMessage('Could not find a MetadataBag for the subject of type "bool".');
         $this->store->getMetadata('title', true);
     }

Tests/Validator/WorkflowValidatorTest.php

@@ -14,7 +14,7 @@ class WorkflowValidatorTest extends TestCase
 
     public function testWorkflowWithInvalidNames()
     {
-        $this->expectException('Symfony\Component\Workflow\Exception\InvalidDefinitionException');
+        $this->expectException(\Symfony\Component\Workflow\Exception\InvalidDefinitionException::class);
         $this->expectExceptionMessage('All transitions for a place must have an unique name. Multiple transitions named "t1" where found for place "a" in workflow "foo".');
         $places = range('a', 'c');
 

Tests/WorkflowTest.php

@@ -24,7 +24,7 @@ class WorkflowTest extends TestCase
 
     public function testGetMarkingWithInvalidStoreReturn()
     {
-        $this->expectException('Symfony\Component\Workflow\Exception\LogicException');
+        $this->expectException(\Symfony\Component\Workflow\Exception\LogicException::class);
         $this->expectExceptionMessage('The value returned by the MarkingStore is not an instance of "Symfony\Component\Workflow\Marking" for workflow "unnamed".');
         $subject = new Subject();
         $workflow = new Workflow(new Definition([], []), $this->getMockBuilder(MarkingStoreInterface::class)->getMock());
@@ -34,7 +34,7 @@ public function testGetMarkingWithInvalidStoreReturn()
 
     public function testGetMarkingWithEmptyDefinition()
     {
-        $this->expectException('Symfony\Component\Workflow\Exception\LogicException');
+        $this->expectException(\Symfony\Component\Workflow\Exception\LogicException::class);
         $this->expectExceptionMessage('The Marking is empty and there is no initial place for workflow "unnamed".');
         $subject = new Subject();
         $workflow = new Workflow(new Definition([], []), new MethodMarkingStore());
@@ -44,7 +44,7 @@ public function testGetMarkingWithEmptyDefinition()
 
     public function testGetMarkingWithImpossiblePlace()
     {
-        $this->expectException('Symfony\Component\Workflow\Exception\LogicException');
+        $this->expectException(\Symfony\Component\Workflow\Exception\LogicException::class);
         $this->expectExceptionMessage('Place "nope" is not valid for workflow "unnamed".');
         $subject = new Subject();
         $subject->setMarking(['nope' => 1]);
@@ -171,7 +171,7 @@ public function testCanWithSameNameTransition()
 
     public function testBuildTransitionBlockerListReturnsUndefinedTransition()
     {
-        $this->expectException('Symfony\Component\Workflow\Exception\UndefinedTransitionException');
+        $this->expectException(UndefinedTransitionException::class);
         $this->expectExceptionMessage('Transition "404 Not Found" is not defined for workflow "unnamed".');
         $definition = $this->createSimpleWorkflowDefinition();
         $subject = new Subject();