Skip to content

Conversation

@Andyg79
Copy link

@Andyg79 Andyg79 commented Mar 27, 2025

Description

Compartment 1 Smokescreen Tests

Context

This pr covers the addition of tests and functions required for the Compartment 1 Smokescreen Tests

Type of changes

This completes the migration of the compartment 1 tests from selenium to playwright

  • Refactoring (non-breaking change)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would change existing functionality)
  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I am familiar with the contributing guidelines
  • I have followed the code style of the project
  • I have added tests to cover my changes (where appropriate)
  • I have updated the documentation accordingly
  • This PR is a result of pair or mob programming

Sensitive Information Declaration

To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.

  • I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

Andyg79 and others added 30 commits January 31, 2025 13:38
Added two packages to requirements.txt
Started database connection
Changed the dropdown option value numbers in the functions to class variables.
Added more page object models and create a few fail scenarios so the test fails with a clear error message
1) Invitations Monitoring Page
2) Invitations Plans Page
3) Create a Plan Page
Andyg79 and others added 14 commits April 2, 2025 21:34
Added Organisations page POM
Added bowel_cancer_screening_page_title_contains_text function on BasePage

# Conflicts:
#	pages/base_page.py
Removed some hard coded values from tests reports page
…_page.py and moving them into utils/screening_subject_page_searcher.py

Creating new POMs were necessary
Removed more hardcoded values from test_reports_page
Updated test_organisations_page to use new page title function
davethepunkyone
davethepunkyone previously approved these changes Apr 3, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 4, 2025

@davethepunkyone
Copy link
Collaborator

Pull request #7 merged, closing this request

@adrianoaru-nhs adrianoaru-nhs deleted the feature/smokescreen_compartment_1 branch May 2, 2025 11:43
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.

8 participants