Cleanup visual-tests script

Author: MichaelDeBoey

packages/components/scripts/visual-tests.js

@@ -2,14 +2,16 @@ const execa = require('execa');
 const git = require('simple-git')();
 
 const hasChanges = async () => {
-  const result = await git.diffSummary(['master...HEAD']);
-  if (result.files.length) return true;
-  return false;
+  const { files } = await git.diffSummary(['master...HEAD']);
+
+  return files.length > 0;
 };
 
 const run = autoAccept => {
   const options = ['chromatic:base'];
-  if (autoAccept) options.push('--auto-accept-changes');
+  if (autoAccept) {
+    options.push('--auto-accept-changes');
+  }
 
   execa('yarn', options).stdout.pipe(process.stdout);
 };
@@ -18,12 +20,14 @@ const start = async () => {
   const { current: currentBranch } = await git.branch();
 
   if (currentBranch === 'master') {
-    run(true);
-  } else {
-    const changes = await hasChanges();
-    if (changes) run();
-    else console.warn('No changes found in branch, skipping visual tests');
+    return run(true);
   }
+
+  if (await hasChanges()) {
+    return run();
+  }
+
+  console.warn('No changes found in branch, skipping visual tests');
 };
 
 start();