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

Remove ec2_vpc_nat_gateway unit tests #407

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Feb 11, 2021

SUMMARY

The ec2_vpc_nat_gateway unit tests rely on hard-coded "check_mode" return values.

We now have integration tests with better (real) coverage than the unit tests which were broken by #372. IMO leaving the unit tests there will only discourage people from making check_mode behave properly.

ISSUE TYPE
  • Tests Pull Request
COMPONENT NAME

ec2_vpc_nat_gateway

ADDITIONAL INFORMATION

Also removed the comment from the integration tests which implied they were based on the fake check_mode behaviour which had hard coded return values.

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review integration tests/integration module module needs_maintainer needs_triage plugins plugin (any type) tests tests needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR and removed community_review labels Feb 11, 2021
Copy link
Collaborator

@jillr jillr left a comment

Choose a reason for hiding this comment

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

LGTM - I don't see any immediate way we could repurpose/revise this unit test without significant rewriting that would be better done from scratch if we wanted units.

@tremble tremble merged commit 701153a into ansible-collections:main Feb 11, 2021
ethemcemozkan pushed a commit to ethemcemozkan/community.aws that referenced this pull request Feb 18, 2021
* Remove ec2_vpc_nat_gateway unit tests
* update comments in ec2_vpc_nat_gateway integration tests - they're not based on the hard coded check_mode results any more
@tremble tremble deleted the ec2_vpc_nat_gateway/unit-tests branch June 29, 2021 12:56
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* Remove ec2_vpc_nat_gateway unit tests
* update comments in ec2_vpc_nat_gateway integration tests - they're not based on the hard coded check_mode results any more
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* Remove ec2_vpc_nat_gateway unit tests
* update comments in ec2_vpc_nat_gateway integration tests - they're not based on the hard coded check_mode results any more
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
* Remove ec2_vpc_nat_gateway unit tests
* update comments in ec2_vpc_nat_gateway integration tests - they're not based on the hard coded check_mode results any more
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
Formally start the ec2 deprecation process

SUMMARY
The ec2 module is based upon the deprecated boto (not boto3/botocore) SDK and most functionality has been replaced by the ec2_instance module.
Formally start the deprecation process and give it 2 major releases.
For practical purposes the ec2 module has a number of limitations (things like not supporting all regions) and we're still getting bugs filed against it for quirky behaviour that's fixed in ec2_instance.  By formally starting the deprecation process we can move most folks over to ec2_instance where possible and avoid trying to support 2 modules at the same time.
The major missing feature with the ec2_instance module is spot instances.  Support for spot instances is underway in ansible-collections#407
Should spot instance support not be available by 3.0.0 we can always extend the deprecation period.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2
ADDITIONAL INFORMATION

Reviewed-by: Jill R <None>
Reviewed-by: None <None>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
* Remove ec2_vpc_nat_gateway unit tests
* update comments in ec2_vpc_nat_gateway integration tests - they're not based on the hard coded check_mode results any more

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@701153a
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
Add initial work on new spot instance module

SUMMARY

Working on a new EC2 spot instance module

ISSUE TYPE


New Module Pull Request

COMPONENT NAME

ec2_spot_instance
ADDITIONAL INFORMATION

Reviewed-by: Jill R <None>
Reviewed-by: Rachana <[email protected]>
Reviewed-by: Gonéri Le Bouder <[email protected]>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration tests/integration module module needs_maintainer needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants