Skip to content

PSP-10899: Playwright and Selenium tests updates#5216

Merged
asanchezr merged 5 commits intobcgov:devfrom
stairaku:playwright-branch
Feb 19, 2026
Merged

PSP-10899: Playwright and Selenium tests updates#5216
asanchezr merged 5 commits intobcgov:devfrom
stairaku:playwright-branch

Conversation

@stairaku
Copy link
Copy Markdown
Collaborator

@stairaku stairaku commented Feb 19, 2026

No description provided.

@stairaku stairaku self-assigned this Feb 19, 2026
@stairaku stairaku added the test-scripts Tag used for functional test scripts label Feb 19, 2026
github-advanced-security[bot]
github-advanced-security AI found potential problems Feb 19, 2026
View reviewed changes
Comment thread testing/playwright-testing/features/worklist/worklist.steps.js
@@ -1,20 +1,14 @@
const { Given, When, Then } = require("@cucumber/cucumber");
const { Given, When, Then, setDefaultTimeout } = require("@cucumber/cucumber");

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note test

Unused variable Given.
Comment thread testing/playwright-testing/support/pages/SharedFileProperties.js
@@ -0,0 +1,29 @@
const { expect } = require("@playwright/test");

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note test

Unused variable expect.
github-advanced-security[bot]
github-advanced-security AI found potential problems Feb 19, 2026
View reviewed changes
Comment thread testing/PIMS.Tests.Automation/Features/Properties.feature
| Yes | 17 |

Scenario: 13._Strata_Properties_Tabs
Given I search for a property by Plan Number from row number 39

Check warning

Code scanning / CodeQL

Useless upcast Warning test

There is no need to upcast from
null
Loading
to
String
Loading
- the conversion can be done implicitly.
Comment thread testing/PIMS.Tests.Automation/Features/Properties.feature
| Yes | 17 |

Scenario: 13._Strata_Properties_Tabs
Given I search for a property by Plan Number from row number 39

Check warning

Code scanning / CodeQL

Useless upcast Warning test

There is no need to upcast from
null
Loading
to
Table
Loading
- the conversion can be done implicitly.
Comment thread testing/PIMS.Tests.Automation/Features/Properties.feature

Scenario: 13._Strata_Properties_Tabs
Given I search for a property by Plan Number from row number 39
When I verify the MultiProperty Tabs

Check warning

Code scanning / CodeQL

Useless upcast Warning test

There is no need to upcast from
null
Loading
to
String
Loading
- the conversion can be done implicitly.
Comment thread testing/PIMS.Tests.Automation/Features/Properties.feature

Scenario: 13._Strata_Properties_Tabs
Given I search for a property by Plan Number from row number 39
When I verify the MultiProperty Tabs

Check warning

Code scanning / CodeQL

Useless upcast Warning test

There is no need to upcast from
null
Loading
to
Table
Loading
- the conversion can be done implicitly.
Comment thread testing/PIMS.Tests.Automation/Features/Properties.feature
Scenario: 13._Strata_Properties_Tabs
Given I search for a property by Plan Number from row number 39
When I verify the MultiProperty Tabs
Then Multiproperty property rendered successfully

Check warning

Code scanning / CodeQL

Useless upcast Warning test

There is no need to upcast from
null
Loading
to
String[]
Loading
- the conversion can be done implicitly.
Comment thread testing/PIMS.Tests.Automation/Features/Properties.feature
Scenario: 13._Strata_Properties_Tabs
Given I search for a property by Plan Number from row number 39
When I verify the MultiProperty Tabs
Then Multiproperty property rendered successfully

Check warning

Code scanning / CodeQL

Useless upcast Warning test

There is no need to upcast from
null
Loading
to
String
Loading
- the conversion can be done implicitly.
Comment thread testing/PIMS.Tests.Automation/Features/Properties.feature
Scenario: 13._Strata_Properties_Tabs
Given I search for a property by Plan Number from row number 39
When I verify the MultiProperty Tabs
Then Multiproperty property rendered successfully

Check warning

Code scanning / CodeQL

Useless upcast Warning test

There is no need to upcast from
null
Loading
to
Table
Loading
- the conversion can be done implicitly.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 19, 2026

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5216

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 19, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
4.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 19, 2026

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5216

@asanchezr asanchezr added this pull request to the merge queue Feb 19, 2026
Merged via the queue into bcgov:dev with commit 3551809 Feb 19, 2026
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@asanchezr asanchezr asanchezr approved these changes

Assignees

@stairaku stairaku

Labels

test-scripts Tag used for functional test scripts

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stairaku @asanchezr @github-advanced-security