Enable Video Structured Data on benchmark pages

helpers/config-node.js

@@ -3,6 +3,7 @@ import fs from 'fs-extra'
 
 import pkg from '~/package.json'
 import { getSiteUrl } from '~/helpers/url.js'
+import { getRouteType } from '~/helpers/app-derived.js'
 
 
 export const siteUrl = getSiteUrl()
@@ -232,6 +233,10 @@ export class PageHead {
         return this.pageUrl.toString()
     }
 
+    get routeType () {
+        return getRouteType( this.pathname )
+    }
+
     get defaultMeta () {
         return nuxtHead.meta
     }
@@ -310,16 +315,24 @@ export class PageHead {
         ].join('')
     }
 
+
     get structuredDataMarkup () {
-        // console.log( 'this.structuredData', this.structuredData )
+        // console.log( 'this.routeType', this.routeType, this.pathname )
 
-        if ( this.structuredData === null ) return ''
+        const hasStructuredData = this.structuredData !== null
+        // Route is primarily for video
+        const videoRouteType = [ 'tv', 'benchmarks' ].includes( this.routeType )
 
+        if ( hasStructuredData && videoRouteType ) {
             const structuredDataJson = JSON.stringify( this.structuredData )
 
             return `<script type="application/ld+json">${ structuredDataJson }</script>`
         }
 
+
+        return ''
+    }
+
     get allHeadMarkup () {
         return [
             this.metaMarkup,

helpers/listing-page.js

@@ -183,10 +183,22 @@ export class ListingDetails {
     }
 
     get structuredData () {
+        // Normal video page with app links
         if ( this.type === 'video' ) {
             return buildVideoStructuredData( this.api, this.api.appLinks )
         }
 
+        // Benchmark page
+        if ( this.hasInitialVideo ) {
+            // Build app links with just the current app
+            const appLinks =  [ {
+                name: this.api.name,
+                endpoint: this.api.endpoint
+            } ]
+
+            return buildVideoStructuredData( this.initialVideo, appLinks )
+        }
+
         return null
     }
 

test/listings/index.js

@@ -40,7 +40,7 @@ const listingsCases = {
         endpoint: '/app/expressvpn/benchmarks/',
         apiEndpointPath: '/api/app/expressvpn.json',
         expectInitialVideo: true,
-        shouldHaveVideoStucturedData: false
+        shouldHaveVideoStucturedData: true
     },
 
     // Express VPN Benchmarks
@@ -193,11 +193,21 @@ test( 'Listings with videos have structured data', async t => {
     for ( const [ caseEndpoint, listingCase ] of listingCaseEntries ) {
 
         const listingDetails = listingsDetails[ caseEndpoint ]
+        const listingPageHead = new PageHead( {
+            ...listingDetails.headOptions,
+            pathname: caseEndpoint
+        })
 
         // Stop here if we're not expecting Video Structured Data
-        if ( !listingCase.shouldHaveVideoStucturedData ) continue
+        if ( !listingCase.shouldHaveVideoStucturedData ) {
+
+            // Check that the non-video listing doesn't have video structured data
+            t.assert( !listingPageHead.structuredDataMarkup.includes('VideoObject'), `${ caseEndpoint } has video structured data` )
+
+            continue
+        }
+
 
-        const listingPageHead = new PageHead( listingDetails.headOptions )
 
         // t.log('listingDetails.initialVideo', listingDetails.initialVideo)
         // t.log( 'caseEndpoint', caseEndpoint )