- hex6Digits to uniqueHex6Digits

- test coverage

Author: Simon Bigelmayr

src/LightcyclerSampleSheet/RandomHexGenerator.php

@@ -4,12 +4,20 @@
 
 class RandomHexGenerator
 {
-    public function hex6Digits(): string
+    /** @var list<string> */
+    private array $generatedHexCodes = [];
+
+    public function uniqueHex6Digits(): string
     {
-        $randomNumber = mt_rand(0, 0xFFFFFF);
-        $hexString = dechex($randomNumber);
-        $paddedHexString = str_pad($hexString, 6, '0', STR_PAD_LEFT);
+        do {
+            $randomNumber = mt_rand(0, 0xFFFFFF);
+            $hexString = dechex($randomNumber);
+            $paddedHexString = str_pad($hexString, 6, '0', STR_PAD_LEFT);
+            $uniqueHexCode = strtoupper($paddedHexString);
+        } while (in_array($uniqueHexCode, $this->generatedHexCodes, true));
+
+        $this->generatedHexCodes[] = $uniqueHexCode;
 
-        return strtoupper($paddedHexString);
+        return $uniqueHexCode;
     }
 }

tests/LightcyclerSampleSheet/RandomHexGeneratorTest.php

@@ -0,0 +1,29 @@
+<?php declare(strict_types=1);
+
+namespace MLL\Utils\Tests\LightcyclerSampleSheet;
+
+use MLL\Utils\LightcyclerSampleSheet\RandomHexGenerator;
+use PHPUnit\Framework\TestCase;
+
+final class RandomHexGeneratorTest extends TestCase
+{
+    public function testGeneratesValidHexCode(): void
+    {
+        $generator = new RandomHexGenerator();
+        $hexCode = $generator->uniqueHex6Digits();
+
+        self::assertMatchesRegularExpression('/^[A-F0-9]{6}$/', $hexCode);
+    }
+
+    public function testGeneratesUniqueHexCodes(): void
+    {
+        $generator = new RandomHexGenerator();
+        $hexCodes = [];
+
+        for ($i = 0; $i < 1000; ++$i) {
+            $hexCodes[] = $generator->uniqueHex6Digits();
+        }
+
+        self::assertCount(1000, array_unique($hexCodes));
+    }
+}

tests/LightcyclerSampleSheet/RelativeQuantificationSheetTest.php

@@ -5,6 +5,8 @@
 use Illuminate\Support\Collection;
 use MLL\Utils\LightcyclerSampleSheet\RelativeQuantificationSample;
 use MLL\Utils\LightcyclerSampleSheet\RelativeQuantificationSheet;
+use MLL\Utils\Microplate\Coordinates;
+use MLL\Utils\Microplate\CoordinateSystem12x8;
 use MLL\Utils\StringUtil;
 use PHPUnit\Framework\TestCase;
 
@@ -17,7 +19,7 @@ public function testGenerate(): void
             'B1' => new RelativeQuantificationSample('Sample 2', '498-640', '4899D1', null),
             'C1' => new RelativeQuantificationSample('Sample 3', '498-640', '8528B9', null),
             'D1' => new RelativeQuantificationSample('Sample 4', '498-640', '8E05D9', null),
-            'E1' => new RelativeQuantificationSample('Sample 5', '498-640', '4080A5', null),
+            'E1' => new RelativeQuantificationSample('Sample 5', '498-640', '4080A5', Coordinates::fromString('C12', new CoordinateSystem12x8())),
         ]);
 
         $sheet = new RelativeQuantificationSheet();
@@ -29,7 +31,7 @@ public function testGenerate(): void
 B1\t"Sample 2"\t""\t498-640\t$004899D1
 C1\t"Sample 3"\t""\t498-640\t$008528B9
 D1\t"Sample 4"\t""\t498-640\t$008E05D9
-E1\t"Sample 5"\t""\t498-640\t$004080A5
+E1\t"Sample 5"\t"C12"\t498-640\t$004080A5
 
 EOT;