From b7a1a131c153225f258cf77be92a6b95b36dc560 Mon Sep 17 00:00:00 2001
From: AlexandrLegchilov <Alexandr_Legchilov@epam.com>
Date: Wed, 2 Oct 2024 16:14:10 +0300
Subject: [PATCH 1/6] add permissions steps

---
 CHANGELOG.md                      |  5 ++++-
 package-lock.json                 |  4 ++--
 package.json                      |  2 +-
 src/actions.ts                    | 32 +++++++++++++++++++++++++++++++
 test-e2e/apps/actions.html        |  9 ++++++++-
 test-e2e/features/actions.feature | 16 ++++++++++++++++
 test-e2e/memory/index.ts          |  5 ++++-
 test-e2e/page_object/index.ts     |  1 +
 8 files changed, 68 insertions(+), 6 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 81d71bc..ceeb797 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -10,7 +10,10 @@ Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how
 :pencil: - chore
 :microscope: - experimental
 
-## [Unreleased]
+## [0.53.0]
+- :rocket: added _I grant {string} browser permissions_ step
+- :rocket: added _I revoke browser permissions_ step
+- :rocket: added _I set {string} geolocation_ step
 - :beetle: improved _I scroll until_ steps to use same locator
 
 ## [0.52.0]
diff --git a/package-lock.json b/package-lock.json
index f0d268b..3f68591 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "@qavajs/steps-playwright",
-  "version": "0.52.0",
+  "version": "0.53.0",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@qavajs/steps-playwright",
-      "version": "0.52.0",
+      "version": "0.53.0",
       "license": "MIT",
       "dependencies": {
         "@playwright/test": "^1.47.0",
diff --git a/package.json b/package.json
index ff80cc8..8afb455 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@qavajs/steps-playwright",
-  "version": "0.52.0",
+  "version": "0.53.0",
   "description": "steps to interact with playwright",
   "main": "./index.js",
   "scripts": {
diff --git a/src/actions.ts b/src/actions.ts
index 0b0e352..2d3b2d7 100644
--- a/src/actions.ts
+++ b/src/actions.ts
@@ -3,6 +3,7 @@ import { getValue, getElement } from './transformers';
 import { po } from '@qavajs/po-playwright';
 import { expect, Browser, BrowserContext, Page } from '@playwright/test';
 import { parseCoords, parseCoordsAsObject, sleep } from './utils/utils';
+import memory from "@qavajs/memory";
 
 declare global {
     var browser: Browser;
@@ -465,3 +466,34 @@ When('I tap {string}', async function (alias: string) {
     const element = await getElement(alias);
     await element.tap();
 });
+
+/**
+ * Grants specified permissions to the browser context.
+ * @param {string} permissionsAlias - permissions array memory alias.
+ * @example I grant '$microphone' browser permissions
+ * where '$location' is memory alias of permissions array - ['geolocation'];
+ * Permissions documentation can be found here https://playwright.dev/docs/api/class-browsercontext#browser-context-grant-permissions-option-permissions
+ */
+When('I grant {string} browser permissions', async function (permissionsAlias: string) {
+    const permissions = await memory.getValue(permissionsAlias);
+    await context.grantPermissions(permissions);
+});
+
+/**
+ * Clears all permission overrides for the browser context.
+ */
+When('I revoke browser permissions', async function () {
+    await context.clearPermissions();
+});
+
+/**
+ * Sets a geolocation for a current context.
+ * @param {string} geolocationAlias - geolocation memory alias.
+ * @example I set '$minsk' geolocation
+ * where '$minsk' is memory alias of location object { latitude: 53.53, longitude: 27.34 };
+ * Passing null or undefined emulates position unavailable.
+ */
+When('I set {string} geolocation', async function (geolocationAlias: string) {
+    const geolocation = await memory.getValue(geolocationAlias);
+    await context.setGeolocation(geolocation);
+});
diff --git a/test-e2e/apps/actions.html b/test-e2e/apps/actions.html
index a264ea4..1305d75 100644
--- a/test-e2e/apps/actions.html
+++ b/test-e2e/apps/actions.html
@@ -76,8 +76,15 @@
 
 <div id="mouseEvent" style="height: 200px; width: 200px; background-color: #0f9e9e"></div>
 <input id="keyboardEvent" style="height: 200px; width: 200px; background-color: #0f9e9e"></input>
-
+<button id="location" onclick="getLocation()">No location</button>
 <script>
+    const locationButton = document.getElementById("location");
+    function getLocation() {
+            navigator.geolocation.watchPosition(showPosition);
+    }
+    function showPosition(position) {
+        locationButton.innerText = `{"latitude":${position.coords.latitude},"longitude":${position.coords.longitude}}`;
+    }
     const action = document.querySelector('#action');
     const button = document.querySelector('#button');
     const buttonTap = document.querySelector('#buttonTap');
diff --git a/test-e2e/features/actions.feature b/test-e2e/features/actions.feature
index bebd81e..5285045 100644
--- a/test-e2e/features/actions.feature
+++ b/test-e2e/features/actions.feature
@@ -241,3 +241,19 @@ Feature: actions
 
   Scenario: scroll until visible in element
     When I scroll in 'Infinite Scroll' until '#row 34 in Infinite Scroll Items' to be visible
+
+  Scenario: set location
+    When I set '$canada' geolocation
+    When I grant '$location' browser permissions
+    When I click 'Location Button'
+    When I expect text of 'Location Button' to equal '$js(JSON.stringify($canada))'
+
+  Scenario: revoke permissions
+    When I set '$canada' geolocation
+    When I grant '$location' browser permissions
+    When I click 'Location Button'
+    When I expect text of 'Location Button' to equal '$js(JSON.stringify($canada))'
+    When I revoke browser permissions
+    When I refresh page
+    When I click 'Location Button'
+    When I expect text of 'Location Button' to equal 'No location'
diff --git a/test-e2e/memory/index.ts b/test-e2e/memory/index.ts
index 9563fae..6f226b0 100644
--- a/test-e2e/memory/index.ts
+++ b/test-e2e/memory/index.ts
@@ -37,5 +37,8 @@ export default class Memory {
     button2 = 'Button2';
 
     userInterceptionPredicate = (response: Response) => response.url().includes('users');
-}
 
+    location = ['geolocation'];
+
+    canada = { latitude: 62.39, longitude: -96.81};
+}
diff --git a/test-e2e/page_object/index.ts b/test-e2e/page_object/index.ts
index 85c39a1..ac862c5 100644
--- a/test-e2e/page_object/index.ts
+++ b/test-e2e/page_object/index.ts
@@ -60,6 +60,7 @@ export default class App {
     PlusButton = $('#plusButton');
     FetchButton = $('#fetchButton');
     FetchResult = $('#fetchResult');
+    LocationButton = $('#location');
 
     // Electron
     OpenNewWindowElectronButton = $('#electronButton');

From dbdeb61fb907d41798836d93220c2441f19cd5ba Mon Sep 17 00:00:00 2001
From: AlexandrLegchilov <Alexandr_Legchilov@epam.com>
Date: Wed, 2 Oct 2024 16:38:31 +0300
Subject: [PATCH 2/6] permissions steps redesign

---
 src/actions.ts                    | 13 ++++++-------
 test-e2e/features/actions.feature |  4 ++--
 2 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/src/actions.ts b/src/actions.ts
index 2d3b2d7..6e6f17c 100644
--- a/src/actions.ts
+++ b/src/actions.ts
@@ -468,15 +468,14 @@ When('I tap {string}', async function (alias: string) {
 });
 
 /**
- * Grants specified permissions to the browser context.
- * @param {string} permissionsAlias - permissions array memory alias.
- * @example I grant '$microphone' browser permissions
- * where '$location' is memory alias of permissions array - ['geolocation'];
+ * Grants specified permission to the browser context.
+ * @param {string} permissionAlias - permission alias.
+ * @example I grant 'geolocation' browser permission
  * Permissions documentation can be found here https://playwright.dev/docs/api/class-browsercontext#browser-context-grant-permissions-option-permissions
  */
-When('I grant {string} browser permissions', async function (permissionsAlias: string) {
-    const permissions = await memory.getValue(permissionsAlias);
-    await context.grantPermissions(permissions);
+When('I grant {string} permission', async function (permissionAlias: string) {
+    const permission = await memory.getValue(permissionAlias);
+    await context.grantPermissions([permission]);
 });
 
 /**
diff --git a/test-e2e/features/actions.feature b/test-e2e/features/actions.feature
index 5285045..82dc3db 100644
--- a/test-e2e/features/actions.feature
+++ b/test-e2e/features/actions.feature
@@ -244,13 +244,13 @@ Feature: actions
 
   Scenario: set location
     When I set '$canada' geolocation
-    When I grant '$location' browser permissions
+    When I grant '$location' permission
     When I click 'Location Button'
     When I expect text of 'Location Button' to equal '$js(JSON.stringify($canada))'
 
   Scenario: revoke permissions
     When I set '$canada' geolocation
-    When I grant '$location' browser permissions
+    When I grant 'geolocation' permission
     When I click 'Location Button'
     When I expect text of 'Location Button' to equal '$js(JSON.stringify($canada))'
     When I revoke browser permissions

From a7eaed0185bdaae42a8a9349377725e41948597d Mon Sep 17 00:00:00 2001
From: AlexandrLegchilov <Alexandr_Legchilov@epam.com>
Date: Wed, 2 Oct 2024 16:40:10 +0300
Subject: [PATCH 3/6] permissions steps redesign

---
 CHANGELOG.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index ceeb797..9affbbd 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -11,7 +11,7 @@ Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how
 :microscope: - experimental
 
 ## [0.53.0]
-- :rocket: added _I grant {string} browser permissions_ step
+- :rocket: added _I grant {string} permission_ step
 - :rocket: added _I revoke browser permissions_ step
 - :rocket: added _I set {string} geolocation_ step
 - :beetle: improved _I scroll until_ steps to use same locator

From a9a68ed2fdf852dd53e25eb3e1a5d083e42c42fa Mon Sep 17 00:00:00 2001
From: AlexandrLegchilov <Alexandr_Legchilov@epam.com>
Date: Wed, 2 Oct 2024 16:46:41 +0300
Subject: [PATCH 4/6] permissions steps redesign

---
 test-e2e/memory/index.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test-e2e/memory/index.ts b/test-e2e/memory/index.ts
index 6f226b0..a08e179 100644
--- a/test-e2e/memory/index.ts
+++ b/test-e2e/memory/index.ts
@@ -38,7 +38,7 @@ export default class Memory {
 
     userInterceptionPredicate = (response: Response) => response.url().includes('users');
 
-    location = ['geolocation'];
+    location = 'geolocation';
 
     canada = { latitude: 62.39, longitude: -96.81};
 }

From e74502cb1a818ba0ee49e86b4f993ead95ed728a Mon Sep 17 00:00:00 2001
From: AlexandrLegchilov <Alexandr_Legchilov@epam.com>
Date: Wed, 2 Oct 2024 16:53:17 +0300
Subject: [PATCH 5/6] permissions steps redesign

---
 src/actions.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/actions.ts b/src/actions.ts
index 6e6f17c..06236a4 100644
--- a/src/actions.ts
+++ b/src/actions.ts
@@ -474,7 +474,7 @@ When('I tap {string}', async function (alias: string) {
  * Permissions documentation can be found here https://playwright.dev/docs/api/class-browsercontext#browser-context-grant-permissions-option-permissions
  */
 When('I grant {string} permission', async function (permissionAlias: string) {
-    const permission = await memory.getValue(permissionAlias);
+    const permission = await getValue(permissionAlias);
     await context.grantPermissions([permission]);
 });
 
@@ -493,6 +493,6 @@ When('I revoke browser permissions', async function () {
  * Passing null or undefined emulates position unavailable.
  */
 When('I set {string} geolocation', async function (geolocationAlias: string) {
-    const geolocation = await memory.getValue(geolocationAlias);
+    const geolocation = await getValue(geolocationAlias);
     await context.setGeolocation(geolocation);
 });

From 4e83284942ed6723c64a120b928158178e10d1bc Mon Sep 17 00:00:00 2001
From: AlexandrLegchilov <Alexandr_Legchilov@epam.com>
Date: Wed, 2 Oct 2024 16:56:09 +0300
Subject: [PATCH 6/6] permissions steps redesign

---
 src/actions.ts | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/actions.ts b/src/actions.ts
index 06236a4..045deb5 100644
--- a/src/actions.ts
+++ b/src/actions.ts
@@ -3,7 +3,6 @@ import { getValue, getElement } from './transformers';
 import { po } from '@qavajs/po-playwright';
 import { expect, Browser, BrowserContext, Page } from '@playwright/test';
 import { parseCoords, parseCoordsAsObject, sleep } from './utils/utils';
-import memory from "@qavajs/memory";
 
 declare global {
     var browser: Browser;
@@ -470,7 +469,7 @@ When('I tap {string}', async function (alias: string) {
 /**
  * Grants specified permission to the browser context.
  * @param {string} permissionAlias - permission alias.
- * @example I grant 'geolocation' browser permission
+ * @example I grant 'geolocation' permission
  * Permissions documentation can be found here https://playwright.dev/docs/api/class-browsercontext#browser-context-grant-permissions-option-permissions
  */
 When('I grant {string} permission', async function (permissionAlias: string) {