update tests for cli metadata

Author: joshedney

build-reporter-plugin.js

@@ -62,11 +62,9 @@ class BugsnagBuildReporterPlugin {
     const optionalOpts = {
       autoAssignRelease: opts.build.autoAssignRelease,
       builderName: opts.build.builderName,
-      metadata: '{' +
-          Object.entries(opts.build.metadata)
-            .map(([key, value]) => `${key}=${value}`)
-            .join(',') +
-          '}',
+      metadata: Object.entries(opts.build.metadata)
+        .map(([key, value]) => `${key}=${value}`)
+        .join(','),
       provider: opts.build.sourceControl.provider,
       repository: opts.build.sourceControl.repository,
       revision: opts.build.sourceControl.revision,

test/build-reporter-plugin.test.mjs

@@ -21,7 +21,7 @@ test('BugsnagBuildReporterPlugin', t => {
 })
 
 test('it sends upon successful build', t => {
-  t.plan(3)
+  t.plan(4)
   const server = createServer((req, res) => {
     let body = ''
     req.on('data', (d) => { body += d })
@@ -37,6 +37,7 @@ test('it sends upon successful build', t => {
       t.ok(j, 'json body was received')
       t.equal(j.appVersion, '1.2.3', 'body should contain app version')
       t.equal(j.apiKey, 'YOUR_API_KEY', 'body should contain api key')
+      t.same(j.metadata, { foo: 'bar', baz: 'qux' }, 'body should contain correct metadata')
     })
   })
   server.listen()

test/fixtures/a/webpack.config.js

@@ -10,7 +10,11 @@ module.exports = {
     new BugsnagBuildReporterPlugin({
       apiKey: 'YOUR_API_KEY',
       appVersion: '1.2.3',
-      endpoint: `http://localhost:${process.env.PORT}`
+      endpoint: `http://localhost:${process.env.PORT}`,
+      metadata: {
+        foo: 'bar',
+        baz: 'qux'
+      }
     })
   ]
 }