Miksus / rocketry

Modern scheduling library for Python
https://rocketry.readthedocs.io
MIT License
3.25k stars 105 forks source link

BUG - since I last pulled, base repo has test failures #158

Open marksmayo opened 1 year ago

marksmayo commented 1 year ago

Describe the bug Latest codebase, pulle down. Run pytest test_build.py is failing, and /schedule/test_core.py runs for 30 min and is still going?

I also tried running them all individually from vscode, got some more intermittent failures.

To Reproduce Latest codebase, pulle down. Run pytest

Expected behavior Tests to execute in full and pass

Screenshots image

image

Desktop (please complete the following information):

Additional context Test failure for build:

image

Miksus commented 1 year ago

Hmm, oh ye. Pass argument "--no-build" with the pytest call. Does that make it to go successfully? If not, I think you can ignore that 👍

I basically made that as a sanity check so that the package gets version dynamically when built (changed versioneer to setuptools_scm and the latter suggests to ignore _version.py). I'm not sure if that's no longer as necessary as every master merge goes to test PyPI anyways.

Perhaps I could change it the other way round: CI passes argument "--check-build" to Pytest and only then this sanity check test is run.

marksmayo commented 1 year ago

pylint: error: Unrecognized option found: no-build - not for me, but I see you've done a PR to fix this so will take a look

marksmayo commented 1 year ago

with new code, still getting:

image

but if it's ok to ignore I will for now

Miksus commented 1 year ago

Ye, you can ignore it 👍 I'm thinking of removing that test altogether, or at least remove the command-line argument. The reason for it was a sanity check the version is generated but I'm not sure if it's fruitful anymore test setuptools_scm itself works correctly.

The PR has issues with pytest. It seems you cannot pass arguments to pytest argparser in built packages without hacks. I possibly change the package layout a bit in the future by putting tests outside the library code but I'll inform well in advance if/when I'm doing that.