Remove useless Shape named constructors

CHANGELOG.md

@@ -7,6 +7,10 @@
 - Requires PHP `8.4`
 - Requires `innmind/filesystem:~9.0`
 
+### Removed
+
+- `Innmind\Graphviz\Edge\Shape` named constructors, use the cases directly
+
 ## 4.0.0 - 2025-04-18
 
 ### Added

src/Edge/Shape.php

@@ -20,105 +20,6 @@ enum Shape
     case tee;
     case vee;
 
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function box(): self
-    {
-        return self::box;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function crow(): self
-    {
-        return self::crow;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function curve(): self
-    {
-        return self::curve;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function icurve(): self
-    {
-        return self::icurve;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function diamond(): self
-    {
-        return self::diamond;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function dot(): self
-    {
-        return self::dot;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function inv(): self
-    {
-        return self::inv;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function none(): self
-    {
-        return self::none;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function normal(): self
-    {
-        return self::normal;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function tee(): self
-    {
-        return self::tee;
-    }
-
-    /**
-     * @psalm-pure
-     */
-    #[\NoDiscard]
-    public static function vee(): self
-    {
-        return self::vee;
-    }
-
     #[\NoDiscard]
     public function toString(): string
     {

tests/Edge/ShapeTest.php

@@ -19,8 +19,8 @@ public function testShape(): BlackBox\Proof
         return $this
             ->forAll($this->shapes())
             ->prove(function(string $shape): void {
-                $this->assertInstanceOf(Shape::class, Shape::$shape());
-                $this->assertSame($shape, Shape::$shape()->toString());
+                $this->assertInstanceOf(Shape::class, Shape::{$shape});
+                $this->assertSame($shape, Shape::{$shape}->toString());
             });
     }
 

tests/EdgeTest.php

@@ -71,10 +71,10 @@ public function testShaped()
             Name::of('b'),
         )
             ->shaped(
-                Shape::box(),
-                Shape::vee(),
-                Shape::tee(),
-                Shape::dot(),
+                Shape::box,
+                Shape::vee,
+                Shape::tee,
+                Shape::dot,
             );
 
         $this->assertSame(1, $edge->attributes()->size());
@@ -92,10 +92,10 @@ public function testShapedWhenBidirectional()
         )
             ->asBidirectional()
             ->shaped(
-                Shape::box(),
-                Shape::vee(),
-                Shape::tee(),
-                Shape::dot(),
+                Shape::box,
+                Shape::vee,
+                Shape::tee,
+                Shape::dot,
             );
 
         $this->assertSame(3, $edge->attributes()->size());