diff options
author | Carlos Tadeu Panato Junior <ctadeu@gmail.com> | 2018-03-13 17:26:56 +0100 |
---|---|---|
committer | Saturnino Abril <saturnino.abril@gmail.com> | 2018-03-14 00:26:56 +0800 |
commit | db4402c40dca7724416f1f6a38e4e256e871c302 (patch) | |
tree | 4f7220368cdba1f7cbe154d0938301ea9c8caee9 /utils/file_backend_s3_test.go | |
parent | caf9c2409935124db85daf5b04d61b7a663ebfc8 (diff) | |
download | chat-db4402c40dca7724416f1f6a38e4e256e871c302.tar.gz chat-db4402c40dca7724416f1f6a38e4e256e871c302.tar.bz2 chat-db4402c40dca7724416f1f6a38e4e256e871c302.zip |
remove s3 region to be mandatory and fix when user call test s3 when the config is saved (#8454)
Diffstat (limited to 'utils/file_backend_s3_test.go')
-rw-r--r-- | utils/file_backend_s3_test.go | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/utils/file_backend_s3_test.go b/utils/file_backend_s3_test.go index ff42a4d19..a8834f226 100644 --- a/utils/file_backend_s3_test.go +++ b/utils/file_backend_s3_test.go @@ -19,14 +19,14 @@ func TestCheckMandatoryS3Fields(t *testing.T) { cfg.AmazonS3Bucket = "test-mm" err = CheckMandatoryS3Fields(&cfg) - if err == nil || err.Message != "api.admin.test_s3.missing_s3_endpoint" { - t.Fatal("should've failed with missing s3 endpoint") + if err != nil { + t.Fatal("should've not failed") } - cfg.AmazonS3Endpoint = "s3.newendpoint.com" + cfg.AmazonS3Endpoint = "" err = CheckMandatoryS3Fields(&cfg) - if err == nil || err.Message != "api.admin.test_s3.missing_s3_region" { - t.Fatal("should've failed with missing s3 region") + if err != nil || cfg.AmazonS3Endpoint != "s3.amazonaws.com" { + t.Fatal("should've not failed because it should set the endpoint to the default") } } |