Merge branch 'develop'

This commit is contained in:
Sam Carlton 2021-05-16 16:05:55 -05:00
commit bf85fbcde1
2 changed files with 42 additions and 0 deletions

30
.github/workflows/run-ava-tests.js.yml vendored Normal file
View file

@ -0,0 +1,30 @@
# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
name: Run Ava Tests
on:
push:
branches: [ master ]
pull_request:
branches: [ master ]
jobs:
build:
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [15.x, 16.x]
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/
steps:
- uses: actions/checkout@v2
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}
- run: npm ci
- run: npm run generate
- run: npm test

View file

@ -65,6 +65,18 @@ test.before(async t => {
t.context.sitemapUrls = sitemap.urlset.url.map( tag => new URL( tag.loc ) )
})
test('Sitemap contains no double slashes in paths', (t) => {
// console.log('t.context.sitemapUrls', t.context.sitemapUrls)
const urlsWithDoubleSlashes = t.context.sitemapUrls.filter( url => url.pathname.includes('//') )
if ( urlsWithDoubleSlashes.length > 0) {
t.fail( `${ urlsWithDoubleSlashes.length } urls with doubles slashes found including ${ urlsWithDoubleSlashes[0] }` )
}
t.pass()
})
test('All Category pages have valid FAQPage structured data', async (t) => {
const categoryUrls = t.context.sitemapUrls.filter( url => url.pathname.startsWith('/kind/') )