Skip to content

Conversation

@scottmries
Copy link
Contributor

@scottmries scottmries commented Aug 28, 2025

No QA required

@scottmries scottmries marked this pull request as ready for review August 28, 2025 20:38
Copilot AI review requested due to automatic review settings August 28, 2025 20:38
@scottmries scottmries requested a review from a team as a code owner August 28, 2025 20:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds missing WebdriverIO (WDIO) smoke tests to the CI pipeline and updates test configurations for better reliability and compatibility.

  • Adds two missing WDIO test directories (global-config and manual-mode) to the GitHub Actions workflow
  • Updates browser configurations across multiple WDIO test files to use Chrome version 138 with additional sandbox flags
  • Fixes a type casting issue in one test file to ensure proper array length checking

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/test.yml Adds missing WDIO test directories to CI pipeline
js/wdio/v7/typescript-basic/setup.ts Updates Chrome configuration with version 138 and no-sandbox flag
js/wdio/typescript-multi-page/setup.ts Updates Chrome configuration with version 138 and no-sandbox flag
js/wdio/manual-mode/setup.ts Updates Chrome configuration and adds RemoteOptions import/cast
js/wdio/global-config/setup.ts Updates Chrome configuration with version 138 and no-sandbox flag
js/wdio/global-config/home.test.ts Fixes array length check by converting NodeList to Array

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@scottmries scottmries marked this pull request as draft September 17, 2025 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants