Fix tests

Author: AydinHassan

src/Factory/ResultRendererFactory.php

@@ -80,6 +80,6 @@ public function create(ResultInterface $result): ResultRendererInterface
      */
     protected function isImplementationNameOfClass(string $implementationName, string $className): bool
     {
-        return is_string($implementationName) && is_subclass_of($implementationName, $className);
+        return is_subclass_of($implementationName, $className);
     }
 }

src/Result/Cgi/CgiResult.php

@@ -4,7 +4,7 @@
 
 use ArrayIterator;
 use IteratorAggregate;
-use PhpSchool\PhpWorkshop\Result\Cli\ResultInterface;
+use PhpSchool\PhpWorkshop\Result\Cgi\ResultInterface;
 use PhpSchool\PhpWorkshop\Result\ResultGroupInterface;
 
 /**

src/UserState.php

@@ -8,9 +8,8 @@
  */
 class UserState
 {
-
     /**
-     * @var string
+     * @var string|null
      */
     private $currentExercise;
 
@@ -57,9 +56,9 @@ public function setCurrentExercise(string $exercise): void
     /**
      * Get the current exercise name.
      *
-     * @return string
+     * @return string|null
      */
-    public function getCurrentExercise(): string
+    public function getCurrentExercise(): ?string
     {
         return $this->currentExercise;
     }

test/Factory/ResultRendererFactoryTest.php

@@ -33,28 +33,6 @@ public function testRegisterRendererRequiresResultRendererInterface(): void
         $factory->registerRenderer($resultClass, $rendererClass);
     }
 
-    public function testRegisterRendererRequiresResultClassToBeString(): void
-    {
-        $this->expectException(InvalidArgumentException::class);
-
-        $resultClass = $this->createMock(ResultInterface::class);
-        $rendererClass = get_class($this->createMock(ResultRendererInterface::class));
-        $factory = new ResultRendererFactory();
-        
-        $factory->registerRenderer($resultClass, $rendererClass);
-    }
-
-    public function testRegisterRendererRequiresRendererClassToBeString(): void
-    {
-        $this->expectException(InvalidArgumentException::class);
-
-        $resultClass = get_class($this->createMock(ResultInterface::class));
-        $rendererClass = $this->createMock(ResultRendererInterface::class);
-        $factory = new ResultRendererFactory();
-        
-        $factory->registerRenderer($resultClass, $rendererClass);
-    }
-
     public function testCreateRequiresMappingToClassName(): void
     {
         $this->expectException(RuntimeException::class);