feat(test): Add traceability requirements #39
Conversation
zpetrace
force-pushed
the
zpetrace/add-traceability-matrix
branch
from
c81d9d9 to
85ff7fc
Compare
|
CCT-1063 was Closed as a duplicate of CCT-1311. Please start using CCT-1311 in all your jira references. |
|
It looks like that there is some incompatibility in unit test on Fedora 42, which contains new version of Python (something like method |
zpetrace
force-pushed
the
zpetrace/add-traceability-matrix
branch
from
85ff7fc to
afffd8c
Compare
zpetrace
force-pushed
the
zpetrace/add-traceability-matrix
branch
from
afffd8c to
510697b
Compare
zpetrace
force-pushed
the
zpetrace/add-traceability-matrix
branch
from
510697b to
ab399e7
Compare
jirihnidek
left a comment
jirihnidek
left a comment
There was a problem hiding this comment.
Testimony validation job failed with error saying that 100% of doc strings is wrong: Total number of invalid docstrings: 74 (100.00%). I do not understand why. Message like this does not say me anything useful:
TestUnicode::test_unicode_with_bom:89
-------------------------------------
* Tokens with invalid values:
Polarion-project-id: RHELSS
type: choice
case sensitive: False
choices: ['test-polarion-id']
Requirement: RHSS-291606
type: choice
case sensitive: False
choices: ['test-requirement']
2e9643b to
6cdb8b4
Compare
zpetrace
force-pushed
the
zpetrace/add-traceability-matrix
branch
from
6cdb8b4 to
d119679
Compare
jirihnidek
left a comment
jirihnidek
left a comment
There was a problem hiding this comment.
Thanks for update. Can you please not remove existing doc string completely that confuses betelgeuse? Can you please convert these "confusing" doc strings to comments? I do not want to lost existing information
zpetrace
force-pushed
the
zpetrace/add-traceability-matrix
branch
from
d119679 to
93c3874
Compare
jirihnidek
left a comment
jirihnidek
left a comment
There was a problem hiding this comment.
Thanks for update 👍 I have one small request.
The intention of CCT-1311 is to add a gating.el10 job for python-iniparse. Before that it would be good to have the traceability matrix to be able to report our tests to Polarion. This PR therefore adds the neccessary docstrings for Betelgeuse to each test. It also adds two github workflows - checks for betelgeuse and for testimony so that on every PR we have a check that the tests have these requirements.
Small change of removing the warning message.
zpetrace
force-pushed
the
zpetrace/add-traceability-matrix
branch
from
93c3874 to
da1ad48
Compare
4 participants
The intention of CCT-1311 is to add a gating.el10 job for python-iniparse. Before that it would be good to have the traceability matrix to be able to report our tests to Polarion. This PR therefore adds the neccessary docstrings for Betelgeuse to each test. It also adds two github workflows - checks for betelgeuse and for testimony so that on every PR we have a check that the tests have these requirements.