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

Add missing testCase.version member to TERCC event #288

Closed
t-persson opened this issue Nov 3, 2021 · 2 comments · Fixed by #286
Closed

Add missing testCase.version member to TERCC event #288

t-persson opened this issue Nov 3, 2021 · 2 comments · Fixed by #286
Assignees
Labels
protocol All protocol changes
Milestone

Comments

@t-persson
Copy link
Contributor

Description

The data.batches.recipes.testCase member in the TERCC event has duplicates of the data.testCase member in the TestCaseTriggered event but missing the version member.

Motivation

Since we have duplicates of all other members, we should add the version member as well.

Exemplification

In ETOS we use the TERCC data.batches.recipes for extra information about a certain test case that we run. This version member would help us trace the version of each testCase that was executed.

Benefits

Both clarity in that these two events have duplicate data, but also the ability to do stuff with the version when publishing a TERCC.

Possible Drawbacks

Duplicate data always hurt a little bit, but I think that should be a discussion for another time

@magnusbaeck
Copy link
Member

We have a URI field that supposedly identifies the testcase. Shouldn't that identification also include the version, making a separate version field superfluous?

@e-backmark-ericsson
Copy link
Member

Could be. But there could also be different ways of describing versions in a URI and in a more free text form, for example human readable.

@magnusbaeck magnusbaeck added this to the Edition Arica milestone Apr 7, 2022
@magnusbaeck magnusbaeck added the protocol All protocol changes label Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
protocol All protocol changes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants