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

Should testSuiteRun be a required field in testCaseRun #166

Open
dan-han-101 opened this issue Nov 3, 2023 · 3 comments
Open

Should testSuiteRun be a required field in testCaseRun #166

dan-han-101 opened this issue Nov 3, 2023 · 3 comments

Comments

@dan-han-101
Copy link
Contributor

testCaseRun currently contains an optional field for testSuiteRun.

For our use cases, we always have a mapping between a test case and suite, so it would be good for us to just make this a required field.

Were there any use cases where test cases are not expected to be part of a test suite?
Thanks!

@e-backmark-ericsson
Copy link
Contributor

The discussions have been around whether there always is a test suite to refer to or not. Some users might not see the need for a test suite at all. Some users might see the pipeline step execution as the test suite and would then instead like to refer the test case executions to that taskRun executions, which could potentially be made possible through #104 .

I'm not a strong opponent of requiring a testSuiteRun to always be sent and thus required to reference from the testCaseRun though.

@afrittoli
Copy link
Contributor

Users that do not rely on that grouping of tests would not know what to provide in terms of testSuiteRun - I guess they could always provide the testRun itself as testSuiteRun in case they didn't have one, but that seems unnecessary.

@dan-han-101 I would like to better understand the benefit you would gain from having testSuiteRun mandatory. Does it solves a specific issue you're facing when using test events?

@dan-han-101
Copy link
Contributor Author

The background / context for this question stems from the use of junit xml test reports.

Many test runners, such as pytest and jenkins report test reports in this common format.

Moreover, this junit format nests test cases under test suites.

Given this common pattern of having all test cases link to some test suite, I thought it might make sense for CDEvents spec to also require this mapping.

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

No branches or pull requests

3 participants