[Workflow] Fixed validator tests and used static function to create workflows

Author: HeahDude

src/Symfony/Component/Workflow/Exception/InvalidDefinitionException.php

@@ -16,6 +16,6 @@
  *
  * @author Tobias Nyholm <[email protected]>
  */
-class InvalidDefinitionException extends \LogicException implements ExceptionInterface
+class InvalidDefinitionException extends LogicException
 {
 }

src/Symfony/Component/Workflow/Tests/Validator/WorkflowValidatorTest.php

@@ -7,15 +7,15 @@
 use Symfony\Component\Workflow\Transition;
 use Symfony\Component\Workflow\Validator\WorkflowValidator;
 
-class WorkflowValidatorTest extends WorkflowTest
+class WorkflowValidatorTest extends \PHPUnit_Framework_TestCase
 {
     /**
      * @expectedException \Symfony\Component\Workflow\Exception\InvalidDefinitionException
      * @expectedExceptionMessage The marking store of workflow "foo" can not store many places.
      */
     public function testSinglePlaceWorkflowValidatorAndComplexWorkflow()
     {
-        $definition = $this->createComplexWorkflow();
+        $definition = WorkflowTest::createComplexWorkflow();
 
         (new WorkflowValidator(true))->validate($definition, 'foo');
     }

src/Symfony/Component/Workflow/Tests/WorkflowTest.php

@@ -47,7 +47,6 @@ public function testGetMarkingWithEmptyDefinition()
     public function testGetMarkingWithImpossiblePlace()
     {
         $subject = new \stdClass();
-        $subject->marking = null;
         $subject->marking = array('nope' => true);
         $workflow = new Workflow(new Definition(array(), array()), new MultipleStateMarkingStore());
 
@@ -210,7 +209,7 @@ public function testGetEnabledTransitions()
         $this->assertSame('t5', $transitions[0]->getName());
     }
 
-    protected function createComplexWorkflow()
+    public static function createComplexWorkflow()
     {
         $builder = new DefinitionBuilder();