commonize test

test/custom-elements/index.ts

@@ -4,7 +4,7 @@ import * as http from 'http';
 import { rollup } from 'rollup';
 import virtual from '@rollup/plugin-virtual';
 import puppeteer from 'puppeteer';
-import { addLineNumbers, loadConfig, loadSvelte, retryAsync } from '../helpers';
+import { addLineNumbers, loadConfig, loadSvelte, retryAsync, executeBrowserTest } from '../helpers';
 import { deepEqual } from 'assert';
 
 const page = `
@@ -124,39 +124,14 @@ describe('custom-elements', function() {
 				}
 			}
 
-			// NOTE: Chromium may exit with SIGSEGV, so retry in that case
+			browser = await executeBrowserTest(
-			let count = 0;
+				browser,
-			do {
+				launchPuppeteer,
-				count++;
+				assertWarnings,
-				try {
+				() => {
-					const page = await browser.newPage();
+					console.log(addLineNumbers(code));
-
-					page.on('console', (type) => {
-						console[type._type](type._text);
-					});
-
-					page.on('error', error => {
-						console.log('>>> an error happened');
-						console.error(error);
-					});
-
-					await page.goto('http://localhost:6789');
-					const result = await page.evaluate(() => test(document.querySelector('main')));
-					if (result) console.log(result);
 					assertWarnings();
-					await page.close();
+				});
-					break;
-				} catch (err) {
-					if (count === 5 || browser.isConnected()) {
-						console.log(addLineNumbers(code));
-						assertWarnings();
-						throw err;
-					}
-					console.debug(err.stack || err);
-					console.log('RESTARTING Chromium...');
-					browser = await launchPuppeteer();
-				}
-			} while (count <= 5);
 		});
 	});
 });

test/helpers.ts

@@ -291,3 +291,38 @@ export async function retryAsync<T>(fn: () => Promise<T>, maxAttempts: number =
 		}
 	}
 }
+
+// NOTE: Chromium may exit with SIGSEGV, so retry in that case
+export async function executeBrowserTest<T>(browser, launchPuppeteer: () => Promise<T>, additionalAssertion: () => void, onError: (err: Error) => void) {
+	let count = 0;
+	do {
+		count++;
+		try {
+			const page = await browser.newPage();
+
+			page.on('console', (type) => {
+				console[type._type](type._text);
+			});
+
+			page.on('error', error => {
+				console.log('>>> an error happened');
+				console.error(error);
+			});
+			await page.goto('http://localhost:6789');
+			const result = await page.evaluate(() => test(document.querySelector('main')));
+			if (result) console.log(result);
+			additionalAssertion();
+			await page.close();
+			break;
+		} catch (err) {
+			if (count === 5 || browser.isConnected()) {
+				onError(err);
+				throw err;
+			}
+			console.debug(err.stack || err);
+			console.log('RESTARTING Chromium...');
+			browser = await launchPuppeteer();
+		}
+	} while (count <= 5);
+	return browser;
+}

test/runtime-puppeteer/index.ts

@@ -10,7 +10,8 @@ import {
 	loadSvelte,
 	mkdirp,
 	prettyPrintPuppeteerAssertionError,
-	retryAsync
+	retryAsync,
+	executeBrowserTest
 } from '../helpers';
 import { deepEqual } from 'assert';
 
@@ -227,40 +228,16 @@ describe('runtime (puppeteer)', function() {
 					throw new Error('Received unexpected warnings');
 				}
 			}
-			// NOTE: Chromium may exit with SIGSEGV, so retry in that case
+
-			let count = 0;
+			browser = await executeBrowserTest(
-			do {
+				browser,
-				count++;
+				launchPuppeteer,
-				try {
+				assertWarnings,
-					const page = await browser.newPage();
+				(err) => {
-
+					failed.add(dir);
-					page.on('console', (type) => {
+					prettyPrintPuppeteerAssertionError(err.message);
-						console[type._type](type._text);
-					});
-
-					page.on('error', error => {
-						console.log('>>> an error happened');
-						console.error(error);
-					});
-
-					await page.goto('http://localhost:6789');
-					const result = await page.evaluate(() => test(document.querySelector('main')));
-					if (result) console.log(result);
 					assertWarnings();
-					await page.close();
+				});
-					break;
-				} catch (err) {
-					if (count === 5 || browser.isConnected()) {
-						failed.add(dir);
-						prettyPrintPuppeteerAssertionError(err.message);
-						assertWarnings();
-						throw err;
-					}
-					console.debug(err.stack || err);
-					console.log('RESTARTING Chromium...');
-					browser = await launchPuppeteer();
-				}
-			} while (count <= 5);
 		});
 	}