search

rm-hull / luma.oled

Python module to drive a SSD1306 / SSD1309 / SSD1322 / SSD1325 / SSD1327 / SSD1331 / SSD1351 / SH1106 OLED
https://luma-oled.readthedocs.io
MIT License
807 stars 161 forks source link

tests: add timeout using pytest-timeout #304

Closed thijstriemstra closed 3 years ago

thijstriemstra commented 3 years ago

This doesn't fix the failing tests but those are related to Pillow 8.0, as seen in other luma projects. This just 'unfreezes' the test suite so it will always finish and not hang for hours on a particular test.

rm-hull commented 3 years ago

So, something must’ve changed recently that is causing this. I wonder if pillow 8 is the cause ... that is the only thing that is used in the display tests and has it has changed recently

thijstriemstra commented 3 years ago

No matter the cause, having a timeout on the testsuite is always a good idea IMO.

rm-hull commented 3 years ago

See comment: https://github.com/rm-hull/luma.lcd/pull/123#issuecomment-711422819