Il 01/08/2012 12:43, Kevin Wolf ha scritto:
>> > +
>> > +    def test_report(self):
>> > +        self.assert_no_active_streams()
>> > +
>> > +        result = self.vm.qmp('block-stream', device='drive0')
>> > +        self.assert_qmp(result, 'return', {})
>> > +
>> > +        completed = False
>> > +        error = False
>> > +        while not completed:
>> > +            for event in self.vm.get_qmp_events(wait=True):
>> > +                if event['event'] == 'BLOCK_JOB_ERROR':
>> > +                    self.assert_qmp(event, 'data/device', 'drive0')
>> > +                    self.assert_qmp(event, 'data/operation', 'read')
> data/action should be asserted as well (same in the other tests).
> 
> What about adding an enospc test as well, once with EIO and once with
> ENOSPC?

Ok.

Paolo


Reply via email to