Add release & environment setting to test command

Fixes #560

Author: stayallive

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Fix incorrect `release` and `environment` values when using the `sentry:test` command
+
 ## 2.12.0
 
 - Add support for normalized route names when using [Laravel Lumen](https://lumen.laravel.com/docs/9.x) (#449)

src/Sentry/Laravel/Console/TestCommand.php

@@ -11,6 +11,7 @@
 use Sentry\State\HubInterface;
 use Sentry\Tracing\SpanContext;
 use Sentry\Tracing\TransactionContext;
+use Throwable;
 
 class TestCommand extends Command
 {
@@ -54,9 +55,20 @@ public function handle(): int
 
         $dsn = $this->option('dsn');
 
+        $laravelClient = null;
+
+        try {
+            $laravelClient = app(HubInterface::class)->getClient();
+        } catch (Throwable $e) {
+            // Ignore any errors related to getting the client from the Laravel container
+            // These errors will surface later in the process but we should not crash here
+        }
+
         // If the DSN was not passed as option to the command we use the registered client to get the DSN from the Laravel config
         if ($dsn === null) {
-            $dsnObject = app(HubInterface::class)->getClient()->getOptions()->getDsn();
+            $dsnObject = $laravelClient === null
+                ? null
+                : $laravelClient->getOptions()->getDsn();
 
             if ($dsnObject !== null) {
                 $dsn = (string)$dsnObject;
@@ -77,6 +89,8 @@ public function handle(): int
         try {
             $clientBuilder = ClientBuilder::create([
                 'dsn' => $dsn,
+                'release' => $laravelClient === null ? null : $laravelClient->getOptions()->getRelease(),
+                'environment' => $laravelClient === null ? null : $laravelClient->getOptions()->getEnvironment(),
                 'traces_sample_rate' => 1.0,
             ]);
         } catch (Exception $e) {