diff options
author | 2023-07-07 16:01:23 -0400 | |
---|---|---|
committer | 2023-07-07 16:01:23 -0400 | |
commit | 9807e4dc22355f0b3b2ff65b0724a95af8e9702d (patch) | |
tree | 723c86670761f97a1bc7b89141521a740e37aff8 /packages/integrations/prefetch/test | |
parent | c135633bf6a84e751249920cba9009f0e394e29a (diff) | |
download | astro-9807e4dc22355f0b3b2ff65b0724a95af8e9702d.tar.gz astro-9807e4dc22355f0b3b2ff65b0724a95af8e9702d.tar.zst astro-9807e4dc22355f0b3b2ff65b0724a95af8e9702d.zip |
Updates prefetch integration to add "only prefetch link on hover/mouseover/focus" option (#6585)
* modifies prefetch to add the option to only prefetch certain pages on hover
* adds new pages to the test website to showcase prefetch-intent functionality
* adds tests to verify prefetch-intent behavior
* adds changelog
* waits until networkidle to check if the prefetching worked instead of waiting on a specific url load
* allows intentSelector to be either a string or array of strings
* Revert "allows intentSelector to be either a string or array of strings"
This reverts commit b0268eb0d5220ad2b08b0b7aee23f43e4caf879f.
* fixes the multiple selector logic and adds tests
* updates docs to include new prefetch-intent integration
* Update packages/integrations/prefetch/README.md
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
* Update packages/integrations/prefetch/README.md
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
* Update packages/integrations/prefetch/README.md
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
* Update .changeset/little-cars-exist.md
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
* Update packages/integrations/prefetch/README.md
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
---------
Co-authored-by: Erika <3019731+Princesseuh@users.noreply.github.com>
Co-authored-by: Nate Moore <natemoo-re@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>
Diffstat (limited to 'packages/integrations/prefetch/test')
6 files changed, 267 insertions, 5 deletions
diff --git a/packages/integrations/prefetch/test/basic-prefetch.test.js b/packages/integrations/prefetch/test/basic-prefetch.test.js index 6bab8a478..5fab536aa 100644 --- a/packages/integrations/prefetch/test/basic-prefetch.test.js +++ b/packages/integrations/prefetch/test/basic-prefetch.test.js @@ -37,18 +37,44 @@ test.describe('Basic prefetch', () => { ).toBeTruthy(); }); }); + + test.describe('prefetches rel="prefetch-intent" links only on hover', () => { + test('prefetches /uses on hover', async ({ page, astro }) => { + const requests = []; + + page.on('request', (request) => requests.push(request.url())); + + await page.goto(astro.resolveUrl('/')); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/uses')), + '/uses was not prefetched' + ).toBeFalsy(); + + await page.hover('a[href="/uses"]'); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/uses')), + '/uses was prefetched on hover' + ).toBeTruthy(); + }); + }); }); test.describe('build', () => { let previewServer; - test.beforeAll(async ({ astro }) => { + test.beforeEach(async ({ astro }) => { await astro.build(); previewServer = await astro.preview(); }); // important: close preview server (free up port and connection) - test.afterAll(async () => { + test.afterEach(async () => { await previewServer.stop(); }); @@ -74,5 +100,31 @@ test.describe('Basic prefetch', () => { ).toBeTruthy(); }); }); + + test.describe('prefetches rel="prefetch-intent" links only on hover', () => { + test('prefetches /uses on hover', async ({ page, astro }) => { + const requests = []; + + page.on('request', (request) => requests.push(request.url())); + + await page.goto(astro.resolveUrl('/')); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/uses')), + '/uses was not prefetched' + ).toBeFalsy(); + + await page.hover('a[href="/uses"]'); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/uses')), + '/uses was prefetched on hover' + ).toBeTruthy(); + }); + }); }); }); diff --git a/packages/integrations/prefetch/test/custom-selectors.test.js b/packages/integrations/prefetch/test/custom-selectors.test.js index 803e1dc3b..ac15d7d5f 100644 --- a/packages/integrations/prefetch/test/custom-selectors.test.js +++ b/packages/integrations/prefetch/test/custom-selectors.test.js @@ -2,11 +2,19 @@ import { expect } from '@playwright/test'; import { testFactory } from './test-utils.js'; import prefetch from '../dist/index.js'; +const customSelector = 'a[href="/contact"]'; +const customIntentSelector = [ + 'a[href][rel~="custom-intent"]', + 'a[href][rel~="customer-intent"]', + 'a[href][rel~="customest-intent"]', +]; + const test = testFactory({ root: './fixtures/basic-prefetch/', integrations: [ prefetch({ - selector: 'a[href="/contact"]', + selector: customSelector, + intentSelector: customIntentSelector, }), ], }); @@ -50,13 +58,13 @@ test.describe('Custom prefetch selectors', () => { test.describe('build', () => { let previewServer; - test.beforeAll(async ({ astro }) => { + test.beforeEach(async ({ astro }) => { await astro.build(); previewServer = await astro.preview(); }); // important: close preview server (free up port and connection) - test.afterAll(async () => { + test.afterEach(async () => { await previewServer.stop(); }); @@ -84,3 +92,167 @@ test.describe('Custom prefetch selectors', () => { }); }); }); + +test.describe('Custom prefetch intent selectors', () => { + test.describe('dev', () => { + let devServer; + + test.beforeEach(async ({ astro }) => { + devServer = await astro.startDevServer(); + }); + + test.afterEach(async () => { + await devServer.stop(); + }); + + test('prefetches custom intent links only on hover if provided an array', async ({ + page, + astro, + }) => { + const requests = []; + + page.on('request', (request) => requests.push(request.url())); + + await page.goto(astro.resolveUrl('/')); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/terms')), + '/terms was not prefetched initially' + ).toBeFalsy(); + + expect( + requests.includes(astro.resolveUrl('/conditions')), + '/conditions was not prefetched initially' + ).toBeFalsy(); + + const combinedIntentSelectors = customIntentSelector.join(','); + const intentElements = await page.$$(combinedIntentSelectors); + + for (const element of intentElements) { + await element.hover(); + } + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/terms')), + '/terms was prefetched on hover' + ).toBeTruthy(); + expect( + requests.includes(astro.resolveUrl('/conditions')), + '/conditions was prefetched on hover' + ).toBeTruthy(); + }); + + test('prefetches custom intent links only on hover if provided a string', async ({ + page, + astro, + }) => { + const requests = []; + + page.on('request', (request) => requests.push(request.url())); + + await page.goto(astro.resolveUrl('/')); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/terms')), + '/terms was not prefetched initially' + ).toBeFalsy(); + + await page.hover(customIntentSelector[0]); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/terms')), + '/terms was prefetched on hover' + ).toBeTruthy(); + }); + }); + + test.describe('build', () => { + let previewServer; + + test.beforeEach(async ({ astro }) => { + await astro.build(); + previewServer = await astro.preview(); + }); + + // important: close preview server (free up port and connection) + test.afterEach(async () => { + await previewServer.stop(); + }); + + test('prefetches custom intent links only on hover if provided an array', async ({ + page, + astro, + }) => { + const requests = []; + + page.on('request', (request) => requests.push(request.url())); + + await page.goto(astro.resolveUrl('/')); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/terms')), + '/terms was not prefetched initially' + ).toBeFalsy(); + + expect( + requests.includes(astro.resolveUrl('/conditions')), + '/conditions was not prefetched initially' + ).toBeFalsy(); + + const combinedIntentSelectors = customIntentSelector.join(','); + const intentElements = await page.$$(combinedIntentSelectors); + + for (const element of intentElements) { + await element.hover(); + } + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/terms')), + '/terms was prefetched on hover' + ).toBeTruthy(); + expect( + requests.includes(astro.resolveUrl('/conditions')), + '/conditions was prefetched on hover' + ).toBeTruthy(); + }); + + test('prefetches custom intent links only on hover if provided a string', async ({ + page, + astro, + }) => { + const requests = []; + + page.on('request', (request) => requests.push(request.url())); + + await page.goto(astro.resolveUrl('/')); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/terms')), + '/terms was not prefetched initially' + ).toBeFalsy(); + + await page.hover(customIntentSelector[0]); + + await page.waitForLoadState('networkidle'); + + expect( + requests.includes(astro.resolveUrl('/terms')), + '/terms was prefetched on hover' + ).toBeTruthy(); + }); + }); +}); diff --git a/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/conditions.astro b/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/conditions.astro new file mode 100644 index 000000000..345abaa38 --- /dev/null +++ b/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/conditions.astro @@ -0,0 +1,11 @@ +--- +--- + +<html> +<head> +<title>Conditions</title> +</head> +<body> + <h1>Conditions</h1> +</body> +</html> diff --git a/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/index.astro b/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/index.astro index 58a864552..b78e39553 100644 --- a/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/index.astro +++ b/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/index.astro @@ -19,11 +19,16 @@ <li> <a href="/admin">Admin</a> </li> + <li> + <a href="/uses" rel="prefetch-intent">Uses</a> + </li> </ul> </nav> <footer> <a href="/contact" rel="prefetch">Contact</a> + <a href="/terms" rel="custom-intent">Terms</a> + <a href="/conditions" rel="customer-intent">Terms</a> </footer> </body> </html> diff --git a/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/terms.astro b/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/terms.astro new file mode 100644 index 000000000..a0fcc1004 --- /dev/null +++ b/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/terms.astro @@ -0,0 +1,11 @@ +--- +--- + +<html> +<head> +<title>Terms</title> +</head> +<body> + <h1>Terms</h1> +</body> +</html> diff --git a/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/uses.astro b/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/uses.astro new file mode 100644 index 000000000..15b050061 --- /dev/null +++ b/packages/integrations/prefetch/test/fixtures/basic-prefetch/src/pages/uses.astro @@ -0,0 +1,11 @@ +--- +--- + +<html> +<head> +<title>Uses</title> +</head> +<body> + <h1>Uses</h1> +</body> +</html> |