[lttng-dev] [PATCH babeltrace 2/2] Test: Validate that enumeration mapping strings are properly escaped

Mathieu Desnoyers mathieu.desnoyers at efficios.com
Fri Nov 29 01:06:07 EST 2013


Both patches merged, thanks!

Mathieu

----- Original Message -----
> From: "Jérémie Galarneau" <jeremie.galarneau at efficios.com>
> To: lttng-dev at lists.lttng.org
> Sent: Thursday, November 28, 2013 9:46:41 PM
> Subject: [lttng-dev] [PATCH babeltrace 2/2] Test: Validate that enumeration	mapping strings are properly escaped
> 
> Signed-off-by: Jérémie Galarneau <jeremie.galarneau at efficios.com>
> ---
>  tests/lib/test_ctf_writer.c | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 
> diff --git a/tests/lib/test_ctf_writer.c b/tests/lib/test_ctf_writer.c
> index d7f7db7..0927a32 100644
> --- a/tests/lib/test_ctf_writer.c
> +++ b/tests/lib/test_ctf_writer.c
> @@ -254,13 +254,30 @@ void append_simple_event(struct bt_ctf_stream_class
> *stream_class,
>  		bt_ctf_field_type_integer_create(12);
>  	struct bt_ctf_field_type *float_type =
>  		bt_ctf_field_type_floating_point_create();
> +	struct bt_ctf_field_type *enum_type =
> +		bt_ctf_field_type_enumeration_create(uint_12_type);
>  	struct bt_ctf_event *simple_event;
>  	struct bt_ctf_field *integer_field;
>  	struct bt_ctf_field *float_field;
> +	struct bt_ctf_field *enum_field;
> +	struct bt_ctf_field *enum_container_field;
>  
>  	bt_ctf_field_type_set_alignment(float_type, 32);
>  	bt_ctf_field_type_floating_point_set_exponent_digits(float_type, 11);
>  	bt_ctf_field_type_floating_point_set_mantissa_digits(float_type, 53);
> +
> +	ok(bt_ctf_field_type_enumeration_add_mapping(enum_type,
> +		"escaping; \"test\"", 0, 0) == 0,
> +		"Accept enumeration mapping strings containing quotes");
> +	ok(bt_ctf_field_type_enumeration_add_mapping(enum_type,
> +		"\tanother \'escaping\'\n test\"", 1, 4) == 0,
> +		"Accept enumeration mapping strings containing special characters");
> +	ok(bt_ctf_field_type_enumeration_add_mapping(enum_type,
> +		"event clock int float", 5, 22) == 0,
> +		"Accept enumeration mapping strings containing reserved keywords");
> +	ok(bt_ctf_event_class_add_field(simple_event_class, enum_type,
> +		"enum_field") == 0, "Add enumeration field to event");
> +
>  	ok(uint_12_type, "Create an unsigned integer type");
>  	bt_ctf_event_class_add_field(simple_event_class, uint_12_type,
>  		"integer_field");
> @@ -281,6 +298,13 @@ void append_simple_event(struct bt_ctf_stream_class
> *stream_class,
>  
>  	float_field = bt_ctf_event_get_payload(simple_event, "float_field");
>  	bt_ctf_field_floating_point_set_value(float_field, 3.1415);
> +	enum_field = bt_ctf_field_create(enum_type);
> +	enum_container_field = bt_ctf_field_enumeration_get_container(
> +		enum_field);
> +	ok(bt_ctf_field_unsigned_integer_set_value(
> +		enum_container_field, 1) == 0,
> +		"Set enumeration container value");
> +	bt_ctf_event_set_payload(simple_event, "enum_field", enum_field);
>  
>  	ok(bt_ctf_clock_set_time(clock, current_time) == 0, "Set clock time");
>  
> @@ -294,8 +318,11 @@ void append_simple_event(struct bt_ctf_stream_class
> *stream_class,
>  	bt_ctf_event_put(simple_event);
>  	bt_ctf_field_type_put(uint_12_type);
>  	bt_ctf_field_type_put(float_type);
> +	bt_ctf_field_type_put(enum_type);
>  	bt_ctf_field_put(integer_field);
>  	bt_ctf_field_put(float_field);
> +	bt_ctf_field_put(enum_field);
> +	bt_ctf_field_put(enum_container_field);
>  }
>  
>  void append_complex_event(struct bt_ctf_stream_class *stream_class,
> --
> 1.8.4.2
> 
> 
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> 

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com



More information about the lttng-dev mailing list