update incorrect comment change for-loop (#413)

* update incorrect comment change for-loop

* don't close test browser if it never started

* prettier

* empty commit

Author: peterbe

scripts/e2e.js

@@ -40,7 +40,7 @@ function openUrl(url, ...options) {
 }
 
 // Simplest form of opening
-openUrl('https://minimalcss.app/').then(spawned => {
+openUrl('https://minimalcss.app/').then((spawned) => {
   const css = spawned.stdout.toString();
   assert(
     css.length > 7000 && css.length < 10000,
@@ -50,7 +50,7 @@ openUrl('https://minimalcss.app/').then(spawned => {
 
 // Open with -o
 openUrl('https://developer.mozilla.org/', '-o', '/tmp/mdn.css').then(
-  spawned => {
+  (spawned) => {
     const stdout = spawned.stdout.toString();
     assert(!stdout.trim(), 'Output should be empty');
     const css = fs.readFileSync('/tmp/mdn.css').toString();
@@ -62,7 +62,7 @@ openUrl('https://developer.mozilla.org/', '-o', '/tmp/mdn.css').then(
 );
 
 // With verbose output
-openUrl('https://developer.mozilla.org/', '--verbose').then(spawned => {
+openUrl('https://developer.mozilla.org/', '--verbose').then((spawned) => {
   const css = spawned.stdout.toString();
   assert(/\/\*\nGenerated /.test(css), 'Expected verbose leading comment');
   assert(

src/run.js

@@ -448,9 +448,7 @@ const minimalcss = async (options) => {
   const skippedUrls = new Set();
 
   try {
-    // Note! This opens one URL at a time synchronous
-    for (let i = 0; i < urls.length; i++) {
-      const pageUrl = urls[i];
+    for (const pageUrl of urls) {
       const page = await browser.newPage();
       if (!enableServiceWorkers) {
         await page._client.send('ServiceWorker.disable');

tests/main.test.js

@@ -41,7 +41,9 @@ beforeAll(async () => {
 
 afterAll(async () => {
   await fastify.close();
-  await browser.close();
+  if (browser) {
+    await browser.close();
+  }
 });
 
 test('handles relative paths', async () => {