PR#266: Support skipping disabled tests in the fmf filter -

standard-test-roles

#266 Support skipping disabled tests in the fmf filter

Merged Oct 25, 2018 by astepano. Opened Oct 23, 2018 by psss.

psss/standard-test-roles disabled into master

psss commented Oct 23, 2018

There is a new CI Metadata attribute proposed for temporarily
disabling broken tests. This commit adds support for this.
https://pagure.io/fedora-ci/metadata/pull-request/1

plautrba commented Oct 24, 2018

Seems to be wrong, maybe leftover from enabled. A test should be skipped if disabled is true

astepano commented Oct 25, 2018

@psss @plautrba should we merge this?

psss commented Oct 25, 2018

@plautrba, why wrong? The ‘False‘ is there just to provide default if ‘disabled‘ attribute is not defined.

plautrba commented Oct 25, 2018

aha. I was interpreted the code wrong way. Please ignore my comment and feel free to merge it.

astepano commented Oct 25, 2018

Commit 7f41fff9 fixes this pull-request

Pull-Request has been merged by astepano

Oct 25, 2018

Pull-Request has been merged by astepano

Oct 25, 2018

Metadata

Assignee

None

Tags

No Tags

standard-test-roles

#266 Support skipping disabled tests in the fmf filter Merged Oct 25, 2018 by astepano. Opened Oct 23, 2018 by psss. psss/standard-test-roles disabled into master

Metadata

#266 Support skipping disabled tests in the fmf filter

Merged Oct 25, 2018 by astepano. Opened Oct 23, 2018 by psss.

psss/standard-test-roles disabled into master