X-Git-Url: https://git.tokkee.org/?a=blobdiff_plain;f=t%2Funit%2Fcore%2Fstore_lookup_test.c;h=1d4bf7aa267bda7a358de48ce58dd788391691aa;hb=40cb189b1d8f15349a57c7215905698ea12fa0c8;hp=e90dd06c1acee054d6388dcbd0ef7e5b99c4e4d0;hpb=383196c820697409670d67f59f180300b4d6bcfa;p=sysdb.git

diff --git a/t/unit/core/store_lookup_test.c b/t/unit/core/store_lookup_test.c
index e90dd06..1d4bf7a 100644
--- a/t/unit/core/store_lookup_test.c
+++ b/t/unit/core/store_lookup_test.c
@@ -85,7 +85,7 @@ populate(void)
 
 START_TEST(test_store_match_name)
 {
-	sdb_store_base_t *obj;
+	sdb_store_obj_t *obj;
 
 	struct {
 		int type;
@@ -170,7 +170,7 @@ END_TEST
 
 START_TEST(test_store_match_attr)
 {
-	sdb_store_base_t *obj;
+	sdb_store_obj_t *obj;
 
 	struct {
 		const char *attr_name;
@@ -184,8 +184,6 @@ START_TEST(test_store_match_attr)
 		{ "1",  NULL,   1, 0 },
 		{ "k3", NULL,   0, 0 },
 		{ "k3", NULL,   1, 0 },
-		{ "k3", NULL,   0, 0 },
-		{ "k3", NULL,   1, 0 },
 		{ "k1", "v1",   0, 1 },
 		{ "k1", "v1",   1, 1 },
 		{ "k1", "^v1$", 1, 1 },
@@ -247,7 +245,7 @@ END_TEST
 
 START_TEST(test_store_cond)
 {
-	sdb_store_base_t *obj;
+	sdb_store_obj_t *obj;
 
 	struct {
 		const char *attr;
@@ -327,7 +325,7 @@ END_TEST
 
 START_TEST(test_store_match_op)
 {
-	sdb_store_base_t *obj;
+	sdb_store_obj_t *obj;
 
 	sdb_store_matcher_t *always = sdb_store_name_matcher(SDB_HOST, "a", 0);
 	sdb_store_matcher_t *never = sdb_store_name_matcher(SDB_HOST, "z", 0);
@@ -397,6 +395,10 @@ END_TEST
 
 START_TEST(test_parse_cmp)
 {
+	sdb_data_t hostname = { SDB_TYPE_STRING, { .string = "hostname" } };
+	sdb_data_t srvname  = { SDB_TYPE_STRING, { .string = "srvname" } };
+	sdb_data_t attrname = { SDB_TYPE_STRING, { .string = "attrname" } };
+
 	sdb_store_matcher_t *check;
 
 	size_t i;
@@ -405,56 +407,80 @@ START_TEST(test_parse_cmp)
 		const char *obj_type;
 		const char *attr;
 		const char *op;
-		const char *value;
+		const sdb_data_t value;
 		int expected;
 	} golden_data[] = {
-		{ "host",      "name", "=",  "hostname", MATCHER_NAME },
-		{ "host",      "name", "!=", "hostname", MATCHER_NOT },
-		{ "host",      "name", "=~", "hostname", MATCHER_NAME },
-		{ "host",      "name", "!~", "hostname", MATCHER_NOT },
-		{ "host",      "attr", "=",  "hostname", -1 },
-		{ "host",      "attr", "!=", "hostname", -1 },
-		{ "host",      "name", "&^", "hostname", -1 },
-		{ "service",   "name", "=",  "srvname",  MATCHER_NAME },
-		{ "service",   "name", "!=", "srvname",  MATCHER_NOT },
-		{ "service",   "name", "=~", "srvname",  MATCHER_NAME },
-		{ "service",   "name", "!~", "srvname",  MATCHER_NOT },
-		{ "service",   "attr", "=",  "srvname",  -1 },
-		{ "service",   "attr", "!=", "srvname",  -1 },
-		{ "service",   "name", "&^", "srvname",  -1 },
-		{ "attribute", "name", "=",  "attrname", MATCHER_NAME },
-		{ "attribute", "name", "!=", "attrname", MATCHER_NOT },
-		{ "attribute", "name", "=~", "attrname", MATCHER_NAME },
-		{ "attribute", "name", "!~", "attrname", MATCHER_NOT },
-		{ "attribute", "attr", "=",  "attrname", MATCHER_ATTR },
-		{ "attribute", "attr", "!=", "attrname", MATCHER_NOT },
-		{ "attribute", "attr", "=~", "attrname", MATCHER_ATTR },
-		{ "attribute", "attr", "!~", "attrname", MATCHER_NOT },
-		{ "attribute", "attr", "&^", "attrname", -1 },
+		{ "host",      "name", "=",  hostname, MATCHER_NAME },
+		{ "host",      "name", "!=", hostname, MATCHER_NOT },
+		{ "host",      "name", "=~", hostname, MATCHER_NAME },
+		{ "host",      "name", "!~", hostname, MATCHER_NOT },
+		{ "host",      "attr", "=",  hostname, -1 },
+		{ "host",      "attr", "!=", hostname, -1 },
+		{ "host",      "name", "&^", hostname, -1 },
+		{ "host",      "name", "<",  hostname, -1 },
+		{ "host",      "name", "<=", hostname, -1 },
+		{ "host",      "name", ">=", hostname, -1 },
+		{ "host",      "name", ">",  hostname, -1 },
+		{ "service",   "name", "=",  srvname,  MATCHER_NAME },
+		{ "service",   "name", "!=", srvname,  MATCHER_NOT },
+		{ "service",   "name", "=~", srvname,  MATCHER_NAME },
+		{ "service",   "name", "!~", srvname,  MATCHER_NOT },
+		{ "service",   "attr", "=",  srvname,  -1 },
+		{ "service",   "attr", "!=", srvname,  -1 },
+		{ "service",   "name", "&^", srvname,  -1 },
+		{ "service",   "name", "<",  srvname,  -1 },
+		{ "service",   "name", "<=", srvname,  -1 },
+		{ "service",   "name", ">=", srvname,  -1 },
+		{ "service",   "name", ">",  srvname,  -1 },
+		{ "attribute", "name", "=",  attrname, MATCHER_NAME },
+		{ "attribute", "name", "!=", attrname, MATCHER_NOT },
+		{ "attribute", "name", "=~", attrname, MATCHER_NAME },
+		{ "attribute", "name", "!~", attrname, MATCHER_NOT },
+		{ "attribute", "name", "<",  attrname, -1 },
+		{ "attribute", "name", "<=", attrname, -1 },
+		{ "attribute", "name", ">=", attrname, -1 },
+		{ "attribute", "name", ">",  attrname, -1 },
+		{ "attribute", "attr", "=",  attrname, MATCHER_ATTR },
+		{ "attribute", "attr", "!=", attrname, MATCHER_NOT },
+		{ "attribute", "attr", "=~", attrname, MATCHER_ATTR },
+		{ "attribute", "attr", "!~", attrname, MATCHER_NOT },
+		{ "attribute", "attr", "&^", attrname, -1 },
+		{ "attribute", "attr", "<",  attrname, MATCHER_LT },
+		{ "attribute", "attr", "<=", attrname, MATCHER_LE },
+/*		{ "attribute", "attr", "=",  attrname, MATCHER_EQ }, */
+		{ "attribute", "attr", ">=", attrname, MATCHER_GE },
+		{ "attribute", "attr", ">",  attrname, MATCHER_GT },
+		{ "foo",       "name", "=",  attrname, -1 },
+		{ "foo",       "attr", "=",  attrname, -1 },
 	};
 
 	for (i = 0; i < SDB_STATIC_ARRAY_LEN(golden_data); ++i) {
+		char buf[1024];
+
 		check = sdb_store_matcher_parse_cmp(golden_data[i].obj_type,
-				golden_data[i].attr, golden_data[i].op, golden_data[i].value);
+				golden_data[i].attr, golden_data[i].op,
+				&golden_data[i].value);
+
+		if (sdb_data_format(&golden_data[i].value,
+					buf, sizeof(buf), SDB_UNQUOTED) < 0)
+			snprintf(buf, sizeof(buf), "ERR");
 
 		if (golden_data[i].expected == -1) {
 			fail_unless(check == NULL,
 					"sdb_store_matcher_parse_cmp(%s, %s, %s, %s) = %p; "
 					"expected: NULL", golden_data[i].obj_type,
-					golden_data[i].attr, golden_data[i].op,
-					golden_data[i].value, check);
+					golden_data[i].attr, golden_data[i].op, buf, check);
 			continue;
 		}
 
 		fail_unless(check != NULL,
 				"sdb_store_matcher_parse_cmp(%s, %s, %s, %s) = %p; "
 				"expected: NULL", golden_data[i].obj_type,
-				golden_data[i].attr, golden_data[i].op,
-				golden_data[i].value, check);
+				golden_data[i].attr, golden_data[i].op, buf, check);
 		fail_unless(M(check)->type == golden_data[i].expected,
 				"sdb_store_matcher_parse_cmp(%s, %s, %s, %s) returned matcher "
 				"of type %d; expected: %d", golden_data[i].obj_type,
-				golden_data[i].attr, golden_data[i].op, golden_data[i].value,
+				golden_data[i].attr, golden_data[i].op, buf,
 				M(check)->type, golden_data[i].expected);
 
 		sdb_object_deref(SDB_OBJ(check));
@@ -463,7 +489,7 @@ START_TEST(test_parse_cmp)
 END_TEST
 
 static int
-lookup_cb(sdb_store_base_t *obj, void *user_data)
+lookup_cb(sdb_store_obj_t *obj, void *user_data)
 {
 	int *i = user_data;
 
@@ -506,6 +532,18 @@ START_TEST(test_lookup)
 			"OBJ\\[attribute\\]\\{ NAME\\{ 'x', \\(nil\\) \\}" },
 		{ "attribute.k1 = 'v1'",   1,
 			"ATTR\\[k1\\]\\{ VALUE\\{ 'v1', \\(nil\\) \\} \\}" },
+		{ "attribute.k2 < 123",    0,
+			"ATTR\\[k2\\]\\{ < 123 \\}" },
+		{ "attribute.k2 <= 123",   1,
+			"ATTR\\[k2\\]\\{ <= 123 \\}" },
+		{ "attribute.k2 >= 123",   1,
+			"ATTR\\[k2\\]\\{ >= 123 \\}" },
+		{ "attribute.k2 > 123",    0,
+			"ATTR\\[k2\\]\\{ > 123 \\}" },
+		{ "attribute.k2 = 123",    1,
+			"ATTR\\[k2\\]\\{ = 123 \\}" },
+		{ "attribute.k2 != 123",   2,
+			"\\(NOT, ATTR\\[k2\\]\\{ = 123 \\}\\)" },
 		{ "attribute.k1 != 'v1'",  2,
 			"\\(NOT, ATTR\\[k1\\]\\{ VALUE\\{ 'v1', \\(nil\\) \\} \\}\\)" },
 		{ "attribute.k1 != 'v2'",  3,