Re: [PATCH] tests: add test for json-streamer.c error recovery

[Markus Armbruster <armbru@redhat.com>]

Paolo Bonzini <pbonzini@redhat.com> writes:

> On Thu, Apr 23, 2026 at 8:56 AM Markus Armbruster <armbru@redhat.com> wrote:
>> > +    } else {
>> > +        qobject_unref(r->result);
>>
>> Unexpected calls to parse_emit() won't be detected as long as the last
>> one yields the expected expression.  Worth detecting?  I'm not quite
>> sure.  As far as I can tell, this is not reached at the moment.
>
> It's only covered in /errors/multiple_values of tests/unit/check-qson.c, yes.
>
>> The stupidest way to detect is to assert r->result is still null.
>
> I'm all for simplicity.
>
>> Can't do that if we want to cover parsing multiple JSON expressions in
>> sequence.  Collect all the results in an array?
>
> Yes, but not now. For now the assert will do.

Makes sense.  Thanks!