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

provider/aws: Fix acceptance tests w/ missing values #11377

Merged
merged 1 commit into from
Jan 24, 2017
Merged

provider/aws: Fix acceptance tests w/ missing values #11377

merged 1 commit into from
Jan 24, 2017

Conversation

radeksimko
Copy link
Member 

$ make testacc TEST=./builtin/providers/aws TESTARGS='-run="(AWSAPIGatewayIntegration_basic|AWSAPIGatewayRestApi_basic|AWSAPIGatewayIntegrat
ionResponse_basic|AWSAutoScalingGroup_enablingMetrics|AWSCodeCommitRepository_create_and_update_default_branch|AWSAccessKey_encrypted|AWSKinesisStream_shardLevelMetrics|AWSLambdaFunction_envVariables|AWSLightsailKeyPair_imported|AWSLightsailKeyPair_encrypted|AWSS3Bucket_basic)"'

==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/01/24 10:52:50 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run="(AWSAPIGatewayIntegration_basic|AWSAPIGatewayRestApi_basic|AWSAPIGatewayIntegrationResponse_basic|AWSAutoScalingGroup_enablingMetrics|AWSCodeCommitRepository_create_and_update_default_branch|AWSAccessKey_encrypted|AWSKinesisStream_shardLevelMetrics|AWSLambdaFunction_envVariables|AWSLightsailKeyPair_imported|AWSLightsailKeyPair_encrypted|AWSS3Bucket_basic)" -timeout 120m
=== RUN   TestAccAWSAPIGatewayIntegrationResponse_basic
--- PASS: TestAccAWSAPIGatewayIntegrationResponse_basic (53.14s)
=== RUN   TestAccAWSAPIGatewayIntegration_basic
--- PASS: TestAccAWSAPIGatewayIntegration_basic (46.95s)
=== RUN   TestAccAWSAPIGatewayRestApi_basic
--- PASS: TestAccAWSAPIGatewayRestApi_basic (44.05s)
=== RUN   TestAccAWSAutoScalingGroup_enablingMetrics
--- PASS: TestAccAWSAutoScalingGroup_enablingMetrics (171.70s)
=== RUN   TestAccAWSCodeCommitRepository_create_and_update_default_branch
--- PASS: TestAccAWSCodeCommitRepository_create_and_update_default_branch (35.67s)
=== RUN   TestAccAWSAccessKey_encrypted
--- PASS: TestAccAWSAccessKey_encrypted (18.79s)
=== RUN   TestAccAWSKinesisStream_shardLevelMetrics
--- PASS: TestAccAWSKinesisStream_shardLevelMetrics (181.25s)
=== RUN   TestAccAWSLambdaFunction_envVariables
--- PASS: TestAccAWSLambdaFunction_envVariables (132.92s)
=== RUN   TestAccAWSLightsailKeyPair_imported
--- PASS: TestAccAWSLightsailKeyPair_imported (29.98s)
=== RUN   TestAccAWSLightsailKeyPair_encrypted
--- PASS: TestAccAWSLightsailKeyPair_encrypted (29.63s)
=== RUN   TestAccAWSS3Bucket_basic
--- PASS: TestAccAWSS3Bucket_basic (46.27s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	790.381s

Related to #11218

@stack72
Copy link
Contributor

stack72 commented Jan 24, 2017

LGTM! Nice :)

@stack72 stack72 merged commit cbaf051 into hashicorp:master Jan 24, 2017
@radeksimko radeksimko deleted the b-aws-fix-acc-tests branch January 24, 2017 12:16
pielu pushed a commit to pielu/terraform that referenced this pull request Jan 24, 2017
Merge commit '439b5507bbe9b00c7154d5b30dd9e35600ef05a2' into sncr
fc54c75 
* commit '439b5507bbe9b00c7154d5b30dd9e35600ef05a2': (688 commits)
  Update CHANGELOG.md
  provider/aws: Added API GW documentation regarding request/response templates (hashicorp#11380)
  Update CHANGELOG.md
  provider/aws: Added a S3 Bucket domain name attribute (hashicorp#10088)
  provider/postgresql: Fix failing acceptance test (hashicorp#11375)
  provider/heroku: Fix failing acceptance test (hashicorp#11374)
  provider/aws: Fix acceptance tests w/ missing values (hashicorp#11377)
  provider/aws: Fix failing ecs acceptance test w/ ALB (hashicorp#11367)
  fix fat-fingered change in test
  provider/aws: Add diff suppress function for aws_db_instance
  Update CHANGELOG.md
  provider/openstack: Volume Attachment Updates (hashicorp#11285)
  Update CHANGELOG.md
  adding (backwards compatible) support for providing multiple source filename and content (hashicorp#11271)
  Correct aws_s3_bucket_object data source acceptance tests. (hashicorp#11346)
  Update CHANGELOG.md
  Add 'aws_vpc_endpoint' data source. (hashicorp#11323)
  Update CHANGELOG.md
  Add 'aws_canonical_user_id' data source. (hashicorp#11332)
  Update CHANGELOG.md
  ...
@ghost
Copy link

ghost commented Apr 17, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 17, 2020
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
bug provider/aws
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@radeksimko @stack72