tests/check_session.c: Fix compiler warning.

[libsigrokdecode.git] / tests / check_session.c

diff --git a/tests/check_session.c b/tests/check_session.c
index 19e7adb376a9fce026e43e50e05ee216e7969a40..f996df5849d391618d788149333b085e432c2be2 100644 (file)
--- a/tests/check_session.c
+++ b/tests/check_session.c
@@ -18,10 +18,12 @@
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
  */
 
-#include "../libsigrokdecode.h" /* First, to avoid compiler warning. */
-#include "../libsigrokdecode-internal.h"
+#include "../libsigrokdecode-internal.h" /* First, to avoid compiler warning. */
+#include "../libsigrokdecode.h"
+#include <stdint.h>
 #include <stdlib.h>
 #include <check.h>
+#include "lib.h"
 
 static void setup(void)
 {
@@ -148,8 +150,8 @@ static void conf_check_ok(struct srd_session *sess, int key, uint64_t x)
 {
        int ret;
 
-       ret = srd_session_config_set(sess, key, g_variant_new_uint64(x));
-       fail_unless(ret == SRD_OK, "srd_session_config_set(%p, %d, %"
+       ret = srd_session_metadata_set(sess, key, g_variant_new_uint64(x));
+       fail_unless(ret == SRD_OK, "srd_session_metadata_set(%p, %d, %"
                PRIu64 ") failed: %d.", sess, key, x, ret);
 }
 
@@ -157,8 +159,8 @@ static void conf_check_fail(struct srd_session *sess, int key, uint64_t x)
 {
        int ret;
 
-       ret = srd_session_config_set(sess, key, g_variant_new_uint64(x));
-       fail_unless(ret != SRD_OK, "srd_session_config_set(%p, %d, %"
+       ret = srd_session_metadata_set(sess, key, g_variant_new_uint64(x));
+       fail_unless(ret != SRD_OK, "srd_session_metadata_set(%p, %d, %"
                PRIu64 ") worked.", sess, key, x);
 }
 
@@ -166,25 +168,25 @@ static void conf_check_fail_null(struct srd_session *sess, int key)
 {
        int ret;
 
-       ret = srd_session_config_set(sess, key, NULL);
+       ret = srd_session_metadata_set(sess, key, NULL);
        fail_unless(ret != SRD_OK,
-               "srd_session_config_set(NULL) for key %d worked.", key);
+               "srd_session_metadata_set(NULL) for key %d worked.", key);
 }
 
 static void conf_check_fail_str(struct srd_session *sess, int key, const char *s)
 {
        int ret;
 
-       ret = srd_session_config_set(sess, key, g_variant_new_string(s));
-       fail_unless(ret != SRD_OK, "srd_session_config_set for key %d "
+       ret = srd_session_metadata_set(sess, key, g_variant_new_string(s));
+       fail_unless(ret != SRD_OK, "srd_session_metadata_set() for key %d "
                "failed: %d.", key, ret);
 }
 
 /*
- * Check whether srd_session_config_set() works.
+ * Check whether srd_session_metadata_set() works.
  * If it returns != SRD_OK (or segfaults) this test will fail.
  */
-START_TEST(test_session_config_set)
+START_TEST(test_session_metadata_set)
 {
        uint64_t i;
        struct srd_session *sess;
@@ -192,25 +194,20 @@ START_TEST(test_session_config_set)
        srd_init(NULL);
        srd_session_new(&sess);
        /* Try a bunch of values. */
-       for (i = 0; i < 1000; i++) {
-               conf_check_ok(sess, SRD_CONF_NUM_PROBES, i);
-               conf_check_ok(sess, SRD_CONF_UNITSIZE, i);
+       for (i = 0; i < 1000; i++)
                conf_check_ok(sess, SRD_CONF_SAMPLERATE, i);
-       }
        /* Try the max. possible value. */
-       conf_check_ok(sess, SRD_CONF_NUM_PROBES, 18446744073709551615ULL);
-       conf_check_ok(sess, SRD_CONF_UNITSIZE, 18446744073709551615ULL);
-       conf_check_ok(sess, SRD_CONF_SAMPLERATE, 18446744073709551615ULL);
+       conf_check_ok(sess, SRD_CONF_SAMPLERATE, UINT64_MAX);
        srd_session_destroy(sess);
        srd_exit();
 }
 END_TEST
 
 /*
- * Check whether srd_session_config_set() fails with invalid input.
+ * Check whether srd_session_metadata_set() fails with invalid input.
  * If it returns SRD_OK (or segfaults) this test will fail.
  */
-START_TEST(test_session_config_set_bogus)
+START_TEST(test_session_metadata_set_bogus)
 {
        struct srd_session *sess;
 
@@ -218,21 +215,13 @@ START_TEST(test_session_config_set_bogus)
        srd_session_new(&sess);
 
        /* Incorrect gvariant type (currently only uint64 is used). */
-       conf_check_fail_str(sess, SRD_CONF_NUM_PROBES, "");
-       conf_check_fail_str(sess, SRD_CONF_UNITSIZE, "");
        conf_check_fail_str(sess, SRD_CONF_SAMPLERATE, "");
-       conf_check_fail_str(sess, SRD_CONF_NUM_PROBES, "Foo");
-       conf_check_fail_str(sess, SRD_CONF_UNITSIZE, "Foo");
        conf_check_fail_str(sess, SRD_CONF_SAMPLERATE, "Foo");
 
        /* NULL data pointer. */
-       conf_check_fail_null(sess, SRD_CONF_NUM_PROBES);
-       conf_check_fail_null(sess, SRD_CONF_UNITSIZE);
        conf_check_fail_null(sess, SRD_CONF_SAMPLERATE);
 
        /* NULL session. */
-       conf_check_fail(NULL, SRD_CONF_NUM_PROBES, 0);
-       conf_check_fail(NULL, SRD_CONF_UNITSIZE, 0);
        conf_check_fail(NULL, SRD_CONF_SAMPLERATE, 0);
 
        /* Invalid keys. */
@@ -263,8 +252,8 @@ Suite *suite_session(void)
 
        tc = tcase_create("config");
        tcase_add_checked_fixture(tc, setup, teardown);
-       tcase_add_test(tc, test_session_config_set);
-       tcase_add_test(tc, test_session_config_set_bogus);
+       tcase_add_test(tc, test_session_metadata_set);
+       tcase_add_test(tc, test_session_metadata_set_bogus);
        suite_add_tcase(s, tc);
 
        return s;