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

The descr meta-data field is not required anymore #63

Closed
rillig opened this issue Apr 26, 2024 · 0 comments · Fixed by #95
Closed

The descr meta-data field is not required anymore #63

rillig opened this issue Apr 26, 2024 · 0 comments · Fixed by #95

Comments

@rillig
Copy link
Contributor

rillig commented Apr 26, 2024

atf-test-case.4 says that descr is required for a test case, while in fact, it was made optional in version 0.8.

While here, atf-test-case.4 promises that descr "will be shown to the user in reports", but when I ran atf-run on NetBSD, none of the test descriptions were listed in the raw report file, which started with application/X-atf-tps; version="3".

ngie-eign added a commit to ngie-eign/atf that referenced this issue Dec 27, 2024
As noted by the reporter, the "descr" metadata property was made
optional back in the 0.8 release. Update the documentation posthumously
per the code change.

Closes:	freebsd#63
Signed-off-by: Enji Cooper <[email protected]>
ngie-eign added a commit to ngie-eign/atf that referenced this issue Dec 27, 2024
As noted by the reporter, the "descr" metadata property was made
optional back in the 0.8 release. Update the documentation posthumously
per the code change.

Closes:	freebsd#63
Reported by:	Roland Illig (@rillig)
Signed-off-by:	Enji Cooper <[email protected]>
ngie-eign added a commit to ngie-eign/atf that referenced this issue Dec 27, 2024
As noted by the reporter, the "descr" metadata property was made
optional back in the 0.8 release
(af4059b to be specific). Update
the documentation posthumously per the code change.

Closes:	freebsd#63
Reported by:	Roland Illig (@rillig)
Signed-off-by:	Enji Cooper <[email protected]>
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 a pull request may close this issue.

1 participant