[PLAT-7801] Add early check for undefined apiKey

CHANGELOG.md

@@ -18,6 +18,7 @@
   - Small performance improvements to `Bugnag.start`
     [bugsnag-android#1680](https://github.com/bugsnag/bugsnag-android/pull/1680)
 - (plugin-react|plugin-vue|plugin-react-navigation|plugin-react-native-navigation) Set `@bugsnag/core` to be an optional peer dependency to avoid unmet peer dependency warnings [#1735](https://github.com/bugsnag/bugsnag-js/pull/1735)
+- (electron) Improved error message when no apiKey is provided to Bugsnag.start() [#1738](https://github.com/bugsnag/bugsnag-js/pull/1738)
 
 ## v7.16.4 (2022-05-03)
 

packages/electron/src/client/main.js

@@ -16,6 +16,12 @@ const { schema } = require('../config/main')
 Event.__type = 'electronnodejs'
 
 module.exports = (opts) => {
+  // check api key has been provided as we need it to create the FileStore
+  // which happens before the API key is validated
+  if (typeof opts.apiKey !== 'string') {
+    throw new Error('No Bugsnag API Key set')
+  }
+
   const filestore = new FileStore(
     opts.apiKey,
     electron.app.getPath('userCache'),

packages/electron/src/client/test/client.test-main.ts

@@ -0,0 +1,13 @@
+import createClient from '../main'
+
+describe('@bugsnag/electron client', () => {
+  describe('createClient', () => {
+    it('throws an error when an apiKey is not provided', () => {
+      expect(
+        () => {
+          createClient({})
+        }
+      ).toThrowError('No Bugsnag API Key set')
+    })
+  })
+})