cleaned up dependencies

CHANGELOG.md

@@ -7,6 +7,10 @@ Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how
 :rocket: - new feature
 :beetle: - bugfix
 :x: - deprecation/removal
+:pencil: - chore
+
+## [Unreleased]
+- :pencil: cleaned up dependencies
 
 ## [0.44.0]
 - :rocket: added option to take full page screenshot (_config.browser.screenshot.fullPage_)

README.md

@@ -95,7 +95,9 @@ module.exports = {
 ```
 
 ## Playwright traces
-@qavajs support capturing playwright traces. https://playwright.dev/docs/next/trace-viewer-intro
+@qavajs support capturing playwright traces. 
+https://playwright.dev/docs/next/trace-viewer-intro
+
 Trace Viewer - https://trace.playwright.dev/
 
 ```typescript

globals.d.ts

@@ -1,4 +1,4 @@
-import { Browser, BrowserContext, Page } from 'playwright';
+import { Browser, BrowserContext, Page } from '@playwright/test';
 import { BrowserManager } from './src/browserManager';
 
 declare global {

package.json

@@ -27,7 +27,7 @@
   "homepage": "https://github.com/qavajs/steps-playwright#readme",
   "devDependencies": {
     "@cucumber/cucumber": "^10.3.1",
-    "@qavajs/cli": "^0.34.2",
+    "@qavajs/cli": "^0.35.0",
     "@qavajs/console-formatter": "^0.7.1",
     "@qavajs/html-formatter": "^0.15.4",
     "@qavajs/memory": "^1.7.0",
@@ -40,12 +40,11 @@
     "electron": "^27.1.3",
     "express": "^4.18.2",
     "ts-node": "^10.9.2",
-    "typescript": "^5.3.3",
+    "typescript": "^5.4.2",
     "vitest": "^1.3.1"
   },
   "dependencies": {
-    "@playwright/test": "^1.41.2",
-    "@qavajs/validation": "^0.7.1",
-    "playwright": "^1.41.2"
+    "@playwright/test": "^1.42.1",
+    "@qavajs/validation": "^0.7.1"
   }
 }

src/actions.ts

@@ -1,9 +1,8 @@
 import { When } from '@cucumber/cucumber';
 import { getValue, getElement } from './transformers';
 import { po } from '@qavajs/po-playwright';
-import { expect } from '@playwright/test';
+import { expect, Browser, BrowserContext, Page } from '@playwright/test';
 import { parseCoords, parseCoordsAsObject, sleep } from './utils/utils';
-import { Browser, BrowserContext, Page } from 'playwright';
 
 declare global {
     var browser: Browser;

src/browserManager.ts

@@ -1,5 +1,5 @@
 import { driverProvider } from './driverProvider';
-import { Browser, BrowserContext, ElectronApplication, Page } from 'playwright';
+import { Browser, BrowserContext, ElectronApplication, Page } from '@playwright/test';
 
 type BrowserDict = {
     [key: string]: Browser;

src/conditionWait.ts

@@ -1,5 +1,4 @@
-import { Locator } from 'playwright';
-import { expect } from '@playwright/test';
+import { Locator, expect } from '@playwright/test';
 import { throwTimeoutError } from './utils/utils';
 
 export const conditionValidations = {
@@ -25,19 +24,19 @@ const waits = {
         reverse: boolean,
         timeout: number,
         timeoutMsg: string
-    ) => element.waitFor({state: reverse ? 'detached' : 'attached', timeout}),
+    ) => element.waitFor({ state: reverse ? 'detached' : 'attached', timeout }),
     [conditionValidations.VISIBLE]: (
         element: Locator,
         reverse: boolean,
         timeout: number,
         timeoutMsg: string
-    ) => element.waitFor({state: reverse ? 'hidden' : 'visible', timeout}),
+    ) => element.waitFor({ state: reverse ? 'hidden' : 'visible', timeout }),
     [conditionValidations.INVISIBLE]: (
         element: Locator,
         reverse: boolean,
         timeout: number,
         timeoutMsg: string
-    ) => element.waitFor({state: reverse ? 'visible' : 'hidden', timeout}),
+    ) => element.waitFor({ state: reverse ? 'visible' : 'hidden', timeout }),
     [conditionValidations.IN_VIEWPORT]: (
         element: Locator,
         reverse: boolean,

src/driverProvider.ts

@@ -1,4 +1,4 @@
-import { chromium, firefox, webkit, _electron as electron, Browser, BrowserType } from 'playwright';
+import { chromium, firefox, webkit, _electron as electron, Browser, BrowserType } from '@playwright/test';
 const browsers: any = { chromium, firefox, webkit, electron };
 
 export async function driverProvider(driverConfig: any): Promise<Browser> {

src/hooks.ts

@@ -9,7 +9,7 @@ import {
     ITestStepHookParameter
 } from '@cucumber/cucumber';
 import defaultTimeouts from './defaultTimeouts';
-import { Browser, BrowserContext, Page } from 'playwright';
+import { Browser, BrowserContext, Page } from '@playwright/test';
 import { po } from '@qavajs/po-playwright';
 import {
     saveScreenshotAfterStep,
@@ -18,7 +18,7 @@ import {
 } from './utils/utils';
 import { readFile } from 'node:fs/promises';
 import { createJSEngine } from './selectorEngines';
-import browserManager, {BrowserManager} from './browserManager';
+import browserManager, { BrowserManager } from './browserManager';
 import tracingManager from './utils/tracingManager';
 
 declare global {
@@ -34,7 +34,7 @@ BeforeAll(async function () {
     await createJSEngine();
 });
 
-Before({name: 'Init'}, async function () {
+Before({ name: 'Init' }, async function () {
     const driverConfig = config.browser ?? config.driver;
     driverConfig.isElectron = driverConfig.capabilities.browserName === 'electron';
     driverConfig.timeout = {
@@ -75,7 +75,7 @@ AfterStep(async function (step: ITestStepHookParameter) {
     }
 });
 
-After({name: 'Teardown'}, async function (scenario: ITestCaseHookParameter) {
+After({ name: 'Teardown' }, async function (scenario: ITestCaseHookParameter) {
     await tracingManager.stop(config.driverConfig, this, scenario);
     await browserManager.teardown({
         reuseSession: config.driverConfig.reuseSession,

src/selectorEngines.ts

@@ -1,4 +1,4 @@
-import { selectors } from 'playwright';
+import { selectors } from '@playwright/test';
 
 export async function createJSEngine() {
     const jsEngine = () => ({

src/transformers.ts

@@ -1,7 +1,7 @@
 import { conditionWait, conditionWaitExtractRegexp } from './conditionWait';
 import { po } from '@qavajs/po-playwright';
 import memory from '@qavajs/memory';
-import { Locator } from 'playwright';
+import { Locator } from '@playwright/test';
 
 export function getValue(alias: string): any {
     return memory.getValue(alias)
@@ -14,7 +14,7 @@ export async function getElement(alias: string): Promise<Locator> {
 export function getConditionWait(condition: string): Function {
     const match = condition.match(conditionWaitExtractRegexp) as RegExpMatchArray;
     if (!match) throw new Error(`${condition} wait is not implemented`);
-    const [ _, reverse, validation ] = match;
+    const [_, reverse, validation] = match;
     return async function (element: Locator, timeout: number) {
         await conditionWait(element, validation, timeout, Boolean(reverse))
     }

src/validations.ts

@@ -1,7 +1,7 @@
 import { Then } from '@cucumber/cucumber';
 import { getValue, getElement, getConditionWait } from './transformers';
 import { getValidation, getPollValidation } from '@qavajs/validation';
-import { Locator } from 'playwright';
+import { Locator } from '@playwright/test';
 
 /**
  * Verify element condition

test-e2e/step-definitions/index.ts

@@ -1,32 +1,31 @@
 import { Then } from '@cucumber/cucumber';
 import memory from '@qavajs/memory';
-import { Page } from 'playwright';
-import { expect } from 'chai';
+import { Page, expect } from '@playwright/test';
 
 declare global {
     var page: Page;
 }
 
-Then('I expect {string} memory value to be equal {string}', async function(actual, expected) {
+Then('I expect {string} memory value to be equal {string}', async function (actual, expected) {
     const actualValue = memory.getValue(actual);
     const expectedValue = memory.getValue(expected);
-    expect(expectedValue).to.eql(actualValue);
+    expect(expectedValue).toEqual(actualValue);
 });
 
-Then('I expect {string} memory value to contain {string}', async function(actual, expected) {
+Then('I expect {string} memory value to contain {string}', async function (actual, expected) {
     const actualValue = memory.getValue(actual);
     const expectedValue = memory.getValue(expected);
-    expect(actualValue).to.contain(expectedValue);
+    expect(actualValue).toContain(expectedValue);
 });
 
-Then('I expect {string} memory value to have type {string}', async function(actual, expected) {
+Then('I expect {string} memory value to have type {string}', async function (actual, expected) {
     const actualValue = memory.getValue(actual);
     const expectedValue = memory.getValue(expected);
-    expect(actualValue).to.be.a(expectedValue);
+    expect(typeof actualValue === expectedValue).toBeTruthy();
 });
 
 Then('I expect viewport size to equal {string}', async function (expectedSize) {
     const expectedValue = await memory.getValue(expectedSize);
     const actualValue = page.viewportSize();
-    expect(actualValue).to.deep.equal(expectedValue, 'Viewport size do not match');
+    expect(actualValue).toEqual(expectedValue);
 })