utils.with_timeout isn't exercised in tests

[malandrina]

nuanced.utils.with_timeout was introduced in #36 but it is replaced with a test double in tests/nuanced/code_graph_test.py and is otherwise not exercised in any automated tests.

We should add some test scenarios for the following cases:

• with_timeout is invoked with a target function that can't be pickled
• with_timeout exposes an error when the timeout is exceeded
• with_timeout exposes the return value of the target function when function execution succeeds before timing out

Resources:

• https://medium.com/@sampsa.riikonen/doing-python-multiprocessing-the-right-way-a54c1880e300
• https://visoft.ro/programming/testing-while-multiprocessing/2022/07/26/

[Angel-Ozz]

Hey!! I am new to OSS and I think I can contribute adding this test scenarios

[malandrina]

Thanks @Angel-Ozz! We'd welcome a PR. Feel free to comment here with questions if you get stuck or need clarification.

[Angel-Ozz]

Heyyy, I have been recently going through the project locally, and apart from the issue, I have noticed that while setting everything up a lot of errors in the src code appear, even slightly errors like "expected 2 blank lines, found 1Flake8(E302)" in line 14 of code_graph.py. I have addressed these issues but obviously those have become a lot of changes, so Idk if i should better ignore them or what to do regarding that

[malandrina]

Good question @Angel-Ozz !

I'm guessing the specific linting error you mentioned, "expected 2 blank lines, found 1Flake8(E302)", is from Flake8 or a tool that uses Flake8 under the hood. This codebase doesn't have a style guide yet but adhering to Flake8 rules seems like a reasonable starting point, so please go ahead and include the code style changes in your PR.