From c70a6e6b152e4a0d9def229219a9b2e927e84985 Mon Sep 17 00:00:00 2001
From: Aiden Bai <aiden.bai05@gmail.com>
Date: Tue, 5 Oct 2021 17:43:19 -0700
Subject: [PATCH] refactor(benchmarks): organize imports

---
 benchmarks/benchmark.mjs                      |  2 +-
 benchmarks/main.jsx                           | 51 ++++++++-----------
 .../suites/appendManyRowsToLargeTable.jsx     |  2 +-
 benchmarks/suites/clearRows.jsx               |  2 +-
 benchmarks/suites/createManyRows.jsx          |  2 +-
 benchmarks/suites/createRows.jsx              |  2 +-
 benchmarks/suites/partialUpdate.jsx           |  2 +-
 benchmarks/suites/removeRow.jsx               |  2 +-
 benchmarks/suites/replaceAllRows.jsx          |  2 +-
 benchmarks/suites/swapRows.jsx                |  2 +-
 10 files changed, 30 insertions(+), 39 deletions(-)

diff --git a/benchmarks/benchmark.mjs b/benchmarks/benchmark.mjs
index d3fbee5b8f..a5ab417341 100644
--- a/benchmarks/benchmark.mjs
+++ b/benchmarks/benchmark.mjs
@@ -1,5 +1,5 @@
-import _ from 'lodash';
 import Benchmark from 'benchmark';
+import _ from 'lodash';
 
 // avoid `Cannot read property 'parentNode' of undefined` error in runScript
 const script = document.createElement('script');
diff --git a/benchmarks/main.jsx b/benchmarks/main.jsx
index d1a2def15e..6b3afdb3c1 100644
--- a/benchmarks/main.jsx
+++ b/benchmarks/main.jsx
@@ -1,7 +1,5 @@
 import 'style.css';
-
-import { m, createElement, style, kebab, patch } from '../src/index';
-
+import { createElement, patch } from '../src/index';
 import appendManyRowsToLargeTable from './suites/appendManyRowsToLargeTable';
 import clearRows from './suites/clearRows';
 import createManyRows from './suites/createManyRows';
@@ -39,6 +37,19 @@ const suites = [
     }),
 );
 
+const generateLogMarkdown = () =>
+  `Benchmarks compiled on \`${new Date().toLocaleString()}\`\n\n**UA:** ${
+    navigator.userAgent
+  }\n\n---\n\n${[...logs]
+    .map((logGroup) => {
+      const logGroupCopy = [...logGroup];
+      const title = logGroupCopy.shift();
+      return `${title.replace('Running: ', '').split(' - ')[0]}\n\n\`\`\`\n${logGroupCopy.join(
+        '\n',
+      )}\n\`\`\`\n`;
+    })
+    .join('\n')}`;
+
 const vnode = () => (
   <div>
     {suites.map((suite) => {
@@ -72,19 +83,7 @@ const vnode = () => (
         </button>{' '}
         <button
           onclick={async () => {
-            navigator.clipboard.writeText(
-              `Benchmarks compiled on \`${new Date().toLocaleString()}\`\n\n**UA:** ${
-                navigator.userAgent
-              }\n\n---\n\n${[...logs]
-                .map((logGroup) => {
-                  const logGroupCopy = [...logGroup];
-                  const title = logGroupCopy.shift();
-                  return `${
-                    title.replace('Running: ', '').split(' - ')[0]
-                  }\n\n\`\`\`\n${logGroupCopy.join('\n')}\n\`\`\`\n`;
-                })
-                .join('\n')}`,
-            );
+            navigator.clipboard.writeText(generateLogMarkdown());
           }}
         >
           📋
@@ -94,7 +93,11 @@ const vnode = () => (
       <div style={{ paddingTop: '20px' }}>
         {logs.map(
           (logGroup) =>
-            logGroup.length && m('pre', undefined, [m('code', undefined, [logGroup.join('\n')])]),
+            logGroup.length && (
+              <pre>
+                <code>{logGroup.join('\n')}</code>
+              </pre>
+            ),
         )}
       </div>
     </details>
@@ -114,19 +117,7 @@ const vnode = () => (
           </button>{' '}
           <button
             onclick={async () => {
-              navigator.clipboard.writeText(
-                `Benchmarks compiled on \`${new Date().toLocaleString()}\`\n\n**UA:** ${
-                  navigator.userAgent
-                }\n\n---\n\n${[...logs]
-                  .map((logGroup) => {
-                    const logGroupCopy = [...logGroup];
-                    const title = logGroupCopy.shift();
-                    return `${
-                      title.replace('Running: ', '').split(' - ')[0]
-                    }\n\n\`\`\`\n${logGroupCopy.join('\n')}\n\`\`\`\n`;
-                  })
-                  .join('\n')}`,
-              );
+              navigator.clipboard.writeText(generateLogMarkdown());
             }}
           >
             📋
diff --git a/benchmarks/suites/appendManyRowsToLargeTable.jsx b/benchmarks/suites/appendManyRowsToLargeTable.jsx
index ba387c568a..d4089f51d7 100644
--- a/benchmarks/suites/appendManyRowsToLargeTable.jsx
+++ b/benchmarks/suites/appendManyRowsToLargeTable.jsx
@@ -3,8 +3,8 @@
  * @description appending 1,000 to a table of 10,000 rows.
  */
 
-import { Suite } from '../benchmark';
 import { createElement, patch } from '../../src/index';
+import { Suite } from '../benchmark';
 import { buildData } from '../data';
 
 const data = buildData(10000);
diff --git a/benchmarks/suites/clearRows.jsx b/benchmarks/suites/clearRows.jsx
index 452ce85ed3..f9a8a2d275 100644
--- a/benchmarks/suites/clearRows.jsx
+++ b/benchmarks/suites/clearRows.jsx
@@ -3,8 +3,8 @@
  * @description clearing a table with 1,000 rows
  */
 
-import { Suite } from '../benchmark';
 import { createElement, patch } from '../../src/index';
+import { Suite } from '../benchmark';
 import { buildData } from '../data';
 
 const data = buildData(1000);
diff --git a/benchmarks/suites/createManyRows.jsx b/benchmarks/suites/createManyRows.jsx
index 3085353c16..4bb49410b0 100644
--- a/benchmarks/suites/createManyRows.jsx
+++ b/benchmarks/suites/createManyRows.jsx
@@ -3,8 +3,8 @@
  * @description creating 10,000 rows
  */
 
-import { Suite } from '../benchmark';
 import { createElement, patch } from '../../src/index';
+import { Suite } from '../benchmark';
 import { buildData } from '../data';
 
 const data = buildData(10000);
diff --git a/benchmarks/suites/createRows.jsx b/benchmarks/suites/createRows.jsx
index 453eb4c3a2..7f9dc898fc 100644
--- a/benchmarks/suites/createRows.jsx
+++ b/benchmarks/suites/createRows.jsx
@@ -3,8 +3,8 @@
  * @description creating 1,000 rows
  */
 
-import { Suite } from '../benchmark';
 import { createElement, patch } from '../../src/index';
+import { Suite } from '../benchmark';
 import { buildData } from '../data';
 
 const data = buildData(10000);
diff --git a/benchmarks/suites/partialUpdate.jsx b/benchmarks/suites/partialUpdate.jsx
index 25d6a953b7..8b7f89b7bc 100644
--- a/benchmarks/suites/partialUpdate.jsx
+++ b/benchmarks/suites/partialUpdate.jsx
@@ -3,8 +3,8 @@
  * @description updating every 10th row for 1,000 rows
  */
 
-import { Suite } from '../benchmark';
 import { createElement, patch, UPDATE } from '../../src/index';
+import { Suite } from '../benchmark';
 import { buildData } from '../data';
 
 const data = buildData(1000);
diff --git a/benchmarks/suites/removeRow.jsx b/benchmarks/suites/removeRow.jsx
index 48e0e1ba99..ffc168a697 100644
--- a/benchmarks/suites/removeRow.jsx
+++ b/benchmarks/suites/removeRow.jsx
@@ -3,8 +3,8 @@
  * @description removing one row
  */
 
+import { createElement, DELETE, patch } from '../../src/index';
 import { Suite } from '../benchmark';
-import { createElement, patch, DELETE } from '../../src/index';
 import { buildData } from '../data';
 
 const data = buildData(1000);
diff --git a/benchmarks/suites/replaceAllRows.jsx b/benchmarks/suites/replaceAllRows.jsx
index da5be6a20a..0c8511c127 100644
--- a/benchmarks/suites/replaceAllRows.jsx
+++ b/benchmarks/suites/replaceAllRows.jsx
@@ -3,8 +3,8 @@
  * @description updating all 1,000 rows
  */
 
-import { Suite } from '../benchmark';
 import { createElement, patch } from '../../src/index';
+import { Suite } from '../benchmark';
 import { buildData } from '../data';
 
 const shuffleArray = (array) => {
diff --git a/benchmarks/suites/swapRows.jsx b/benchmarks/suites/swapRows.jsx
index a946c0b2a0..f52c5bf7d0 100644
--- a/benchmarks/suites/swapRows.jsx
+++ b/benchmarks/suites/swapRows.jsx
@@ -3,8 +3,8 @@
  * @description swap 2 rows for table with 1,000 rows
  */
 
+import { createElement, patch, UPDATE } from '../../src/index';
 import { Suite } from '../benchmark';
-import { m, createElement, patch, VFlags, UPDATE } from '../../src/index';
 import { buildData } from '../data';
 
 const data = buildData(1000);