_Bastler/partey_workadventure
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Refactoring tests

Browse Source
This commit is contained in:
David Négrier 2021-12-06 11:21:18 +01:00
parent 05f805445a
commit 94762c86c5
3 changed files with 32 additions and 45 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tests/tests/reconnect.ts
View File

@ -3,7 +3,7 @@ import {assertLogMessage} from "./utils/log";
const fs = require('fs'); const fs = require('fs');
const Docker = require('dockerode'); const Docker = require('dockerode');
import { Selector } from 'testcafe'; import { Selector } from 'testcafe';
import {userAlice} from "./utils/roles"; import {login} from "./utils/roles";
import {findContainer, rebootBack, rebootPusher, resetRedis, startContainer, stopContainer} from "./utils/containers"; import {findContainer, rebootBack, rebootPusher, resetRedis, startContainer, stopContainer} from "./utils/containers";
fixture `Reconnection` fixture `Reconnection`
@ -22,7 +22,7 @@ test("Test that connection can succeed even if WorkAdventure starts while pusher
await startContainer(container); await startContainer(container);
await t.useRole(userAlice); await login(t, 'http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/mousewheel.json');
t.ctx.passed = true; t.ctx.passed = true;
}).after(async t => { }).after(async t => {

36
tests/tests/utils/roles.ts
View File

@ -1,31 +1,15 @@
import { Role } from 'testcafe'; import { Role } from 'testcafe';
export const userAliceOnPage = (url: string) => Role(url, async t => { export function login(t: TestController, url: string, userName: string = "Alice", characterNumber: number = 2) {
await t t = t
.typeText('input[name="loginSceneName"]', 'Alice') .navigateTo(url)
.click('button.loginSceneFormSubmit') .typeText('input[name="loginSceneName"]', userName)
.click('button.selectCharacterButtonRight') .click('button.loginSceneFormSubmit');
.click('button.selectCharacterButtonRight')
.click('button.selectCharacterSceneFormSubmit')
.click('button.letsgo');
});
for (let i = 0; i < characterNumber; i++) {
t = t.click('button.selectCharacterButtonRight');
}
export const userAlice = Role('http://play.workadventure.localhost/', async t => { return t.click('button.selectCharacterSceneFormSubmit')
await t
.typeText('input[name="loginSceneName"]', 'Alice')
.click('button.loginSceneFormSubmit')
.click('button.selectCharacterButtonRight')
.click('button.selectCharacterButtonRight')
.click('button.selectCharacterSceneFormSubmit')
.click('button.letsgo'); .click('button.letsgo');
}); }
export const userBob = Role('http://play.workadventure.localhost/', async t => {
await t
.typeText('input[name="loginSceneName"]', 'Bob')
.click('button.loginSceneFormSubmit')
.click('button.selectCharacterButtonRight')
.click('button.selectCharacterSceneFormSubmit')
.click('button.letsgo');
});

37
tests/tests/variables.ts
View File

@ -3,7 +3,7 @@ import {assertLogMessage} from "./utils/log";
const fs = require('fs'); const fs = require('fs');
const Docker = require('dockerode'); const Docker = require('dockerode');
import { Selector } from 'testcafe'; import { Selector } from 'testcafe';
import {userAlice, userAliceOnPage} from "./utils/roles"; import {login} from "./utils/roles";
import { import {
findContainer, findContainer,
rebootBack, rebootBack,
@ -32,7 +32,9 @@ test("Test that variables storage works", async (t: TestController) => {
//const mainWindow = await t.getCurrentWindow(); //const mainWindow = await t.getCurrentWindow();
await t.useRole(userAliceOnPage('http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/shared_variables.json?somerandomparam=1')) login(t, 'http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/shared_variables.json?somerandomparam=1');
await t //.useRole(userAliceOnPage('http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/shared_variables.json?somerandomparam=1'))
.switchToIframe("#cowebsite-buffer iframe") .switchToIframe("#cowebsite-buffer iframe")
.expect(variableInput.value).eql('default value') .expect(variableInput.value).eql('default value')
.selectText(variableInput) .selectText(variableInput)
@ -48,6 +50,7 @@ test("Test that variables storage works", async (t: TestController) => {
.expect(variableInput.value).eql('new value') .expect(variableInput.value).eql('new value')
//.debug() //.debug()
.switchToMainWindow() .switchToMainWindow()
//.wait(5000)
//.switchToWindow(mainWindow) //.switchToWindow(mainWindow)
/* /*
.wait(5000) .wait(5000)
@ -58,12 +61,16 @@ test("Test that variables storage works", async (t: TestController) => {
// Now, let's kill the reverse proxy to cut the connexion // Now, let's kill the reverse proxy to cut the connexion
//console.log('Rebooting traefik'); //console.log('Rebooting traefik');
await rebootTraefik(); rebootTraefik();
//console.log('Rebooting done'); //console.log('Rebooting done');
// Maybe we should:
// 1: stop Traefik
// 2: detect reconnecting screen
// 3: start Traefik again
await t await t
.wait(10000) // Note sure why but maybe this will solve problem in CI/CD env
.switchToIframe("#cowebsite-buffer iframe") .switchToIframe("#cowebsite-buffer iframe")
.expect(variableInput.value).eql('new value') .expect(variableInput.value).eql('new value')
@ -142,19 +149,10 @@ test("Test that variables storage works", async (t: TestController) => {
.switchToMainWindow() .switchToMainWindow()
/*
const backContainer = await findContainer('back');
await stopContainer(backContainer);
await startContainer(backContainer);
*/
t.ctx.passed = true; t.ctx.passed = true;
}).after(async t => { }).after(async t => {
if (!t.ctx.passed) { if (!t.ctx.passed) {
console.log("Test failed. Browser logs:") console.log("Test 'Test that variables storage works' failed. Browser logs:")
try { try {
console.log(await t.getBrowserConsoleMessages()); console.log(await t.getBrowserConsoleMessages());
} catch (e) { } catch (e) {
@ -174,7 +172,10 @@ test("Test that variables cache in the back don't prevent setting a variable in
// Let's start by visiting a map that DOES not have the variable. // Let's start by visiting a map that DOES not have the variable.
fs.copyFileSync('../maps/tests/Variables/Cache/variables_cache_1.json', '../maps/tests/Variables/Cache/variables_tmp.json'); fs.copyFileSync('../maps/tests/Variables/Cache/variables_cache_1.json', '../maps/tests/Variables/Cache/variables_tmp.json');
await t.useRole(userAlice); //const aliceOnPageTmp = userAliceOnPage('http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/Cache/variables_tmp.json');
await login(t, 'http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/Cache/variables_tmp.json', 'Alice', 2);
//.takeScreenshot('before_switch.png'); //.takeScreenshot('before_switch.png');
// Let's REPLACE the map by a map that has a new variable // Let's REPLACE the map by a map that has a new variable
@ -182,7 +183,9 @@ test("Test that variables cache in the back don't prevent setting a variable in
fs.copyFileSync('../maps/tests/Variables/Cache/variables_cache_2.json', '../maps/tests/Variables/Cache/variables_tmp.json'); fs.copyFileSync('../maps/tests/Variables/Cache/variables_cache_2.json', '../maps/tests/Variables/Cache/variables_tmp.json');
await t.openWindow('http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/Cache/variables_tmp.json') await t.openWindow('http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/Cache/variables_tmp.json')
.resizeWindow(960, 800) .resizeWindow(960, 800)
.useRole(userAlice);
await login(t, 'http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/Cache/variables_tmp.json', 'Bob', 3);
//.takeScreenshot('after_switch.png'); //.takeScreenshot('after_switch.png');
// Let's check we successfully manage to save the variable value. // Let's check we successfully manage to save the variable value.
@ -191,7 +194,7 @@ test("Test that variables cache in the back don't prevent setting a variable in
t.ctx.passed = true; t.ctx.passed = true;
}).after(async t => { }).after(async t => {
if (!t.ctx.passed) { if (!t.ctx.passed) {
console.log("Test failed. Browser logs:") console.log("Test 'Test that variables cache in the back don't prevent setting a variable in case the map changes' failed. Browser logs:")
console.log(await t.getBrowserConsoleMessages()); console.log(await t.getBrowserConsoleMessages());
} }
}); });