<div dir="ltr">Merged, thanks!<div><br></div><div>Jérémie</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 18, 2015 at 10:18 AM, Mathieu Desnoyers <span dir="ltr"><<a href="mailto:mathieu.desnoyers@efficios.com" target="_blank">mathieu.desnoyers@efficios.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">bump: Jérémie, I know you are busy with Babeltrace, but can you quickly<br>
look into merging this fix ? It would repair the CI.<br>
<br>
Thanks,<br>
<br>
Mathieu<br>
<div><div class="h5"><br>
----- On Jun 10, 2015, at 11:27 AM, Jonathan Rajotte <a href="mailto:jonathan.rajotte-julien@efficios.com">jonathan.rajotte-julien@efficios.com</a> wrote:<br>
<br>
> Since the number of events present in the tests apps in prone to change it<br>
> make more sense to look for a minimum of events. This is less rigourous<br>
> but more flexible to change.<br>
><br>
> Signed-off-by: Jonathan Rajotte <<a href="mailto:jonathan.rajotte-julien@efficios.com">jonathan.rajotte-julien@efficios.com</a>><br>
> ---<br>
> tests/regression/tools/mi/test_mi | 15 ++++++++++-----<br>
> 1 file changed, 10 insertions(+), 5 deletions(-)<br>
><br>
> diff --git a/tests/regression/tools/mi/test_mi<br>
> b/tests/regression/tools/mi/test_mi<br>
> index ccf9236..c44b598 100755<br>
> --- a/tests/regression/tools/mi/test_mi<br>
> +++ b/tests/regression/tools/mi/test_mi<br>
> @@ -328,7 +328,7 @@ function test_ust_lttng_event ()<br>
>       $XML_VALIDATE $OUTPUT_DEST<br>
>       ok $? "Mi test: Multiple enable ust event with inside fail xsd validation"<br>
><br>
> -     #Check for 2 success = true et 1 false<br>
> +     #Check for 2 success = true and 1 success = false<br>
>       extract_xml $OUTPUT_DEST $XPATH_ENABLE_EVENT_SUCCESS result<br>
>       num_ocur=$(grep -o "true" <<< "$result" | wc -l)<br>
>       test "$num_ocur" -eq "2"<br>
> @@ -511,16 +511,21 @@ function test_list_ust_event ()<br>
>       $XML_VALIDATE $OUTPUT_DEST<br>
>       ok $? "Mi test: list ust event xsd validation"<br>
><br>
> -     #Extract events 5 events should be present<br>
> +     #Extract events<br>
>       extract_xml $OUTPUT_DEST "$XPATH_LIST_UST_EVENT""/name/text()" result<br>
>       num=$(echo "$result" | wc -l)<br>
> -     test "$num" -eq "5"<br>
> +<br>
> +     #Since the number of events is prone to change we only look for >= from a<br>
> +     #base number. An alternative would be to look for each events but this<br>
> +     #is complicated for nothing.<br>
> +     test "$num" -ge "5"<br>
>       ok $? "Mi test: $num / 5 ust events discovered"<br>
><br>
> -     #Extract field from tp:tptest event. Expect 12<br>
> +     #Extract field from tp:tptest event. Expect >= to 12 because number of<br>
> +     #field is prone to change.<br>
>       extract_xml $OUTPUT_DEST "$XPATH_LIST_UST_EVENT""[./name =<br>
>       'tp:tptest']/event_fields/event_field/name/text()" result<br>
>       num=$(echo "$result" | wc -l)<br>
> -     test "$num" -eq "12"<br>
> +     test "$num" -ge "12"<br>
>       ok $? "Mi test: $num / 12 ust event fields discovered"<br>
><br>
>       #Wait for all background processes<br>
> --<br>
> 2.1.4<br>
><br>
><br>
</div></div>> _______________________________________________<br>
> lttng-dev mailing list<br>
> <a href="mailto:lttng-dev@lists.lttng.org">lttng-dev@lists.lttng.org</a><br>
> <a href="http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev" rel="noreferrer" target="_blank">http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Mathieu Desnoyers<br>
EfficiOS Inc.<br>
<a href="http://www.efficios.com" rel="noreferrer" target="_blank">http://www.efficios.com</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Jérémie Galarneau<br>EfficiOS Inc.<br><a href="http://www.efficios.com" target="_blank">http://www.efficios.com</a></div>
</div>