-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Suggested workflow for testing a build on a CI service? #166
Comments
Have you considered using |
I presume that installs the package in a venv? How is it different from install? The documentation doesn't seem to mention it? |
Yes it does, but |
But does it install it in a PS: Just from looking at it, I expected |
Exactly. See this issue for more information: #34
You're testing the installed package rather than the installation routine itself with that.
Hmmm, maybe |
Maybe that's good enough. My intention here is to be sure that a package actually installs and works. I got bitten by a Python2/3 Unicode issue in a setup.py once, it worked on Py2 but not on Py3 -- and there was no CI with Py3 set up to catch this before the package hit PyPI. |
FYI Travis CI lets you pick the Python version. For example: https://github.com/moigagoo/cliar/blob/develop/.travis.yml#L2 |
I know, we have since improved ;) |
Also, wouldn't this run the tests against the directory instead of the installed package:
? |
I think this will run the tests against the directory, but with the package installed. It's just that in Python's path resolution algorithm current dir comes before other paths, so the package from the current dir will be imported before the one installed with However, should you import the package from a file that doesn't start with But it's the same version anyway, so the result will be the same. |
fyi, i currently employ this snippet for the config of Travis CI: install:
- travis_retry pip install poetry
- travis_retry poetry install
script:
- poetry run make $TARGET |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Closing this issue automatically because it has not had any activity since it has been marked as stale. If you think it is still relevant and should be addressed, feel free to open a new one. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
I want to install a wheel package and have the test suit run over it, like tox does. The purpose is to test that an installed package actually works.
I have a GitLab CI script like this:
The test stage receives
dist/*.whl
, which we can install. I have to manually specify the development dependencies though. Doable, but not very elegant. Is there a better workflow with poetry?The text was updated successfully, but these errors were encountered: