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

fix tests for commands molecule init role and molecule init scenario #3924

Closed
wants to merge 1 commit into from

Conversation

ziegenberg
Copy link
Contributor

@ziegenberg ziegenberg commented May 25, 2023

Currently, the tests fail for test_command_init_role and test_command_init_scneario. The tests execute ansible-lint on the newly-created roles and the linter recently got more restrictive, so we now have to adapt.

There are still two failures with Python 3.11 and ansible-core devel. ansible-compat calls ansible-galaxy collection list --format=json internally (https://github.com/ansible/ansible-compat/blob/d30965cd023b4116f6e959ace0e4dbf219677952/src/ansible_compat/runtime.py#LL235C20-L235C20) and due to some unknown reason it is executed with elevated verbosity producing broken json output. The output begins with:

Using molecule/.ansible.cfg as config file
{"/home/daniel/.ansible/collections/ansible_collections": {"ansible.posix": {"version": "1.5.1"},

.
.
.

@ziegenberg ziegenberg requested review from a team as code owners May 25, 2023 14:51
@ziegenberg ziegenberg requested review from greg-hellings, sky-joker, ssbarnea, cidrblock and shatakshiiii and removed request for a team May 25, 2023 14:51
@github-actions github-actions bot added the bug label May 25, 2023
Currently the tests fail for `test_command_init_role` and
`test_command_init_scneario`. The tests execute `ansible-lint` on the
newly-created roles and the linter recently got more restrictive, so we
now have to adapt.
@ziegenberg
Copy link
Contributor Author

And @ssbarnea fixed the remaining issue in ansible/ansible-compat#284 :)

@ziegenberg
Copy link
Contributor Author

This change is now included in PR #3904, so closing this one.

@ziegenberg ziegenberg closed this Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant