[libcamera-devel] [PATCH v2 06/13] py: unittests: make typechecker happy
Tomi Valkeinen
tomi.valkeinen at ideasonboard.com
Wed May 18 08:46:33 CEST 2022
On 17/05/2022 19:08, Laurent Pinchart wrote:
> Hi Tomi,
>
> Thank you for the patch.
>
> On Tue, May 17, 2022 at 05:33:18PM +0300, Tomi Valkeinen wrote:
>> Add some annotations and self.assertIsNotNone() calls to remove the
>> typechecker warnings.
>>
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen at ideasonboard.com>
>> ---
>> test/py/unittests.py | 12 ++++++++++--
>> 1 file changed, 10 insertions(+), 2 deletions(-)
>>
>> diff --git a/test/py/unittests.py b/test/py/unittests.py
>> index 4c214f0a..2ea5ca35 100755
>> --- a/test/py/unittests.py
>> +++ b/test/py/unittests.py
>> @@ -10,6 +10,7 @@ import libcamera as libcam
>> import os
>> import selectors
>> import time
>> +import typing
>> import unittest
>> import weakref
>>
>> @@ -70,6 +71,9 @@ class SimpleTestMethods(BaseTestCase):
>>
>>
>> class CameraTesterBase(BaseTestCase):
>> + cm: typing.Any
>> + cam: typing.Any
>> +
>> def setUp(self):
>> self.cm = libcam.CameraManager.singleton()
>> self.cam = next((cam for cam in self.cm.cameras if 'platform/vimc' in cam.id), None)
>> @@ -131,6 +135,7 @@ class AllocatorTestMethods(CameraTesterBase):
>> wr_allocator = weakref.ref(allocator)
>>
>> buffers = allocator.buffers(stream)
>> + self.assertIsNotNone(buffers)
>> buffers = None
>>
>> buffer = allocator.buffers(stream)[0]
>> @@ -166,6 +171,8 @@ class SimpleCaptureMethods(CameraTesterBase):
>>
>> streamconfig = camconfig.at(0)
>> fmts = streamconfig.formats
>> + self.assertIsNotNone(fmts)
>> + fmts = None
>>
>> ret = cam.configure(camconfig)
>> self.assertZero(ret)
>> @@ -225,6 +232,7 @@ class SimpleCaptureMethods(CameraTesterBase):
>>
>> streamconfig = camconfig.at(0)
>> fmts = streamconfig.formats
>> + self.assertIsNotNone(fmts)
>
> Is there a reason why there's no fmts = None here ? Apart from that,
No, I can add it.
Tomi
More information about the libcamera-devel
mailing list