wip

Author: driesvints

src/Console/KeysCommand.php

@@ -29,7 +29,7 @@ class KeysCommand extends Command
     /**
      * Execute the console command.
      *
-     * @return void
+     * @return int
      */
     public function handle()
     {
@@ -40,6 +40,8 @@ public function handle()
 
         if ((file_exists($publicKey) || file_exists($privateKey)) && ! $this->option('force')) {
             $this->error('Encryption keys already exist. Use the --force option to overwrite them.');
+
+            return 1;
         } else {
             if (class_exists(LegacyRSA::class)) {
                 $keys = (new LegacyRSA)->createKey($this->input ? (int) $this->option('length') : 4096);
@@ -55,5 +57,7 @@ public function handle()
 
             $this->info('Encryption keys generated successfully.');
         }
+
+        return 0;
     }
 }

tests/Feature/KeysCommandTest.php

@@ -0,0 +1,29 @@
+<?php
+
+namespace Laravel\Passport\Tests\Feature;
+
+use Mockery as m;
+
+class KeysCommandTest extends PassportTestCase
+{
+    protected function tearDown(): void
+    {
+        m::close();
+
+        @unlink(self::PUBLIC_KEY);
+        @unlink(self::PRIVATE_KEY);
+    }
+
+    public function testPrivateAndPublicKeysAreGenerated()
+    {
+        $this->assertFileExists(self::PUBLIC_KEY);
+        $this->assertFileExists(self::PRIVATE_KEY);
+    }
+
+    public function testPrivateAndPublicKeysShouldNotBeGeneratedTwice()
+    {
+        $this->artisan('passport:keys')
+            ->assertFailed()
+            ->expectsOutput('Encryption keys already exist. Use the --force option to overwrite them.');
+    }
+}

tests/Feature/PassportTestCase.php

@@ -12,7 +12,7 @@ abstract class PassportTestCase extends TestCase
 {
     use RefreshDatabase;
 
-    const KEYS = __DIR__.'/keys';
+    const KEYS = __DIR__.'/../keys';
     const PUBLIC_KEY = self::KEYS.'/oauth-public.key';
     const PRIVATE_KEY = self::KEYS.'/oauth-private.key';
 
@@ -24,6 +24,8 @@ protected function setUp(): void
 
         Passport::routes();
 
+        Passport::loadKeysFrom(self::KEYS);
+
         @unlink(self::PUBLIC_KEY);
         @unlink(self::PRIVATE_KEY);