X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=tests%2Fstrutil.c;h=f4bbe9ec64537987d84290e0d7369ad166561eff;hb=74eeebfb6da68d128575eaeebdeeb588f7787808;hp=82856f1dd4a4d8e719bc0c5b5b9326fa6334a2a7;hpb=74d915331a41d521eb48edf5f734a1ea2ec82df9;p=libsigrok.git

diff --git a/tests/strutil.c b/tests/strutil.c
index 82856f1d..f4bbe9ec 100644
--- a/tests/strutil.c
+++ b/tests/strutil.c
@@ -121,10 +121,19 @@ static void test_rational(const char *input, struct sr_rational expected)
 	fail_unless(ret == SR_OK, "Unexpected rc for '%s': %d, errno %d.",
 		input, ret, errno);
 	fail_unless((expected.p == rational.p) && (expected.q == rational.q),
-		    "Invalid result for '%s': %ld/%ld'.",
+		    "Invalid result for '%s': %" PRIi64 "/%" PRIu64 "'.",
 		    input, rational.p, rational.q);
 }
 
+static void test_rational_fail(const char *input)
+{
+	int ret;
+	struct sr_rational rational;
+
+	ret = sr_parse_rational(input, &rational);
+	fail_unless(ret != SR_OK, "Unexpected success for '%s'.", input);
+}
+
 static void test_voltage(uint64_t v_p, uint64_t v_q, const char *expected)
 {
 	char *s;
@@ -145,7 +154,7 @@ START_TEST(test_locale)
 	fprintf(stderr, "Old locale = %s\n", old_locale);
 	/* Copy the name so it won’t be clobbered by setlocale. */
 	saved_locale = g_strdup(old_locale);
-	ck_assert_msg(saved_locale != NULL);
+	ck_assert(saved_locale != NULL);
 
 #ifdef _WIN32
 	/*
@@ -395,6 +404,9 @@ START_TEST(test_fractional)
 	test_rational("-.1", (struct sr_rational){-1, 10});
 	test_rational(" .1", (struct sr_rational){1, 10});
 	test_rational("+.1", (struct sr_rational){1, 10});
+	test_rational_fail(".");
+	test_rational_fail(".e");
+	test_rational_fail(".e1");
 }
 END_TEST