diff --git a/ui/desktop/tests/end2end/authentication.spec.js b/ui/desktop/tests/end2end/authentication.spec.js index 93540c0f7..3ba899d13 100644 --- a/ui/desktop/tests/end2end/authentication.spec.js +++ b/ui/desktop/tests/end2end/authentication.spec.js @@ -4,12 +4,19 @@ const { _electron: electron } = require('playwright'); const helpers = require('./test-helpers'); let electronApp = null; +// Subdirectory within tests/end2end/screenshots where the screenshots of this test suite will be stored. const screenshotsDirectory = 'authentication/'; +// Path where the Boundary Desktop client binary is generated const executablePath = helpers.returnExecutablePath( process.platform, process.arch ); +// Set login variables +const originValue = 'http://localhost:9200'; +const loginUsername = 'admin'; +const loginPassword = 'password'; + test.beforeEach(async () => { electronApp = await electron.launch({ executablePath: executablePath, @@ -34,29 +41,15 @@ test.describe('Authentication end to end test suite', async () => { await boundaryWindow.evaluate(() => window.localStorage.setItem('desktop:origin', null) ); - const originValue = 'http://localhost:9200'; - const authLoginNameValue = 'admin'; - const authLoginPasswordValue = 'password'; - // Fill the origin input - await boundaryWindow.waitForSelector('[name=host]'); - await boundaryWindow.fill('[name=host]', originValue); - // Take screenshot - await boundaryWindow.screenshot({ - path: helpers.generateScreenshotPath( - screenshotsDirectory, - 'fillOrigin' - ), - fullPage: true, - }); + // Perform login + await helpers.login( + boundaryWindow, + originValue, + loginUsername, + loginPassword + ); - // Click the submit button - await boundaryWindow.waitForSelector('button[type="submit"]'); - await helpers.click(boundaryWindow, 'button[type="submit"]'); - - // Fill user & password - await boundaryWindow.fill('[name="identification"]', authLoginNameValue); - await boundaryWindow.fill('[name="password"]', authLoginPasswordValue); // Take screenshot await boundaryWindow.screenshot({ path: helpers.generateScreenshotPath( @@ -126,32 +119,13 @@ test.describe('Authentication end to end test suite', async () => { await boundaryWindow.evaluate(() => window.localStorage.setItem('desktop:origin', null) ); - const originValue = 'http://localhost:9200'; - const authLoginNameValue = 'admin'; - const authLoginPasswordValue = '123456'; - - // Fill the origin input - await boundaryWindow.fill('[name=host]', originValue); - // Take screenshot - await boundaryWindow.screenshot({ - path: helpers.generateScreenshotPath( - screenshotsDirectory, - 'fillOrigin' - ), - fullPage: true, - }); - - // Click the submit button - await boundaryWindow.waitForSelector('button[type="submit"]'); - await helpers.click(boundaryWindow, 'button[type="submit"]'); - - // Fill user & password - await boundaryWindow.fill('[name="identification"]', authLoginNameValue); - await boundaryWindow.fill('[name="password"]', authLoginPasswordValue); - - // Click submit - await boundaryWindow.waitForSelector('button[type="submit"]'); - await helpers.click(boundaryWindow, 'button[type="submit"]'); + // Perform login + await helpers.login( + boundaryWindow, + originValue, + loginUsername, + '123456789' + ); // Wait for the notification await boundaryWindow.waitForSelector('.rose-notification-body'); diff --git a/ui/desktop/tests/end2end/targets.spec.js b/ui/desktop/tests/end2end/targets.spec.js index e71211cf1..3baa15199 100644 --- a/ui/desktop/tests/end2end/targets.spec.js +++ b/ui/desktop/tests/end2end/targets.spec.js @@ -4,12 +4,19 @@ const { _electron: electron } = require('playwright'); const helpers = require('./test-helpers'); let electronApp = null; +// Subdirectory within tests/end2end/screenshots where the screenshots of this test suite will be stored. const screenshotsDirectory = 'targets/'; +// Path where the Boundary Desktop client binary is generated const executablePath = helpers.returnExecutablePath( process.platform, process.arch ); +// Set login variables +const originValue = 'http://localhost:9200'; +const loginUsername = 'admin'; +const loginPassword = 'password'; + test.beforeEach(async () => { electronApp = await electron.launch({ executablePath: executablePath, @@ -28,30 +35,19 @@ test.afterEach(async () => { test.describe('Targets end to end test suite', async () => { test('Connects to a target', async () => { const boundaryWindow = await electronApp.firstWindow(); // The window that contains the app. + // Override local storage origin await boundaryWindow.evaluate(() => window.localStorage.setItem('desktop:origin', null) ); - const originValue = 'http://localhost:9200'; - const authLoginNameValue = 'admin'; - const authLoginPasswordValue = 'password'; - - // Fill the origin input - await boundaryWindow.waitForSelector('[name=host]'); - await boundaryWindow.fill('[name=host]', originValue); - - // Click the submit button - await boundaryWindow.waitForSelector('button[type="submit"]'); - await helpers.click(boundaryWindow, 'button[type="submit"]'); - - // Fill user & password - await boundaryWindow.fill('[name="identification"]', authLoginNameValue); - await boundaryWindow.fill('[name="password"]', authLoginPasswordValue); - - // Click submit - await boundaryWindow.waitForSelector('button[type="submit"]'); - await helpers.click(boundaryWindow, 'button[type="submit"]'); + // Perform the login + await helpers.login( + boundaryWindow, + originValue, + loginUsername, + loginPassword + ); // Check we are in Targets await boundaryWindow.waitForURL('**/#/scopes/global/projects/targets'); diff --git a/ui/desktop/tests/end2end/test-helpers.js b/ui/desktop/tests/end2end/test-helpers.js index ba7e739fd..da35f237e 100644 --- a/ui/desktop/tests/end2end/test-helpers.js +++ b/ui/desktop/tests/end2end/test-helpers.js @@ -46,9 +46,34 @@ exports.returnExecutablePath = (platform, arch) => { /** * Performs a click with a workarouind due to an error with plain playwright click. * More info about the issue here: https://github.com/microsoft/playwright/issues/1808 - * @param {*} appWindow The current electron window. - * @param {*} selector The DOM element selector. + * @param {object} appWindow The electron window to perform the action. + * @param {string} selector The DOM element selector. */ exports.click = async (appWindow, selector) => { await appWindow.$eval(selector, (element) => element.click()); }; + +/** + * Performs login on the app window provided with the credentials provided. + * @param {object} appWindow The electron window to perform the action. + * @param {string} origin URL origin to set. + * @param {string} username Username to login. + * @param {string} password Password to login. + */ +exports.login = async (appWindow, origin, username, password) => { + // Fill the origin input + await appWindow.waitForSelector('[name=host]'); + await appWindow.fill('[name=host]', origin); + + // Click the set origin submit button + await appWindow.waitForSelector('button[type="submit"]'); + await this.click(appWindow, 'button[type="submit"]'); + + // Fill user & password + await appWindow.fill('[name="identification"]', username); + await appWindow.fill('[name="password"]', password); + + // Click submit + await appWindow.waitForSelector('button[type="submit"]'); + await this.click(appWindow, 'button[type="submit"]'); +};