provider/aws: Fixed the need of sending S3 Replication StorageClass when not set

[Ninir]

Description

Fixes #10909

Acceptance test

$ make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSS3Bucket_ReplicationWithoutStorageClass'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2016/12/24 11:43:54 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSS3Bucket_ReplicationWithoutStorageClass -timeout 120m
=== RUN   TestAccAWSS3Bucket_ReplicationWithoutStorageClass
--- PASS: TestAccAWSS3Bucket_ReplicationWithoutStorageClass (80.27s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	80.302s

Merry Christmas all! 😄 🎄 🎅

[jen20]

Hi @Ninir! This looks like a valid change and test, but could you give a brief description of what the issue is if this patch is not applied? Thanks!

[jen20]

Actually I just found the bug report in #10909 - disregard my previous comment. Thanks!

[ghost]

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.