[tokkee]

tokkee.org

Code

projects / sysdb.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
parser_test: Add testcases covering operand type checks.
[sysdb.git] / t / unit / frontend / parser_test.c
diff --git a/t/unit/frontend/parser_test.c b/t/unit/frontend/parser_test.c
index 2368bcc4d3efe821a5342aa25ee07316ff69e7e1..4926f7662271cf21459bd886631b93cf64ddbdf5 100644 (file)
--- a/t/unit/frontend/parser_test.c
+++ b/t/unit/frontend/parser_test.c
@@ -1,6 +1,6 @@
 /*
  * SysDB - t/unit/frontend/parser_test.c
- * Copyright (C) 2013 Sebastian 'tokkee' Harl <sh@tokkee.org>
+ * Copyright (C) 2013-2015 Sebastian 'tokkee' Harl <sh@tokkee.org>
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -52,7 +52,7 @@ START_TEST(test_parse)
                { ";",                   -1,  0, 0 },
                { ";;",                  -1,  0, 0 },
 
-               /* valid commands */
+               /* FETCH commands */
                { "FETCH host 'host'",   -1,  1, SDB_CONNECTION_FETCH  },
                { "FETCH host 'host' FILTER "
                  "age > 60s",           -1,  1, SDB_CONNECTION_FETCH  },
@@ -61,6 +61,7 @@ START_TEST(test_parse)
                { "FETCH metric "
                  "'host'.'metric'",     -1,  1, SDB_CONNECTION_FETCH  },
 
+               /* LIST commands */
                { "LIST hosts",          -1,  1, SDB_CONNECTION_LIST   },
                { "LIST hosts -- foo",   -1,  1, SDB_CONNECTION_LIST   },
                { "LIST hosts;",         -1,  1, SDB_CONNECTION_LIST   },
@@ -74,6 +75,7 @@ START_TEST(test_parse)
                { "LIST metrics FILTER "
                  "age > 60s",           -1,  1, SDB_CONNECTION_LIST   },
 
+               /* LOOKUP commands */
                { "LOOKUP hosts",        -1,  1, SDB_CONNECTION_LOOKUP },
                { "LOOKUP hosts MATCHING "
                  "name = 'host'",       -1,  1, SDB_CONNECTION_LOOKUP },
@@ -107,13 +109,24 @@ START_TEST(test_parse)
                  "name =~ 'p' "
                  "FILTER age>"
                  "interval",           -1,   1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "host.name =~ 'p'",   -1,   1, SDB_CONNECTION_LOOKUP },
                { "LOOKUP services",    -1,   1, SDB_CONNECTION_LOOKUP },
                { "LOOKUP services MATCHING ANY "
                  "attribute =~ 'a'",   -1,   1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP services MATCHING "
+                 "host.name = 'p'",    -1,   1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP services MATCHING "
+                 "service.name = 'p'", -1,   1, SDB_CONNECTION_LOOKUP },
                { "LOOKUP metrics",     -1,   1, SDB_CONNECTION_LOOKUP },
                { "LOOKUP metrics MATCHING ANY "
                  "attribute =~ 'a'",   -1,   1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP metrics MATCHING "
+                 "host.name = 'p'",    -1,   1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP metrics MATCHING "
+                 "metric.name = 'p'",  -1,   1, SDB_CONNECTION_LOOKUP },
 
+               /* TIMESERIES commands */
                { "TIMESERIES 'host'.'metric' "
                  "START 2014-01-01 "
                  "END 2014-12-31 "
@@ -126,6 +139,51 @@ START_TEST(test_parse)
                { "TIMESERIES "
                  "'host'.'metric'",     -1,  1, SDB_CONNECTION_TIMESERIES },
 
+               /* STORE commands */
+               { "STORE host 'host'",   -1,  1, SDB_CONNECTION_STORE_HOST },
+               { "STORE host 'host' "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1,  1, SDB_CONNECTION_STORE_HOST },
+               { "STORE host attribute "
+                 "'host'.'key' 123",    -1,  1, SDB_CONNECTION_STORE_ATTRIBUTE },
+               { "STORE host attribute "
+                 "'host'.'key' 123 "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1,  1, SDB_CONNECTION_STORE_ATTRIBUTE },
+               { "STORE service "
+                 "'host'.'svc'",        -1,  1, SDB_CONNECTION_STORE_SERVICE },
+               { "STORE service "
+                 "'host'.'svc' "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1,  1, SDB_CONNECTION_STORE_SERVICE },
+               { "STORE service attribute "
+                 "'host'.'svc'.'key' "
+                 "123",                 -1,  1, SDB_CONNECTION_STORE_ATTRIBUTE },
+               { "STORE service attribute "
+                 "'host'.'svc'.'key' "
+                 "123 "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1,  1, SDB_CONNECTION_STORE_ATTRIBUTE },
+               { "STORE metric "
+                 "'host'.'metric'",     -1,  1, SDB_CONNECTION_STORE_METRIC },
+               { "STORE metric "
+                 "'host'.'metric' "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1,  1, SDB_CONNECTION_STORE_METRIC },
+               { "STORE metric "
+                 "'host'.'metric' "
+                 "STORE 'typ' 'id' "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1,  1, SDB_CONNECTION_STORE_METRIC },
+               { "STORE metric attribute "
+                 "'host'.'metric'.'key' "
+                 "123",                 -1,  1, SDB_CONNECTION_STORE_ATTRIBUTE },
+               { "STORE metric attribute "
+                 "'host'.'metric'.'key' "
+                 "123 "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1,  1, SDB_CONNECTION_STORE_ATTRIBUTE },
+
                /* string constants */
                { "LOOKUP hosts MATCHING "
                  "name = ''''",         -1,  1, SDB_CONNECTION_LOOKUP },
@@ -186,6 +244,30 @@ START_TEST(test_parse)
                  "1Y42D",               -1,  1, SDB_CONNECTION_LOOKUP },
                 */
 
+               /* array constants */
+               { "LOOKUP hosts MATCHING "
+                 "backend = ['foo']",   -1,  1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "backend = ['a','b']", -1,  1, SDB_CONNECTION_LOOKUP },
+
+               /* valid operand types */
+               { "LOOKUP hosts MATCHING "
+                 "age * 1 > 0s",        -1,  1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "age / 1 > 0s",        -1,  1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "name > ''",           -1,  1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "name >= ''",          -1,  1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "name != ''",          -1,  1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "name = ''",           -1,  1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "name <= ''",          -1,  1, SDB_CONNECTION_LOOKUP },
+               { "LOOKUP hosts MATCHING "
+                 "name < ''",           -1,  1, SDB_CONNECTION_LOOKUP },
+
                /* NULL */
                { "LOOKUP hosts MATCHING "
                  "attribute['foo'] "
@@ -226,6 +308,37 @@ START_TEST(test_parse)
                { "LOOKUP hosts MATCHING "
                  "attr['foo'] IS NULL", -1, -1, 0 },
 
+               /* type mismatches */
+               { "LOOKUP hosts MATCHING "
+                 "age > 0",             -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "age >= 0",            -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "age = 0",             -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "age != 0",            -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "age <= 0",            -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "age < 0",             -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "age + 1 > 0s",        -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "age - 1 > 0s",        -1, -1, 0 },
+               /* datetime <mul/div> integer is allowed */
+               { "LOOKUP hosts MATCHING "
+                 "age || 1 > 0s",       -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "name + 1 = ''",       -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "name - 1 = ''",       -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "name * 1 = ''",       -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "name / 1 = ''",       -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "name % 1 = ''",       -1, -1, 0 },
+
                /* comments */
                { "/* some comment */",  -1,  0, 0 },
                { "-- another comment",  -1,  0, 0 },
@@ -237,13 +350,15 @@ START_TEST(test_parse)
                { "LIST hosts; INVALID", -1, -1, 0 },
                { "/* some incomplete",  -1, -1, 0 },
 
-               /* invalid commands */
+               /* invalid LIST commands */
                { "LIST",                -1, -1, 0 },
                { "LIST foo",            -1, -1, 0 },
                { "LIST hosts MATCHING "
                  "name = 'host'",       -1, -1, 0 },
                { "LIST foo FILTER "
                  "age > 60s",           -1, -1, 0 },
+
+               /* invalid FETCH commands */
                { "FETCH host 'host' MATCHING "
                  "name = 'host'",       -1, -1, 0 },
                { "FETCH service 'host'",-1, -1, 0 },
@@ -254,6 +369,7 @@ START_TEST(test_parse)
                { "FETCH foo 'host' FILTER "
                  "age > 60s",           -1, -1, 0 },
 
+               /* invalid LOOKUP commands */
                { "LOOKUP foo",          -1, -1, 0 },
                { "LOOKUP foo MATCHING "
                  "name = 'host'",       -1, -1, 0 },
@@ -270,18 +386,48 @@ START_TEST(test_parse)
                  "'f' || oo",           -1, -1, 0 },
                { "LOOKUP hosts MATCHING "
                  "ANY host = 'host'",   -1, -1, 0 },
+               { "LOOKUP hosts MATCHING "
+                 "service.name = 's'",  -1, -1, 0 },
                { "LOOKUP services MATCHING "
                  "ANY host = 'host'",   -1, -1, 0 },
                { "LOOKUP services MATCHING "
                  "ANY service = 'svc'", -1, -1, 0 },
                { "LOOKUP services MATCHING "
                  "ANY metric = 'm'",    -1, -1, 0 },
+               { "LOOKUP services MATCHING "
+                 "metric.name = 'm'",   -1, -1, 0 },
                { "LOOKUP metrics MATCHING "
                  "ANY host = 'host'",   -1, -1, 0 },
                { "LOOKUP metrics MATCHING "
                  "ANY service = 'svc'", -1, -1, 0 },
                { "LOOKUP metrics MATCHING "
                  "ANY metric = 'm'",    -1, -1, 0 },
+               { "LOOKUP metrics MATCHING "
+                 "service.name = 'm'",  -1, -1, 0 },
+
+               /* invalid STORE commands */
+               { "STORE host "
+                 "'obj'.'host'",        -1, -1, 0 },
+               { "STORE host attribute "
+                 ".'key' 123",          -1, -1, 0 },
+               { "STORE host attribute "
+                 "'o'.'h'.'key' 123",   -1, -1, 0 },
+               { "STORE service 'svc'", -1, -1, 0 },
+               { "STORE service "
+                 "'host'.'svc' "
+                 "STORE 'typ' 'id' "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1, -1, 0 },
+               { "STORE service attribute "
+                 "'svc'.'key' 123",     -1, -1, 0 },
+               { "STORE metric 'm'",    -1, -1, 0 },
+               { "STORE metric "
+                 "'host'.'metric' "
+                 "STORE 'typ'.'id' "
+                 "LAST UPDATE "
+                 "2015-02-01",          -1, -1, 0 },
+               { "STORE metric attribute "
+                 "'metric'.'key' 123",  -1, -1, 0 },
        };
 
        sdb_strbuf_t *errbuf = sdb_strbuf_create(64);
@@ -452,6 +598,14 @@ START_TEST(test_parse_matcher)
                 * IS NULL currently maps to an equality matcher */
                { "attribute['foo'] IS NULL",     -1,  MATCHER_ISNULL },
                { "attribute['foo'] IS NOT NULL", -1,  MATCHER_ISNNULL },
+               /* array expressions */
+               { "backend < ['a']",              -1,  MATCHER_LT },
+               { "backend <= ['a']",             -1,  MATCHER_LE },
+               { "backend = ['a']",              -1,  MATCHER_EQ },
+               { "backend != ['a']",             -1,  MATCHER_NE },
+               { "backend >= ['a']",             -1,  MATCHER_GE },
+               { "backend > ['a']",              -1,  MATCHER_GT },
+               { "backend &^ ['a']",             -1,  -1 },
 
                /* object field matchers */
                { "name < 'a'",                   -1,  MATCHER_LT },
System DataBase (SysDB)