Skip to content

Commit

Permalink
Remove skip from the ResultInterpret enumeration (#3269)
Browse files Browse the repository at this point in the history
As discussed today, the `skip` value was never documented in the
test `result` key specification and was added by an accident.

There seems to be a use case to support skipping selected tests
but that functionality is substantially different from the result
interpretation configured by the `result` key which happens only
after test execution is completed so most probably a new test key
should be introduced for it.
  • Loading branch information
psss authored Oct 10, 2024
1 parent 3803a44 commit f9642cf
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion tmt/result.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@ class ResultInterpret(enum.Enum):
INFO = 'info'
WARN = 'warn'
ERROR = 'error'
SKIP = 'skip'

# Special interpret values
RESPECT = 'respect'
Expand Down
1 change: 0 additions & 1 deletion tmt/schemas/common.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -478,7 +478,6 @@ definitions:
- warn
- error
- fail
- skip
- custom
- restraint

Expand Down

0 comments on commit f9642cf

Please sign in to comment.