Commit 48796673 authored by Michael Hanselmann's avatar Michael Hanselmann
Browse files

Add result checks for OpTags*


Signed-off-by: default avatarMichael Hanselmann <hansmi@google.com>
Reviewed-by: default avatarRené Nussbaumer <rn@google.com>
parent 415feb2e
......@@ -1741,6 +1741,7 @@ class OpTagsGet(OpCode):
# Name is only meaningful for nodes and instances
("name", ht.NoDefault, ht.TMaybeString, None),
]
OP_RESULT = ht.TListOf(ht.TNonEmptyString)
class OpTagsSearch(OpCode):
......@@ -1749,6 +1750,10 @@ class OpTagsSearch(OpCode):
OP_PARAMS = [
("pattern", ht.NoDefault, ht.TNonEmptyString, None),
]
OP_RESULT = ht.TListOf(ht.TAnd(ht.TIsLength(2), ht.TItems([
ht.TNonEmptyString,
ht.TNonEmptyString,
])))
class OpTagsSet(OpCode):
......@@ -1759,6 +1764,7 @@ class OpTagsSet(OpCode):
# Name is only meaningful for nodes and instances
("name", ht.NoDefault, ht.TMaybeString, None),
]
OP_RESULT = ht.TNone
class OpTagsDel(OpCode):
......@@ -1769,6 +1775,7 @@ class OpTagsDel(OpCode):
# Name is only meaningful for nodes and instances
("name", ht.NoDefault, ht.TMaybeString, None),
]
OP_RESULT = ht.TNone
# Test opcodes
......
......@@ -38,10 +38,6 @@ import testutils
#: Unless an opcode is included in the following list it must have a result
#: check of some sort
MISSING_RESULT_CHECK = frozenset([
opcodes.OpTagsDel,
opcodes.OpTagsGet,
opcodes.OpTagsSearch,
opcodes.OpTagsSet,
opcodes.OpTestAllocator,
opcodes.OpTestDelay,
opcodes.OpTestDummy,
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment