Skip to content
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

Migrate to poetry #7

Merged
merged 4 commits into from
Jan 3, 2022
Merged

Migrate to poetry #7

merged 4 commits into from
Jan 3, 2022

Conversation

missingcharacter
Copy link
Member

tests output

$ ./run_tests.sh
Poe => python -m pytest -v --log-file=tests.log --log-file-level=INFO --cov=. tests/ -o junit_family=xunit2 --junitxml=tmp/test-output/pytest.xml
/home/ricardo/.asdf/installs/python/3.10.0/lib/python3.10/site-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
  warnings.warn(
==================================================================================== test session starts =====================================================================================
platform linux -- Python 3.10.0, pytest-6.2.5, py-1.11.0, pluggy-1.0.0 -- /home/ricardo/.cache/pypoetry/virtualenvs/ivy-accounts-tools-81zYZnac-py3.10/bin/python
cachedir: .pytest_cache
rootdir: /home/ricardo/src/ivy-accounts-tools
plugins: cov-3.0.0
collected 12 items                                                                                                                                                                           

tests/test_setup_account.py::test_sub_account_parser PASSED                                                                                                                            [  8%]
tests/test_setup_account.py::test_sub_account_creation PASSED                                                                                                                          [ 16%]
tests/test_setup_account.py::test_sub_account_duplicate PASSED                                                                                                                         [ 25%]
tests/test_setup_account.py::test_account_setup_parser PASSED                                                                                                                          [ 33%]
tests/test_setup_account.py::test_account_setup PASSED                                                                                                                                 [ 41%]
tests/test_setup_account.py::test_account_setup_duplicate PASSED                                                                                                                       [ 50%]
tests/test_setup_account.py::test_vpc_cleaner PASSED                                                                                                                                   [ 58%]
tests/test_setup_account.py::test_infra_buckets_argparser PASSED                                                                                                                       [ 66%]
tests/test_setup_account.py::test_infra_buckets_argparser_with_regions PASSED                                                                                                          [ 75%]
tests/test_setup_account.py::test_infra_buckets_creator_on_default_region PASSED                                                                                                       [ 83%]
tests/test_setup_account.py::test_infra_buckets_creator_duplicate PASSED                                                                                                               [ 91%]
tests/test_setup_account.py::test_infra_buckets_creator_on_regions PASSED                                                                                                              [100%]

====================================================================================== warnings summary ======================================================================================
../../.cache/pypoetry/virtualenvs/ivy-accounts-tools-81zYZnac-py3.10/lib/python3.10/site-packages/botocore/httpsession.py:57: 4 warnings
tests/test_setup_account.py: 88 warnings
  /home/ricardo/.cache/pypoetry/virtualenvs/ivy-accounts-tools-81zYZnac-py3.10/lib/python3.10/site-packages/botocore/httpsession.py:57: DeprecationWarning: ssl.PROTOCOL_TLS is deprecated
    context = SSLContext(ssl_version or ssl.PROTOCOL_SSLv23)

-- Docs: https://docs.pytest.org/en/stable/warnings.html
---------------------------------------------------- generated xml file: /home/ricardo/src/ivy-accounts-tools/tmp/test-output/pytest.xml -----------------------------------------------------

---------- coverage: platform linux, python 3.10.0-final-0 -----------
Name                                 Stmts   Miss  Cover
--------------------------------------------------------
infra_buckets/infra_buckets.py          73     13    82%
new_sub_account/new_sub_account.py      68     21    69%
setup_sso/setup_sso.py                 124     23    81%
vpc_cleaner/vpc_cleaner.py             154     35    77%
--------------------------------------------------------
TOTAL                                  419     92    78%

============================================================================== 12 passed, 92 warnings in 9.95s ===============================================================================

Copy link
Contributor

@abvikg abvikg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not fully know this repo, but here's my approval for what I've seen!

s3transfer = "==0.3.1"
six = "==1.14.0"
urllib3 = "==1.25.8"
boto3-stubs = {extras = ["iam", "organizations"], version = "==1.14.60.0"}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did not know this was a thing 🤔

pyproject.toml Outdated Show resolved Hide resolved
@missingcharacter missingcharacter merged commit cf0f6b3 into master Jan 3, 2022
@missingcharacter missingcharacter deleted the migrate-to-poetry branch January 3, 2022 17:27
@missingcharacter
Copy link
Member Author

Thanks @abvikg !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants