Skip to content

VED-1109: Remove hard coded values and update the tests#1312

Merged
Thomas-Boyle merged 3 commits intomasterfrom
1109-remove-hardcoded-values
Mar 20, 2026
Merged

VED-1109: Remove hard coded values and update the tests#1312
Thomas-Boyle merged 3 commits intomasterfrom
1109-remove-hardcoded-values

Conversation

@Thomas-Boyle
Copy link
Contributor

  • Updated scenarios in search.feature to specify patient identifier system and target-disease system for clarity.
  • Modified test_search_steps.py to accept parameters for patient identifier and target-disease system in relevant step definitions, improving flexibility and maintainability of the tests.

…isease system in scenarios

- Updated scenarios in search.feature to specify patient identifier system and target-disease system for clarity.
- Modified test_search_steps.py to accept parameters for patient identifier and target-disease system in relevant step definitions, improving flexibility and maintainability of the tests.
@shivani-rana-kainos shivani-rana-kainos self-requested a review March 19, 2026 16:13
@sonarqubecloud
Copy link

@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox March 20, 2026 08:58 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle merged commit 7a2a83a into master Mar 20, 2026
17 checks passed
@Thomas-Boyle Thomas-Boyle deleted the 1109-remove-hardcoded-values branch March 20, 2026 09:15
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