wip(tests): unit tests enhancements

Author: FranckFreiburger

tests/basic.test.js

@@ -1,51 +1,54 @@
-const {	defaultFiles,	getPage } = require('./testsTools.js');
+const { defaultFiles, createPage } = require('./testsTools.js');
 
-test('text-only tamplate', async () => {
 
-	const page = await getPage({
+test('text-only template', async () => {
+
+	const { page, output } = await createPage({
 		files: {
 			...defaultFiles,
 			'/component.vue': `
 				<template>
-					Hello World ! <b id="done"/>
+					Hello World !
 				</template>
 			`
 		}
 	});
 
-	await page.waitForSelector('#done');
 	await expect(page.$eval('#app', el => el.textContent.trim())).resolves.toBe('Hello World !');
+
 	await page.close();
 });
 
 
 test('properly detect and reports errors in template', async () => {
 
-	const page = await getPage({
+	const { page, output } = await createPage({
 		files: {
 			...defaultFiles,
 			'/component.vue': `
 				<template>
-					Hello World ! {{ msg } <b id="done"/>
+					Hello World ! {{ msg }
 				</template>
 			`
 		}
 	});
 
-	await page.waitForSelector('#done');
-	await expect(page.console.some(e => e.type === 'error' && e.content[0] === 'SFC template')).toBe(true);
+	//await page.waitForSelector('#done');
+	await expect(output.some(e => e.type === 'error' && e.content[0] === 'SFC template')).toBe(true);
+	//await new Promise(resolve => page.on('consoleValues', ({ type, args }) => type === 'error' && args[0] === 'SFC template' && resolve() ));
+
 	await page.close();
 });
 
 
 test('properly detect and reports errors in style', async () => {
 
-	const page = await getPage({
+	const { page, output } = await createPage({
 		files: {
 			...defaultFiles,
 			'/component.vue': `
 				<template>
-					Hello World ! <b id="done"/>
+					Hello World !
 				</template>
 				<style>
 					body
@@ -56,15 +59,15 @@ test('properly detect and reports errors in style', async () => {
 		}
 	});
 
-	await page.waitForSelector('#done');
-	await expect(page.console.some(e => e.type === 'error' && e.content[0] === 'SFC style')).toBe(true);
+	await expect(output.some(e => e.type === 'error' && e.content[0] === 'SFC style')).toBe(true);
+
 	await page.close();
 });
 
 
 test('properly detect and reports errors in script', async () => {
 
-	const page = await getPage({
+	const { page, output } = await createPage({
 		files: {
 			...defaultFiles,
 			'/component.vue': `
@@ -75,21 +78,21 @@ test('properly detect and reports errors in script', async () => {
 		}
 	});
 
-	await new Promise(resolve => setTimeout(resolve, 500));
-	await expect(page.console.some(e => e.type === 'error' && e.content[0] === 'SFC script')).toBe(true);
+	await expect(output.some(e => e.type === 'error' && e.content[0] === 'SFC script')).toBe(true);
+
 	await page.close();
 });
 
 
 
 test('all blocks', async () => {
 
-	const page = await getPage({
+	const { page, output } = await createPage({
 		files: {
 			...defaultFiles,
 			'/component.vue': `
 				<template>
-					<b>Hello {{ msg }} !</b><b id="done"/>
+					<b>Hello {{ msg }} !</b>
 				</template>
 				<style scoped>
 					b { color: red; }
@@ -103,15 +106,15 @@ test('all blocks', async () => {
 		}
 	});
 
-	await new Promise(resolve => setTimeout(resolve, 500));
-	await expect(!page.console.some(e => e.type === 'error')).toBe(true);
+	await expect(!output.some(e => e.type === 'error')).toBe(true);
+
 	await page.close();
 });
 
 
 test('invalid require', async () => {
 
-	const page = await getPage({
+	const { page, output } = await createPage({
 		files: {
 			...defaultFiles,
 			'/component.vue': `
@@ -122,8 +125,8 @@ test('invalid require', async () => {
 		}
 	});
 
-	await new Promise(resolve => setTimeout(resolve, 1500));
-	await expect(page.console.some(e => e.type === 'pageerror' && String(e.content).includes('HttpError') )).toBe(true);
+	await expect(output.some(e => e.type === 'pageerror' && String(e.content).includes('HttpError') )).toBe(true);
+
 	await page.close();
 });
 

tests/testsTools.js

@@ -6,7 +6,7 @@ const mime = require('mime-types');
 
 const local = new URL('http://local/');
 
-async function getPage({ files }) {
+async function createPage({ files }) {
 
 	async function getFile(url, encoding) {
 
@@ -53,27 +53,32 @@ async function getPage({ files }) {
 		}
 	});
 
-	page.console = [];
+	const output = [];
 
-	//page.on('console', msg => page.console.push({ type: msg.type(), content: msg.text() }) );
-	page.on('console', async msg => page.console.push({ type: msg.type(), content: await Promise.all( msg.args().map(e => e.jsonValue()) ) }) );
-	page.on('pageerror', error => page.console.push({ type: 'pageerror', content: error }) );
+	page.on('console', async msg => output.push({ type: msg.type(), content: await Promise.all( msg.args().map(e => e.jsonValue()) ) }) );
+	page.on('pageerror', error => output.push({ type: 'pageerror', content: error }) );
 
 	page.on('error', msg => console.log('ERROR', msg));
 
 	//page.done = new Promise(resolve => page.exposeFunction('_done', resolve));
 
 	await page.goto(new URL('/index.html', local));
 
-	return page;
+	await new Promise(resolve => setTimeout(resolve, 250));
+
+	return { page, output };
 }
 
 let browser;
 
 beforeAll(async () => {
 
+	if ( browser )
+		return browser;
+
 	browser = await puppeteer.launch({
-		headless: !false,
+		headless: true,
+		pipe: true,
 		args: [
 			'--incognito',
 			'--disable-gpu',
@@ -94,11 +99,12 @@ afterAll(async () => {
 
 const defaultFiles = {
 	'/vue3-sfc-loader.js': Fs.readFileSync(Path.join(__dirname, '../dist/vue3-sfc-loader.js'), { encoding: 'utf-8' }),
+	'/vue': Fs.readFileSync(Path.join(__dirname, '../node_modules/vue/dist/vue.global.js'), { encoding: 'utf-8' }),
 	'/index.html': `
 		<!DOCTYPE html>
 		<html><body>
 			<div id="app"></div>
-			<script src="https://unpkg.com/vue@next"></script>
+			<script src="vue"></script>
 			<script src="vue3-sfc-loader.js"></script>
 			<script>
 
@@ -114,9 +120,6 @@ const defaultFiles = {
 							name: {
 								value: this.constructor.name,
 							},
-							statusCode: {
-								value: res.statusCode,
-							},
 							url: {
 								value: url,
 							},
@@ -165,5 +168,5 @@ const defaultFiles = {
 
 module.exports = {
 	defaultFiles,
-	getPage,
+	createPage,
 }