Skip to content

Conversation

@eriknw
Copy link
Member

@eriknw eriknw commented May 5, 2023

We somehow forgot to include this. Still need to verify that including this allows us to run tests with custom options, but I think it should.

@eriknw eriknw added hygiene Improve code quality and reduce maintenance overhead tests Unit tests and/or continuous integration labels May 5, 2023
@coveralls
Copy link

coveralls commented May 5, 2023

Coverage Status

Coverage: 98.99% (-0.5%) from 99.54% when pulling 756dc68 on eriknw:include_conftest into 0eeebfc on python-graphblas:main.

@eriknw
Copy link
Member Author

eriknw commented May 5, 2023

Okay, so adding conftest.py to MANIFEST.in actually doesn't add it to the wheel--it's only added to the source distribution. It belongs in the source distribution, so this change is still good.

I just added some logic so that slow tests will run (and not emit warnings) when run via pytest --pyargs graphblas. It also runs "randomly". I don't know how to allow custom cli arguments to be passed in when pytest is invoked in this way.

@eriknw eriknw merged commit f140f9c into python-graphblas:main May 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hygiene Improve code quality and reduce maintenance overhead tests Unit tests and/or continuous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants