Upgrade Optimus package

composer.json

@@ -34,11 +34,11 @@
         "docs": "https://github.com/cybercog/laravel-optimus/wiki"
     },
     "require": {
-        "php": "^7.1.3|^8.0",
+        "php": "^8.0",
         "graham-campbell/manager": "^4.0",
-        "illuminate/contracts": "^5.5|^6.0|^7.0|^8.0|^9.0",
-        "illuminate/support": "^5.5|^6.0|^7.0|^8.0|^9.0",
-        "jenssegers/optimus": "^0.2.2"
+        "illuminate/contracts": "^9.0",
+        "illuminate/support": "^9.0",
+        "jenssegers/optimus": "^1.0"
     },
     "require-dev": {
         "graham-campbell/testbench": "^5.0",

config/optimus.php

@@ -35,6 +35,11 @@
     | configuration has been included, but you may add as many connections as
     | you would like.
     |
+    | Laravel Optimus depends on jenssegers/optimus, we need three values:
+    | - A large `prime` number lower than 2147483647
+    | - The `inverse` prime so that (PRIME * INVERSE) & MAXID == 1
+    | - A large `random` integer lower than 2147483647
+    |
     */
 
     'connections' => [

phpunit.xml.dist

@@ -13,7 +13,6 @@
     <testsuites>
         <testsuite name="Package Test Suite">
             <directory suffix=".php">tests/</directory>
-            <exclude>tests/database/factories/</exclude>
         </testsuite>
     </testsuites>
     <php>

src/OptimusFactory.php

@@ -13,7 +13,7 @@
 
 namespace Cog\Laravel\Optimus;
 
-use Illuminate\Support\Arr;
+use InvalidArgumentException;
 use Jenssegers\Optimus\Optimus;
 
 class OptimusFactory
@@ -35,16 +35,36 @@ public function make(array $config): Optimus
      * Get the configuration data.
      *
      * @param array $config
-     * @return array
+     * @return array<string, int>
      *
-     * @throws \InvalidArgumentException
+     * @throws InvalidArgumentException
      */
     protected function getConfig(array $config): array
     {
+        $prime = $config['prime'] ?? null;
+        $inverse = $config['inverse'] ?? null;
+        $random = $config['random'] ?? null;
+
+        if (is_int($prime) === false) {
+            throw new InvalidArgumentException(
+                "Optimus `prime` value must be integer but `$prime` provided",
+            );
+        }
+        if (is_int($prime) === false) {
+            throw new InvalidArgumentException(
+                "Optimus `inverse` value must be integer `$inverse` provided",
+            );
+        }
+        if (is_int($random) === false) {
+            throw new InvalidArgumentException(
+                "Optimus `random` value must be integer `$random` provided",
+            );
+        }
+
         return [
-            'prime' => Arr::get($config, 'prime', 0),
-            'inverse' => Arr::get($config, 'inverse', 0),
-            'random' => Arr::get($config, 'random', 0),
+            'prime' => $prime,
+            'inverse' => $inverse,
+            'random' => $random,
         ];
     }
 

tests/OptimusFactoryTest.php

@@ -23,9 +23,9 @@ public function testMakeStandard(): void
         $factory = $this->getOptimusFactory();
 
         $return = $factory->make([
-            'prime' => 'your-prime-integer',
-            'inverse' => 'your-inverse-integer',
-            'random' => 'your-random-integer',
+            'prime' => 0,
+            'inverse' => 0,
+            'random' => 0,
         ]);
 
         $this->assertInstanceOf(Optimus::class, $return);

tests/Providers/ServiceProviderTest.php

@@ -35,6 +35,11 @@ public function testOptimusManagerIsInjectable(): void
 
     public function testBindings(): void
     {
+        $appConfig = $this->app->get('config');
+        $appConfig->set('optimus.connections.main.prime', 0);
+        $appConfig->set('optimus.connections.main.inverse', 0);
+        $appConfig->set('optimus.connections.main.random', 0);
+
         $this->assertIsInjectable(Optimus::class);
 
         $original = $this->app['optimus.connection'];