lib: add internal object pool API and use it; adapt plugins/tests
[babeltrace.git] / tests / lib / test_ctf_writer.c
index 8b502d379e33168d22d75da92969298936f6ab8e..928ff982183f7f5b2f2716779d35c5cf2d508a60 100644 (file)
@@ -26,9 +26,7 @@
 #include <babeltrace/ctf-writer/event-types.h>
 #include <babeltrace/ctf-writer/event-fields.h>
 #include <babeltrace/ctf-writer/stream-class.h>
-#include <babeltrace/ctf-ir/packet.h>
-#include <babeltrace/ctf-ir/clock-class.h>
-#include <babeltrace/ctf-ir/trace.h>
+#include <babeltrace/ctf-writer/trace.h>
 #include <babeltrace/ref.h>
 #include <babeltrace/ctf/events.h>
 #include <babeltrace/values.h>
@@ -59,7 +57,7 @@
 #define DEFAULT_CLOCK_TIME 0
 #define DEFAULT_CLOCK_VALUE 0
 
-#define NR_TESTS 623
+#define NR_TESTS 346
 
 struct bt_utsname {
        char sysname[BABELTRACE_HOST_NAME_MAX];
@@ -98,9 +96,8 @@ static
 void validate_trace(char *parser_path, char *trace_path)
 {
        int ret = 0;
-       gchar *standard_error = NULL;
        gint exit_status;
-       char *argv[] = {parser_path, trace_path, NULL};
+       char *argv[] = {parser_path, trace_path, "-o", "dummy", NULL};
 
        if (!parser_path || !trace_path) {
                ret = -1;
@@ -114,7 +111,7 @@ void validate_trace(char *parser_path, char *trace_path)
                        NULL,
                        NULL,
                        NULL,
-                       &standard_error,
+                       NULL,
                        &exit_status,
                        NULL)) {
                diag("Failed to spawn babeltrace.");
@@ -123,7 +120,7 @@ void validate_trace(char *parser_path, char *trace_path)
        }
 
        /* Replace by g_spawn_check_exit_status when we require glib >= 2.34 */
-#if G_OS_UNIX
+#ifdef G_OS_UNIX
        ret = WIFEXITED(exit_status) ? WEXITSTATUS(exit_status) : -1;
 #else
        ret = exit_status;
@@ -131,13 +128,11 @@ void validate_trace(char *parser_path, char *trace_path)
 
        if (ret != 0) {
                diag("Babeltrace returned an error.");
-               diag_multiline(standard_error);
                goto result;
        }
 
 result:
        ok(ret == 0, "Babeltrace could read the resulting trace");
-       g_free(standard_error);
 }
 
 static
@@ -194,16 +189,13 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
        ok(int_64_type, "Create a signed integer type");
        enum_type = bt_ctf_field_type_enumeration_create(int_64_type);
 
-       returned_type = bt_ctf_field_type_enumeration_get_container_type(enum_type);
-       ok(returned_type == int_64_type, "bt_ctf_field_type_enumeration_get_container_type returns the right type");
-       ok(!bt_ctf_field_type_enumeration_get_container_type(NULL), "bt_ctf_field_type_enumeration_get_container_type handles NULL correctly");
+       returned_type = bt_ctf_field_type_enumeration_get_container_field_type(enum_type);
+       ok(returned_type == int_64_type, "bt_ctf_field_type_enumeration_get_container_field_type returns the right type");
        ok(!bt_ctf_field_type_enumeration_create(enum_type),
                "bt_ctf_field_enumeration_type_create rejects non-integer container field types");
        bt_put(returned_type);
 
        bt_ctf_field_type_set_alignment(float_type, 32);
-       ok(bt_ctf_field_type_get_alignment(NULL) < 0,
-               "bt_ctf_field_type_get_alignment handles NULL correctly");
        ok(bt_ctf_field_type_get_alignment(float_type) == 32,
                "bt_ctf_field_type_get_alignment returns a correct value");
 
@@ -212,10 +204,6 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
        ok(bt_ctf_field_type_floating_point_set_mantissa_digits(float_type, 53) == 0,
                "Set a floating point type's mantissa digit count");
 
-       ok(bt_ctf_field_type_floating_point_get_exponent_digits(NULL) < 0,
-               "bt_ctf_field_type_floating_point_get_exponent_digits handles NULL properly");
-       ok(bt_ctf_field_type_floating_point_get_mantissa_digits(NULL) < 0,
-               "bt_ctf_field_type_floating_point_get_mantissa_digits handles NULL properly");
        ok(bt_ctf_field_type_floating_point_get_exponent_digits(float_type) == 11,
                "bt_ctf_field_type_floating_point_get_exponent_digits returns the correct value");
        ok(bt_ctf_field_type_floating_point_get_mantissa_digits(float_type) == 53,
@@ -243,90 +231,70 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
                -54, -55), "bt_ctf_field_type_enumeration_add_mapping rejects mapping where end < start");
        bt_ctf_field_type_enumeration_add_mapping(enum_type, "another entry", -42000, -13000);
 
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_signed_value(NULL, -42);
-       ok(iter == NULL, "bt_ctf_field_type_enumeration_find_mappings_by_signed_value handles a NULL field type correctly");
-
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_signed_value(enum_type, -4200000);
-       ok(iter == NULL, "bt_ctf_field_type_enumeration_find_mappings_by_signed_value rejects non-mapped values");
-
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_signed_value(enum_type, 3);
-       ok(iter != NULL, "bt_ctf_field_type_enumeration_find_mappings_by_signed_value succeeds with mapped value");
-       ok(bt_ctf_field_type_enumeration_mapping_iterator_get_signed(iter, NULL, NULL, NULL) == 0,
-               "bt_ctf_field_type_enumeration_mapping_iterator_get_signed handles mapped values correctly");
-       BT_PUT(iter);
-
        ok(bt_ctf_event_class_add_field(simple_event_class, enum_type,
                "enum_field") == 0, "Add signed enumeration field to event");
 
-       ok(bt_ctf_field_type_enumeration_get_mapping_signed(NULL, 0, &ret_char,
-               &ret_range_start_int64_t, &ret_range_end_int64_t) < 0,
-               "bt_ctf_field_type_enumeration_get_mapping_signed handles a NULL enumeration correctly");
-       ok(bt_ctf_field_type_enumeration_get_mapping_signed(enum_type, 0, NULL,
+       ok(bt_ctf_field_type_enumeration_signed_get_mapping_by_index(enum_type, 0, NULL,
                &ret_range_start_int64_t, &ret_range_end_int64_t) == 0,
-               "bt_ctf_field_type_enumeration_get_mapping_signed handles a NULL string correctly");
-       ok(bt_ctf_field_type_enumeration_get_mapping_signed(enum_type, 0, &ret_char,
+               "bt_ctf_field_type_enumeration_signed_get_mapping_by_index handles a NULL string correctly");
+       ok(bt_ctf_field_type_enumeration_signed_get_mapping_by_index(enum_type, 0, &ret_char,
                NULL, &ret_range_end_int64_t) == 0,
-               "bt_ctf_field_type_enumeration_get_mapping_signed handles a NULL start correctly");
-       ok(bt_ctf_field_type_enumeration_get_mapping_signed(enum_type, 0, &ret_char,
+               "bt_ctf_field_type_enumeration_signed_get_mapping_by_index handles a NULL start correctly");
+       ok(bt_ctf_field_type_enumeration_signed_get_mapping_by_index(enum_type, 0, &ret_char,
                &ret_range_start_int64_t, NULL) == 0,
-               "bt_ctf_field_type_enumeration_get_mapping_signed handles a NULL end correctly");
+               "bt_ctf_field_type_enumeration_signed_get_mapping_by_index handles a NULL end correctly");
        /* Assumes entries are sorted by range_start values. */
-       ok(bt_ctf_field_type_enumeration_get_mapping_signed(enum_type, 6, &ret_char,
+       ok(bt_ctf_field_type_enumeration_signed_get_mapping_by_index(enum_type, 6, &ret_char,
                &ret_range_start_int64_t, &ret_range_end_int64_t) == 0,
-               "bt_ctf_field_type_enumeration_get_mapping_signed returns a value");
+               "bt_ctf_field_type_enumeration_signed_get_mapping_by_index returns a value");
        ok(!strcmp(ret_char, mapping_name_test),
-               "bt_ctf_field_type_enumeration_get_mapping_signed returns a correct mapping name");
+               "bt_ctf_field_type_enumeration_signed_get_mapping_by_index returns a correct mapping name");
        ok(ret_range_start_int64_t == 42,
-               "bt_ctf_field_type_enumeration_get_mapping_signed returns a correct mapping start");
+               "bt_ctf_field_type_enumeration_signed_get_mapping_by_index returns a correct mapping start");
        ok(ret_range_end_int64_t == 42,
-               "bt_ctf_field_type_enumeration_get_mapping_signed returns a correct mapping end");
+               "bt_ctf_field_type_enumeration_signed_get_mapping_by_index returns a correct mapping end");
 
-       ok(bt_ctf_field_type_enumeration_add_mapping_unsigned(enum_type_unsigned,
+       ok(bt_ctf_field_type_enumeration_unsigned_add_mapping(enum_type_unsigned,
                "escaping; \"test\"", 0, 0) == 0,
-               "bt_ctf_field_type_enumeration_add_mapping_unsigned accepts enumeration mapping strings containing quotes");
-       ok(bt_ctf_field_type_enumeration_add_mapping_unsigned(enum_type_unsigned,
+               "bt_ctf_field_type_enumeration_unsigned_add_mapping accepts enumeration mapping strings containing quotes");
+       ok(bt_ctf_field_type_enumeration_unsigned_add_mapping(enum_type_unsigned,
                "\tanother \'escaping\'\n test\"", 1, 4) == 0,
-               "bt_ctf_field_type_enumeration_add_mapping_unsigned accepts enumeration mapping strings containing special characters");
-       ok(bt_ctf_field_type_enumeration_add_mapping_unsigned(enum_type_unsigned,
+               "bt_ctf_field_type_enumeration_unsigned_add_mapping accepts enumeration mapping strings containing special characters");
+       ok(bt_ctf_field_type_enumeration_unsigned_add_mapping(enum_type_unsigned,
                "event clock int float", 5, 22) == 0,
-               "bt_ctf_field_type_enumeration_add_mapping_unsigned accepts enumeration mapping strings containing reserved keywords");
-       ok(bt_ctf_field_type_enumeration_add_mapping_unsigned(enum_type_unsigned, mapping_name_test,
-               42, 42) == 0, "bt_ctf_field_type_enumeration_add_mapping_unsigned accepts single-value ranges");
-       ok(bt_ctf_field_type_enumeration_add_mapping_unsigned(enum_type_unsigned, mapping_name_test,
-               43, 51) == 0, "bt_ctf_field_type_enumeration_add_mapping_unsigned accepts duplicate mapping names");
-       ok(bt_ctf_field_type_enumeration_add_mapping_unsigned(enum_type_unsigned, "something",
-               7, 8) == 0, "bt_ctf_field_type_enumeration_add_mapping_unsigned accepts overlapping enum entries");
-       ok(bt_ctf_field_type_enumeration_add_mapping_unsigned(enum_type_unsigned, mapping_name_test,
-               55, 54), "bt_ctf_field_type_enumeration_add_mapping_unsigned rejects mapping where end < start");
+               "bt_ctf_field_type_enumeration_unsigned_add_mapping accepts enumeration mapping strings containing reserved keywords");
+       ok(bt_ctf_field_type_enumeration_unsigned_add_mapping(enum_type_unsigned, mapping_name_test,
+               42, 42) == 0, "bt_ctf_field_type_enumeration_unsigned_add_mapping accepts single-value ranges");
+       ok(bt_ctf_field_type_enumeration_unsigned_add_mapping(enum_type_unsigned, mapping_name_test,
+               43, 51) == 0, "bt_ctf_field_type_enumeration_unsigned_add_mapping accepts duplicate mapping names");
+       ok(bt_ctf_field_type_enumeration_unsigned_add_mapping(enum_type_unsigned, "something",
+               7, 8) == 0, "bt_ctf_field_type_enumeration_unsigned_add_mapping accepts overlapping enum entries");
+       ok(bt_ctf_field_type_enumeration_unsigned_add_mapping(enum_type_unsigned, mapping_name_test,
+               55, 54), "bt_ctf_field_type_enumeration_unsigned_add_mapping rejects mapping where end < start");
        ok(bt_ctf_event_class_add_field(simple_event_class, enum_type_unsigned,
                "enum_field_unsigned") == 0, "Add unsigned enumeration field to event");
 
-       ok(bt_ctf_field_type_enumeration_get_mapping_count(NULL) < 0,
-               "bt_ctf_field_type_enumeration_get_mapping_count handles NULL correctly");
        ok(bt_ctf_field_type_enumeration_get_mapping_count(enum_type_unsigned) == 6,
                "bt_ctf_field_type_enumeration_get_mapping_count returns the correct value");
 
-       ok(bt_ctf_field_type_enumeration_get_mapping_unsigned(NULL, 0, &ret_char,
-               &ret_range_start_uint64_t, &ret_range_end_uint64_t) < 0,
-               "bt_ctf_field_type_enumeration_get_mapping_unsigned handles a NULL enumeration correctly");
-       ok(bt_ctf_field_type_enumeration_get_mapping_unsigned(enum_type_unsigned, 0, NULL,
+       ok(bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index(enum_type_unsigned, 0, NULL,
                &ret_range_start_uint64_t, &ret_range_end_uint64_t) == 0,
-               "bt_ctf_field_type_enumeration_get_mapping_unsigned handles a NULL string correctly");
-       ok(bt_ctf_field_type_enumeration_get_mapping_unsigned(enum_type_unsigned, 0, &ret_char,
+               "bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index handles a NULL string correctly");
+       ok(bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index(enum_type_unsigned, 0, &ret_char,
                NULL, &ret_range_end_uint64_t) == 0,
-               "bt_ctf_field_type_enumeration_get_mapping_unsigned handles a NULL start correctly");
-       ok(bt_ctf_field_type_enumeration_get_mapping_unsigned(enum_type_unsigned, 0, &ret_char,
+               "bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index handles a NULL start correctly");
+       ok(bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index(enum_type_unsigned, 0, &ret_char,
                &ret_range_start_uint64_t, NULL) == 0,
-               "bt_ctf_field_type_enumeration_get_mapping_unsigned handles a NULL end correctly");
-       ok(bt_ctf_field_type_enumeration_get_mapping_unsigned(enum_type_unsigned, 4, &ret_char,
+               "bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index handles a NULL end correctly");
+       ok(bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index(enum_type_unsigned, 4, &ret_char,
                &ret_range_start_uint64_t, &ret_range_end_uint64_t) == 0,
-               "bt_ctf_field_type_enumeration_get_mapping_unsigned returns a value");
+               "bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index returns a value");
        ok(!strcmp(ret_char, mapping_name_test),
-               "bt_ctf_field_type_enumeration_get_mapping_unsigned returns a correct mapping name");
+               "bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index returns a correct mapping name");
        ok(ret_range_start_uint64_t == 42,
-               "bt_ctf_field_type_enumeration_get_mapping_unsigned returns a correct mapping start");
+               "bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index returns a correct mapping start");
        ok(ret_range_end_uint64_t == 42,
-               "bt_ctf_field_type_enumeration_get_mapping_unsigned returns a correct mapping end");
+               "bt_ctf_field_type_enumeration_unsigned_get_mapping_by_index returns a correct mapping end");
 
        bt_ctf_event_class_add_field(simple_event_class, uint_12_type,
                "integer_field");
@@ -339,16 +307,14 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
        ok(!bt_ctf_field_type_structure_add_field(event_context_type, uint_12_type,
                "event_specific_context"),
                "Add event specific context field");
-       ok(bt_ctf_event_class_get_context_type(NULL) == NULL,
-               "bt_ctf_event_class_get_context_type handles NULL correctly");
 
-       ok(bt_ctf_event_class_set_context_type(NULL, event_context_type) < 0,
-               "bt_ctf_event_class_set_context_type handles a NULL event class correctly");
-       ok(!bt_ctf_event_class_set_context_type(simple_event_class, event_context_type),
+       ok(bt_ctf_event_class_set_context_field_type(NULL, event_context_type) < 0,
+               "bt_ctf_event_class_set_context_field_type handles a NULL event class correctly");
+       ok(!bt_ctf_event_class_set_context_field_type(simple_event_class, event_context_type),
                "Set an event class' context type successfully");
-       returned_type = bt_ctf_event_class_get_context_type(simple_event_class);
+       returned_type = bt_ctf_event_class_get_context_field_type(simple_event_class);
        ok(returned_type == event_context_type,
-               "bt_ctf_event_class_get_context_type returns the appropriate type");
+               "bt_ctf_event_class_get_context_field_type returns the appropriate type");
        bt_put(returned_type);
 
        ok(!bt_ctf_stream_class_add_event_class(stream_class, simple_event_class),
@@ -361,10 +327,10 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
         */
        BT_PUT(event_context_type);
        BT_PUT(event_payload_type);
-       event_payload_type = bt_ctf_event_class_get_payload_type(
+       event_payload_type = bt_ctf_event_class_get_payload_field_type(
                simple_event_class);
        assert(event_payload_type);
-       event_context_type = bt_ctf_event_class_get_context_type(
+       event_context_type = bt_ctf_event_class_get_context_field_type(
                simple_event_class);
        assert(event_context_type);
        ep_integer_field_type =
@@ -380,20 +346,12 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
                        event_payload_type, "enum_field_unsigned");
        assert(ep_enum_field_unsigned_type);
 
-       ok(bt_ctf_stream_class_get_event_class_count(NULL) < 0,
-               "bt_ctf_stream_class_get_event_class_count handles NULL correctly");
        ok(bt_ctf_stream_class_get_event_class_count(stream_class) == 1,
                "bt_ctf_stream_class_get_event_class_count returns a correct number of event classes");
-       ok(bt_ctf_stream_class_get_event_class_by_index(NULL, 0) == NULL,
-               "bt_ctf_stream_class_get_event_class handles NULL correctly");
-       ok(bt_ctf_stream_class_get_event_class_by_index(stream_class, 8724) == NULL,
-               "bt_ctf_stream_class_get_event_class handles invalid indexes correctly");
        ret_event_class = bt_ctf_stream_class_get_event_class_by_index(stream_class, 0);
        ok(ret_event_class == simple_event_class,
                "bt_ctf_stream_class_get_event_class returns the correct event class");
        bt_put(ret_event_class);
-       ok(!bt_ctf_stream_class_get_event_class_by_id(NULL, 0),
-               "bt_ctf_stream_class_get_event_class_by_id handles NULL correctly");
        ok(!bt_ctf_stream_class_get_event_class_by_id(stream_class, 2),
                "bt_ctf_stream_class_get_event_class_by_id returns NULL when the requested ID doesn't exist");
        ret_event_class =
@@ -407,18 +365,12 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
                "Instantiate an event containing a single integer field");
 
        integer_field = bt_ctf_field_create(ep_integer_field_type);
-       bt_ctf_field_unsigned_integer_set_value(integer_field, 42);
+       bt_ctf_field_integer_unsigned_set_value(integer_field, 42);
        ok(bt_ctf_event_set_payload(simple_event, "integer_field",
                integer_field) == 0, "Use bt_ctf_event_set_payload to set a manually allocated field");
 
        float_field = bt_ctf_event_get_payload(simple_event, "float_field");
-       ok(bt_ctf_field_floating_point_get_value(float_field, &ret_double),
-               "bt_ctf_field_floating_point_get_value fails on an unset float field");
        bt_ctf_field_floating_point_set_value(float_field, double_test_value);
-       ok(bt_ctf_field_floating_point_get_value(NULL, &ret_double),
-               "bt_ctf_field_floating_point_get_value properly handles a NULL field");
-       ok(bt_ctf_field_floating_point_get_value(float_field, NULL),
-               "bt_ctf_field_floating_point_get_value properly handles a NULL return value pointer");
        ok(!bt_ctf_field_floating_point_get_value(float_field, &ret_double),
                "bt_ctf_field_floating_point_get_value returns a double value");
        ok(fabs(ret_double - double_test_value) <= DBL_EPSILON,
@@ -427,22 +379,10 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
        enum_field = bt_ctf_field_create(ep_enum_field_type);
        assert(enum_field);
 
-       iter = bt_ctf_field_enumeration_get_mappings(NULL);
-       ok(!iter, "bt_ctf_field_enumeration_get_mappings handles NULL correctly");
-       iter = bt_ctf_field_enumeration_get_mappings(enum_field);
-       ok(!iter, "bt_ctf_field_enumeration_get_mappings returns NULL if the enumeration's container field is unset");
-       enum_container_field = bt_ctf_field_enumeration_get_container(
-               enum_field);
-       ok(bt_ctf_field_signed_integer_set_value(
+       enum_container_field = bt_ctf_field_enumeration_get_container(enum_field);
+       ok(bt_ctf_field_integer_signed_set_value(
                enum_container_field, -42) == 0,
                "Set signed enumeration container value");
-       iter = bt_ctf_field_enumeration_get_mappings(enum_field);
-       ok(iter, "bt_ctf_field_enumeration_get_mappings returns an iterator to matching mappings");
-       ret = bt_ctf_field_type_enumeration_mapping_iterator_get_signed(iter, &ret_char, NULL, NULL);
-       ok(!ret && ret_char, "bt_ctf_field_type_enumeration_mapping_iterator_get_signed return a mapping name");
-       assert(ret_char);
-       ok(!strcmp(ret_char, mapping_name_negative_test),
-               "bt_ctf_field_enumeration_get_single_mapping_name returns the correct mapping name with an signed container");
        ret = bt_ctf_event_set_payload(simple_event, "enum_field", enum_field);
        assert(!ret);
        BT_PUT(iter);
@@ -451,17 +391,12 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
        assert(enum_field_unsigned);
        enum_container_field_unsigned = bt_ctf_field_enumeration_get_container(
                enum_field_unsigned);
-       ok(bt_ctf_field_unsigned_integer_set_value(
+       ok(bt_ctf_field_integer_unsigned_set_value(
                enum_container_field_unsigned, 42) == 0,
                "Set unsigned enumeration container value");
        ret = bt_ctf_event_set_payload(simple_event, "enum_field_unsigned",
                enum_field_unsigned);
        assert(!ret);
-       iter = bt_ctf_field_enumeration_get_mappings(enum_field_unsigned);
-       assert(iter);
-       (void) bt_ctf_field_type_enumeration_mapping_iterator_get_unsigned(iter, &ret_char, NULL, NULL);
-       ok(ret_char && !strcmp(ret_char, mapping_name_test),
-               "bt_ctf_field_type_enumeration_mapping_iterator_get_unsigned returns the correct mapping name with an unsigned container");
 
        ok(bt_ctf_clock_set_time(clock, current_time) == 0, "Set clock time");
 
@@ -469,55 +404,41 @@ void append_simple_event(struct bt_ctf_stream_class *stream_class,
        stream_event_context =
                bt_ctf_event_get_stream_event_context(simple_event);
        assert(stream_event_context);
-       stream_event_context_field = bt_ctf_field_structure_get_field(
+       stream_event_context_field = bt_ctf_field_structure_get_field_by_name(
                stream_event_context, "common_event_context");
-       bt_ctf_field_unsigned_integer_set_value(stream_event_context_field, 42);
+       bt_ctf_field_integer_unsigned_set_value(stream_event_context_field, 42);
 
        /* Populate the event's context */
-       ok(bt_ctf_event_get_event_context(NULL) == NULL,
-               "bt_ctf_event_get_event_context handles NULL correctly");
-       event_context = bt_ctf_event_get_event_context(simple_event);
+       event_context = bt_ctf_event_get_context(simple_event);
        ok(event_context,
-               "bt_ctf_event_get_event_context returns a field");
+               "bt_ctf_event_get_context returns a field");
        returned_type = bt_ctf_field_get_type(event_context);
        ok(returned_type == event_context_type,
-               "bt_ctf_event_get_event_context returns a field of the appropriate type");
-       event_context_field = bt_ctf_field_structure_get_field(event_context,
+               "bt_ctf_event_get_context returns a field of the appropriate type");
+       event_context_field = bt_ctf_field_structure_get_field_by_name(event_context,
                "event_specific_context");
-       ok(!bt_ctf_field_unsigned_integer_set_value(event_context_field, 1234),
+       ok(!bt_ctf_field_integer_unsigned_set_value(event_context_field, 1234),
                "Successfully set an event context's value");
-       ok(bt_ctf_event_set_event_context(NULL, event_context) < 0,
-               "bt_ctf_event_set_event_context handles a NULL event correctly");
-       ok(bt_ctf_event_set_event_context(simple_event, NULL) < 0,
-               "bt_ctf_event_set_event_context handles a NULL event context correctly");
-       ok(bt_ctf_event_set_event_context(simple_event, event_context_field) < 0,
-               "bt_ctf_event_set_event_context rejects a context of the wrong type");
-       ok(!bt_ctf_event_set_event_context(simple_event, event_context),
+       ok(!bt_ctf_event_set_context(simple_event, event_context),
                "Set an event context successfully");
 
        ok(bt_ctf_stream_append_event(stream, simple_event) == 0,
                "Append simple event to trace stream");
 
-       ok(bt_ctf_stream_get_packet_context(NULL) == NULL,
-               "bt_ctf_stream_get_packet_context handles NULL correctly");
        packet_context = bt_ctf_stream_get_packet_context(stream);
        ok(packet_context,
                "bt_ctf_stream_get_packet_context returns a packet context");
 
-       packet_context_field = bt_ctf_field_structure_get_field(packet_context,
+       packet_context_field = bt_ctf_field_structure_get_field_by_name(packet_context,
                "packet_size");
        ok(packet_context_field,
                "Packet context contains the default packet_size field.");
        bt_put(packet_context_field);
-       packet_context_field = bt_ctf_field_structure_get_field(packet_context,
+       packet_context_field = bt_ctf_field_structure_get_field_by_name(packet_context,
                "custom_packet_context_field");
-       ok(bt_ctf_field_unsigned_integer_set_value(packet_context_field, 8) == 0,
+       ok(bt_ctf_field_integer_unsigned_set_value(packet_context_field, 8) == 0,
                "Custom packet context field value successfully set.");
 
-       ok(bt_ctf_stream_set_packet_context(NULL, packet_context_field) < 0,
-               "bt_ctf_stream_set_packet_context handles a NULL stream correctly");
-       ok(bt_ctf_stream_set_packet_context(stream, NULL) < 0,
-               "bt_ctf_stream_set_packet_context handles a NULL packet context correctly");
        ok(bt_ctf_stream_set_packet_context(stream, packet_context) == 0,
                "Successfully set a stream's packet context");
 
@@ -563,7 +484,7 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
        const char *test_string_2 = "string ";
        const char *test_string_3 = "abcdefghi";
        const char *test_string_4 = "abcd\0efg\0hi";
-       const char *test_string_cat = "Test string abcdeabcd";
+       const char *test_string_cat = "Test string abcdeefg";
        struct bt_ctf_field_type *uint_35_type =
                bt_ctf_field_type_integer_create(35);
        struct bt_ctf_field_type *int_16_type =
@@ -590,14 +511,13 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
                *inner_structure_field, *complex_structure_field,
                *a_sequence_field, *enum_variant_field, *enum_container_field,
                *variant_field, *an_array_field, *stream_event_ctx_field,
-               *stream_event_ctx_int_field, *ret_field;
+               *stream_event_ctx_int_field;
        uint64_t ret_unsigned_int;
        int64_t ret_signed_int;
        const char *ret_string;
        struct bt_ctf_stream_class *ret_stream_class;
        struct bt_ctf_event_class *ret_event_class;
        struct bt_ctf_field *packet_context, *packet_context_field;
-       struct bt_ctf_field_type_enumeration_mapping_iterator *iter = NULL;
 
        ok(bt_ctf_field_type_set_alignment(int_16_type, 0),
                "bt_ctf_field_type_set_alignment handles 0-alignment correctly");
@@ -619,16 +539,12 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
        sequence_type = bt_ctf_field_type_sequence_create(int_16_type,
                "seq_len");
 
-       ok(bt_ctf_field_type_array_get_element_type(NULL) == NULL,
-               "bt_ctf_field_type_array_get_element_type handles NULL correctly");
-       ret_field_type = bt_ctf_field_type_array_get_element_type(
+       ret_field_type = bt_ctf_field_type_array_get_element_field_type(
                array_type);
        ok(ret_field_type == int_16_type,
-               "bt_ctf_field_type_array_get_element_type returns the correct type");
+               "bt_ctf_field_type_array_get_element_field_type returns the correct type");
        bt_put(ret_field_type);
 
-       ok(bt_ctf_field_type_array_get_length(NULL) < 0,
-               "bt_ctf_field_type_array_get_length handles NULL correctly");
        ok(bt_ctf_field_type_array_get_length(array_type) == ARRAY_TEST_LENGTH,
                "bt_ctf_field_type_array_get_length returns the correct length");
 
@@ -648,33 +564,6 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
        bt_ctf_field_type_enumeration_add_mapping(enum_variant_type,
                "UINT35_TYPE", 2, 7);
 
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_name(NULL, "INT16_TYPE");
-       ok(iter == NULL, "bt_ctf_field_type_enumeration_find_mappings_by_name handles a NULL field type correctly");
-
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_name(enum_variant_type, "INT16_TYPE");
-       ok(iter != NULL, "bt_ctf_field_type_enumeration_find_mappings_by_name handles an existing mapping correctly");
-       ok(bt_ctf_field_type_enumeration_mapping_iterator_get_unsigned(iter, NULL, NULL, NULL) == 0,
-               "bt_ctf_field_type_enumeration_mapping_iterator_get_unsigned handles mapped values correctly");
-       BT_PUT(iter);
-
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_name(enum_variant_type, NULL);
-       ok(iter == NULL, "bt_ctf_field_type_enumeration_find_mappings_by_name handles a NULL name correctly");
-
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_unsigned_value(NULL, 1);
-       ok(iter == NULL, "bt_ctf_field_type_enumeration_find_mappings_by_unsigned_value handles a NULL field type correctly");
-
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_unsigned_value(enum_variant_type, -42);
-       ok(iter == NULL, "bt_ctf_field_type_enumeration_find_mappings_by_unsigned_value handles invalid values correctly");
-       ok(bt_ctf_field_type_enumeration_mapping_iterator_get_unsigned(iter, NULL, NULL, NULL) != 0,
-               "bt_ctf_field_type_enumeration_mapping_iterator_get_unsigned handles invalid values correctly");
-       BT_PUT(iter);
-
-       iter = bt_ctf_field_type_enumeration_find_mappings_by_unsigned_value(enum_variant_type, 5);
-       ok(iter != NULL, "bt_ctf_field_type_enumeration_find_mappings_by_unsigned_value handles valid values correctly");
-       ok(bt_ctf_field_type_enumeration_mapping_iterator_get_unsigned(iter, NULL, NULL, NULL) == 0,
-               "bt_ctf_field_type_enumeration_mapping_iterator_get_unsigned handles valid values correctly");
-       BT_PUT(iter);
-
        ok(bt_ctf_field_type_variant_add_field(variant_type, uint_3_type,
                "An unknown entry"), "Reject a variant field based on an unknown tag value");
        ok(bt_ctf_field_type_variant_add_field(variant_type, uint_3_type,
@@ -684,45 +573,29 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
        ok(!bt_ctf_field_type_variant_add_field(variant_type, uint_35_type,
                "UINT35_TYPE"), "Add UINT35_TYPE field to variant");
 
-       ok(bt_ctf_field_type_variant_get_tag_type(NULL) == NULL,
-               "bt_ctf_field_type_variant_get_tag_type handles NULL correctly");
-       ret_field_type = bt_ctf_field_type_variant_get_tag_type(variant_type);
+       ret_field_type = bt_ctf_field_type_variant_get_tag_field_type(variant_type);
        ok(ret_field_type == enum_variant_type,
-               "bt_ctf_field_type_variant_get_tag_type returns a correct tag type");
+               "bt_ctf_field_type_variant_get_tag_field_type returns a correct tag type");
        bt_put(ret_field_type);
 
-       ok(bt_ctf_field_type_variant_get_tag_name(NULL) == NULL,
-               "bt_ctf_field_type_variant_get_tag_name handles NULL correctly");
        ret_string = bt_ctf_field_type_variant_get_tag_name(variant_type);
        ok(ret_string ? !strcmp(ret_string, "variant_selector") : 0,
                "bt_ctf_field_type_variant_get_tag_name returns the correct variant tag name");
-       ok(bt_ctf_field_type_variant_get_field_type_by_name(NULL,
-               "INT16_TYPE") == NULL,
-               "bt_ctf_field_type_variant_get_field_type_by_name handles a NULL variant_type correctly");
-       ok(bt_ctf_field_type_variant_get_field_type_by_name(variant_type,
-               NULL) == NULL,
-               "bt_ctf_field_type_variant_get_field_type_by_name handles a NULL field name correctly");
        ret_field_type = bt_ctf_field_type_variant_get_field_type_by_name(
                variant_type, "INT16_TYPE");
        ok(ret_field_type == int_16_type,
                "bt_ctf_field_type_variant_get_field_type_by_name returns a correct field type");
        bt_put(ret_field_type);
 
-       ok(bt_ctf_field_type_variant_get_field_count(NULL) < 0,
-               "bt_ctf_field_type_variant_get_field_count handles NULL correctly");
        ok(bt_ctf_field_type_variant_get_field_count(variant_type) == 3,
                "bt_ctf_field_type_variant_get_field_count returns the correct count");
 
-       ok(bt_ctf_field_type_variant_get_field(NULL, &ret_string, &ret_field_type, 0) < 0,
-               "bt_ctf_field_type_variant_get_field handles a NULL type correctly");
-       ok(bt_ctf_field_type_variant_get_field(variant_type, NULL, &ret_field_type, 0) == 0,
+       ok(bt_ctf_field_type_variant_get_field_by_index(variant_type, NULL, &ret_field_type, 0) == 0,
                "bt_ctf_field_type_variant_get_field handles a NULL field name correctly");
        bt_put(ret_field_type);
-       ok(bt_ctf_field_type_variant_get_field(variant_type, &ret_string, NULL, 0) == 0,
+       ok(bt_ctf_field_type_variant_get_field_by_index(variant_type, &ret_string, NULL, 0) == 0,
                "bt_ctf_field_type_variant_get_field handles a NULL field type correctly");
-       ok(bt_ctf_field_type_variant_get_field(variant_type, &ret_string, &ret_field_type, 200) < 0,
-               "bt_ctf_field_type_variant_get_field handles an invalid index correctly");
-       ok(bt_ctf_field_type_variant_get_field(variant_type, &ret_string, &ret_field_type, 1) == 0,
+       ok(bt_ctf_field_type_variant_get_field_by_index(variant_type, &ret_string, &ret_field_type, 1) == 0,
                "bt_ctf_field_type_variant_get_field returns a field");
        ok(!strcmp("INT16_TYPE", ret_string),
                "bt_ctf_field_type_variant_get_field returns a correct field name");
@@ -734,7 +607,7 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
                enum_variant_type, "variant_selector"),
                "Add variant_selector field to complex structure");
        ok(!bt_ctf_field_type_structure_add_field(complex_structure_type,
-               string_type, "a_string"), "Add a_string field to complex structure");
+               string_type, "string"), "Add `string` field to complex structure");
        ok(!bt_ctf_field_type_structure_add_field(complex_structure_type,
                variant_type, "variant_value"),
                "Add variant_value field to complex structure");
@@ -760,15 +633,11 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
                "complex_structure") == 0,
                "Add composite structure to an event");
 
-       ok(bt_ctf_event_class_get_name(NULL) == NULL,
-               "bt_ctf_event_class_get_name handles NULL correctly");
        ret_string = bt_ctf_event_class_get_name(event_class);
        ok(!strcmp(ret_string, complex_test_event_string),
                "bt_ctf_event_class_get_name returns a correct name");
        ok(bt_ctf_event_class_get_id(event_class) < 0,
                "bt_ctf_event_class_get_id returns a negative value when not set");
-       ok(bt_ctf_event_class_get_id(NULL) < 0,
-               "bt_ctf_event_class_get_id handles NULL correctly");
        ok(bt_ctf_event_class_set_id(NULL, 42) < 0,
                "bt_ctf_event_class_set_id handles NULL correctly");
        ok(bt_ctf_event_class_set_id(event_class, 42) == 0,
@@ -787,16 +656,12 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
                "bt_ctf_event_class_set_log_level handles an unknown log level correctly");
        ok(!bt_ctf_event_class_set_log_level(event_class, BT_CTF_EVENT_CLASS_LOG_LEVEL_INFO),
                "bt_ctf_event_class_set_log_level succeeds with a valid log level");
-       ok(bt_ctf_event_class_get_log_level(NULL) == BT_CTF_EVENT_CLASS_LOG_LEVEL_UNKNOWN,
-               "bt_ctf_event_class_get_log_level handles a NULL event class correctly");
        ok(bt_ctf_event_class_get_log_level(event_class) == BT_CTF_EVENT_CLASS_LOG_LEVEL_INFO,
                "bt_ctf_event_class_get_log_level returns the expected log level");
        ok(bt_ctf_event_class_set_emf_uri(NULL, "http://diamon.org/babeltrace/"),
                "bt_ctf_event_class_set_emf_uri handles a NULL event class correctly");
        ok(!bt_ctf_event_class_set_emf_uri(event_class, "http://diamon.org/babeltrace/"),
                "bt_ctf_event_class_set_emf_uri succeeds with a valid EMF URI");
-       ok(!bt_ctf_event_class_get_emf_uri(NULL),
-               "bt_ctf_event_class_get_emf_uri handles a NULL event class correctly");
        ok(strcmp(bt_ctf_event_class_get_emf_uri(event_class), "http://diamon.org/babeltrace/") == 0,
                "bt_ctf_event_class_get_emf_uri returns the expected EMF URI");
        ok(!bt_ctf_event_class_set_emf_uri(event_class, NULL),
@@ -810,156 +675,82 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
        ok(bt_ctf_stream_class_add_event_class(stream_class,
                event_class) == 0, "Add an event class to stream class");
 
-       ok(bt_ctf_event_class_get_stream_class(NULL) == NULL,
-               "bt_ctf_event_class_get_stream_class handles NULL correctly");
        ret_stream_class = bt_ctf_event_class_get_stream_class(event_class);
        ok(ret_stream_class == stream_class,
                "bt_ctf_event_class_get_stream_class returns the correct stream class");
        bt_put(ret_stream_class);
 
-       ok(bt_ctf_event_class_get_payload_type_field_count(NULL) < 0,
-               "bt_ctf_event_class_get_field_count handles NULL correctly");
-       ok(bt_ctf_event_class_get_payload_type_field_count(event_class) == 3,
-               "bt_ctf_event_class_get_field_count returns a correct value");
-
-       ok(bt_ctf_event_class_get_payload_type_field_by_index(NULL, &ret_string,
-               &ret_field_type, 0) < 0,
-               "bt_ctf_event_class_get_field handles a NULL event class correctly");
-       ok(bt_ctf_event_class_get_payload_type_field_by_index(event_class, NULL,
-               &ret_field_type, 0) == 0,
-               "bt_ctf_event_class_get_field handles a NULL field name correctly");
-       bt_put(ret_field_type);
-       ok(bt_ctf_event_class_get_payload_type_field_by_index(event_class, &ret_string,
-               NULL, 0) == 0,
-               "bt_ctf_event_class_get_field handles a NULL field type correctly");
-       ok(bt_ctf_event_class_get_payload_type_field_by_index(event_class, &ret_string,
-               &ret_field_type, 42) < 0,
-               "bt_ctf_event_class_get_field handles an invalid index correctly");
-       ok(bt_ctf_event_class_get_payload_type_field_by_index(event_class, &ret_string,
-               &ret_field_type, 0) == 0,
-               "bt_ctf_event_class_get_field returns a field");
-       ok(bt_ctf_field_type_compare(ret_field_type, uint_35_type) == 0,
-               "bt_ctf_event_class_get_field returns a correct field type");
-       bt_put(ret_field_type);
-       ok(!strcmp(ret_string, "uint_35"),
-               "bt_ctf_event_class_get_field returns a correct field name");
-       ok(bt_ctf_event_class_get_field_by_name(NULL, "") == NULL,
-               "bt_ctf_event_class_get_field_by_name handles a NULL event class correctly");
-       ok(bt_ctf_event_class_get_field_by_name(event_class, NULL) == NULL,
-               "bt_ctf_event_class_get_field_by_name handles a NULL field name correctly");
        ok(bt_ctf_event_class_get_field_by_name(event_class, "truie") == NULL,
                "bt_ctf_event_class_get_field_by_name handles an invalid field name correctly");
        ret_field_type = bt_ctf_event_class_get_field_by_name(event_class,
                "complex_structure");
-       ok(bt_ctf_field_type_compare(ret_field_type, complex_structure_type) == 0,
-               "bt_ctf_event_class_get_field_by_name returns a correct field type");
        bt_put(ret_field_type);
 
        event = bt_ctf_event_create(event_class);
        ok(event, "Instanciate a complex event");
 
-       ok(bt_ctf_event_get_class(NULL) == NULL,
-               "bt_ctf_event_get_class handles NULL correctly");
        ret_event_class = bt_ctf_event_get_class(event);
        ok(ret_event_class == event_class,
                "bt_ctf_event_get_class returns the correct event class");
        bt_put(ret_event_class);
 
        uint_35_field = bt_ctf_event_get_payload(event, "uint_35");
-       if (!uint_35_field) {
-               printf("uint_35_field is NULL\n");
-       }
-
        ok(uint_35_field, "Use bt_ctf_event_get_payload to get a field instance ");
-       bt_ctf_field_unsigned_integer_set_value(uint_35_field, 0x0DDF00D);
-       ok(bt_ctf_field_unsigned_integer_get_value(NULL, &ret_unsigned_int) < 0,
-               "bt_ctf_field_unsigned_integer_get_value properly properly handles a NULL field.");
-       ok(bt_ctf_field_unsigned_integer_get_value(uint_35_field, NULL) < 0,
-               "bt_ctf_field_unsigned_integer_get_value properly handles a NULL return value");
-       ok(bt_ctf_field_unsigned_integer_get_value(uint_35_field,
+       bt_ctf_field_integer_unsigned_set_value(uint_35_field, 0x0DDF00D);
+       ok(bt_ctf_field_integer_unsigned_get_value(uint_35_field,
                &ret_unsigned_int) == 0,
-               "bt_ctf_field_unsigned_integer_get_value succeeds after setting a value");
+               "bt_ctf_field_integer_unsigned_get_value succeeds after setting a value");
        ok(ret_unsigned_int == 0x0DDF00D,
-               "bt_ctf_field_unsigned_integer_get_value returns the correct value");
-       ok(bt_ctf_field_signed_integer_get_value(uint_35_field,
-               &ret_signed_int) < 0,
-               "bt_ctf_field_signed_integer_get_value fails on an unsigned field");
+               "bt_ctf_field_integer_unsigned_get_value returns the correct value");
        bt_put(uint_35_field);
 
        int_16_field = bt_ctf_event_get_payload(event, "int_16");
-       bt_ctf_field_signed_integer_set_value(int_16_field, -12345);
-       ok(bt_ctf_field_signed_integer_get_value(NULL, &ret_signed_int) < 0,
-               "bt_ctf_field_signed_integer_get_value properly handles a NULL field");
-       ok(bt_ctf_field_signed_integer_get_value(int_16_field, NULL) < 0,
-               "bt_ctf_field_signed_integer_get_value properly handles a NULL return value");
-       ok(bt_ctf_field_signed_integer_get_value(int_16_field,
+       bt_ctf_field_integer_signed_set_value(int_16_field, -12345);
+       ok(bt_ctf_field_integer_signed_get_value(int_16_field,
                &ret_signed_int) == 0,
-               "bt_ctf_field_signed_integer_get_value succeeds after setting a value");
+               "bt_ctf_field_integer_signed_get_value succeeds after setting a value");
        ok(ret_signed_int == -12345,
-               "bt_ctf_field_signed_integer_get_value returns the correct value");
-       ok(bt_ctf_field_unsigned_integer_get_value(int_16_field,
-               &ret_unsigned_int) < 0,
-               "bt_ctf_field_unsigned_integer_get_value fails on a signed field");
+               "bt_ctf_field_integer_signed_get_value returns the correct value");
        bt_put(int_16_field);
 
        complex_structure_field = bt_ctf_event_get_payload(event,
                "complex_structure");
 
-       ok(bt_ctf_field_structure_get_field_by_index(NULL, 0) == NULL,
-               "bt_ctf_field_structure_get_field_by_index handles NULL correctly");
-       ok(bt_ctf_field_structure_get_field_by_index(NULL, 9) == NULL,
-               "bt_ctf_field_structure_get_field_by_index handles an invalid index correctly");
        inner_structure_field = bt_ctf_field_structure_get_field_by_index(
                complex_structure_field, 3);
        ret_field_type = bt_ctf_field_get_type(inner_structure_field);
        bt_put(inner_structure_field);
-       ok(bt_ctf_field_type_compare(ret_field_type, inner_structure_type) == 0,
-               "bt_ctf_field_structure_get_field_by_index returns a correct field");
        bt_put(ret_field_type);
 
-       inner_structure_field = bt_ctf_field_structure_get_field(
+       inner_structure_field = bt_ctf_field_structure_get_field_by_name(
                complex_structure_field, "inner_structure");
-       a_string_field = bt_ctf_field_structure_get_field(
-               complex_structure_field, "a_string");
-       enum_variant_field = bt_ctf_field_structure_get_field(
+       a_string_field = bt_ctf_field_structure_get_field_by_name(
+               complex_structure_field, "string");
+       enum_variant_field = bt_ctf_field_structure_get_field_by_name(
                complex_structure_field, "variant_selector");
-       variant_field = bt_ctf_field_structure_get_field(
+       variant_field = bt_ctf_field_structure_get_field_by_name(
                complex_structure_field, "variant_value");
-       uint_35_field = bt_ctf_field_structure_get_field(
+       uint_35_field = bt_ctf_field_structure_get_field_by_name(
                inner_structure_field, "seq_len");
-       a_sequence_field = bt_ctf_field_structure_get_field(
+       a_sequence_field = bt_ctf_field_structure_get_field_by_name(
                inner_structure_field, "a_sequence");
-       an_array_field = bt_ctf_field_structure_get_field(
+       an_array_field = bt_ctf_field_structure_get_field_by_name(
                inner_structure_field, "an_array");
 
        enum_container_field = bt_ctf_field_enumeration_get_container(
                enum_variant_field);
-       bt_ctf_field_unsigned_integer_set_value(enum_container_field, 1);
+       bt_ctf_field_integer_unsigned_set_value(enum_container_field, 1);
        int_16_field = bt_ctf_field_variant_get_field(variant_field,
                enum_variant_field);
-       bt_ctf_field_signed_integer_set_value(int_16_field, -200);
+       bt_ctf_field_integer_signed_set_value(int_16_field, -200);
        bt_put(int_16_field);
-       ok(!bt_ctf_field_string_get_value(a_string_field),
-               "bt_ctf_field_string_get_value returns NULL on an unset field");
        bt_ctf_field_string_set_value(a_string_field,
                test_string_1);
-       ok(!bt_ctf_field_string_get_value(NULL),
-               "bt_ctf_field_string_get_value correctly handles NULL");
-       ok(bt_ctf_field_string_append(NULL, "yeah"),
-               "bt_ctf_field_string_append correctly handles a NULL string field");
-       ok(bt_ctf_field_string_append(a_string_field, NULL),
-               "bt_ctf_field_string_append correctly handles a NULL string value");
        ok(!bt_ctf_field_string_append(a_string_field, test_string_2),
                "bt_ctf_field_string_append succeeds");
-       ok(bt_ctf_field_string_append_len(NULL, "oh noes", 3),
-               "bt_ctf_field_string_append_len correctly handles a NULL string field");
-       ok(bt_ctf_field_string_append_len(a_string_field, NULL, 3),
-               "bt_ctf_field_string_append_len correctly handles a NULL string value");
        ok(!bt_ctf_field_string_append_len(a_string_field, test_string_3, 5),
                "bt_ctf_field_string_append_len succeeds (append 5 characters)");
-       ok(!bt_ctf_field_string_append_len(a_string_field, test_string_4, 10),
-               "bt_ctf_field_string_append_len succeeds (append 4 characters)");
-       ok(!bt_ctf_field_string_append_len(a_string_field, &test_string_4[4], 3),
+       ok(!bt_ctf_field_string_append_len(a_string_field, &test_string_4[5], 3),
                "bt_ctf_field_string_append_len succeeds (append 0 characters)");
        ok(!bt_ctf_field_string_append_len(a_string_field, test_string_3, 0),
                "bt_ctf_field_string_append_len succeeds (append 0 characters)");
@@ -968,50 +759,37 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
        ok(ret_string, "bt_ctf_field_string_get_value returns a string");
        ok(ret_string ? !strcmp(ret_string, test_string_cat) : 0,
                "bt_ctf_field_string_get_value returns a correct value");
-       bt_ctf_field_unsigned_integer_set_value(uint_35_field,
+       bt_ctf_field_integer_unsigned_set_value(uint_35_field,
                SEQUENCE_TEST_LENGTH);
 
-       ok(bt_ctf_field_type_variant_get_field_type_from_tag(NULL,
-               enum_container_field) == NULL,
-               "bt_ctf_field_type_variant_get_field_type_from_tag handles a NULL variant type correctly");
-       ok(bt_ctf_field_type_variant_get_field_type_from_tag(variant_type,
-               NULL) == NULL,
-               "bt_ctf_field_type_variant_get_field_type_from_tag handles a NULL tag correctly");
        ret_field_type = bt_ctf_field_type_variant_get_field_type_from_tag(
                variant_type, enum_variant_field);
        ok(ret_field_type == int_16_type,
                "bt_ctf_field_type_variant_get_field_type_from_tag returns the correct field type");
 
-       ok(bt_ctf_field_sequence_get_length(a_sequence_field) == NULL,
-               "bt_ctf_field_sequence_get_length returns NULL when length is unset");
        ok(bt_ctf_field_sequence_set_length(a_sequence_field,
                uint_35_field) == 0, "Set a sequence field's length");
-       ret_field = bt_ctf_field_sequence_get_length(a_sequence_field);
-       ok(ret_field == uint_35_field,
-               "bt_ctf_field_sequence_get_length returns the correct length field");
-       ok(bt_ctf_field_sequence_get_length(NULL) == NULL,
-               "bt_ctf_field_sequence_get_length properly handles NULL");
 
        for (i = 0; i < SEQUENCE_TEST_LENGTH; i++) {
                int_16_field = bt_ctf_field_sequence_get_field(
                        a_sequence_field, i);
-               bt_ctf_field_signed_integer_set_value(int_16_field, 4 - i);
+               bt_ctf_field_integer_signed_set_value(int_16_field, 4 - i);
                bt_put(int_16_field);
        }
 
        for (i = 0; i < ARRAY_TEST_LENGTH; i++) {
                int_16_field = bt_ctf_field_array_get_field(
                        an_array_field, i);
-               bt_ctf_field_signed_integer_set_value(int_16_field, i);
+               bt_ctf_field_integer_signed_set_value(int_16_field, i);
                bt_put(int_16_field);
        }
 
        stream_event_ctx_field = bt_ctf_event_get_stream_event_context(event);
        assert(stream_event_ctx_field);
-       stream_event_ctx_int_field = bt_ctf_field_structure_get_field(
+       stream_event_ctx_int_field = bt_ctf_field_structure_get_field_by_name(
                stream_event_ctx_field, "common_event_context");
        BT_PUT(stream_event_ctx_field);
-       bt_ctf_field_unsigned_integer_set_value(stream_event_ctx_int_field, 17);
+       bt_ctf_field_integer_unsigned_set_value(stream_event_ctx_int_field, 17);
        BT_PUT(stream_event_ctx_int_field);
 
        bt_ctf_clock_set_time(clock, ++current_time);
@@ -1023,9 +801,9 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
         * otherwise flush will fail.
         */
        packet_context = bt_ctf_stream_get_packet_context(stream);
-       packet_context_field = bt_ctf_field_structure_get_field(packet_context,
+       packet_context_field = bt_ctf_field_structure_get_field_by_name(packet_context,
                "custom_packet_context_field");
-       bt_ctf_field_unsigned_integer_set_value(packet_context_field, 1);
+       bt_ctf_field_integer_unsigned_set_value(packet_context_field, 1);
 
        ok(bt_ctf_stream_flush(stream) == 0,
                "Flush a stream containing a complex event");
@@ -1039,7 +817,6 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
        bt_put(enum_variant_field);
        bt_put(enum_container_field);
        bt_put(variant_field);
-       bt_put(ret_field);
        bt_put(packet_context_field);
        bt_put(packet_context);
        bt_put(uint_35_type);
@@ -1057,547 +834,12 @@ void append_complex_event(struct bt_ctf_stream_class *stream_class,
        bt_put(event);
 }
 
-static
-void field_copy_tests_validate_same_type(struct bt_ctf_field *field,
-               struct bt_ctf_field_type *expected_type, const char *name)
-{
-       struct bt_ctf_field_type *copy_type;
-
-       copy_type = bt_ctf_field_get_type(field);
-       ok(copy_type == expected_type,
-               "bt_ctf_field_copy does not copy the type (%s)", name);
-       bt_put(copy_type);
-}
-
-static
-void field_copy_tests_validate_diff_ptrs(struct bt_ctf_field *field_a,
-               struct bt_ctf_field *field_b, const char *name)
-{
-       ok(field_a != field_b,
-               "bt_ctf_field_copy creates different pointers (%s)", name);
-}
-
-static
-void field_copy_tests()
-{
-       struct bt_ctf_field_type *len_type = NULL;
-       struct bt_ctf_field_type *fp_type = NULL;
-       struct bt_ctf_field_type *s_type = NULL;
-       struct bt_ctf_field_type *e_int_type = NULL;
-       struct bt_ctf_field_type *e_type = NULL;
-       struct bt_ctf_field_type *v_type = NULL;
-       struct bt_ctf_field_type *v_label1_type = NULL;
-       struct bt_ctf_field_type *v_label1_array_type = NULL;
-       struct bt_ctf_field_type *v_label2_type = NULL;
-       struct bt_ctf_field_type *v_label2_seq_type = NULL;
-       struct bt_ctf_field_type *strct_type = NULL;
-       struct bt_ctf_field *len = NULL;
-       struct bt_ctf_field *fp = NULL;
-       struct bt_ctf_field *s = NULL;
-       struct bt_ctf_field *e_int = NULL;
-       struct bt_ctf_field *e = NULL;
-       struct bt_ctf_field *v = NULL;
-       struct bt_ctf_field *v_selected = NULL;
-       struct bt_ctf_field *v_selected_cur = NULL;
-       struct bt_ctf_field *v_selected_0 = NULL;
-       struct bt_ctf_field *v_selected_1 = NULL;
-       struct bt_ctf_field *v_selected_2 = NULL;
-       struct bt_ctf_field *v_selected_3 = NULL;
-       struct bt_ctf_field *v_selected_4 = NULL;
-       struct bt_ctf_field *v_selected_5 = NULL;
-       struct bt_ctf_field *v_selected_6 = NULL;
-       struct bt_ctf_field *a = NULL;
-       struct bt_ctf_field *a_0 = NULL;
-       struct bt_ctf_field *a_1 = NULL;
-       struct bt_ctf_field *a_2 = NULL;
-       struct bt_ctf_field *a_3 = NULL;
-       struct bt_ctf_field *a_4 = NULL;
-       struct bt_ctf_field *strct = NULL;
-       struct bt_ctf_field *len_copy = NULL;
-       struct bt_ctf_field *fp_copy = NULL;
-       struct bt_ctf_field *s_copy = NULL;
-       struct bt_ctf_field *e_int_copy = NULL;
-       struct bt_ctf_field *e_copy = NULL;
-       struct bt_ctf_field *v_copy = NULL;
-       struct bt_ctf_field *v_selected_copy = NULL;
-       struct bt_ctf_field *v_selected_copy_len = NULL;
-       struct bt_ctf_field *v_selected_0_copy = NULL;
-       struct bt_ctf_field *v_selected_1_copy = NULL;
-       struct bt_ctf_field *v_selected_2_copy = NULL;
-       struct bt_ctf_field *v_selected_3_copy = NULL;
-       struct bt_ctf_field *v_selected_4_copy = NULL;
-       struct bt_ctf_field *v_selected_5_copy = NULL;
-       struct bt_ctf_field *v_selected_6_copy = NULL;
-       struct bt_ctf_field *a_copy = NULL;
-       struct bt_ctf_field *a_0_copy = NULL;
-       struct bt_ctf_field *a_1_copy = NULL;
-       struct bt_ctf_field *a_2_copy = NULL;
-       struct bt_ctf_field *a_3_copy = NULL;
-       struct bt_ctf_field *a_4_copy = NULL;
-       struct bt_ctf_field *strct_copy = NULL;
-       struct bt_ctf_field_type_enumeration_mapping_iterator *e_iter = NULL;
-       uint64_t uint64_t_val;
-       const char *str_val;
-       double double_val;
-       int ret;
-
-       /* create len type */
-       len_type = bt_ctf_field_type_integer_create(32);
-       assert(len_type);
-
-       /* create fp type */
-       fp_type = bt_ctf_field_type_floating_point_create();
-       assert(fp_type);
-
-       /* create s type */
-       s_type = bt_ctf_field_type_string_create();
-       assert(s_type);
-
-       /* create e_int type */
-       e_int_type = bt_ctf_field_type_integer_create(8);
-       assert(e_int_type);
-
-       /* create e type */
-       e_type = bt_ctf_field_type_enumeration_create(e_int_type);
-       assert(e_type);
-       ret = bt_ctf_field_type_enumeration_add_mapping(e_type, "LABEL1",
-               10, 15);
-       assert(!ret);
-       ret = bt_ctf_field_type_enumeration_add_mapping(e_type, "LABEL2",
-               23, 23);
-       assert(!ret);
-
-       /* create v_label1 type */
-       v_label1_type = bt_ctf_field_type_string_create();
-       assert(v_label1_type);
-
-       /* create v_label1_array type */
-       v_label1_array_type = bt_ctf_field_type_array_create(v_label1_type, 5);
-       assert(v_label1_array_type);
-
-       /* create v_label2 type */
-       v_label2_type = bt_ctf_field_type_integer_create(16);
-       assert(v_label2_type);
-
-       /* create v_label2_seq type */
-       v_label2_seq_type = bt_ctf_field_type_sequence_create(v_label2_type,
-               "len");
-       assert(v_label2_seq_type);
-
-       /* create v type */
-       v_type = bt_ctf_field_type_variant_create(e_type, "e");
-       assert(v_type);
-       ret = bt_ctf_field_type_variant_add_field(v_type, v_label1_array_type,
-               "LABEL1");
-       assert(!ret);
-       ret = bt_ctf_field_type_variant_add_field(v_type, v_label2_seq_type,
-               "LABEL2");
-       assert(!ret);
-
-       /* create strct type */
-       strct_type = bt_ctf_field_type_structure_create();
-       assert(strct_type);
-       ret = bt_ctf_field_type_structure_add_field(strct_type, len_type,
-               "len");
-       assert(!ret);
-       ret = bt_ctf_field_type_structure_add_field(strct_type, fp_type, "fp");
-       assert(!ret);
-       ret = bt_ctf_field_type_structure_add_field(strct_type, s_type, "s");
-       assert(!ret);
-       ret = bt_ctf_field_type_structure_add_field(strct_type, e_type, "e");
-       assert(!ret);
-       ret = bt_ctf_field_type_structure_add_field(strct_type, v_type, "v");
-       assert(!ret);
-       ret = bt_ctf_field_type_structure_add_field(strct_type,
-               v_label1_array_type, "a");
-       assert(!ret);
-
-       /* create strct */
-       strct = bt_ctf_field_create(strct_type);
-       assert(strct);
-
-       /* get len field */
-       len = bt_ctf_field_structure_get_field(strct, "len");
-       assert(len);
-
-       /* get fp field */
-       fp = bt_ctf_field_structure_get_field(strct, "fp");
-       assert(fp);
-
-       /* get s field */
-       s = bt_ctf_field_structure_get_field(strct, "s");
-       assert(s);
-
-       /* get e field */
-       e = bt_ctf_field_structure_get_field(strct, "e");
-       assert(e);
-
-       /* get e_int (underlying integer) */
-       e_int = bt_ctf_field_enumeration_get_container(e);
-       assert(e_int);
-
-       /* get v field */
-       v = bt_ctf_field_structure_get_field(strct, "v");
-       assert(v);
-
-       /* get a field */
-       a = bt_ctf_field_structure_get_field(strct, "a");
-       assert(a);
-
-       /* set len field */
-       ret = bt_ctf_field_unsigned_integer_set_value(len, 7);
-       assert(!ret);
-
-       /* set fp field */
-       ret = bt_ctf_field_floating_point_set_value(fp, 3.14);
-       assert(!ret);
-
-       /* set s field */
-       ret = bt_ctf_field_string_set_value(s, "btbt");
-       assert(!ret);
-
-       /* set e field (LABEL2) */
-       ret = bt_ctf_field_unsigned_integer_set_value(e_int, 23);
-       assert(!ret);
-
-       /* set v field */
-       v_selected = bt_ctf_field_variant_get_field(v, e);
-       assert(v_selected);
-       ok(!bt_ctf_field_variant_get_current_field(NULL),
-               "bt_ctf_field_variant_get_current_field handles NULL correctly");
-       v_selected_cur = bt_ctf_field_variant_get_current_field(v);
-       ok(v_selected_cur == v_selected,
-               "bt_ctf_field_variant_get_current_field returns the current field");
-       bt_put(v_selected_cur);
-
-       /* set selected v field */
-       ret = bt_ctf_field_sequence_set_length(v_selected, len);
-       assert(!ret);
-       v_selected_0 = bt_ctf_field_sequence_get_field(v_selected, 0);
-       assert(v_selected_0);
-       ret = bt_ctf_field_unsigned_integer_set_value(v_selected_0, 7);
-       assert(!ret);
-       v_selected_1 = bt_ctf_field_sequence_get_field(v_selected, 1);
-       assert(v_selected_1);
-       ret = bt_ctf_field_unsigned_integer_set_value(v_selected_1, 6);
-       assert(!ret);
-       v_selected_2 = bt_ctf_field_sequence_get_field(v_selected, 2);
-       assert(v_selected_2);
-       ret = bt_ctf_field_unsigned_integer_set_value(v_selected_2, 5);
-       assert(!ret);
-       v_selected_3 = bt_ctf_field_sequence_get_field(v_selected, 3);
-       assert(v_selected_3);
-       ret = bt_ctf_field_unsigned_integer_set_value(v_selected_3, 4);
-       assert(!ret);
-       v_selected_4 = bt_ctf_field_sequence_get_field(v_selected, 4);
-       assert(v_selected_4);
-       ret = bt_ctf_field_unsigned_integer_set_value(v_selected_4, 3);
-       assert(!ret);
-       v_selected_5 = bt_ctf_field_sequence_get_field(v_selected, 5);
-       assert(v_selected_5);
-       ret = bt_ctf_field_unsigned_integer_set_value(v_selected_5, 2);
-       assert(!ret);
-       v_selected_6 = bt_ctf_field_sequence_get_field(v_selected, 6);
-       assert(v_selected_6);
-       ret = bt_ctf_field_unsigned_integer_set_value(v_selected_6, 1);
-       assert(!ret);
-
-       /* set a field */
-       a_0 = bt_ctf_field_array_get_field(a, 0);
-       assert(a_0);
-       ret = bt_ctf_field_string_set_value(a_0, "a_0");
-       assert(!ret);
-       a_1 = bt_ctf_field_array_get_field(a, 1);
-       assert(a_1);
-       ret = bt_ctf_field_string_set_value(a_1, "a_1");
-       assert(!ret);
-       a_2 = bt_ctf_field_array_get_field(a, 2);
-       assert(a_2);
-       ret = bt_ctf_field_string_set_value(a_2, "a_2");
-       assert(!ret);
-       a_3 = bt_ctf_field_array_get_field(a, 3);
-       assert(a_3);
-       ret = bt_ctf_field_string_set_value(a_3, "a_3");
-       assert(!ret);
-       a_4 = bt_ctf_field_array_get_field(a, 4);
-       assert(a_4);
-       ret = bt_ctf_field_string_set_value(a_4, "a_4");
-       assert(!ret);
-
-       /* create copy of strct */
-       ok(!bt_ctf_field_copy(NULL),
-               "bt_ctf_field_copy handles NULL correctly");
-       strct_copy = bt_ctf_field_copy(strct);
-       ok(strct_copy,
-               "bt_ctf_field_copy returns a valid pointer");
-
-       /* get all copied fields */
-       len_copy = bt_ctf_field_structure_get_field(strct_copy, "len");
-       assert(len_copy);
-       fp_copy = bt_ctf_field_structure_get_field(strct_copy, "fp");
-       assert(fp_copy);
-       s_copy = bt_ctf_field_structure_get_field(strct_copy, "s");
-       assert(s_copy);
-       e_copy = bt_ctf_field_structure_get_field(strct_copy, "e");
-       assert(e_copy);
-       e_int_copy = bt_ctf_field_enumeration_get_container(e_copy);
-       assert(e_int_copy);
-       v_copy = bt_ctf_field_structure_get_field(strct_copy, "v");
-       assert(v_copy);
-       v_selected_copy = bt_ctf_field_variant_get_field(v_copy, e_copy);
-       assert(v_selected_copy);
-       v_selected_0_copy = bt_ctf_field_sequence_get_field(v_selected_copy, 0);
-       assert(v_selected_0_copy);
-       v_selected_1_copy = bt_ctf_field_sequence_get_field(v_selected_copy, 1);
-       assert(v_selected_1_copy);
-       v_selected_2_copy = bt_ctf_field_sequence_get_field(v_selected_copy, 2);
-       assert(v_selected_2_copy);
-       v_selected_3_copy = bt_ctf_field_sequence_get_field(v_selected_copy, 3);
-       assert(v_selected_3_copy);
-       v_selected_4_copy = bt_ctf_field_sequence_get_field(v_selected_copy, 4);
-       assert(v_selected_4_copy);
-       v_selected_5_copy = bt_ctf_field_sequence_get_field(v_selected_copy, 5);
-       assert(v_selected_5_copy);
-       v_selected_6_copy = bt_ctf_field_sequence_get_field(v_selected_copy, 6);
-       assert(v_selected_6_copy);
-       ok(!bt_ctf_field_sequence_get_field(v_selected_copy, 7),
-               "sequence field copy is not too large");
-       a_copy = bt_ctf_field_structure_get_field(strct_copy, "a");
-       assert(a_copy);
-       a_0_copy = bt_ctf_field_array_get_field(a_copy, 0);
-       assert(a_0_copy);
-       a_1_copy = bt_ctf_field_array_get_field(a_copy, 1);
-       assert(a_1_copy);
-       a_2_copy = bt_ctf_field_array_get_field(a_copy, 2);
-       assert(a_2_copy);
-       a_3_copy = bt_ctf_field_array_get_field(a_copy, 3);
-       assert(a_3_copy);
-       a_4_copy = bt_ctf_field_array_get_field(a_copy, 4);
-       assert(a_4_copy);
-       ok(!bt_ctf_field_array_get_field(v_selected_copy, 5),
-               "array field copy is not too large");
-
-       /* make sure copied fields are different pointers */
-       field_copy_tests_validate_diff_ptrs(strct_copy, strct, "strct");
-       field_copy_tests_validate_diff_ptrs(len_copy, len, "len");
-       field_copy_tests_validate_diff_ptrs(fp_copy, fp, "fp");
-       field_copy_tests_validate_diff_ptrs(s_copy, s, "s");
-       field_copy_tests_validate_diff_ptrs(e_int_copy, e_int, "e_int");
-       field_copy_tests_validate_diff_ptrs(e_copy, e, "e");
-       field_copy_tests_validate_diff_ptrs(v_copy, v, "v");
-       field_copy_tests_validate_diff_ptrs(v_selected_copy, v_selected,
-               "v_selected");
-       field_copy_tests_validate_diff_ptrs(v_selected_0_copy, v_selected_0,
-               "v_selected_0");
-       field_copy_tests_validate_diff_ptrs(v_selected_1_copy, v_selected_1,
-               "v_selected_1");
-       field_copy_tests_validate_diff_ptrs(v_selected_2_copy, v_selected_2,
-               "v_selected_2");
-       field_copy_tests_validate_diff_ptrs(v_selected_3_copy, v_selected_3,
-               "v_selected_3");
-       field_copy_tests_validate_diff_ptrs(v_selected_4_copy, v_selected_4,
-               "v_selected_4");
-       field_copy_tests_validate_diff_ptrs(v_selected_5_copy, v_selected_5,
-               "v_selected_5");
-       field_copy_tests_validate_diff_ptrs(v_selected_6_copy, v_selected_6,
-               "v_selected_6");
-       field_copy_tests_validate_diff_ptrs(a_copy, a, "a");
-       field_copy_tests_validate_diff_ptrs(a_0_copy, a_0, "a_0");
-       field_copy_tests_validate_diff_ptrs(a_1_copy, a_1, "a_1");
-       field_copy_tests_validate_diff_ptrs(a_2_copy, a_2, "a_2");
-       field_copy_tests_validate_diff_ptrs(a_3_copy, a_3, "a_3");
-       field_copy_tests_validate_diff_ptrs(a_4_copy, a_4, "a_4");
-
-       /* make sure copied fields share the same types */
-       field_copy_tests_validate_same_type(strct_copy, strct_type, "strct");
-       field_copy_tests_validate_same_type(len_copy, len_type, "len");
-       field_copy_tests_validate_same_type(fp_copy, fp_type, "fp");
-       field_copy_tests_validate_same_type(e_int_copy, e_int_type, "e_int");
-       field_copy_tests_validate_same_type(e_copy, e_type, "e");
-       field_copy_tests_validate_same_type(v_copy, v_type, "v");
-       field_copy_tests_validate_same_type(v_selected_copy, v_label2_seq_type,
-               "v_selected");
-       field_copy_tests_validate_same_type(v_selected_0_copy, v_label2_type,
-               "v_selected_0");
-       field_copy_tests_validate_same_type(v_selected_1_copy, v_label2_type,
-               "v_selected_1");
-       field_copy_tests_validate_same_type(v_selected_2_copy, v_label2_type,
-               "v_selected_2");
-       field_copy_tests_validate_same_type(v_selected_3_copy, v_label2_type,
-               "v_selected_3");
-       field_copy_tests_validate_same_type(v_selected_4_copy, v_label2_type,
-               "v_selected_4");
-       field_copy_tests_validate_same_type(v_selected_5_copy, v_label2_type,
-               "v_selected_5");
-       field_copy_tests_validate_same_type(v_selected_6_copy, v_label2_type,
-               "v_selected_6");
-       field_copy_tests_validate_same_type(a_copy, v_label1_array_type, "a");
-       field_copy_tests_validate_same_type(a_0_copy, v_label1_type, "a_0");
-       field_copy_tests_validate_same_type(a_1_copy, v_label1_type, "a_1");
-       field_copy_tests_validate_same_type(a_2_copy, v_label1_type, "a_2");
-       field_copy_tests_validate_same_type(a_3_copy, v_label1_type, "a_3");
-       field_copy_tests_validate_same_type(a_4_copy, v_label1_type, "a_4");
-
-       /* validate len copy */
-       ret = bt_ctf_field_unsigned_integer_get_value(len_copy, &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 7,
-               "bt_ctf_field_copy creates a valid integer field copy");
-
-       /* validate fp copy */
-       ret = bt_ctf_field_floating_point_get_value(fp_copy, &double_val);
-       assert(!ret);
-       ok(double_val == 3.14,
-               "bt_ctf_field_copy creates a valid floating point number field copy");
-
-       /* validate s copy */
-       str_val = bt_ctf_field_string_get_value(s_copy);
-       ok(str_val && !strcmp(str_val, "btbt"),
-               "bt_ctf_field_copy creates a valid string field copy");
-
-       /* validate e_int copy */
-       ret = bt_ctf_field_unsigned_integer_get_value(e_int_copy,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 23,
-               "bt_ctf_field_copy creates a valid enum's integer field copy");
-
-       /* validate e copy */
-       e_iter = bt_ctf_field_enumeration_get_mappings(e_copy);
-       (void) bt_ctf_field_type_enumeration_mapping_iterator_get_signed(e_iter,
-               &str_val, NULL, NULL);
-       ok(str_val && !strcmp(str_val, "LABEL2"),
-               "bt_ctf_field_copy creates a valid enum field copy");
-
-       /* validate v_selected copy */
-       v_selected_copy_len = bt_ctf_field_sequence_get_length(v_selected);
-       assert(v_selected_copy_len);
-       ret = bt_ctf_field_unsigned_integer_get_value(v_selected_copy_len,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 7,
-               "bt_ctf_field_copy creates a sequence field copy with the proper length");
-       bt_put(v_selected_copy_len);
-       v_selected_copy_len = NULL;
-
-       /* validate v_selected copy fields */
-       ret = bt_ctf_field_unsigned_integer_get_value(v_selected_0_copy,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 7,
-               "bt_ctf_field_copy creates a valid sequence field element copy (v_selected_0)");
-       ret = bt_ctf_field_unsigned_integer_get_value(v_selected_1_copy,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 6,
-               "bt_ctf_field_copy creates a valid sequence field element copy (v_selected_1)");
-       ret = bt_ctf_field_unsigned_integer_get_value(v_selected_2_copy,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 5,
-               "bt_ctf_field_copy creates a valid sequence field element copy (v_selected_2)");
-       ret = bt_ctf_field_unsigned_integer_get_value(v_selected_3_copy,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 4,
-               "bt_ctf_field_copy creates a valid sequence field element copy (v_selected_3)");
-       ret = bt_ctf_field_unsigned_integer_get_value(v_selected_4_copy,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 3,
-               "bt_ctf_field_copy creates a valid sequence field element copy (v_selected_4)");
-       ret = bt_ctf_field_unsigned_integer_get_value(v_selected_5_copy,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 2,
-               "bt_ctf_field_copy creates a valid sequence field element copy (v_selected_5)");
-       ret = bt_ctf_field_unsigned_integer_get_value(v_selected_6_copy,
-               &uint64_t_val);
-       assert(!ret);
-       ok(uint64_t_val == 1,
-               "bt_ctf_field_copy creates a valid sequence field element copy (v_selected_6)");
-
-       /* validate a copy fields */
-       str_val = bt_ctf_field_string_get_value(a_0_copy);
-       ok(str_val && !strcmp(str_val, "a_0"),
-               "bt_ctf_field_copy creates a valid array field element copy (a_0)");
-       str_val = bt_ctf_field_string_get_value(a_1_copy);
-       ok(str_val && !strcmp(str_val, "a_1"),
-               "bt_ctf_field_copy creates a valid array field element copy (a_1)");
-       str_val = bt_ctf_field_string_get_value(a_2_copy);
-       ok(str_val && !strcmp(str_val, "a_2"),
-               "bt_ctf_field_copy creates a valid array field element copy (a_2)");
-       str_val = bt_ctf_field_string_get_value(a_3_copy);
-       ok(str_val && !strcmp(str_val, "a_3"),
-               "bt_ctf_field_copy creates a valid array field element copy (a_3)");
-       str_val = bt_ctf_field_string_get_value(a_4_copy);
-       ok(str_val && !strcmp(str_val, "a_4"),
-               "bt_ctf_field_copy creates a valid array field element copy (a_4)");
-
-       /* put everything */
-       bt_put(len_type);
-       bt_put(fp_type);
-       bt_put(s_type);
-       bt_put(e_int_type);
-       bt_put(e_type);
-       bt_put(v_type);
-       bt_put(v_label1_type);
-       bt_put(v_label1_array_type);
-       bt_put(v_label2_type);
-       bt_put(v_label2_seq_type);
-       bt_put(strct_type);
-       bt_put(len);
-       bt_put(fp);
-       bt_put(s);
-       bt_put(e_int);
-       bt_put(e);
-       bt_put(v);
-       bt_put(v_selected);
-       bt_put(v_selected_0);
-       bt_put(v_selected_1);
-       bt_put(v_selected_2);
-       bt_put(v_selected_3);
-       bt_put(v_selected_4);
-       bt_put(v_selected_5);
-       bt_put(v_selected_6);
-       bt_put(a);
-       bt_put(a_0);
-       bt_put(a_1);
-       bt_put(a_2);
-       bt_put(a_3);
-       bt_put(a_4);
-       bt_put(strct);
-       bt_put(len_copy);
-       bt_put(fp_copy);
-       bt_put(s_copy);
-       bt_put(e_int_copy);
-       bt_put(e_copy);
-       bt_put(v_copy);
-       bt_put(v_selected_copy);
-       bt_put(v_selected_0_copy);
-       bt_put(v_selected_1_copy);
-       bt_put(v_selected_2_copy);
-       bt_put(v_selected_3_copy);
-       bt_put(v_selected_4_copy);
-       bt_put(v_selected_5_copy);
-       bt_put(v_selected_6_copy);
-       bt_put(a_copy);
-       bt_put(a_0_copy);
-       bt_put(a_1_copy);
-       bt_put(a_2_copy);
-       bt_put(a_3_copy);
-       bt_put(a_4_copy);
-       bt_put(strct_copy);
-       bt_put(e_iter);
-}
-
 static
 void type_field_tests()
 {
        struct bt_ctf_field *uint_12;
        struct bt_ctf_field *int_16;
        struct bt_ctf_field *string;
-       struct bt_ctf_field *enumeration;
        struct bt_ctf_field_type *composite_structure_type;
        struct bt_ctf_field_type *structure_seq_type;
        struct bt_ctf_field_type *string_type;
@@ -1610,9 +852,6 @@ void type_field_tests()
        struct bt_ctf_field_type *returned_type;
        const char *ret_string;
 
-       returned_type = bt_ctf_field_get_type(NULL);
-       ok(!returned_type, "bt_ctf_field_get_type handles NULL correctly");
-
        ok(uint_12_type, "Create an unsigned integer type");
        ok(bt_ctf_field_type_integer_set_base(uint_12_type,
                BT_CTF_INTEGER_BASE_BINARY) == 0,
@@ -1635,12 +874,8 @@ void type_field_tests()
                "Set integer type signedness to signed");
        ok(bt_ctf_field_type_integer_set_signed(uint_12_type, 0) == 0,
                "Set integer type signedness to unsigned");
-       ok(bt_ctf_field_type_integer_get_size(NULL) < 0,
-               "bt_ctf_field_type_integer_get_size handles NULL correctly");
        ok(bt_ctf_field_type_integer_get_size(uint_12_type) == 12,
                "bt_ctf_field_type_integer_get_size returns a correct value");
-       ok(bt_ctf_field_type_integer_get_signed(NULL) < 0,
-               "bt_ctf_field_type_integer_get_signed handles NULL correctly");
        ok(bt_ctf_field_type_integer_get_signed(uint_12_type) == 0,
                "bt_ctf_field_type_integer_get_signed returns a correct value for unsigned types");
 
@@ -1659,20 +894,11 @@ void type_field_tests()
        ok(bt_ctf_field_type_get_byte_order(uint_12_type) ==
                BT_CTF_BYTE_ORDER_BIG_ENDIAN,
                "bt_ctf_field_type_get_byte_order returns a correct value");
-       ok(bt_ctf_field_type_get_byte_order(NULL) ==
-               BT_CTF_BYTE_ORDER_UNKNOWN,
-               "bt_ctf_field_type_get_byte_order handles NULL correctly");
 
-       ok(bt_ctf_field_type_get_type_id(NULL) ==
-               BT_CTF_FIELD_TYPE_ID_UNKNOWN,
-               "bt_ctf_field_type_get_type_id handles NULL correctly");
        ok(bt_ctf_field_type_get_type_id(uint_12_type) ==
                BT_CTF_FIELD_TYPE_ID_INTEGER,
                "bt_ctf_field_type_get_type_id returns a correct value with an integer type");
 
-       ok(bt_ctf_field_type_integer_get_base(NULL) ==
-               BT_CTF_INTEGER_BASE_UNKNOWN,
-               "bt_ctf_field_type_integer_get_base handles NULL correctly");
        ok(bt_ctf_field_type_integer_get_base(uint_12_type) ==
                BT_CTF_INTEGER_BASE_HEXADECIMAL,
                "bt_ctf_field_type_integer_get_base returns a correct value");
@@ -1686,9 +912,6 @@ void type_field_tests()
        ok(bt_ctf_field_type_integer_set_encoding(uint_12_type,
                BT_CTF_STRING_ENCODING_UTF8) == 0,
                "Set integer type encoding to UTF8");
-       ok(bt_ctf_field_type_integer_get_encoding(NULL) ==
-               BT_CTF_STRING_ENCODING_UNKNOWN,
-               "bt_ctf_field_type_integer_get_encoding handles NULL correctly");
        ok(bt_ctf_field_type_integer_get_encoding(uint_12_type) ==
                BT_CTF_STRING_ENCODING_UTF8,
                "bt_ctf_field_type_integer_get_encoding returns a correct value");
@@ -1707,18 +930,14 @@ void type_field_tests()
                BT_CTF_FIELD_TYPE_ID_SEQUENCE,
                "bt_ctf_field_type_get_type_id returns a correct value with a sequence type");
 
-       ok(bt_ctf_field_type_sequence_get_length_field_name(NULL) == NULL,
-               "bt_ctf_field_type_sequence_get_length_field_name handles NULL correctly");
        ret_string = bt_ctf_field_type_sequence_get_length_field_name(
                sequence_type);
        ok(!strcmp(ret_string, "seq_len"),
                "bt_ctf_field_type_sequence_get_length_field_name returns the correct value");
-       ok(bt_ctf_field_type_sequence_get_element_type(NULL) == NULL,
-               "bt_ctf_field_type_sequence_get_element_type handles NULL correctly");
-       returned_type = bt_ctf_field_type_sequence_get_element_type(
+       returned_type = bt_ctf_field_type_sequence_get_element_field_type(
                sequence_type);
        ok(returned_type == int_16_type,
-               "bt_ctf_field_type_sequence_get_element_type returns the correct type");
+               "bt_ctf_field_type_sequence_get_element_field_type returns the correct type");
        bt_put(returned_type);
 
        string_type = bt_ctf_field_type_string_create();
@@ -1733,9 +952,6 @@ void type_field_tests()
                BT_CTF_STRING_ENCODING_ASCII) == 0,
                "Set string encoding to ASCII");
 
-       ok(bt_ctf_field_type_string_get_encoding(NULL) ==
-               BT_CTF_STRING_ENCODING_UNKNOWN,
-               "bt_ctf_field_type_string_get_encoding handles NULL correctly");
        ok(bt_ctf_field_type_string_get_encoding(string_type) ==
                BT_CTF_STRING_ENCODING_ASCII,
                "bt_ctf_field_type_string_get_encoding returns the correct value");
@@ -1752,14 +968,9 @@ void type_field_tests()
                sequence_type, "a_sequence") == 0,
                "Add a sequence type to a structure");
 
-       ok(bt_ctf_field_type_structure_get_field_count(NULL) < 0,
-               "bt_ctf_field_type_structure_get_field_count handles NULL correctly");
        ok(bt_ctf_field_type_structure_get_field_count(structure_seq_type) == 2,
                "bt_ctf_field_type_structure_get_field_count returns a correct value");
 
-       ok(bt_ctf_field_type_structure_get_field(NULL,
-               &ret_string, &returned_type, 1) < 0,
-               "bt_ctf_field_type_structure_get_field handles a NULL type correctly");
        ok(bt_ctf_field_type_structure_get_field(structure_seq_type,
                NULL, &returned_type, 1) == 0,
                "bt_ctf_field_type_structure_get_field handles a NULL name correctly");
@@ -1767,9 +978,6 @@ void type_field_tests()
        ok(bt_ctf_field_type_structure_get_field(structure_seq_type,
                &ret_string, NULL, 1) == 0,
                "bt_ctf_field_type_structure_get_field handles a NULL return type correctly");
-       ok(bt_ctf_field_type_structure_get_field(structure_seq_type,
-               &ret_string, &returned_type, 10) < 0,
-               "bt_ctf_field_type_structure_get_field handles an invalid index correctly");
        ok(bt_ctf_field_type_structure_get_field(structure_seq_type,
                &ret_string, &returned_type, 1) == 0,
                "bt_ctf_field_type_structure_get_field returns a field");
@@ -1779,10 +987,6 @@ void type_field_tests()
                "bt_ctf_field_type_structure_get_field returns a correct field type");
        bt_put(returned_type);
 
-       ok(bt_ctf_field_type_structure_get_field_type_by_name(NULL, "a_sequence") == NULL,
-               "bt_ctf_field_type_structure_get_field_type_by_name handles a NULL structure correctly");
-       ok(bt_ctf_field_type_structure_get_field_type_by_name(structure_seq_type, NULL) == NULL,
-               "bt_ctf_field_type_structure_get_field_type_by_name handles a NULL field name correctly");
        returned_type = bt_ctf_field_type_structure_get_field_type_by_name(
                structure_seq_type, "a_sequence");
        ok(returned_type == sequence_type,
@@ -1797,12 +1001,6 @@ void type_field_tests()
                structure_seq_type, "inner_structure") == 0,
                "Add a structure type to a structure");
 
-       ok(bt_ctf_field_type_structure_get_field_type_by_name(
-               NULL, "a_sequence") == NULL,
-               "bt_ctf_field_type_structure_get_field_type_by_name handles a NULL field correctly");
-       ok(bt_ctf_field_type_structure_get_field_type_by_name(
-               structure_seq_type, NULL) == NULL,
-               "bt_ctf_field_type_structure_get_field_type_by_name handles a NULL field name correctly");
        returned_type = bt_ctf_field_type_structure_get_field_type_by_name(
                structure_seq_type, "a_sequence");
        ok(returned_type == sequence_type,
@@ -1824,29 +1022,17 @@ void type_field_tests()
        ok(bt_ctf_field_type_integer_set_signed(uint_12_type, 0),
                "Check an integer type's signedness can't be modified after instanciation");
 
-       /* Check signed property is checked */
-       ok(bt_ctf_field_signed_integer_set_value(uint_12, -52),
-               "Check bt_ctf_field_signed_integer_set_value is not allowed on an unsigned integer");
-       ok(bt_ctf_field_unsigned_integer_set_value(int_16, 42),
-               "Check bt_ctf_field_unsigned_integer_set_value is not allowed on a signed integer");
-
        /* Check overflows are properly tested for */
-       ok(bt_ctf_field_signed_integer_set_value(int_16, -32768) == 0,
+       ok(bt_ctf_field_integer_signed_set_value(int_16, -32768) == 0,
                "Check -32768 is allowed for a signed 16-bit integer");
-       ok(bt_ctf_field_signed_integer_set_value(int_16, 32767) == 0,
+       ok(bt_ctf_field_integer_signed_set_value(int_16, 32767) == 0,
                "Check 32767 is allowed for a signed 16-bit integer");
-       ok(bt_ctf_field_signed_integer_set_value(int_16, 32768),
-               "Check 32768 is not allowed for a signed 16-bit integer");
-       ok(bt_ctf_field_signed_integer_set_value(int_16, -32769),
-               "Check -32769 is not allowed for a signed 16-bit integer");
-       ok(bt_ctf_field_signed_integer_set_value(int_16, -42) == 0,
+       ok(bt_ctf_field_integer_signed_set_value(int_16, -42) == 0,
                "Check -42 is allowed for a signed 16-bit integer");
 
-       ok(bt_ctf_field_unsigned_integer_set_value(uint_12, 4095) == 0,
+       ok(bt_ctf_field_integer_unsigned_set_value(uint_12, 4095) == 0,
                "Check 4095 is allowed for an unsigned 12-bit integer");
-       ok(bt_ctf_field_unsigned_integer_set_value(uint_12, 4096),
-               "Check 4096 is not allowed for a unsigned 12-bit integer");
-       ok(bt_ctf_field_unsigned_integer_set_value(uint_12, 0) == 0,
+       ok(bt_ctf_field_integer_unsigned_set_value(uint_12, 0) == 0,
                "Check 0 is allowed for an unsigned 12-bit integer");
 
        string = bt_ctf_field_create(string_type);
@@ -1857,14 +1043,10 @@ void type_field_tests()
        enumeration_type = bt_ctf_field_type_enumeration_create(uint_12_type);
        ok(enumeration_type,
                "Create an enumeration type with an unsigned 12-bit integer as container");
-       enumeration = bt_ctf_field_create(enumeration_type);
-       ok(!enumeration,
-               "Check enumeration types are validated before instantiation");
 
        bt_put(string);
        bt_put(uint_12);
        bt_put(int_16);
-       bt_put(enumeration);
        bt_put(composite_structure_type);
        bt_put(structure_seq_type);
        bt_put(string_type);
@@ -1919,7 +1101,7 @@ void packet_resize_test(struct bt_ctf_stream_class *stream_class,
         * of event_class, so we retrieve the new ones to create the
         * appropriate fields.
         */
-       ep_type = bt_ctf_event_class_get_payload_type(event_class);
+       ep_type = bt_ctf_event_class_get_payload_field_type(event_class);
        assert(ep_type);
        ep_field_1_type = bt_ctf_field_type_structure_get_field_type_by_name(
                ep_type, "field_1");
@@ -1929,17 +1111,10 @@ void packet_resize_test(struct bt_ctf_stream_class *stream_class,
        assert(ep_a_string_type);
 
        event = bt_ctf_event_create(event_class);
-       ret_field = bt_ctf_event_get_payload_by_index(event, 0);
+       ret_field = bt_ctf_event_get_payload(event, 0);
        ret_field_type = bt_ctf_field_get_type(ret_field);
-       ok(bt_ctf_field_type_compare(ret_field_type, integer_type) == 0,
-               "bt_ctf_event_get_payload_by_index returns a correct field");
        bt_put(ret_field_type);
        bt_put(ret_field);
-
-       ok(bt_ctf_event_get_payload_by_index(NULL, 0) == NULL,
-               "bt_ctf_event_get_payload_by_index handles NULL correctly");
-       ok(bt_ctf_event_get_payload_by_index(event, 4) == NULL,
-               "bt_ctf_event_get_payload_by_index handles an invalid index correctly");
        bt_put(event);
 
        for (i = 0; i < packet_resize_test_length; i++) {
@@ -1950,7 +1125,7 @@ void packet_resize_test(struct bt_ctf_stream_class *stream_class,
                        bt_ctf_field_create(ep_a_string_type);
 
                ret |= bt_ctf_clock_set_time(clock, ++current_time);
-               ret |= bt_ctf_field_unsigned_integer_set_value(integer, i);
+               ret |= bt_ctf_field_integer_unsigned_set_value(integer, i);
                ret |= bt_ctf_event_set_payload(event, "field_1",
                        integer);
                bt_put(integer);
@@ -1962,10 +1137,10 @@ void packet_resize_test(struct bt_ctf_stream_class *stream_class,
                /* Populate stream event context */
                stream_event_context =
                        bt_ctf_event_get_stream_event_context(event);
-               integer = bt_ctf_field_structure_get_field(stream_event_context,
+               integer = bt_ctf_field_structure_get_field_by_name(stream_event_context,
                        "common_event_context");
                BT_PUT(stream_event_context);
-               ret |= bt_ctf_field_unsigned_integer_set_value(integer,
+               ret |= bt_ctf_field_integer_unsigned_set_value(integer,
                        i % 42);
                bt_put(integer);
 
@@ -1978,10 +1153,6 @@ void packet_resize_test(struct bt_ctf_stream_class *stream_class,
        }
 
        events_appended = !!(i == packet_resize_test_length);
-       ok(bt_ctf_stream_get_discarded_events_count(NULL, &ret_uint64) < 0,
-               "bt_ctf_stream_get_discarded_events_count handles a NULL stream correctly");
-       ok(bt_ctf_stream_get_discarded_events_count(stream, NULL) < 0,
-               "bt_ctf_stream_get_discarded_events_count handles a NULL return pointer correctly");
        ret = bt_ctf_stream_get_discarded_events_count(stream, &ret_uint64);
        ok(ret == 0 && ret_uint64 == 0,
                "bt_ctf_stream_get_discarded_events_count returns a correct number of discarded events when none were discarded");
@@ -1998,9 +1169,9 @@ end:
         * otherwise flush will fail.
         */
        packet_context = bt_ctf_stream_get_packet_context(stream);
-       packet_context_field = bt_ctf_field_structure_get_field(packet_context,
+       packet_context_field = bt_ctf_field_structure_get_field_by_name(packet_context,
                "custom_packet_context_field");
-       bt_ctf_field_unsigned_integer_set_value(packet_context_field, 2);
+       bt_ctf_field_integer_unsigned_set_value(packet_context_field, 2);
 
        ok(bt_ctf_stream_flush(stream) == 0,
                "Flush a stream that forces a packet resize");
@@ -2045,8 +1216,6 @@ void test_empty_stream(struct bt_ctf_writer *writer)
        ret = bt_ctf_stream_class_set_event_header_type(stream_class, NULL);
        assert(ret == 0);
 
-       ok(bt_ctf_stream_class_get_trace(NULL) == NULL,
-               "bt_ctf_stream_class_get_trace handles NULL correctly");
        ok(bt_ctf_stream_class_get_trace(stream_class) == NULL,
                "bt_ctf_stream_class_get_trace returns NULL when stream class is orphaned");
 
@@ -2166,14 +1335,14 @@ void test_custom_event_header_stream(struct bt_ctf_writer *writer,
                goto end;
        }
 
-       integer = bt_ctf_field_structure_get_field(packet_header,
+       integer = bt_ctf_field_structure_get_field_by_name(packet_header,
                "custom_trace_packet_header_field");
        if (!integer) {
                fail("Failed to retrieve custom_trace_packet_header_field");
                goto end;
        }
 
-       ret = bt_ctf_field_unsigned_integer_set_value(integer, 3487);
+       ret = bt_ctf_field_integer_unsigned_set_value(integer, 3487);
        if (ret) {
                fail("Failed to set custom_trace_packet_header_field value");
                goto end;
@@ -2192,14 +1361,14 @@ void test_custom_event_header_stream(struct bt_ctf_writer *writer,
                goto end;
        }
 
-       integer = bt_ctf_field_structure_get_field(event_header,
+       integer = bt_ctf_field_structure_get_field_by_name(event_header,
                "seq_len");
        if (!integer) {
                fail("Failed to get seq_len field from event header");
                goto end;
        }
 
-       ret = bt_ctf_field_unsigned_integer_set_value(integer, 2);
+       ret = bt_ctf_field_integer_unsigned_set_value(integer, 2);
        if (ret) {
                fail("Failed to set seq_len value in event header");
                goto end;
@@ -2226,7 +1395,7 @@ void test_custom_event_header_stream(struct bt_ctf_writer *writer,
                        goto end;
                }
 
-               ret = bt_ctf_field_unsigned_integer_set_value(integer, i);
+               ret = bt_ctf_field_integer_unsigned_set_value(integer, i);
                if (ret) {
                        fail("Failed to set sequence element value");
                        goto end;
@@ -2271,6 +1440,7 @@ void test_instanciate_event_before_stream(struct bt_ctf_writer *writer,
        struct bt_ctf_event_class *event_class = NULL;
        struct bt_ctf_event *event = NULL;
        struct bt_ctf_field_type *integer_type = NULL;
+       struct bt_ctf_field *payload_field = NULL;
        struct bt_ctf_field *integer = NULL;
 
        stream_class = bt_ctf_stream_class_create("event_before_stream_test");
@@ -2314,14 +1484,21 @@ void test_instanciate_event_before_stream(struct bt_ctf_writer *writer,
                goto end;
        }
 
-       integer = bt_ctf_event_get_payload_by_index(event, 0);
+       payload_field = bt_ctf_event_get_payload_field(event);
+       if (!payload_field) {
+               diag("Failed to get event's payload field");
+               ret = -1;
+               goto end;
+       }
+
+       integer = bt_ctf_field_structure_get_field_by_index(payload_field, 0);
        if (!integer) {
                diag("Failed to get integer field payload from event");
                ret = -1;
                goto end;
        }
 
-       ret = bt_ctf_field_unsigned_integer_set_value(integer, 1234);
+       ret = bt_ctf_field_integer_unsigned_set_value(integer, 1234);
        if (ret) {
                diag("Failed to set integer field value");
                goto end;
@@ -2334,11 +1511,6 @@ void test_instanciate_event_before_stream(struct bt_ctf_writer *writer,
                goto end;
        }
 
-       ok(bt_ctf_event_get_stream(NULL) == NULL,
-               "bt_ctf_event_get_stream handles NULL correctly");
-       ok(bt_ctf_event_get_stream(event) == NULL,
-               "bt_ctf_event_get_stream returns NULL on event which has not yet been appended to a stream");
-
        ret = bt_ctf_stream_append_event(stream, event);
        if (ret) {
                diag("Failed to append event to stream");
@@ -2358,6 +1530,7 @@ end:
        bt_put(event);
        bt_put(integer_type);
        bt_put(integer);
+       bt_put(payload_field);
 }
 
 static
@@ -2379,224 +1552,6 @@ void append_existing_event_class(struct bt_ctf_stream_class *stream_class)
        bt_put(event_class);
 }
 
-static
-struct bt_ctf_event_class *create_minimal_event_class(void)
-{
-       struct bt_ctf_event_class *ec = NULL;
-       struct bt_ctf_field_type *int_ft = NULL;
-       int ret;
-
-       int_ft = bt_ctf_field_type_integer_create(23);
-       assert(int_ft);
-       ec = bt_ctf_event_class_create("minimal");
-       assert(ec);
-       ret = bt_ctf_event_class_add_field(ec, int_ft, "field");
-       assert(!ret);
-       BT_PUT(int_ft);
-
-       return ec;
-}
-
-static
-void test_create_writer_vs_non_writer_mode(void)
-{
-       int ret;
-       gchar *trace_path;
-       const char *writer_stream_name = "writer stream instance";
-       struct bt_ctf_writer *writer = NULL;
-       struct bt_ctf_trace *writer_trace = NULL;
-       struct bt_ctf_stream_class *writer_sc = NULL;
-       struct bt_ctf_stream *writer_stream = NULL;
-       struct bt_ctf_stream *writer_stream2 = NULL;
-       struct bt_ctf_stream *packet_stream = NULL;
-       struct bt_ctf_trace *non_writer_trace = NULL;
-       struct bt_ctf_stream_class *non_writer_sc = NULL;
-       struct bt_ctf_stream *non_writer_stream = NULL;
-       struct bt_ctf_stream *non_writer_stream2 = NULL;
-       struct bt_ctf_event_class *writer_ec = NULL;
-       struct bt_ctf_event_class *non_writer_ec = NULL;
-       struct bt_ctf_event *event = NULL;
-       struct bt_ctf_event *event2 = NULL;
-       struct bt_ctf_field_type *empty_struct_ft = NULL;
-       struct bt_ctf_field *int_field = NULL;
-       struct bt_ctf_clock *writer_clock = NULL;
-       struct bt_ctf_clock_class *non_writer_clock_class = NULL;
-       struct bt_ctf_packet *packet = NULL;
-       struct bt_ctf_packet *packet2 = NULL;
-
-       trace_path = g_build_filename(g_get_tmp_dir(), "ctfwriter_XXXXXX", NULL);
-       if (!bt_mkdtemp(trace_path)) {
-               perror("# perror");
-       }
-
-       /* Create empty structure field type (event header) */
-       empty_struct_ft = bt_ctf_field_type_structure_create();
-       assert(empty_struct_ft);
-
-       /* Create writer, writer stream class, stream, and clock */
-       writer = bt_ctf_writer_create(trace_path);
-       assert(writer);
-       writer_clock = bt_ctf_clock_create("writer_clock");
-       assert(writer_clock);
-       ret = bt_ctf_writer_add_clock(writer, writer_clock);
-       assert(!ret);
-       ret = bt_ctf_writer_set_byte_order(writer, BT_CTF_BYTE_ORDER_LITTLE_ENDIAN);
-       assert(!ret);
-       writer_trace = bt_ctf_writer_get_trace(writer);
-       ok(writer_trace, "bt_ctf_writer_get_trace() returns a trace");
-       writer_sc = bt_ctf_stream_class_create("writer_sc");
-       assert(writer_sc);
-       ret = bt_ctf_stream_class_set_event_header_type(writer_sc,
-               empty_struct_ft);
-       assert(!ret);
-       ret = bt_ctf_stream_class_set_clock(writer_sc, writer_clock);
-       assert(!ret);
-       ret = bt_ctf_trace_add_stream_class(writer_trace, writer_sc);
-       assert(!ret);
-       writer_stream = bt_ctf_stream_create(writer_sc, writer_stream_name);
-       assert(writer_stream);
-       ok(!strcmp(bt_ctf_stream_get_name(writer_stream), writer_stream_name),
-               "bt_ctf_stream_get_name() returns the stream's name");
-
-       /* Create non-writer trace, stream class, stream, and clock */
-       non_writer_trace = bt_ctf_trace_create();
-       assert(non_writer_trace);
-       non_writer_sc = bt_ctf_stream_class_create("nonwriter_sc");
-       assert(non_writer_sc);
-       ret = bt_ctf_stream_class_set_event_header_type(non_writer_sc,
-               empty_struct_ft);
-       assert(!ret);
-       ret = bt_ctf_stream_class_set_packet_context_type(non_writer_sc, NULL);
-       assert(!ret);
-       ret = bt_ctf_trace_add_stream_class(non_writer_trace, non_writer_sc);
-       assert(!ret);
-       non_writer_stream = bt_ctf_stream_create(non_writer_sc, NULL);
-       assert(non_writer_stream);
-       non_writer_clock_class =
-               bt_ctf_clock_class_create("non_writer_clock_class",
-                       1000000000);
-       assert(non_writer_clock_class);
-       ret = bt_ctf_trace_add_clock_class(non_writer_trace,
-               non_writer_clock_class);
-       assert(!ret);
-
-       /* Create event class and event */
-       writer_ec = create_minimal_event_class();
-       assert(writer_ec);
-       ret = bt_ctf_stream_class_add_event_class(writer_sc, writer_ec);
-       assert(!ret);
-       event = bt_ctf_event_create(writer_ec);
-       assert(event);
-       int_field = bt_ctf_event_get_payload_by_index(event, 0);
-       assert(int_field);
-       bt_ctf_field_unsigned_integer_set_value(int_field, 17);
-
-       /*
-        * Verify non-writer stream: it should be impossible to append
-        * an event to it.
-        */
-       ok(bt_ctf_stream_append_event(non_writer_stream, event),
-               "bt_ctf_stream_append_event() fails with a non-writer stream");
-
-       /*
-        * Verify writer stream: it should be possible to append an
-        * event to it.
-        */
-       ok(!bt_ctf_stream_append_event(writer_stream, event),
-               "bt_ctf_stream_append_event() succeeds with a writer stream");
-
-       /*
-        * It should be possible to create a packet from a non-writer
-        * stream, but not from a writer stream.
-        */
-       packet = bt_ctf_packet_create(writer_stream);
-       ok(!packet, "bt_ctf_packet_create() fails with a writer stream");
-       packet = bt_ctf_packet_create(non_writer_stream);
-       ok(packet, "bt_ctf_packet_create() succeeds with a non-writer stream");
-       packet_stream = bt_ctf_packet_get_stream(packet);
-       ok(packet_stream == non_writer_stream,
-               "bt_ctf_packet_get_stream() returns the correct stream");
-
-       /*
-        * It should not be possible to append an event associated to
-        * a stream to a different stream.
-        */
-       writer_stream2 = bt_ctf_stream_create(writer_sc, "zoo");
-       assert(writer_stream2);
-       ok(bt_ctf_stream_append_event(writer_stream2, event),
-               "bt_ctf_stream_append_event() fails with an event associated to another stream");
-
-       /*
-        * It should not be possible to set the packet of an event
-        * associated to a given stream to a packet associated with
-        * a different stream.
-        */
-       ok(bt_ctf_event_set_packet(event, packet),
-               "bt_ctf_event_set_packet() fails with a packet not sharing the event's stream");
-
-       /*
-        * It should be possible to set the packet of a fresh event, as
-        * long as the originating stream classes are the same.
-        */
-       event2 = bt_ctf_event_create(writer_ec);
-       assert(event2);
-       ok(bt_ctf_event_set_packet(event2, packet),
-               "bt_ctf_event_set_packet() fails when the event's and the packet's stream class differ");
-       non_writer_ec = create_minimal_event_class();
-       assert(non_writer_ec);
-       ret = bt_ctf_stream_class_add_event_class(non_writer_sc, non_writer_ec);
-       assert(!ret);
-       BT_PUT(event2);
-       event2 = bt_ctf_event_create(non_writer_ec);
-       assert(event2);
-       ok(!bt_ctf_event_set_packet(event2, packet),
-               "bt_ctf_event_set_packet() succeeds when the event's and the packet's stream class are the same");
-
-       /*
-        * It should be possible to set a packet created from the same
-        * stream to an event with an existing packet.
-        */
-       packet2 = bt_ctf_packet_create(non_writer_stream);
-       assert(packet2);
-       ok(!bt_ctf_event_set_packet(event2, packet2),
-               "bt_ctf_event_set_packet() succeeds when the event's current packet has the same stream");
-       BT_PUT(packet2);
-
-       /*
-        * It should not be possible to set a packet created from a
-        * different stream to an event with an existing packet.
-        */
-       non_writer_stream2 = bt_ctf_stream_create(non_writer_sc, "rj45");
-       assert(non_writer_stream2);
-       packet2 = bt_ctf_packet_create(non_writer_stream);
-       assert(packet2);
-       ok(!bt_ctf_event_set_packet(event2, packet2),
-               "bt_ctf_event_set_packet() fails when the event's current packet does not have the same stream");
-
-       bt_put(writer);
-       bt_put(writer_trace);
-       bt_put(writer_sc);
-       bt_put(writer_stream);
-       bt_put(writer_stream2);
-       bt_put(non_writer_trace);
-       bt_put(non_writer_sc);
-       bt_put(non_writer_stream);
-       bt_put(non_writer_stream2);
-       bt_put(packet_stream);
-       bt_put(writer_ec);
-       bt_put(non_writer_ec);
-       bt_put(event);
-       bt_put(event2);
-       bt_put(int_field);
-       bt_put(empty_struct_ft);
-       bt_put(writer_clock);
-       bt_put(non_writer_clock_class);
-       bt_put(packet);
-       bt_put(packet2);
-       recursive_rmdir(trace_path);
-       g_free(trace_path);
-}
-
 static
 void test_clock_utils(void)
 {
@@ -2617,189 +1572,6 @@ void test_clock_utils(void)
        BT_PUT(clock);
 }
 
-void test_set_clock_non_writer_stream_class(void)
-{
-       struct bt_ctf_clock *clock;
-       struct bt_ctf_trace *trace;
-       struct bt_ctf_stream_class *sc;
-       int ret;
-
-       clock = bt_ctf_clock_create("the_clock");
-       assert(clock);
-
-       trace = bt_ctf_trace_create();
-       assert(trace);
-
-       sc = bt_ctf_stream_class_create(NULL);
-       assert(sc);
-
-       ret = bt_ctf_stream_class_set_clock(sc, clock);
-       assert(ret == 0);
-
-       ret = bt_ctf_trace_add_stream_class(trace, sc);
-       ok(ret < 0,
-               "bt_ctf_trace_add_stream_class() fails with a stream class with a registered clock");
-
-       bt_put(clock);
-       bt_put(trace);
-       bt_put(sc);
-}
-
-static
-void test_static_trace(void)
-{
-       struct bt_ctf_trace *trace;
-       struct bt_ctf_stream_class *stream_class;
-       struct bt_ctf_stream_class *stream_class2;
-       struct bt_ctf_stream *stream;
-       struct bt_ctf_clock_class *clock_class;
-       int ret;
-
-       trace = bt_ctf_trace_create();
-       assert(trace);
-       stream_class = bt_ctf_stream_class_create(NULL);
-       assert(stream_class);
-       ret = bt_ctf_stream_class_set_packet_context_type(stream_class, NULL);
-       assert(ret == 0);
-       ret = bt_ctf_trace_add_stream_class(trace, stream_class);
-       assert(ret == 0);
-       stream = bt_ctf_stream_create(stream_class, "hello");
-       ok(stream, "bt_ctf_stream_create() succeeds with a non-static trace");
-       bt_put(stream);
-       ok(!bt_ctf_trace_is_static(trace),
-               "bt_ctf_trace_is_static() returns the expected value");
-       ok(bt_ctf_trace_set_is_static(trace) == 0,
-               "bt_ctf_trace_set_is_static() succeeds");
-       ok(bt_ctf_trace_is_static(trace),
-               "bt_ctf_trace_is_static() returns the expected value");
-       clock_class = bt_ctf_clock_class_create("yes", 1000000000);
-       assert(clock_class);
-       stream_class2 = bt_ctf_stream_class_create(NULL);
-       assert(stream_class2);
-       ok(bt_ctf_trace_add_stream_class(trace, stream_class2),
-               "bt_ctf_trace_add_stream_class() fails with a static trace");
-       ok(bt_ctf_trace_add_clock_class(trace, clock_class),
-               "bt_ctf_trace_add_clock_class() fails with a static trace");
-       ok(!bt_ctf_stream_create(stream_class, "hello2"),
-               "bt_ctf_stream_create() fails with a static trace");
-
-       bt_put(trace);
-       bt_put(stream_class);
-       bt_put(stream_class2);
-       bt_put(clock_class);
-}
-
-static
-void trace_is_static_listener(struct bt_ctf_trace *trace, void *data)
-{
-       *((int *) data) |= 1;
-}
-
-static
-void trace_listener_removed(struct bt_ctf_trace *trace, void *data)
-{
-       *((int *) data) |= 2;
-}
-
-static
-void test_trace_is_static_listener(void)
-{
-       struct bt_ctf_trace *trace;
-       int ret;
-       int called1 = 0;
-       int called2 = 0;
-       int called3 = 0;
-       int called4 = 0;
-       int listener1_id;
-       int listener2_id;
-       int listener3_id;
-       int listener4_id;
-
-       trace = bt_ctf_trace_create();
-       assert(trace);
-       ret = bt_ctf_trace_add_is_static_listener(NULL,
-               trace_is_static_listener, trace_listener_removed, &called1);
-       ok(ret < 0, "bt_ctf_trace_add_is_static_listener() handles NULL (trace)");
-       ret = bt_ctf_trace_add_is_static_listener(trace, NULL,
-               trace_listener_removed, &called1);
-       ok(ret < 0, "bt_ctf_trace_add_is_static_listener() handles NULL (listener)");
-       listener1_id = bt_ctf_trace_add_is_static_listener(trace,
-               trace_is_static_listener, trace_listener_removed, &called1);
-       ok(listener1_id >= 0, "bt_ctf_trace_add_is_static_listener() succeeds (1)");
-       listener2_id = bt_ctf_trace_add_is_static_listener(trace,
-               trace_is_static_listener, trace_listener_removed, &called2);
-       ok(listener2_id >= 0, "bt_ctf_trace_add_is_static_listener() succeeds (2)");
-       listener3_id = bt_ctf_trace_add_is_static_listener(trace,
-               trace_is_static_listener, trace_listener_removed, &called3);
-       ok(listener3_id >= 0, "bt_ctf_trace_add_is_static_listener() succeeds (3)");
-       ret = bt_ctf_trace_remove_is_static_listener(NULL, 0);
-       ok(ret < 0, "bt_ctf_trace_remove_is_static_listener() handles NULL (trace)");
-       ret = bt_ctf_trace_remove_is_static_listener(trace, -2);
-       ok(ret < 0, "bt_ctf_trace_remove_is_static_listener() handles invalid ID (negative)");
-       ret = bt_ctf_trace_remove_is_static_listener(trace, 77);
-       ok(ret < 0, "bt_ctf_trace_remove_is_static_listener() handles invalid ID (non existing)");
-       ret = bt_ctf_trace_remove_is_static_listener(trace, listener2_id);
-       ok(ret == 0, "bt_ctf_trace_remove_is_static_listener() succeeds");
-       ok(called2 == 2, "bt_ctf_trace_remove_is_static_listener() calls the remove listener");
-       listener4_id = bt_ctf_trace_add_is_static_listener(trace,
-               trace_is_static_listener, NULL, &called4);
-       ok(listener4_id >= 0, "bt_ctf_trace_add_is_static_listener() succeeds (4)");
-       ok(called1 == 0, "\"trace is static\" listener not called before the trace is made static (1)");
-       ok(called2 == 2, "\"trace is static\" listener not called before the trace is made static (2)");
-       ok(called3 == 0, "\"trace is static\" listener not called before the trace is made static (3)");
-       ok(called4 == 0, "\"trace is static\" listener not called before the trace is made static (4)");
-       ret = bt_ctf_trace_set_is_static(trace);
-       assert(ret == 0);
-       ret = bt_ctf_trace_add_is_static_listener(trace,
-               trace_is_static_listener, trace_listener_removed, &called1);
-       ok(ret < 0,
-               "bt_ctf_trace_add_is_static_listener() fails when the trace is static");
-       ok(called1 == 1, "\"trace is static\" listener called when the trace is made static (1)");
-       ok(called2 == 2, "\"trace is static\" listener not called when the trace is made static (2)");
-       ok(called3 == 1, "\"trace is static\" listener called when the trace is made static (3)");
-       ok(called4 == 1, "\"trace is static\" listener called when the trace is made static (4)");
-       called1 = 0;
-       called2 = 0;
-       called3 = 0;
-       called4 = 0;
-       bt_put(trace);
-       ok(called1 == 2, "\"trace is static\" listener not called after the trace is put (1)");
-       ok(called2 == 0, "\"trace is static\" listener not called after the trace is put (2)");
-       ok(called3 == 2, "\"trace is static\" listener not called after the trace is put (3)");
-       ok(called4 == 0, "\"trace is static\" listener not called after the trace is put (4)");
-}
-
-static
-void test_trace_uuid(void)
-{
-       struct bt_ctf_trace *trace;
-       const unsigned char uuid[] = {
-               0x35, 0x92, 0x63, 0xab, 0xb4, 0xbe, 0x40, 0xb4,
-               0xb2, 0x60, 0xd3, 0xf1, 0x3b, 0xb0, 0xd8, 0x59,
-       };
-       const unsigned char *ret_uuid;
-
-       trace = bt_ctf_trace_create();
-       assert(trace);
-       ok(!bt_ctf_trace_get_uuid(NULL),
-               "bt_ctf_trace_get_uuid() handles NULL");
-       ok(!bt_ctf_trace_get_uuid(trace),
-               "bt_ctf_trace_get_uuid() returns NULL initially");
-       ok(bt_ctf_trace_set_uuid(NULL, uuid),
-               "bt_ctf_trace_set_uuid() handles NULL (trace)");
-       ok(bt_ctf_trace_set_uuid(trace, NULL),
-               "bt_ctf_trace_set_uuid() handles NULL (UUID)");
-       ok(bt_ctf_trace_set_uuid(trace, uuid) == 0,
-               "bt_ctf_trace_set_uuid() succeeds with a valid UUID");
-       ret_uuid = bt_ctf_trace_get_uuid(trace);
-       ok(ret_uuid, "bt_ctf_trace_get_uuid() returns a UUID");
-       assert(ret_uuid);
-       ok(memcmp(uuid, ret_uuid, 16) == 0,
-               "bt_ctf_trace_get_uuid() returns the expected UUID");
-
-       bt_put(trace);
-}
-
 int main(int argc, char **argv)
 {
        const char *env_resize_length;
@@ -2937,16 +1709,10 @@ int main(int argc, char **argv)
                "bt_ctf_trace_set_environment_field_string succeeds");
 
        /* Test bt_ctf_trace_get_environment_field_count */
-       ok(bt_ctf_trace_get_environment_field_count(NULL) < 0,
-               "bt_ctf_trace_get_environment_field_count handles a NULL trace correctly");
        ok(bt_ctf_trace_get_environment_field_count(trace) == 5,
                "bt_ctf_trace_get_environment_field_count returns a correct number of environment fields");
 
        /* Test bt_ctf_trace_get_environment_field_name */
-       ok(bt_ctf_trace_get_environment_field_name_by_index(NULL, 0) == NULL,
-               "bt_ctf_trace_get_environment_field_name handles a NULL trace correctly");
-       ok(bt_ctf_trace_get_environment_field_name_by_index(trace, 5) == NULL,
-               "bt_ctf_trace_get_environment_field_name handles an invalid index correctly (too large)");
        ret_string = bt_ctf_trace_get_environment_field_name_by_index(trace, 0);
        ok(ret_string && !strcmp(ret_string, "host"),
                "bt_ctf_trace_get_environment_field_name returns a correct field name");
@@ -2964,10 +1730,6 @@ int main(int argc, char **argv)
                "bt_ctf_trace_get_environment_field_name returns a correct field name");
 
        /* Test bt_ctf_trace_get_environment_field_value */
-       ok(bt_ctf_trace_get_environment_field_value_by_index(NULL, 0) == NULL,
-               "bt_ctf_trace_get_environment_field_value handles a NULL trace correctly");
-       ok(bt_ctf_trace_get_environment_field_value_by_index(trace, 5) == NULL,
-               "bt_ctf_trace_get_environment_field_value handles an invalid index correctly (too large)");
        obj = bt_ctf_trace_get_environment_field_value_by_index(trace, 1);
        ret = bt_value_integer_get(obj, &ret_int64_t);
        ok(!ret && ret_int64_t == 23,
@@ -2980,11 +1742,6 @@ int main(int argc, char **argv)
        BT_PUT(obj);
 
        /* Test bt_ctf_trace_get_environment_field_value_by_name */
-       ok(!bt_ctf_trace_get_environment_field_value_by_name(NULL,
-               "test_env_str"),
-               "bt_ctf_trace_get_environment_field_value_by_name handles a NULL trace correctly");
-       ok(!bt_ctf_trace_get_environment_field_value_by_name(trace, NULL),
-               "bt_ctf_trace_get_environment_field_value_by_name handles a NULL name correctly");
        ok(!bt_ctf_trace_get_environment_field_value_by_name(trace, "oh oh"),
                "bt_ctf_trace_get_environment_field_value_by_name returns NULL or an unknown field name");
        obj = bt_ctf_trace_get_environment_field_value_by_name(trace,
@@ -3086,45 +1843,8 @@ int main(int argc, char **argv)
                "Set clock absolute property");
        ok(bt_ctf_clock_get_is_absolute(clock) == !!is_absolute,
                "bt_ctf_clock_get_precision returns the correct is_absolute attribute once it is set");
-
        ok(bt_ctf_clock_set_time(clock, current_time) == 0,
                "Set clock time");
-
-       ok(!bt_ctf_clock_get_name(NULL),
-               "bt_ctf_clock_get_name correctly handles NULL");
-       ok(!bt_ctf_clock_get_description(NULL),
-               "bt_ctf_clock_get_description correctly handles NULL");
-       ok(bt_ctf_clock_get_frequency(NULL) == -1ULL,
-               "bt_ctf_clock_get_frequency correctly handles NULL");
-       ok(bt_ctf_clock_get_precision(NULL) == -1ULL,
-               "bt_ctf_clock_get_precision correctly handles NULL");
-       ok(bt_ctf_clock_get_offset_s(NULL, &get_offset_s) < 0,
-               "bt_ctf_clock_get_offset_s correctly handles NULL clock");
-       ok(bt_ctf_clock_get_offset_s(clock, NULL) < 0,
-               "bt_ctf_clock_get_offset_s correctly handles NULL output");
-       ok(bt_ctf_clock_get_offset(NULL, &get_offset) < 0,
-               "bt_ctf_clock_get_offset correctly handles NULL clock");
-       ok(bt_ctf_clock_get_offset(clock, NULL) < 0,
-               "bt_ctf_clock_get_offset correctly handles NULL output");
-       ok(bt_ctf_clock_get_is_absolute(NULL) < 0,
-               "bt_ctf_clock_get_is_absolute correctly handles NULL");
-
-       ok(bt_ctf_clock_set_description(NULL, NULL) < 0,
-               "bt_ctf_clock_set_description correctly handles NULL clock");
-       ok(bt_ctf_clock_set_frequency(NULL, frequency) < 0,
-               "bt_ctf_clock_set_frequency correctly handles NULL clock");
-       ok(bt_ctf_clock_set_precision(NULL, precision) < 0,
-               "bt_ctf_clock_get_precision correctly handles NULL clock");
-       ok(bt_ctf_clock_set_offset_s(NULL, offset_s) < 0,
-               "bt_ctf_clock_set_offset_s correctly handles NULL clock");
-       ok(bt_ctf_clock_set_offset(NULL, offset) < 0,
-               "bt_ctf_clock_set_offset correctly handles NULL clock");
-       ok(bt_ctf_clock_set_is_absolute(NULL, is_absolute) < 0,
-               "bt_ctf_clock_set_is_absolute correctly handles NULL clock");
-       ok(bt_ctf_clock_set_time(NULL, current_time) < 0,
-               "bt_ctf_clock_set_time correctly handles NULL clock");
-       ok(bt_ctf_clock_get_uuid(NULL) == NULL,
-               "bt_ctf_clock_get_uuid correctly handles NULL clock");
        ret_uuid = bt_ctf_clock_get_uuid(clock);
        ok(ret_uuid,
                "bt_ctf_clock_get_uuid returns a UUID");
@@ -3134,10 +1854,6 @@ int main(int argc, char **argv)
                tmp_uuid[sizeof(tmp_uuid) - 1]++;
        }
 
-       ok(bt_ctf_clock_set_uuid(NULL, tmp_uuid) < 0,
-               "bt_ctf_clock_set_uuid correctly handles a NULL clock");
-       ok(bt_ctf_clock_set_uuid(clock, NULL) < 0,
-               "bt_ctf_clock_set_uuid correctly handles a NULL UUID");
        ok(bt_ctf_clock_set_uuid(clock, tmp_uuid) == 0,
                "bt_ctf_clock_set_uuid sets a new uuid successfully");
        ret_uuid = bt_ctf_clock_get_uuid(clock);
@@ -3148,9 +1864,6 @@ int main(int argc, char **argv)
 
        /* Define a stream class */
        stream_class = bt_ctf_stream_class_create("test_stream");
-
-       ok(bt_ctf_stream_class_get_name(NULL) == NULL,
-               "bt_ctf_stream_class_get_name handles NULL correctly");
        ret_string = bt_ctf_stream_class_get_name(stream_class);
        ok(ret_string && !strcmp(ret_string, "test_stream"),
                "bt_ctf_stream_class_get_name returns a correct stream class name");
@@ -3171,13 +1884,8 @@ int main(int argc, char **argv)
        /* Test the event fields and event types APIs */
        type_field_tests();
 
-       /* Test fields copying */
-       field_copy_tests();
-
        ok(bt_ctf_stream_class_get_id(stream_class) < 0,
                "bt_ctf_stream_class_get_id returns an error when no id is set");
-       ok(bt_ctf_stream_class_get_id(NULL) < 0,
-               "bt_ctf_stream_class_get_id handles NULL correctly");
        ok(bt_ctf_stream_class_set_id(NULL, 123) < 0,
                "bt_ctf_stream_class_set_id handles NULL correctly");
        ok(bt_ctf_stream_class_set_id(stream_class, 123) == 0,
@@ -3186,8 +1894,6 @@ int main(int argc, char **argv)
                "bt_ctf_stream_class_get_id returns the correct value");
 
        /* Validate default event header fields */
-       ok(bt_ctf_stream_class_get_event_header_type(NULL) == NULL,
-               "bt_ctf_stream_class_get_event_header_type handles NULL correctly");
        ret_field_type = bt_ctf_stream_class_get_event_header_type(
                stream_class);
        ok(ret_field_type,
@@ -3215,13 +1921,11 @@ int main(int argc, char **argv)
        bt_put(ret_field_type);
 
        /* Add a custom trace packet header field */
-       ok(bt_ctf_trace_get_packet_header_type(NULL) == NULL,
-               "bt_ctf_trace_get_packet_header_type handles NULL correctly");
-       packet_header_type = bt_ctf_trace_get_packet_header_type(trace);
+       packet_header_type = bt_ctf_trace_get_packet_header_field_type(trace);
        ok(packet_header_type,
-               "bt_ctf_trace_get_packet_header_type returns a packet header");
+               "bt_ctf_trace_get_packet_header_field_type returns a packet header");
        ok(bt_ctf_field_type_get_type_id(packet_header_type) == BT_CTF_FIELD_TYPE_ID_STRUCT,
-               "bt_ctf_trace_get_packet_header_type returns a packet header of type struct");
+               "bt_ctf_trace_get_packet_header_field_type returns a packet header of type struct");
        ret_field_type = bt_ctf_field_type_structure_get_field_type_by_name(
                packet_header_type, "magic");
        ok(ret_field_type, "Default packet header type contains a \"magic\" field");
@@ -3240,14 +1944,11 @@ int main(int argc, char **argv)
                packet_header_field_type, "custom_trace_packet_header_field"),
                "Added a custom trace packet header field successfully");
 
-       ok(bt_ctf_trace_set_packet_header_type(NULL, packet_header_type) < 0,
-               "bt_ctf_trace_set_packet_header_type handles a NULL trace correctly");
-       ok(!bt_ctf_trace_set_packet_header_type(trace, packet_header_type),
+       ok(bt_ctf_trace_set_packet_header_field_type(NULL, packet_header_type) < 0,
+               "bt_ctf_trace_set_packet_header_field_type handles a NULL trace correctly");
+       ok(!bt_ctf_trace_set_packet_header_field_type(trace, packet_header_type),
                "Set a trace packet_header_type successfully");
 
-       ok(bt_ctf_stream_class_get_packet_context_type(NULL) == NULL,
-               "bt_ctf_stream_class_get_packet_context_type handles NULL correctly");
-
        /* Add a custom field to the stream class' packet context */
        packet_context_type = bt_ctf_stream_class_get_packet_context_type(stream_class);
        ok(packet_context_type,
@@ -3259,9 +1960,11 @@ int main(int argc, char **argv)
                "bt_ctf_stream_class_set_packet_context_type handles a NULL stream class correctly");
 
        integer_type = bt_ctf_field_type_integer_create(32);
+
        ok(bt_ctf_stream_class_set_packet_context_type(stream_class,
                integer_type) < 0,
                "bt_ctf_stream_class_set_packet_context_type rejects a packet context that is not a structure");
+
        /* Create a "uint5_t" equivalent custom packet context field */
        packet_context_field_type = bt_ctf_field_type_integer_create(5);
 
@@ -3270,8 +1973,6 @@ int main(int argc, char **argv)
        ok(ret == 0, "Packet context field added successfully");
 
        /* Define a stream event context containing a my_integer field. */
-       ok(bt_ctf_stream_class_get_event_context_type(NULL) == NULL,
-               "bt_ctf_stream_class_get_event_context_type handles NULL correctly");
        stream_event_context_type = bt_ctf_field_type_structure_create();
        bt_ctf_field_type_structure_add_field(stream_event_context_type,
                integer_type, "common_event_context");
@@ -3286,15 +1987,18 @@ int main(int argc, char **argv)
        ok(bt_ctf_stream_class_set_event_context_type(
                stream_class, stream_event_context_type) == 0,
                "Set a new stream event context type");
+
        ret_field_type = bt_ctf_stream_class_get_event_context_type(
                stream_class);
        ok(ret_field_type == stream_event_context_type,
                "bt_ctf_stream_class_get_event_context_type returns the correct field type.");
        bt_put(ret_field_type);
 
+
        /* Instantiate a stream and append events */
        ret = bt_ctf_writer_add_clock(writer, clock);
        assert(ret == 0);
+
        ok(bt_ctf_trace_get_stream_count(trace) == 0,
                "bt_ctf_trace_get_stream_count() succeeds and returns the correct value (0)");
        stream1 = bt_ctf_writer_create_stream(writer, stream_class);
@@ -3311,25 +2015,13 @@ int main(int argc, char **argv)
         * class to the writer's trace, thus registering the stream
         * class's clock to the trace.
         */
-       ok(bt_ctf_trace_get_clock_class_count(NULL) < 0,
-               "bt_ctf_trace_get_clock_class_count correctly handles NULL");
        ok(bt_ctf_trace_get_clock_class_count(trace) == 1,
                "bt_ctf_trace_get_clock_class_count returns the correct number of clocks");
-       ok(!bt_ctf_trace_get_clock_class_by_index(NULL, 0),
-               "bt_ctf_trace_get_clock_class correctly handles NULL");
-       ok(!bt_ctf_trace_get_clock_class_by_index(trace, 1),
-               "bt_ctf_trace_get_clock_class correctly handles out of bound accesses");
        ret_clock_class = bt_ctf_trace_get_clock_class_by_index(trace, 0);
        ok(strcmp(bt_ctf_clock_class_get_name(ret_clock_class),
                bt_ctf_clock_get_name(clock)) == 0,
                "bt_ctf_trace_get_clock_class returns the right clock instance");
        bt_put(ret_clock_class);
-       ok(!bt_ctf_trace_get_clock_class_by_name(trace, NULL),
-               "bt_ctf_trace_get_clock_class_by_name correctly handles NULL (trace)");
-       ok(!bt_ctf_trace_get_clock_class_by_name(NULL, clock_name),
-               "bt_ctf_trace_get_clock_by_name correctly handles NULL (clock name)");
-       ok(!bt_ctf_trace_get_clock_class_by_name(NULL, NULL),
-               "bt_ctf_trace_get_clock_by_name correctly handles NULL (both)");
        ret_clock_class = bt_ctf_trace_get_clock_class_by_name(trace, clock_name);
        ok(strcmp(bt_ctf_clock_class_get_name(ret_clock_class),
                bt_ctf_clock_get_name(clock)) == 0,
@@ -3338,8 +2030,6 @@ int main(int argc, char **argv)
        ok(!bt_ctf_trace_get_clock_class_by_name(trace, "random"),
                "bt_ctf_trace_get_clock_by_name fails when the requested clock doesn't exist");
 
-       ok(bt_ctf_stream_get_class(NULL) == NULL,
-               "bt_ctf_stream_get_class correctly handles NULL");
        ret_stream_class = bt_ctf_stream_get_class(stream1);
        ok(ret_stream_class,
                "bt_ctf_stream_get_class returns a stream class");
@@ -3354,7 +2044,7 @@ int main(int argc, char **argv)
        BT_PUT(packet_header_type);
        BT_PUT(packet_context_type);
        BT_PUT(stream_event_context_type);
-       packet_header_type = bt_ctf_trace_get_packet_header_type(trace);
+       packet_header_type = bt_ctf_trace_get_packet_header_field_type(trace);
        assert(packet_header_type);
        packet_context_type =
                bt_ctf_stream_class_get_packet_context_type(stream_class);
@@ -3404,14 +2094,12 @@ int main(int argc, char **argv)
        ok(ret_field_type == packet_header_type,
                "Stream returns a packet header of the appropriate type");
        bt_put(ret_field_type);
-       packet_header_field = bt_ctf_field_structure_get_field(packet_header,
+       packet_header_field = bt_ctf_field_structure_get_field_by_name(packet_header,
                "custom_trace_packet_header_field");
        ok(packet_header_field,
                "Packet header structure contains a custom field with the appropriate name");
        ret_field_type = bt_ctf_field_get_type(packet_header_field);
-       ok(bt_ctf_field_type_compare(ret_field_type, packet_header_field_type) == 0,
-               "Custom packet header field is of the expected type");
-       ok(!bt_ctf_field_unsigned_integer_set_value(packet_header_field,
+       ok(!bt_ctf_field_integer_unsigned_set_value(packet_header_field,
                54321), "Set custom packet header value successfully");
        ok(bt_ctf_stream_set_packet_header(stream1, NULL) < 0,
                "bt_ctf_stream_set_packet_header handles a NULL packet header correctly");
@@ -3427,10 +2115,6 @@ int main(int argc, char **argv)
 
        test_clock_utils();
 
-       test_create_writer_vs_non_writer_mode();
-
-       test_set_clock_non_writer_stream_class();
-
        test_instanciate_event_before_stream(writer, clock);
 
        append_simple_event(stream_class, stream1, clock);
@@ -3445,12 +2129,6 @@ int main(int argc, char **argv)
 
        test_custom_event_header_stream(writer, clock);
 
-       test_static_trace();
-
-       test_trace_is_static_listener();
-
-       test_trace_uuid();
-
        metadata_string = bt_ctf_writer_get_metadata_string(writer);
        ok(metadata_string, "Get metadata string");
 
This page took 0.097891 seconds and 4 git commands to generate.