[lttng-dev] [PATCH lttng-tools] Tests: Add filtering tests for uncovered cases

[Christian Babeux]

While investigating the code coverage of the filtering feature,
a couple of possible tests cases were uncovered:

Error tests:
* Strings can't be IR root node
* Unary ! not allowed on string type
* Comparison with string type not allowed
* Logical operator not allowed with string types
* Nesting of binary operator not allowed

Valid tests:
* Cover all left/right operands permutations with
  fields ref. and numeric values.

Signed-off-by: Christian Babeux <christian.babeux at efficios.com>
---
 tests/tools/filtering/invalid-filters | 15 +++++++++++++++
 tests/tools/filtering/valid-filters   |  4 ++++
 2 files changed, 19 insertions(+)

diff --git a/tests/tools/filtering/invalid-filters b/tests/tools/filtering/invalid-filters
index d0777e5..b653705 100755
--- a/tests/tools/filtering/invalid-filters
+++ b/tests/tools/filtering/invalid-filters
@@ -118,6 +118,21 @@ INVALID_FILTERS=(
 		"!a.f.d"
 		"asdf.asdfsd.sadf < 4"
 		"asdfasdf->asdfasdf < 2"
+		# String can't be root node
+		"\"somestring\""
+		# Unary op on string not allowed
+		"!\"somestring\""
+		# Comparison with string type not allowed
+		"\"somestring\" > 42"
+		"\"somestring\" > 42.0"
+		"42 > \"somestring\""
+		"42.0 > \"somestring\""
+		# Logical operator with string type not allowed
+		"\"somestring\" || 1"
+		"1 || \"somestring\""
+		# Nesting of binary operator not allowed
+		"1 | (1 | (1 | 1))"
+		"1 > (1 > (1 > 1))"
 		)
 
 start_lttng_sessiond
diff --git a/tests/tools/filtering/valid-filters b/tests/tools/filtering/valid-filters
index b48b6ed..d32a60d 100755
--- a/tests/tools/filtering/valid-filters
+++ b/tests/tools/filtering/valid-filters
@@ -361,6 +361,8 @@ FILTERS=("intfield"                                                     #1
 	 "intfield < 0x2"                                               #24
 	 "intfield < 02"                                                #25
 	 "stringfield2 == \"\\\*\""                                     #26
+	 "1.0 || intfield || 1.0"                                       #27
+	 "1 < intfield"                                                 #28
 )
 
 VALIDATOR=("validator_intfield"                     #1
@@ -389,6 +391,8 @@ VALIDATOR=("validator_intfield"                     #1
 	   "validator_intfield_lt"                  #24
 	   "validator_intfield_lt"                  #25
 	   "validator_true_statement"               #26
+	   "validator_true_statement"               #27
+	   "validator_intfield_gt"                  #28
 )
 
 FILTER_COUNT=${#FILTERS[@]}
-- 
1.8.0