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

Potential Issue Parsing Heredoc's #3840

Closed
stack72 opened this issue Nov 10, 2015 · 2 comments
Closed

Potential Issue Parsing Heredoc's #3840

stack72 opened this issue Nov 10, 2015 · 2 comments

Comments

@stack72
Copy link
Contributor

stack72 commented Nov 10, 2015

On master, I am trying to run the Iam Role tests

I get the following:

make testacc TEST=./builtin/providers/aws TESTARGS='-run=AWSRole' 2>~/tf.log
go generate ./...
TF_ACC=1 go test ./builtin/providers/aws -v -run=AWSRole -timeout 90m
=== RUN   TestAccAWSRole_basic
--- PASS: TestAccAWSRole_basic (4.22s)
=== RUN   TestAccAWSRole_testNameChange
--- FAIL: TestAccAWSRole_testNameChange (0.00s)
    testing.go:137: Step 0 error: Error loading configuration: Error parsing /var/folders/gd/yv86v7pd1cd5dpzjrw8702k00000gn/T/tf-test624339930/main.tf: 5:24: illegal char
FAIL
exit status 1
FAIL    github.com/hashicorp/terraform/builtin/providers/aws    4.242s

I have made no changes to these tests.

jen20 added a commit that referenced this issue Nov 10, 2015
This test reproduces the issue which is likely the root cause of #3840.
Test is currently failing with an "illegal character" message
corresponding with the location of the heredoc, which is also seen in
various acceptance tests for providers.
jen20 added a commit to hashicorp/hcl that referenced this issue Nov 10, 2015
This is likely the root cause of hashicorp/terraform#3840.
jen20 added a commit that referenced this issue Nov 10, 2015
This test reproduces the issue which is likely the root cause of #3840.
Test is currently failing with an "illegal character" message
corresponding with the location of the heredoc, which is also seen in
various acceptance tests for providers.
@stack72
Copy link
Contributor Author

stack72 commented Nov 10, 2015

@jen20 this is now fixed upstream - thanks @mitchellh

make testacc TEST=./builtin/providers/aws TESTARGS='-run=AWSRole' 2>~/tf.log
go generate ./...
TF_ACC=1 go test ./builtin/providers/aws -v -run=AWSRole -timeout 90m
=== RUN   TestAccAWSRole_basic
--- PASS: TestAccAWSRole_basic (5.75s)
=== RUN   TestAccAWSRole_testNameChange
--- PASS: TestAccAWSRole_testNameChange (15.92s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    21.688s

@stack72 stack72 closed this as completed Nov 10, 2015
dpetzold pushed a commit to clearcare/terraform that referenced this issue Nov 12, 2015
This test reproduces the issue which is likely the root cause of hashicorp#3840.
Test is currently failing with an "illegal character" message
corresponding with the location of the heredoc, which is also seen in
various acceptance tests for providers.
omeid pushed a commit to omeid/terraform that referenced this issue Mar 30, 2018
…domain_identity-trailing-period

resource/aws_ses_domain_identity: Support trailing period in domain name
omeid pushed a commit to omeid/terraform that referenced this issue Mar 30, 2018
@ghost
Copy link

ghost commented Apr 30, 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 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants