Skip to content

Conversation

@Ravikirana-Microsoft
Copy link

@Ravikirana-Microsoft Ravikirana-Microsoft commented Jun 6, 2025

Purpose

  • ...

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

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 migrates and consolidates end-to-end automation test cases for the BYOc Research Assistant into a new structure. Key changes include adding new test scripts, page object classes, and configuration files needed for authentication and environment setup.

Reviewed Changes

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

Show a summary per file
File Description
tests/e2e-tests/tests/test_poc_byoc_research_assistant.py Adds multiple E2E tests for Articles, Grants, and Draft pages.
tests/e2e-tests/tests/conftest.py Provides fixtures and pytest hooks for test setup and reporting.
tests/e2e-tests/sample_dotenv_file.txt Sample environment configuration file.
tests/e2e-tests/requirements.txt Lists required packages for E2E testing.
tests/e2e-tests/pages/loginPage.py Implements login functionality using page objects.
tests/e2e-tests/pages/grantsPage.py Implements Grants page interactions.
tests/e2e-tests/pages/draftPage.py Implements Draft page interactions; contains a naming typo.
tests/e2e-tests/pages/articlesPage.py Implements Articles page interactions.
tests/e2e-tests/config/constants.py Defines constants and test data.
tests/e2e-tests/base/base.py Provides common utility methods for page objects.
tests/e2e-tests/README.md Documentation for setting up and running the tests.
tests/e2e-tests/.gitignore Lists files and directories to be ignored in version control.
Comments suppressed due to low confidence (2)

tests/e2e-tests/pages/loginPage.py:10

  • [nitpick] The YES_BUTTON locator is identical to SIGNIN_BUTTON. Consider using a single variable or renaming one to clearly reflect its purpose if it targets a different UI element.
YES_BUTTON = "//input[@id='idSIButton9']"

tests/e2e-tests/pages/draftPage.py:15

  • There is a spelling mistake in the variable name 'DRAFT_AURTHOR_NAME'; it should likely be 'DRAFT_AUTHOR_NAME' to improve clarity.
DRAFT_AURTHOR_NAME = "//input[@placeholder='Name']"

@Avijit-Microsoft Avijit-Microsoft changed the title automation test cases moved test: automation test cases moved Jun 6, 2025
@Avijit-Microsoft Avijit-Microsoft merged commit 3ef1294 into byoc-researcher Jun 6, 2025
5 of 7 checks passed
@Ravikirana-Microsoft Ravikirana-Microsoft deleted the moving-automation-testing branch July 18, 2025 04:05
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.

3 participants