Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: integration testing portability enhancement and schema cleanup #1024

Merged
merged 7 commits into from
Dec 18, 2024

Conversation

urangel
Copy link
Contributor

@urangel urangel commented Dec 17, 2024

Description

These changes support testing decoupling from the HarnessDetails struct so that testing is more portable and does not need to be contained within this repository.

In addition, two ADCS edges are removed from the cue schema as they are not currently implemented and including them in certain lists of edges results in counts that are not correct when using those lists for testing purposes.

Motivation and Context

This PR addresses: BED-5167

We want to keep some tests in the BHE repo while still using a lot of the testing infrastructure and strategies that have only been availablein BHCE.

How Has This Been Tested?

Tests continue to pass. Changes are largely contained to testing updates.

Screenshots (optional):

Types of changes

  • Chore (a change that does not modify the application functionality)

Checklist:

@urangel urangel self-assigned this Dec 18, 2024
@urangel urangel added api A pull request containing changes affecting the API code. tooling This updates developer tooling labels Dec 18, 2024
@urangel urangel changed the title Bed 5167 chore: integration testing portability enhancement and schema cleanup Dec 18, 2024
@urangel urangel merged commit 51b29a4 into stage/v6.3.1 Dec 18, 2024
4 checks passed
@urangel urangel deleted the BED-5167 branch December 18, 2024 18:48
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api A pull request containing changes affecting the API code. tooling This updates developer tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants