As discussed on the Best Practices meeting, this pull request is proposing a new attribute to allow disabling individual tests.
@mvadkert, @plautrba, @jheger, @pcahyna, @vdolezal could you please review? Thanks.
rebased onto ae8be78d236b76e2308d39c8ea89cb0e657e4a84
Added default value. @plautrba, thanks for the idea.
Looks good to me :thumbsup:
Purpose, motivation and stories talk about disabling tests, while the attribute is called "enabled". It's a bit strange for me, but I have no real objection.
hmm, maybe I would even say @plautrba is right here, and would politely opt-in for disabled
rebased onto 06e369beb202cf3c0b3842d3d3216b854f97f94e
Makes sense. Sounds better to me as well. I've updated the pull request accordingly.
Pull request created to add support for disabling tests in standard-test-roles: https://pagure.io/standard-test-roles/pull-request/266
👍
Metadata Update from @psss: - Request assigned
Pull-Request has been merged by psss
Metadata Update from @psss: - Pull-request tagged with: test metadata
As discussed on the Best Practices meeting, this pull request is proposing a new attribute to allow disabling individual tests.