🐛 Fix client and env info normalization (#588)

percy · Oct 19, 2021 · ee04cc7 · ee04cc7
1 parent b213554
commit ee04cc7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/packages/core/src/snapshot.js b/packages/core/src/snapshot.js
@@ -20,8 +20,8 @@ export function getSnapshotConfig(percy, options) {
   let log = logger('core:snapshot');
 
   // migrate deprecated snapshot config options
-  let { clientInfo, environmentInfo, ...opts } = options;
-  let snapshot = PercyConfig.migrate(opts, '/snapshot');
+  let { clientInfo, environmentInfo, ...snapshot } = (
+    PercyConfig.migrate(options, '/snapshot'));
 
   // throw an error when missing required widths
   if (!(snapshot.widths ?? percy.config.snapshot.widths)?.length) {

diff --git a/packages/core/test/snapshot.test.js b/packages/core/test/snapshot.test.js
@@ -209,13 +209,13 @@ describe('Snapshot', () => {
     await percy.snapshot({
       name: 'test snapshot',
       url: 'http://localhost:8000',
-      clientInfo: 'test client info',
-      environmentInfo: 'test env info',
+      client_info: 'test client info',
+      environment_info: 'test env info',
       widths: [400, 1200],
       discovery: {
-        allowedHostnames: ['example.com'],
-        requestHeaders: { 'X-Foo': 'Bar' },
-        disableCache: true
+        'allowed-hostnames': ['example.com'],
+        'request-headers': { 'X-Foo': 'Bar' },
+        'disable-cache': true
       },
       additionalSnapshots: [
         { prefix: 'foo ', waitForTimeout: 100 },