added has name trait test

Author: transistive

src/Traits/HasNameTrait.php

@@ -2,13 +2,16 @@
 
 namespace WikibaseSolutions\CypherDSL\Traits;
 
+use LogicException;
 use function bin2hex;
 use function ceil;
 use function openssl_random_pseudo_bytes;
 use function substr;
 
 trait HasNameTrait
 {
+    use ErrorTrait;
+
     private string $name;
 
     /**
@@ -28,6 +31,10 @@ public static function automaticVariableLength(): int
      */
     public function getName(): string
     {
+        if (!isset($this->name)) {
+            throw new LogicException('Name is not yet configured. Please call `configureName` to setup a name.');
+        }
+
         return $this->name;
     }
 

tests/Unit/Traits/HasNameTraitTest.php

@@ -0,0 +1,42 @@
+<?php
+
+namespace WikibaseSolutions\CypherDSL\Tests\Unit\Traits;
+
+use LogicException;
+use PHPUnit\Framework\TestCase;
+use WikibaseSolutions\CypherDSL\Traits\HasNameTrait;
+
+class HasNameTraitTest extends TestCase
+{
+    private $hasName;
+
+    public function setUp(): void
+    {
+        $this->hasName = new class {
+            use HasNameTrait {
+                configureName as public;
+                generateName as public;
+            }
+        };
+    }
+
+    public function testHasName(): void
+    {
+        $this->expectException(LogicException::class);
+
+        $this->hasName->getName();
+    }
+
+    public function testGenerateName(): void
+    {
+        $this->assertMatchesRegularExpression('/var\w{32}/', $this->hasName->generateName());
+        $this->assertMatchesRegularExpression('/x\w{16}/', $this->hasName->generateName('x', 16));
+    }
+
+    public function testConfigureName(): void
+    {
+        $this->hasName->configureName(null, 'y', 16);
+
+        $this->assertMatchesRegularExpression('/y\w{16}/', $this->hasName->getName());
+    }
+}

tests/Unit/Traits/HasVariableTraitTest.php

@@ -0,0 +1,42 @@
+<?php
+
+namespace WikibaseSolutions\CypherDSL\Tests\Unit\Traits;
+
+use PHPUnit\Framework\TestCase;
+use WikibaseSolutions\CypherDSL\Traits\HasVariableTrait;
+
+class HasVariableTraitTest extends TestCase
+{
+    private $hasVariable;
+
+    public function setUp(): void
+    {
+        $this->hasVariable = new class {
+            use HasVariableTrait;
+        };
+    }
+
+    public function testDefaultGeneration(): void
+    {
+        self::assertNull($this->hasVariable->getVariable());
+        self::assertNotNull($this->hasVariable->getName());
+
+        self::assertMatchesRegularExpression('/var\w{32}/', $this->hasVariable->getVariable()->getName());
+    }
+
+    public function testNamed(): void
+    {
+        $this->hasVariable->named('x');
+
+        self::assertSame($this->hasVariable->getVariable(), $this->hasVariable->getName());
+        self::assertEquals('x', $this->hasVariable->getVariable()->getName());
+    }
+
+    public function testSetName(): void
+    {
+        $this->hasVariable->setName('x');
+
+        self::assertSame($this->hasVariable->getVariable(), $this->hasVariable->getName());
+        self::assertEquals('x', $this->hasVariable->getVariable()->getName());
+    }
+}