Refactored version

README.md

@@ -48,7 +48,7 @@ isTrue(false);               // returns null - WTF?
 And now the **same code with Enum** instead of Constants:
 
 ```php
-class MyBoolean extends \PetrKnap\Php\Enum\AbstractEnum
+class MyBoolean extends \PetrKnap\Php\Enum\Enum
 {
     protected function members()
     {
@@ -82,7 +82,7 @@ isTrue(false);                 // uncaught type error - OK
 
 ### Enum declaration
 ```php
-class DayOfWeekEnum extends \PetrKnap\Php\Enum\AbstractEnum
+class DayOfWeekEnum extends \PetrKnap\Php\Enum\Enum
 {
     protected function members()
     {

composer.json

@@ -24,11 +24,11 @@
     "phpunit/phpunit": "4.*"
   },
   "scripts": {
-    "test": [
-      "./vendor/bin/phpunit"
+    "tests/": [
+      "sudo docker run -v $(pwd):/app --rm php:5.3-cli bash -c 'cd /app && php ./vendor/bin/phpunit'"
     ],
     "post-autoload-dump": [
-      "@test"
+      "@tests/"
     ]
   },
   "autoload": {

src/Enum/AbstractEnum.php → src/Enum/Enum.php

@@ -2,6 +2,8 @@
 
 namespace PetrKnap\Php\Enum;
 
+use PetrKnap\Php\Enum\Exception\EnumNotFoundException;
+
 /**
  * Abstract enum object
  *
@@ -10,7 +12,7 @@
  * @package PetrKnap\Php\Enum
  * @license https://github.com/petrknap/php-enum/blob/master/LICENSE MIT
  */
-abstract class AbstractEnum
+abstract class Enum implements EnumInterface
 {
     /**
      * @var self[][]
@@ -34,7 +36,6 @@ abstract class AbstractEnum
 
     /**
      * @param string $memberName
-     * @throws EnumException
      */
     protected function __construct($memberName)
     {
@@ -77,19 +78,15 @@ public static function __callStatic($memberName, array $args)
     }
 
     /**
-     * Returns member name
-     *
-     * @return string
+     * @inheritdoc
      */
     public function getName()
     {
         return $this->memberName;
     }
 
     /**
-     * Returns member value
-     *
-     * @return mixed
+     * @inheritdoc
      */
     public function getValue()
     {
@@ -137,18 +134,17 @@ private function exists($memberName)
     /**
      * @param string $memberName
      * @return mixed
-     * @throws EnumException
+     * @throws EnumNotFoundException
      */
     private function get($memberName)
     {
         if (!$this->exists($memberName)) {
-            throw new EnumException(
+            throw new EnumNotFoundException(
                 sprintf(
                     "%s does not exist in %s",
                     $memberName,
                     get_called_class()
-                ),
-                EnumException::OUT_OF_RANGE
+                )
             );
         }
 
@@ -158,21 +154,28 @@ private function get($memberName)
     /**
      * @param mixed $value
      * @return self
-     * @throws EnumException
+     * @throws EnumNotFoundException
      */
-    public static function findByValue($value)
+    public static function getEnumByValue($value)
     {
         foreach (self::getMembers() as $n => $v) {
             if ($value === $v) {
                 return self::__callStatic($n, array());
             }
         }
-        throw new EnumException(
+        throw new EnumNotFoundException(
             sprintf(
                 "Value not found in %s",
                 get_called_class()
-            ),
-            EnumException::OUT_OF_RANGE
+            )
         );
     }
+
+    /**
+     * @return string
+     */
+    public function __toString()
+    {
+        return sprintf("%s::%s", get_called_class(), $this->getName());
+    }
 }

src/Enum/EnumInterface.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace PetrKnap\Php\Enum;
+
+interface EnumInterface
+{
+    /**
+     * Returns member name
+     *
+     * @return string
+     */
+    public function getName();
+
+    /**
+     * Returns member value
+     *
+     * @return mixed
+     */
+    public function getValue();
+}

src/Enum/Exception/EnumException.php

@@ -0,0 +1,8 @@
+<?php
+
+namespace PetrKnap\Php\Enum\Exception;
+
+abstract class EnumException extends \Exception
+{
+    // Empty class
+}

src/Enum/Exception/EnumNotFoundException.php

@@ -0,0 +1,8 @@
+<?php
+
+namespace PetrKnap\Php\Enum\Exception;
+
+class EnumNotFoundException extends EnumException
+{
+    // Empty class
+}

tests/Enum/EnumTest.php

@@ -2,59 +2,60 @@
 
 namespace PetrKnap\Php\Enum\Test;
 
-use PetrKnap\Php\Enum\EnumException;
+use PetrKnap\Php\Enum\Enum;
+use PetrKnap\Php\Enum\Exception\EnumException;
+use PetrKnap\Php\Enum\Exception\EnumNotFoundException;
 use PetrKnap\Php\Enum\Test\EnumTest\MyBoolean;
 
 class EnumTest extends \PHPUnit_Framework_TestCase
 {
-    public function goodKeyProvider()
+    /**
+     * @dataProvider dataCallStaticsWorks
+     * @param string $name
+     * @param mixed|EnumException $expectedValue
+     */
+    public function testCallStaticsWorks($name, $expectedValue)
     {
-        return array(array("MY_TRUE", 1), array("MY_FALSE", 2));
+        if ($expectedValue instanceof EnumException) {
+            $this->setExpectedException(get_class($expectedValue));
+        }
+
+        $this->assertSame($expectedValue, MyBoolean::__callStatic($name, array())->getValue());
     }
 
-    public function wrongKeyProvider()
+    public function dataCallStaticsWorks()
     {
-        return array(array("MY_NULL"), array("MY_VOID"));
+        return array(
+            array("MY_TRUE", 1),
+            array("MY_FALSE", 2),
+            array("MY_NULL", new EnumNotFoundException())
+        );
     }
 
     /**
-     * @covers       EnumMock::__callStatic
-     * @dataProvider goodKeyProvider
-     *
-     * @param string $name
+     * @dataProvider dataGetEnumByValueWorks
      * @param mixed $value
+     * @param Enum|EnumException $expectedEnum
      */
-    public function testMagicConstruction_GoodKey($name, $value)
+    public function testGetEnumByValueWorks($value, $expectedEnum)
     {
-        /** @var MyBoolean $enum */
-        $enum = MyBoolean::$name();
+        if ($expectedEnum instanceof EnumException) {
+            $this->setExpectedException(get_class($expectedEnum));
+        }
 
-        $this->assertInstanceOf(MyBoolean::getClass(), $enum);
-        $this->assertSame($name, $enum->getName());
-        $this->assertSame($value, $enum->getValue());
+        $this->assertSame($expectedEnum, MyBoolean::getEnumByValue($value));
     }
 
-    /**
-     * @covers       EnumMock::__callStatic
-     * @dataProvider wrongKeyProvider
-     *
-     * @param string $name
-     */
-    public function testMagicConstruction_WrongKey($name)
+    public function dataGetEnumByValueWorks()
     {
-        $this->setExpectedException(
-            get_class(new EnumException()),
-            "",
-            EnumException::OUT_OF_RANGE
+        return array(
+            array(1, MyBoolean::MY_TRUE()),
+            array(2, MyBoolean::MY_FALSE()),
+            array(3, new EnumNotFoundException())
         );
-
-        MyBoolean::$name();
     }
 
-    /**
-     * @covers EnumMock::__callStatic
-     */
-    public function testComparable()
+    public function testComparableWorks()
     {
         $this->assertSame(MyBoolean::MY_TRUE(), MyBoolean::MY_TRUE());
         $this->assertNotSame(MyBoolean::MY_TRUE(), MyBoolean::MY_FALSE());
@@ -63,41 +64,32 @@ public function testComparable()
         $this->assertFalse(MyBoolean::MY_TRUE() == MyBoolean::MY_FALSE());
     }
 
-    /**
-     * @covers EnumMock::getMembers
-     * @runInSeparateProcess
-     */
-    public function testGetMembers()
+    public function testGetMembersWorks()
     {
-        $members = MyBoolean::getMembers();
-
-        $this->assertInternalType("array", $members);
-        $this->assertCount(2, $members);
-        $this->assertArrayHasKey("MY_TRUE", $members);
-        $this->assertEquals(1, $members["MY_TRUE"]);
-        $this->assertArrayHasKey("MY_FALSE", $members);
-        $this->assertEquals(2, $members["MY_FALSE"]);
+        $this->assertEquals(array(
+            "MY_TRUE" => 1,
+            "MY_FALSE" => 2
+        ), MyBoolean::getMembers());
     }
 
     /**
-     * @dataProvider dataFindByValue
-     * @param mixed $value
-     * @param mixed $expected
+     * @dataProvider dataToStringWorks
+     * @param Enum $enum
+     * @param string $expectedString
      */
-    public function testFindByValue($value, $expected)
+    public function testToStringWorks(Enum $enum, $expectedString)
     {
-        if ($expected instanceof \Exception) {
-            $this->setExpectedException(get_class($expected));
-        }
-        $this->assertSame($expected, MyBoolean::findByValue($value));
+        $this->assertSame($expectedString, $enum->__toString());
+        $this->assertSame($expectedString, (string) $enum);
+        $this->assertSame($expectedString, "{$enum}");
+        $this->assertSame($expectedString, $enum . "");
     }
 
-    public function dataFindByValue()
+    public function dataToStringWorks()
     {
         return array(
-            array(1, MyBoolean::MY_TRUE()),
-            array(2, MyBoolean::MY_FALSE()),
-            array(3, new EnumException())
+            array(MyBoolean::MY_TRUE(), MyBoolean::getClass() . "::MY_TRUE"),
+            array(MyBoolean::MY_FALSE(), MyBoolean::getClass() . "::MY_FALSE")
         );
     }
 }

tests/Enum/EnumTest/MyBoolean.php

@@ -2,13 +2,13 @@
 
 namespace PetrKnap\Php\Enum\Test\EnumTest;
 
-use PetrKnap\Php\Enum\AbstractEnum;
+use PetrKnap\Php\Enum\Enum;
 
 /**
  * @method static MyBoolean MY_TRUE()
  * @method static MyBoolean MY_FALSE()
  */
-class MyBoolean extends AbstractEnum
+class MyBoolean extends Enum
 {
     protected function members()
     {