[PATCH wayland 4/4] tests: add code, public-code and private-code tests

Emil Velikov emil.l.velikov at gmail.com
Thu Feb 22 11:23:22 UTC 2018


Hi Pekka,

Thanks for having a look!

On 22 February 2018 at 10:26, Pekka Paalanen <ppaalanen at gmail.com> wrote:

>> +# The existing "code" must produce result identical to "public-code"
>> +generate_and_compare "code" "small.xml" "small-public-code.c"
>> +generate_and_compare "public-code" "small.xml" "small-public-code.c"
>> +generate_and_compare "private-code" "small.xml" "small-private-code.c"
>>  exit $RETCODE
>
> Hi Emil,
>
> is there a reason why you introduced small-public-code.c as a new
> expected output file? It is by definition identical to small-code.c.
> Even the generate_and_compare line is identical.
>

I opted to have all three combinations split out and distinct from the
existing tests.
One would do something like the following. I don't have strong
preference, as long as we have some tests.

# The existing "code" must produce result identical to "public-code"
generate_and_compare "code" "small.xml" "small-code.c"
generate_and_compare "public-code" "small.xml" "small-code.c"
generate_and_compare "private-code" "small.xml" "small-private-code.c"

Updated patch coming in a second - feel free to pick the more
favourable solution.

Thanks
Emil


More information about the wayland-devel mailing list