index bca86452dffc44afe22f2fb5b3418eaa7e6f42f3..075a4bc0e475791fba0cf8ace1ebc7de60eb206e 100644 (file)
#include "libsysdb_test.h"
#include <check.h>
#include "libsysdb_test.h"
#include <check.h>
+#include <limits.h>
/*
* tests
/*
* tests
{ ";;", -1, 0, 0 },
/* valid commands */
{ ";;", -1, 0, 0 },
/* valid commands */
- { "FETCH 'host'", -1, 1, CONNECTION_FETCH },
- { "FETCH 'host' FILTER "
+ { "FETCH host 'host'", -1, 1, CONNECTION_FETCH },
+ { "FETCH host 'host' FILTER "
"host = 'host'", -1, 1, CONNECTION_FETCH },
"host = 'host'", -1, 1, CONNECTION_FETCH },
- { "LIST", -1, 1, CONNECTION_LIST },
- { "LIST -- comment", -1, 1, CONNECTION_LIST },
- { "LIST;", -1, 1, CONNECTION_LIST },
- { "LIST; INVALID", 5, 1, CONNECTION_LIST },
- { "LIST FILTER "
+ { "LIST hosts", -1, 1, CONNECTION_LIST },
+ { "LIST hosts -- foo", -1, 1, CONNECTION_LIST },
+ { "LIST hosts;", -1, 1, CONNECTION_LIST },
+ { "LIST hosts; INVALID", 11, 1, CONNECTION_LIST },
+ { "LIST hosts FILTER "
"host = 'host'", -1, 1, CONNECTION_LIST },
"host = 'host'", -1, 1, CONNECTION_LIST },
+ { "LIST services", -1, 1, CONNECTION_LIST },
+ { "LIST services FILTER "
+ "service = 'svc'", -1, 1, CONNECTION_LIST },
+ { "LIST metrics", -1, 1, CONNECTION_LIST },
+ { "LIST metrics FILTER "
+ "metric = 'metric'", -1, 1, CONNECTION_LIST },
{ "LOOKUP hosts", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
{ "LOOKUP hosts", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"service =~ 'r'", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"host =~ 'p' "
"service =~ 'r'", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"host =~ 'p' "
- "FILTER :age > 1D", -1, 1, CONNECTION_LOOKUP },
+ "FILTER .age > 1D", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"host =~ 'p' "
{ "LOOKUP hosts MATCHING "
"host =~ 'p' "
- "FILTER :age > 1D AND "
- ":interval < 240s" , -1, 1, CONNECTION_LOOKUP },
+ "FILTER .age > 1D AND "
+ ".interval < 240s" , -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"host =~ 'p' "
{ "LOOKUP hosts MATCHING "
"host =~ 'p' "
- "FILTER NOT :age>1D", -1, 1, CONNECTION_LOOKUP },
+ "FILTER NOT .age>1D", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"host =~ 'p' "
{ "LOOKUP hosts MATCHING "
"host =~ 'p' "
- "FILTER :age>"
- ":interval", -1, 1, CONNECTION_LOOKUP },
+ "FILTER .age>"
+ ".interval", -1, 1, CONNECTION_LOOKUP },
+
+ { "TIMESERIES 'host'.'metric' "
+ "START 2014-01-01 "
+ "END 2014-12-31 "
+ "23:59:59", -1, 1, CONNECTION_TIMESERIES },
+ { "TIMESERIES 'host'.'metric' "
+ "START 2014-02-02 "
+ "14:02", -1, 1, CONNECTION_TIMESERIES },
+ { "TIMESERIES 'host'.'metric' "
+ "END 2014-02-02", -1, 1, CONNECTION_TIMESERIES },
+ { "TIMESERIES "
+ "'host'.'metric'", -1, 1, CONNECTION_TIMESERIES },
+
+ /* string constants */
+ { "LOOKUP hosts MATCHING "
+ "host = ''''", -1, 1, CONNECTION_LOOKUP },
+ { "LOOKUP hosts MATCHING "
+ "host = '''foo'", -1, 1, CONNECTION_LOOKUP },
+ { "LOOKUP hosts MATCHING "
+ "host = 'f''oo'", -1, 1, CONNECTION_LOOKUP },
+ { "LOOKUP hosts MATCHING "
+ "host = 'foo'''", -1, 1, CONNECTION_LOOKUP },
+ { "LOOKUP hosts MATCHING "
+ "host = '''", -1, -1, 0 },
/* numeric constants */
{ "LOOKUP hosts MATCHING "
/* numeric constants */
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"1234", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"1234", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo != "
+ "attribute['foo'] != "
"+234", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"+234", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo < "
+ "attribute['foo'] < "
"-234", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"-234", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo > "
+ "attribute['foo'] > "
"12.4", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"12.4", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo <= "
+ "attribute['foo'] <= "
"12. + .3", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"12. + .3", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo >= "
+ "attribute['foo'] <= "
+ "'f' || 'oo'", -1, 1, CONNECTION_LOOKUP },
+ { "LOOKUP hosts MATCHING "
+ "attribute['foo'] >= "
".4", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
".4", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"+12e3", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"+12e3", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"+12e-3", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"+12e-3", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"-12e+3", -1, 1, CONNECTION_LOOKUP },
/* date, time, interval constants */
{ "LOOKUP hosts MATCHING "
"-12e+3", -1, 1, CONNECTION_LOOKUP },
/* date, time, interval constants */
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"1 Y 42D", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"1 Y 42D", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"1s 42D", -1, 1, CONNECTION_LOOKUP },
/*
* TODO: Something like 1Y42D should work as well but it doesn't since
* the scanner will tokenize it into {digit}{identifier} :-/
*
{ "LOOKUP hosts MATCHING "
"1s 42D", -1, 1, CONNECTION_LOOKUP },
/*
* TODO: Something like 1Y42D should work as well but it doesn't since
* the scanner will tokenize it into {digit}{identifier} :-/
*
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"1Y42D", -1, 1, CONNECTION_LOOKUP },
*/
/* NULL */
{ "LOOKUP hosts MATCHING "
"1Y42D", -1, 1, CONNECTION_LOOKUP },
*/
/* NULL */
{ "LOOKUP hosts MATCHING "
- "attribute.foo "
+ "attribute['foo'] "
"IS NULL", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"IS NULL", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "attribute.foo "
+ "attribute['foo'] "
"IS NOT NULL", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"IS NOT NULL", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
- "NOT attribute.foo "
+ "NOT attribute['foo'] "
"IS NULL", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"host IS NULL", -1, -1, 0 },
"IS NULL", -1, 1, CONNECTION_LOOKUP },
{ "LOOKUP hosts MATCHING "
"host IS NULL", -1, -1, 0 },
/* invalid numeric constants */
{ "LOOKUP hosts MATCHING "
/* invalid numeric constants */
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"+-12e+3", -1, -1, 0 },
{ "LOOKUP hosts MATCHING "
"+-12e+3", -1, -1, 0 },
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"-12e-+3", -1, -1, 0 },
{ "LOOKUP hosts MATCHING "
"-12e-+3", -1, -1, 0 },
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"e+3", -1, -1, 0 },
{ "LOOKUP hosts MATCHING "
"e+3", -1, -1, 0 },
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"3e", -1, -1, 0 },
/* following SQL standard, we don't support hex numbers */
{ "LOOKUP hosts MATCHING "
"3e", -1, -1, 0 },
/* following SQL standard, we don't support hex numbers */
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"0x12", -1, -1, 0 },
/* invalid expressions */
{ "LOOKUP hosts MATCHING "
"0x12", -1, -1, 0 },
/* invalid expressions */
{ "LOOKUP hosts MATCHING "
- "attribute.foo = "
+ "attribute['foo'] = "
"1.23 + 'foo'", -1, -1, 0 },
"1.23 + 'foo'", -1, -1, 0 },
+ { "LOOKUP hosts MATCHING "
+ "attr['foo'] = 1.23", -1, -1, 0 },
+ { "LOOKUP hosts MATCHING "
+ "attr['foo'] IS NULL", -1, -1, 0 },
/* comments */
{ "/* some comment */", -1, 0, 0 },
/* comments */
{ "/* some comment */", -1, 0, 0 },
/* syntax errors */
{ "INVALID", -1, -1, 0 },
{ "FETCH host", -1, -1, 0 },
/* syntax errors */
{ "INVALID", -1, -1, 0 },
{ "FETCH host", -1, -1, 0 },
- { "LIST; INVALID", 8, -1, 0 },
+ { "FETCH 'host'", -1, -1, 0 },
+ { "LIST hosts; INVALID", -1, -1, 0 },
{ "/* some incomplete", -1, -1, 0 },
{ "/* some incomplete", -1, -1, 0 },
- { "LIST MATCHING "
+ { "LIST", -1, -1, 0 },
+ { "LIST foo", -1, -1, 0 },
+ { "LIST hosts MATCHING "
+ "host = 'host'", -1, -1, 0 },
+ { "LIST foo FILTER "
"host = 'host'", -1, -1, 0 },
"host = 'host'", -1, -1, 0 },
- { "FETCH 'host' MATCHING "
+ { "FETCH host 'host' MATCHING "
+ "host = 'host'", -1, -1, 0 },
+ { "FETCH foo 'host'", -1, -1, 0 },
+ { "FETCH foo 'host' FILTER "
"host = 'host'", -1, -1, 0 },
{ "LOOKUP foo", -1, -1, 0 },
"host = 'host'", -1, -1, 0 },
{ "LOOKUP foo", -1, -1, 0 },
{ "LOOKUP foo MATCHING "
"host = 'host' FILTER "
"host = 'host'", -1, -1, 0 },
{ "LOOKUP foo MATCHING "
"host = 'host' FILTER "
"host = 'host'", -1, -1, 0 },
+ { "LOOKUP hosts MATCHING "
+ "attribute['foo'] <= "
+ "f || 'oo'", -1, -1, 0 },
+ { "LOOKUP hosts MATCHING "
+ "attribute['foo'] <= "
+ "'f' || oo", -1, -1, 0 },
};
size_t i;
};
size_t i;
int expected;
} golden_data[] = {
/* empty expressions */
int expected;
} golden_data[] = {
/* empty expressions */
- { NULL, -1, -1 },
- { "", -1, -1 },
+ { NULL, -1, -1 },
+ { "", -1, -1 },
/* valid expressions */
/* valid expressions */
- { "host = 'localhost'", -1, MATCHER_NAME },
- { "host != 'localhost'", -1, MATCHER_NOT },
- { "host =~ 'host'", -1, MATCHER_NAME },
- { "host !~ 'host'", -1, MATCHER_NOT },
- { "host = 'localhost' -- foo", -1, MATCHER_NAME },
- { "host = 'host' <garbage>", 13, MATCHER_NAME },
+ { "host = 'localhost'", -1, MATCHER_NAME },
+ { "host != 'localhost'", -1, MATCHER_NOT },
+ { "host =~ 'host'", -1, MATCHER_NAME },
+ { "host !~ 'host'", -1, MATCHER_NOT },
+ { "host = 'localhost' -- foo", -1, MATCHER_NAME },
+ { "host = 'host' <garbage>", 13, MATCHER_NAME },
/* match hosts by service */
/* match hosts by service */
- { "service = 'name'", -1, MATCHER_NAME },
- { "service != 'name'", -1, MATCHER_NOT },
- { "service =~ 'pattern'", -1, MATCHER_NAME },
- { "service !~ 'pattern'", -1, MATCHER_NOT },
+ { "service = 'name'", -1, MATCHER_NAME },
+ { "service != 'name'", -1, MATCHER_NOT },
+ { "service =~ 'pattern'", -1, MATCHER_NAME },
+ { "service !~ 'pattern'", -1, MATCHER_NOT },
/* match hosts by attribute */
/* match hosts by attribute */
- { "attribute = 'name'", -1, MATCHER_NAME },
- { "attribute != 'name'", -1, MATCHER_NOT },
- { "attribute =~ 'pattern'", -1, MATCHER_NAME },
- { "attribute !~ 'pattern'", -1, MATCHER_NOT },
+ { "attribute = 'name'", -1, MATCHER_NAME },
+ { "attribute != 'name'", -1, MATCHER_NOT },
+ { "attribute =~ 'pattern'", -1, MATCHER_NAME },
+ { "attribute !~ 'pattern'", -1, MATCHER_NOT },
/* composite expressions */
{ "host =~ 'pattern' AND "
/* composite expressions */
{ "host =~ 'pattern' AND "
- "service =~ 'pattern'", -1, MATCHER_AND },
+ "service =~ 'pattern'", -1, MATCHER_AND },
{ "host =~ 'pattern' OR "
{ "host =~ 'pattern' OR "
- "service =~ 'pattern'", -1, MATCHER_OR },
- { "NOT host = 'host'", -1, MATCHER_NOT },
+ "service =~ 'pattern'", -1, MATCHER_OR },
+ { "NOT host = 'host'", -1, MATCHER_NOT },
/* numeric expressions */
/* numeric expressions */
- { "attribute.foo < 123", -1, MATCHER_LT },
- { "attribute.foo <= 123", -1, MATCHER_LE },
- { "attribute.foo = 123", -1, MATCHER_EQ },
- { "attribute.foo >= 123", -1, MATCHER_GE },
- { "attribute.foo > 123", -1, MATCHER_GT },
+ { "attribute['foo'] < 123", -1, MATCHER_LT },
+ { "attribute['foo'] <= 123", -1, MATCHER_LE },
+ { "attribute['foo'] = 123", -1, MATCHER_EQ },
+ { "attribute['foo'] >= 123", -1, MATCHER_GE },
+ { "attribute['foo'] > 123", -1, MATCHER_GT },
+ /* datetime expressions */
+ { "attribute['foo'] = "
+ "2014-08-16", -1, MATCHER_EQ },
+ { "attribute['foo'] = "
+ "17:23", -1, MATCHER_EQ },
+ { "attribute['foo'] = "
+ "17:23:53", -1, MATCHER_EQ },
+ { "attribute['foo'] = "
+ "17:23:53.123", -1, MATCHER_EQ },
+ { "attribute['foo'] = "
+ "17:23:53.123456789", -1, MATCHER_EQ },
+ { "attribute['foo'] = "
+ "2014-08-16 17:23", -1, MATCHER_EQ },
+ { "attribute['foo'] = "
+ "2014-08-16 17:23:53", -1, MATCHER_EQ },
/* NULL; while this is an implementation detail,
* IS NULL currently maps to an equality matcher */
/* NULL; while this is an implementation detail,
* IS NULL currently maps to an equality matcher */
- { "attribute.foo IS NULL", -1, MATCHER_ISNULL },
- { "attribute.foo IS NOT NULL", -1, MATCHER_NOT },
+ { "attribute['foo'] IS NULL", -1, MATCHER_ISNULL },
+ { "attribute['foo'] IS NOT NULL", -1, MATCHER_ISNNULL },
/* object field matchers */
/* object field matchers */
- { ":last_update < 10s", -1, MATCHER_LT },
- { ":AGE <= 1m", -1, MATCHER_LE },
- { ":interval = 10h", -1, MATCHER_EQ },
- { ":Last_Update >= 24D", -1, MATCHER_GE },
- { ":age > 1M", -1, MATCHER_GT },
- { ":age != 20Y", -1, MATCHER_NOT },
- { ":backend != 'be'", -1, MATCHER_NOT },
- { ":age <= 2 * :interval", -1, MATCHER_LE },
+ { ".last_update < 10s", -1, MATCHER_LT },
+ { ".AGE <= 1m", -1, MATCHER_LE },
+ { ".interval = 10h", -1, MATCHER_EQ },
+ { ".Last_Update >= 24D", -1, MATCHER_GE },
+ { ".age > 1M", -1, MATCHER_GT },
+ { ".age != 20Y", -1, MATCHER_NOT },
+ { ".age <= 2 * .interval", -1, MATCHER_LE },
+ { "'be' IN .backend", -1, MATCHER_IN },
/* check operator precedence */
{ "host = 'name' OR "
"service = 'name' AND "
"attribute = 'name' OR "
/* check operator precedence */
{ "host = 'name' OR "
"service = 'name' AND "
"attribute = 'name' OR "
- "attribute.foo = 'bar'", -1, MATCHER_OR },
+ "attribute['foo'] = 'bar'", -1, MATCHER_OR },
{ "host = 'name' AND "
"service = 'name' AND "
"attribute = 'name' OR "
{ "host = 'name' AND "
"service = 'name' AND "
"attribute = 'name' OR "
- "attribute.foo = 'bar'", -1, MATCHER_OR },
+ "attribute['foo'] = 'bar'", -1, MATCHER_OR },
{ "host = 'name' AND "
"service = 'name' OR "
"attribute = 'name' AND "
{ "host = 'name' AND "
"service = 'name' OR "
"attribute = 'name' AND "
- "attribute.foo = 'bar'", -1, MATCHER_OR },
+ "attribute['foo'] = 'bar'", -1, MATCHER_OR },
{ "(host = 'name' OR "
"service = 'name') AND "
"(attribute = 'name' OR "
{ "(host = 'name' OR "
"service = 'name') AND "
"(attribute = 'name' OR "
- "attribute.foo = 'bar')", -1, MATCHER_AND },
+ "attribute['foo'] = 'bar')", -1, MATCHER_AND },
{ "NOT host = 'name' OR "
{ "NOT host = 'name' OR "
- "service = 'name'", -1, MATCHER_OR },
+ "service = 'name'", -1, MATCHER_OR },
{ "NOT host = 'name' OR "
{ "NOT host = 'name' OR "
- "NOT service = 'name'", -1, MATCHER_OR },
+ "NOT service = 'name'", -1, MATCHER_OR },
{ "NOT (host = 'name' OR "
{ "NOT (host = 'name' OR "
- "NOT service = 'name')", -1, MATCHER_NOT },
+ "NOT service = 'name')", -1, MATCHER_NOT },
/* syntax errors */
/* syntax errors */
- { "LIST", -1, -1 },
- { "foo &^ bar", -1, -1 },
+ { "LIST", -1, -1 },
+ { "foo &^ bar", -1, -1 },
+ { ".invalid", -1, -1 },
};
size_t i;
};
size_t i;
}
END_TEST
}
END_TEST
+START_TEST(test_parse_expr)
+{
+ struct {
+ const char *expr;
+ int len;
+ int expected;
+ } golden_data[] = {
+ /* empty expressions */
+ { NULL, -1, INT_MAX },
+ { "", -1, INT_MAX },
+
+ /* constant expressions */
+ { "'localhost'", -1, 0 },
+ { "123", -1, 0 },
+ { "2014-08-16", -1, 0 },
+ { "17:23", -1, 0 },
+ { "17:23:53", -1, 0 },
+ { "17:23:53.123", -1, 0 },
+ { "17:23:53.123456789", -1, 0 },
+ { "2014-08-16 17:23", -1, 0 },
+ { "2014-08-16 17:23:53", -1, 0 },
+ { "10s", -1, 0 },
+ { "60m", -1, 0 },
+ { "10Y 24D 1h", -1, 0 },
+
+ { "123 + 456", -1, 0 },
+ { "'foo' || 'bar'", -1, 0 },
+ { "456 - 123", -1, 0 },
+ { "1.2 * 3.4", -1, 0 },
+ { "1.2 / 3.4", -1, 0 },
+ { "5 % 2", -1, 0 },
+
+ /* queryable fields */
+ { ".last_update", -1, FIELD_VALUE },
+ { ".AGE", -1, FIELD_VALUE },
+ { ".interval", -1, FIELD_VALUE },
+ { ".Last_Update", -1, FIELD_VALUE },
+ { ".backend", -1, FIELD_VALUE },
+
+ /* attributes */
+ { "attribute['foo']", -1, ATTR_VALUE },
+
+ /* arithmetic expressions */
+ { ".age + .age", -1, SDB_DATA_ADD },
+ { ".age - .age", -1, SDB_DATA_SUB },
+ { ".age * .age", -1, SDB_DATA_MUL },
+ { ".age / .age", -1, SDB_DATA_DIV },
+ { ".age % .age", -1, SDB_DATA_MOD },
+ { ".age || .age", -1, SDB_DATA_CONCAT },
+
+ /* operator precedence */
+ { ".age + .age * .age", -1, SDB_DATA_ADD },
+ { ".age * .age + .age", -1, SDB_DATA_ADD },
+ { ".age + .age - .age", -1, SDB_DATA_SUB },
+ { ".age - .age + .age", -1, SDB_DATA_ADD },
+ { "(.age + .age) * .age", -1, SDB_DATA_MUL },
+ { ".age + (.age * .age)", -1, SDB_DATA_ADD },
+
+ /* syntax errors */
+ { "LIST", -1, INT_MAX },
+ { "foo &^ bar", -1, INT_MAX },
+ { ".invalid", -1, INT_MAX },
+ };
+
+ size_t i;
+
+ for (i = 0; i < SDB_STATIC_ARRAY_LEN(golden_data); ++i) {
+ sdb_store_expr_t *e;
+ e = sdb_fe_parse_expr(golden_data[i].expr, golden_data[i].len);
+
+ if (golden_data[i].expected == INT_MAX) {
+ fail_unless(e == NULL,
+ "sdb_fe_parse_expr(%s) = %p; expected: NULL",
+ golden_data[i].expr, e);
+ continue;
+ }
+
+ fail_unless(e != NULL, "sdb_fe_parse_expr(%s) = NULL; "
+ "expected: <expr>", golden_data[i].expr);
+ fail_unless(e->type == golden_data[i].expected,
+ "sdb_fe_parse_expr(%s) returned expression of type %d; "
+ "expected: %d", golden_data[i].expr, e->type,
+ golden_data[i].expected);
+
+ sdb_object_deref(SDB_OBJ(e));
+ }
+}
+END_TEST
+
Suite *
fe_parser_suite(void)
{
Suite *
fe_parser_suite(void)
{
tc = tcase_create("core");
tcase_add_test(tc, test_parse);
tcase_add_test(tc, test_parse_matcher);
tc = tcase_create("core");
tcase_add_test(tc, test_parse);
tcase_add_test(tc, test_parse_matcher);
+ tcase_add_test(tc, test_parse_expr);
suite_add_tcase(s, tc);
return s;
suite_add_tcase(s, tc);
return s;