Part 1 of a better error message when measure without navigating (#3980)

sitespeedio · Oct 13, 2023 · 75a1214 · 75a1214
1 parent 6fa10f9
commit 75a1214
Showing 1 changed file with 21 additions and 7 deletions.
diff --git a/lib/plugins/browsertime/index.js b/lib/plugins/browsertime/index.js
@@ -281,16 +281,30 @@ export default class BrowsertimePlugin extends SitespeedioPlugin {
                 try {
                   run.har = pickAPage(result.har, harIndex);
                 } catch (harError) {
-                  log.error('Couldnt get the right page for the HAR', harError);
-                  super.sendMessage(
-                    'error',
-                    'Could not get the right page for the HAR, the page is missing',
-                    {
+                  if (message.type === 'browsertime.navigationScripts') {
+                    const message =
+                      'There are pages that misses data. This is in almost all cases caused by that you try to measure a page view but you do not navigate to a new page.';
+                    log.error(message, harError);
+                    super.sendMessage('error', message, {
                       url,
                       runIndex,
                       iteration: runIndex + 1
-                    }
-                  );
+                    });
+                  } else {
+                    log.error(
+                      'Couldnt get the right page for the HAR',
+                      harError
+                    );
+                    super.sendMessage(
+                      'error',
+                      'Could not get the right page for the HAR, the page is missing',
+                      {
+                        url,
+                        runIndex,
+                        iteration: runIndex + 1
+                      }
+                    );
+                  }
                 }
               } else {
                 // If we do not have a HAR, use browser info from the result