From 6455b007420918294df942a5ae435edd8d7e7232 Mon Sep 17 00:00:00 2001 From: ramki-bruno Date: Mon, 14 Apr 2025 13:33:22 +0530 Subject: [PATCH] Removed old Playwright setup --- package-lock.json | 64 ---------------------- package.json | 3 -- playwright.config.js | 110 -------------------------------------- tests/home.spec.js | 48 ----------------- tests/pages/home.page.js | 86 ----------------------------- tests/utils/data-faker.js | 6 --- 6 files changed, 317 deletions(-) delete mode 100644 playwright.config.js delete mode 100644 tests/home.spec.js delete mode 100644 tests/pages/home.page.js delete mode 100644 tests/utils/data-faker.js diff --git a/package-lock.json b/package-lock.json index 8bf023cfd..1fa51c2c1 100644 --- a/package-lock.json +++ b/package-lock.json @@ -23,7 +23,6 @@ "devDependencies": { "@faker-js/faker": "^7.6.0", "@jest/globals": "^29.2.0", - "@playwright/test": "^1.27.1", "@types/jest": "^29.5.11", "@types/lodash-es": "^4.17.12", "concurrently": "^8.2.2", @@ -6104,22 +6103,6 @@ "node": ">=14" } }, - "node_modules/@playwright/test": { - "version": "1.49.1", - "resolved": "https://registry.npmjs.org/@playwright/test/-/test-1.49.1.tgz", - "integrity": "sha512-Ky+BVzPz8pL6PQxHqNRW1k3mIyv933LML7HktS8uik0bUXNCdPhoS/kLihiO1tMf/egaJb4IutXd7UywvXEW+g==", - "dev": true, - "license": "Apache-2.0", - "dependencies": { - "playwright": "1.49.1" - }, - "bin": { - "playwright": "cli.js" - }, - "engines": { - "node": ">=18" - } - }, "node_modules/@popperjs/core": { "version": "2.11.8", "resolved": "https://registry.npmjs.org/@popperjs/core/-/core-2.11.8.tgz", @@ -21299,53 +21282,6 @@ "integrity": "sha512-fnWVljUchTro6RiCFvCXBbNhJc2NijN7oIQxbwsyL0buWJPG85v81ehlHI9fXrJsMNgTofEoWIQeClKpgxFLrg==", "license": "MIT" }, - "node_modules/playwright": { - "version": "1.49.1", - "resolved": "https://registry.npmjs.org/playwright/-/playwright-1.49.1.tgz", - "integrity": "sha512-VYL8zLoNTBxVOrJBbDuRgDWa3i+mfQgDTrL8Ah9QXZ7ax4Dsj0MSq5bYgytRnDVVe+njoKnfsYkH3HzqVj5UZA==", - "dev": true, - "license": "Apache-2.0", - "dependencies": { - "playwright-core": "1.49.1" - }, - "bin": { - "playwright": "cli.js" - }, - "engines": { - "node": ">=18" - }, - "optionalDependencies": { - "fsevents": "2.3.2" - } - }, - "node_modules/playwright-core": { - "version": "1.49.1", - "resolved": "https://registry.npmjs.org/playwright-core/-/playwright-core-1.49.1.tgz", - "integrity": "sha512-BzmpVcs4kE2CH15rWfzpjzVGhWERJfmnXmniSyKeRZUs9Ws65m+RGIi7mjJK/euCegfn3i7jvqWeWyHe9y3Vgg==", - "dev": true, - "license": "Apache-2.0", - "bin": { - "playwright-core": "cli.js" - }, - "engines": { - "node": ">=18" - } - }, - "node_modules/playwright/node_modules/fsevents": { - "version": "2.3.2", - "resolved": "https://registry.npmjs.org/fsevents/-/fsevents-2.3.2.tgz", - "integrity": "sha512-xiqMQR4xAeHTuB9uWm+fFRcIOgKBMiOBP+eXiyT7jsgVCq1bkVygt00oASowB7EdtpOHaaPgKt812P9ab+DDKA==", - "dev": true, - "hasInstallScript": true, - "license": "MIT", - "optional": true, - "os": [ - "darwin" - ], - "engines": { - "node": "^8.16.0 || ^10.6.0 || >=11.0.0" - } - }, "node_modules/plist": { "version": "3.1.0", "resolved": "https://registry.npmjs.org/plist/-/plist-3.1.0.tgz", diff --git a/package.json b/package.json index 30d41bb01..4bd03c657 100644 --- a/package.json +++ b/package.json @@ -20,7 +20,6 @@ "devDependencies": { "@faker-js/faker": "^7.6.0", "@jest/globals": "^29.2.0", - "@playwright/test": "^1.27.1", "@types/jest": "^29.5.11", "@types/lodash-es": "^4.17.12", "concurrently": "^8.2.2", @@ -59,8 +58,6 @@ "build:electron:snap": "./scripts/build-electron.sh snap", "watch:common": "npm run watch --workspace=packages/bruno-common", "test:codegen": "npm run dev:web & node ./scripts/playwright-codegen.js", - "test:e2e": "npx playwright test", - "test:report": "npx playwright show-report", "test:prettier:web": "npm run test:prettier --workspace=packages/bruno-app", "prepare": "husky install", "lint": "npx eslint ./" diff --git a/playwright.config.js b/playwright.config.js deleted file mode 100644 index 63afd88d4..000000000 --- a/playwright.config.js +++ /dev/null @@ -1,110 +0,0 @@ -// @ts-check -const { devices } = require('@playwright/test'); - -/** - * Read environment variables from file. - * https://github.com/motdotla/dotenv - */ -// require('dotenv').config(); - -process.env.PLAYWRIGHT = "1"; - -/** - * @see https://playwright.dev/docs/test-configuration - * @type {import('@playwright/test').PlaywrightTestConfig} - */ -const config = { - testDir: './tests', - /* Maximum time one test can run for. */ - timeout: 30 * 1000, - expect: { - /** - * Maximum time expect() should wait for the condition to be met. - * For example in `await expect(locator).toHaveText();` - */ - timeout: 5000 - }, - /* Run tests in files in parallel */ - fullyParallel: true, - /* Fail the build on CI if you accidentally left test.only in the source code. */ - forbidOnly: !!process.env.CI, - /* Retry on CI only */ - retries: process.env.CI ? 2 : 0, - /* Opt out of parallel tests on CI. */ - workers: process.env.CI ? 1 : undefined, - /* Reporter to use. See https://playwright.dev/docs/test-reporters */ - reporter: 'html', - /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */ - use: { - /* Maximum time each action such as `click()` can take. Defaults to 0 (no limit). */ - actionTimeout: 0, - /* Base URL to use in actions like `await page.goto('/')`. */ - baseURL: 'http://localhost:3000', - - /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */ - trace: 'retain-on-failure', - }, - - /* Configure projects for major browsers */ - projects: [ - { - name: 'chromium', - use: { - ...devices['Desktop Chrome'], - }, - }, - - { - name: 'firefox', - use: { - ...devices['Desktop Firefox'], - }, - }, - - { - name: 'webkit', - use: { - ...devices['Desktop Safari'], - }, - }, - - /* Test against mobile viewports. */ - // { - // name: 'Mobile Chrome', - // use: { - // ...devices['Pixel 5'], - // }, - // }, - // { - // name: 'Mobile Safari', - // use: { - // ...devices['iPhone 12'], - // }, - // }, - - /* Test against branded browsers. */ - // { - // name: 'Microsoft Edge', - // use: { - // channel: 'msedge', - // }, - // }, - // { - // name: 'Google Chrome', - // use: { - // channel: 'chrome', - // }, - // }, - ], - - /* Folder for test artifacts such as screenshots, videos, traces, etc. */ - // outputDir: 'test-results/', - - /* Run your local dev server before starting the tests */ - webServer: { - command: 'npm run dev:web', - port: 3000, - }, -}; - -module.exports = config; diff --git a/tests/home.spec.js b/tests/home.spec.js deleted file mode 100644 index 0ab8a5652..000000000 --- a/tests/home.spec.js +++ /dev/null @@ -1,48 +0,0 @@ -const { test, expect } = require('@playwright/test'); -const { HomePage } = require('../tests/pages/home.page'); -const { faker } = require('./utils/data-faker'); - -test.describe('bruno e2e test', () => { - let homePage; - - test.beforeEach(async ({ page }) => { - homePage = new HomePage(page); - - await homePage.open(); - await expect(page).toHaveURL('/'); - await expect(page).toHaveTitle(/bruno/); - }); - - test('user should be able to create new collection & new request', async () => { - await homePage.createNewCollection(faker.randomWords); - await expect(homePage.createNewCollectionSuccessToast).toBeVisible(); - - // using fake data to simulate negative case - await homePage.createNewRequest(faker.randomVerb, faker.randomHttpMethod, faker.randomUrl); - await expect(homePage.networkErrorToast).toBeVisible(); - - // using real data to simulate positive case - await homePage.createNewRequest('Single User', 'GET', 'https://reqres.in/api/users/2'); - await expect(homePage.statusRequestSuccess).toBeVisible(); - }); - - test('user should be able to load & use sample collection', async () => { - await homePage.loadSampleCollection(); - await expect(homePage.loadSampleCollectionSuccessToast).toBeVisible(); - - await homePage.getUsers(); - await expect(homePage.statusRequestSuccess).toBeVisible(); - - await homePage.getSingleUser(); - await expect(homePage.statusRequestSuccess).toBeVisible(); - - await homePage.getUserNotFound(); - await expect(homePage.statusRequestNotFound).toBeVisible(); - - await homePage.createUser(); - await expect(homePage.statusRequestCreated).toBeVisible(); - - await homePage.updateUser(); - await expect(homePage.statusRequestSuccess).toBeVisible(); - }); -}); diff --git a/tests/pages/home.page.js b/tests/pages/home.page.js deleted file mode 100644 index 4aff24ce1..000000000 --- a/tests/pages/home.page.js +++ /dev/null @@ -1,86 +0,0 @@ -exports.HomePage = class HomePage { - constructor(page) { - this.page = page; - - // welcome - this.createCollectionSelector = page.locator('#create-collection'); - this.addCollectionSelector = page.locator('#add-collection'); - this.importCollectionSelector = page.locator('#import-collection'); - this.loadSampleCollectionSelector = page.locator('#load-sample-collection'); - - // sample collection - this.loadSampleCollectionSuccessToast = page.getByText('Sample Collection loaded successfully'); - this.sampleCollectionSelector = page.locator('#sidebar-collection-name'); - this.getUsersSelector = page.getByText('Users'); - this.getSingleUserSelector = page.getByText('Single User'); - this.getUserNotFoundSelector = page.getByText('User Not Found'); - this.postCreateSelector = page.getByText('Create'); - this.putUpdateSelector = page.getByText('Update'); - - // request panel - this.sendRequestButton = page.locator('#send-request'); - this.statusRequestSuccess = page.getByText('200 OK'); - this.statusRequestNotFound = page.getByText('404 Not Found'); - this.statusRequestCreated = page.getByText('201 Created'); - - // create collection - this.collectionNameField = page.locator('#collection-name'); - this.submitButton = page.locator(`button[type='submit']`); - this.createNewCollectionSuccessToast = page.getByText('Collection created'); - this.createNewTab = page.locator('#create-new-tab'); - this.requestNameField = page.locator('input[name="requestName"]'); - this.methodName = page.locator('#create-new-request-method').first(); - this.requestUrlField = page.locator('#request-url'); - this.networkErrorToast = page.getByText('Network Error'); - } - - async open() { - await this.page.goto('/'); - } - - async loadSampleCollection() { - await this.loadSampleCollectionSelector.click(); - } - - async getUsers() { - await this.sampleCollectionSelector.click(); - await this.getUsersSelector.click(); - await this.sendRequestButton.click(); - } - - async getSingleUser() { - await this.getSingleUserSelector.click(); - await this.sendRequestButton.click(); - } - - async getUserNotFound() { - await this.getUserNotFoundSelector.click(); - await this.sendRequestButton.click(); - } - - async createUser() { - await this.postCreateSelector.click(); - await this.sendRequestButton.click(); - } - - async updateUser() { - await this.putUpdateSelector.click(); - await this.sendRequestButton.click(); - } - - async createNewCollection(collectionName) { - await this.createCollectionSelector.click(); - await this.collectionNameField.fill(collectionName); - await this.submitButton.click(); - } - - async createNewRequest(name, method, endpoint) { - await this.createNewTab.click(); - await this.requestNameField.fill(name); - await this.methodName.click(); - await this.page.click(`text=${method}`); - await this.requestUrlField.fill(endpoint); - await this.submitButton.click(); - await this.sendRequestButton.click(); - } -}; diff --git a/tests/utils/data-faker.js b/tests/utils/data-faker.js deleted file mode 100644 index 2674b6244..000000000 --- a/tests/utils/data-faker.js +++ /dev/null @@ -1,6 +0,0 @@ -const { faker } = require('@faker-js/faker'); - -export let randomWords = faker.random.words(); -export let randomVerb = faker.hacker.verb(); -export let randomHttpMethod = faker.internet.httpMethod(); -export let randomUrl = faker.internet.url();