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.

Sign up for GitHub

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

Jump to bottom

Overhaul ansible-test integration tests. #76111

Merged
merged 7 commits into from Oct 25, 2021
Merged

Overhaul ansible-test integration tests. #76111

merged 7 commits into from Oct 25, 2021

Conversation

mattclay
Copy link
Member

SUMMARY

Overhaul ansible-test integration tests.

This updates the tests to use all available Python versions and extends the tests to run on the default container.

ISSUE TYPE

Test Pull Request

COMPONENT NAME

ansible-test integration tests

@ansibot ansibot added affects_2.13 community_review In order to be merged, this PR must follow the community review workflow. needs_triage Needs a first human triage before being processed. support:community This issue/PR relates to code supported by the Ansible community. labels Oct 21, 2021
@jborean93 jborean93 removed the needs_triage Needs a first human triage before being processed. label Oct 21, 2021
@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed community_review In order to be merged, this PR must follow the community review workflow. labels Oct 21, 2021
@ansibot ansibot added core_review In order to be merged, this PR must follow the core review workflow. support:core This issue/PR relates to code supported by the Ansible Engineering Team. test This PR relates to tests. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Oct 21, 2021
@ansibot
Copy link
Contributor

ansibot commented Oct 22, 2021

The test ansible-test sanity --test pylint [explain] failed with 1 error:

test/lib/ansible_test/_internal/commands/sanity/import.py:5:0: unused-import: Unused import tempfile

click here for bot help

@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed core_review In order to be merged, this PR must follow the core review workflow. labels Oct 22, 2021
@samdoran samdoran added the ci_verified Changes made in this PR are causing tests to fail. label Oct 25, 2021
@ansibot ansibot added core_review In order to be merged, this PR must follow the core review workflow. and removed ci_verified Changes made in this PR are causing tests to fail. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Oct 25, 2021
@mattclay mattclay merged commit cae7d2a into ansible:devel Oct 25, 2021
@mattclay mattclay deleted the at-integration-tests branch October 25, 2021 23:32
mattclay added a commit to mattclay/ansible that referenced this pull request Oct 25, 2021
@mattclay
Overhaul ansible-test integration tests. (ansible#76111)
d08ff47 
* Overhaul ansible-test integration tests.
* ansible-test - Fix import test pyyaml usage.
* ansible-test - Remove unused import.
* ansible-test - Fix traceback when pip is unavailable.
* ansible-test - Fix typo in port forwarding message.
* ansible-test - Fix controller logic in requirements install.
* Fix unit tests in ansible-test integration test.

Unit tests are now run for available Python versions which
provide `virtualenv` (Python 2.x) or `venv` (Python 3.x).

(cherry picked from commit cae7d2a)
@mattclay mattclay mentioned this pull request Oct 25, 2021
Merged
mattclay added a commit that referenced this pull request Nov 8, 2021
@mattclay
[stable-2.12] ansible-test - Multiple related backports. (#76136)
6cba4c1 
* [stable-2.12] ansible-test - Fix Python real prefix detection.

(cherry picked from commit b4cbe1a)

* ansible-test - Remove duplicate code.

(cherry picked from commit b9694ce)

* Overhaul ansible-test integration tests. (#76111)

* Overhaul ansible-test integration tests.
* ansible-test - Fix import test pyyaml usage.
* ansible-test - Remove unused import.
* ansible-test - Fix traceback when pip is unavailable.
* ansible-test - Fix typo in port forwarding message.
* ansible-test - Fix controller logic in requirements install.
* Fix unit tests in ansible-test integration test.

Unit tests are now run for available Python versions which
provide `virtualenv` (Python 2.x) or `venv` (Python 3.x).

(cherry picked from commit cae7d2a)

* ansible-test - Fix import test on old Pythons. (#76137)

(cherry picked from commit cd64e0b)

* Skip ansible-test tests on Python 2.6.
@ansible ansible locked and limited conversation to collaborators Nov 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
affects_2.13 core_review In order to be merged, this PR must follow the core review workflow. support:community This issue/PR relates to code supported by the Ansible community. support:core This issue/PR relates to code supported by the Ansible Engineering Team. test This PR relates to tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@mattclay @ansibot @samdoran @jborean93