feat(series): redefine page views so that prerenders (that never are …

…turned into navigation events) are no longer counted

series.js

@@ -27,7 +27,16 @@ import { reclassifyAcquisition } from './utils.js';
  * @param {Bundle} bundle a series of events that belong to the same page view
  * @returns {number} the number of page views
  */
-export const pageViews = (bundle) => bundle.weight;
+export const pageViews = (bundle) => {
+  const isBundle = true;
+  const isPrerender = bundle.events.find((evt) =>
+    evt.checkpoint === 'prerender'
+  );
+  const isPrerenderThenNavigate = bundle.events.find((evt) =>
+    evt.checkpoint === 'navigate'
+    && evt.target === 'prerender');
+  return isBundle && (!isPrerender || isPrerenderThenNavigate) ? bundle.weight : 0;
+};
 
 /**
  * A visit is a page view that does not follow an internal link. This means a visit starts

test/series.test.js

@@ -20,7 +20,10 @@ import {
 
 describe('series:pageViews', () => {
   it('pageViews:bare', () => {
-    assert.equal(pageViews({ weight: 1 }), 1);
+    assert.equal(pageViews({ weight: 1, events: [] }), 1);
+    assert.equal(pageViews({ weight: 1, events: [{ checkpoint: 'prerender' }] }), 0);
+    assert.equal(pageViews({ weight: 1, events: [{ checkpoint: 'navigate', target: 'prerender' }] }), 1);
+    assert.equal(pageViews({ weight: 1, events: [{ checkpoint: 'prerender' }, { checkpoint: 'navigate', target: 'prerender' }] }), 1);
   });
 
   it('pageViews:DataChunks', () => {