fix: check if api key is set in Honeybadger::event (#198)

subzero10 · web-flow · commit 9fb13908b7d2 · 2024-08-28T09:05:49.000+03:00
diff --git a/src/Honeybadger.php b/src/Honeybadger.php
@@ -216,7 +216,7 @@ public function clear(): Reporter
      */
     public function event($eventTypeOrPayload, array $payload = null): void
     {
-        if (!$this->config['events']['enabled']) {
+        if (empty($this->config['api_key']) || ! $this->config['events']['enabled']) {
             return;
         }
 
diff --git a/tests/LogEventHandlerTest.php b/tests/LogEventHandlerTest.php
@@ -30,6 +30,7 @@ public function it_formats_a_log_for_events_api()
     {
         $client = $this->createMock(HoneybadgerClient::class);
         $config = new Config([
+            'api_key' => '1234',
             'events' => [
                 'enabled' => true
             ]
@@ -68,6 +69,7 @@ public function it_ignores_logs_below_its_minimum_level()
     {
         $client = $this->createMock(HoneybadgerClient::class);
         $config = new Config([
+            'api_key' => '1234',
             'events' => [
                 'enabled' => true
             ]

Original file line number	Diff line number	Diff line change
`@@ -216,7 +216,7 @@ public function clear(): Reporter`
`216`	`216`	`*/`
`217`	`217`	`public function event($eventTypeOrPayload, array $payload = null): void`
`218`	`218`	`{`
`219`		`- if (!$this->config['events']['enabled']) {`
	`219`	`+ if (empty($this->config['api_key']) \|\| ! $this->config['events']['enabled']) {`
`220`	`220`	`return;`
`221`	`221`	`}`
`222`	`222`
Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,7 @@ public function it_formats_a_log_for_events_api()`
`30`	`30`	`{`
`31`	`31`	`$client = $this->createMock(HoneybadgerClient::class);`
`32`	`32`	`$config = new Config([`
	`33`	`+ 'api_key' => '1234',`
`33`	`34`	`'events' => [`
`34`	`35`	`'enabled' => true`
`35`	`36`	`]`
`@@ -68,6 +69,7 @@ public function it_ignores_logs_below_its_minimum_level()`
`68`	`69`	`{`
`69`	`70`	`$client = $this->createMock(HoneybadgerClient::class);`
`70`	`71`	`$config = new Config([`
	`72`	`+ 'api_key' => '1234',`
`71`	`73`	`'events' => [`
`72`	`74`	`'enabled' => true`
`73`	`75`	`]`