Moooaare tests

src/Lens/Lens.php

@@ -49,11 +49,11 @@ public static function identity(): self
              */
             static fn ($s) => $s,
             /**
-             * @param I $s
-             * @param mixed $_
+             * @param I $_
+             * @param I $a
              * @returns I
              */
-            static fn ($s, $_) => $s
+            static fn ($_, $a) => $a
         );
     }
 

src/Reflect/Internal/reflect_object.php

@@ -2,20 +2,11 @@
 namespace VeeWee\Reflecta\Reflect;
 
 use ReflectionObject;
-use Throwable;
-use VeeWee\Reflecta\Reflect\Exception\UnreflectableException;
 
 /**
  * @psalm-internal VeeWee\Reflecta
- *
- *
- * @throws UnreflectableException
  */
 function reflect_object(object $object): ReflectionObject
 {
-    try {
-        return new ReflectionObject($object);
-    } catch (Throwable $previous) {
-        throw UnreflectableException::unknownClass(get_debug_type($object), $previous);
-    }
+    return new ReflectionObject($object);
 }

tests/unit/Lens/ComposeTest.php

@@ -11,8 +11,6 @@ final class ComposeTest extends TestCase
 {
     public function test_it_can_compose_lenses(): void
     {
-        static::markTestIncomplete('Identity doesnt seem to work as expected');
-
         $greetLens = index('greet');
         $messageLens = index('message');
         $composed = compose($greetLens, $messageLens);

tests/unit/Lens/LensTest.php

@@ -61,7 +61,7 @@ public function test_it_can_have_identity(): void
         $lens = Lens::identity();
 
         static::assertSame('hello', $lens->get('hello'));
-        static::assertSame('hello', $lens->set('hello', 'ignored'));
+        static::assertSame('hello', $lens->set('ignored', 'hello'));
     }
 
 

tests/unit/Reflect/InstantiateTest.php

@@ -3,6 +3,7 @@
 
 namespace VeeWee\Reflecta\UnitTests\Reflect;
 
+use Generator;
 use PHPUnit\Framework\TestCase;
 use VeeWee\Reflecta\Reflect\Exception\UnreflectableException;
 use VeeWee\Reflecta\TestFixtures\Boom;
@@ -12,12 +13,10 @@
 final class InstantiateTest extends TestCase
 {
 
-
     public function test_it_errors(): void
     {
-        static::markTestIncomplete('TODO : find final internal class');
         $this->expectException(UnreflectableException::class);
-        instantiate('internalfinalclass');
+        instantiate(Generator::class);
     }
 
     public function it_returns_instance()

tests/unit/Reflect/Internal/ReflectObjectTest.php

@@ -5,34 +5,27 @@
 
 use PHPUnit\Framework\TestCase;
 use ReflectionObject;
-use VeeWee\Reflecta\Reflect\Exception\UnreflectableException;
+use stdClass;
 use VeeWee\Reflecta\TestFixtures\X;
 use function VeeWee\Reflecta\Reflect\reflect_object;
 
 final class ReflectObjectTest extends TestCase
 {
-
-    public function test_it_errors(): void
-    {
-        static::markTestIncomplete('TODO : find an unreflectable object');
-
-        $this->expectException(UnreflectableException::class);
-        reflect_object(null);
-    }
-
-    public function it_returns_reflection_class()
+
+    public function test_it_returns_reflection_class()
     {
         $rc = reflect_object(new X());
 
         static::assertInstanceOf(ReflectionObject::class, $rc);
         static::assertSame(X::class, $rc->getName());
     }
 
-    public function it_can_deal_with_stdClass()
+
+    public function test_it_can_deal_with_std_class()
     {
-        $rc = reflect_object(new StdClass());
+        $rc = reflect_object(new stdClass());
 
         static::assertInstanceOf(ReflectionObject::class, $rc);
-        static::assertSame(StdClass::class, $rc->getName());
+        static::assertSame(stdClass::class, $rc->getName());
     }
 }