2019-01-29 12:27:42 -05:00
|
|
|
package common
|
|
|
|
|
|
|
|
import (
|
|
|
|
"fmt"
|
|
|
|
"reflect"
|
|
|
|
"testing"
|
|
|
|
)
|
|
|
|
|
|
|
|
func testOMIConfig() *OMIConfig {
|
|
|
|
return &OMIConfig{
|
|
|
|
OMIName: "foo",
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func getFakeAccessConfig(region string) *AccessConfig {
|
|
|
|
c := testAccessConfig()
|
|
|
|
c.RawRegion = region
|
|
|
|
return c
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestOMIConfigPrepare_name(t *testing.T) {
|
|
|
|
c := testOMIConfig()
|
|
|
|
accessConf := testAccessConfig()
|
|
|
|
if err := c.Prepare(accessConf, nil); err != nil {
|
|
|
|
t.Fatalf("shouldn't have err: %s", err)
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIName = ""
|
|
|
|
if err := c.Prepare(accessConf, nil); err == nil {
|
|
|
|
t.Fatal("should have error")
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestOMIConfigPrepare_regions(t *testing.T) {
|
|
|
|
c := testOMIConfig()
|
|
|
|
c.OMIRegions = nil
|
|
|
|
|
|
|
|
var errs []error
|
|
|
|
var err error
|
|
|
|
accessConf := testAccessConfig()
|
|
|
|
mockConn := &mockOAPIClient{}
|
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatalf("shouldn't have err: %#v", errs)
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIRegions, err = listOAPIRegions(mockConn)
|
|
|
|
if err != nil {
|
|
|
|
t.Fatalf("shouldn't have err: %s", err.Error())
|
|
|
|
}
|
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatalf("shouldn't have err: %#v", errs)
|
|
|
|
}
|
|
|
|
errs = errs[:0]
|
|
|
|
|
|
|
|
c.OMIRegions = []string{"us-east-1", "us-west-1", "us-east-1"}
|
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatalf("bad: %s", errs[0])
|
|
|
|
}
|
|
|
|
|
|
|
|
expected := []string{"us-east-1", "us-west-1"}
|
|
|
|
if !reflect.DeepEqual(c.OMIRegions, expected) {
|
|
|
|
t.Fatalf("bad: %#v", c.OMIRegions)
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIRegions = []string{"custom"}
|
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatal("shouldn't have error")
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIRegions = []string{"us-east-1", "us-east-2", "us-west-1"}
|
2019-02-05 14:18:44 -05:00
|
|
|
|
2019-01-29 12:27:42 -05:00
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatal(fmt.Sprintf("shouldn't have error: %s", errs[0]))
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIRegions = []string{"us-east-1", "us-east-2", "us-west-1"}
|
2019-02-05 14:18:44 -05:00
|
|
|
|
2019-01-29 12:27:42 -05:00
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatal("should have passed; we are able to use default KMS key if not sharing")
|
|
|
|
}
|
|
|
|
|
2019-02-05 14:18:44 -05:00
|
|
|
c.SnapshotAccountIDs = []string{"user-foo", "user-bar"}
|
2019-01-29 12:27:42 -05:00
|
|
|
c.OMIRegions = []string{"us-east-1", "us-east-2", "us-west-1"}
|
2019-02-05 14:18:44 -05:00
|
|
|
|
2019-01-29 12:27:42 -05:00
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatal("should have an error b/c can't use default KMS key if sharing")
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIRegions = []string{"us-east-1", "us-west-1"}
|
2019-02-05 14:18:44 -05:00
|
|
|
|
2019-01-29 12:27:42 -05:00
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatal("should have error b/c theres a region in the key map that isn't in omi_regions")
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIRegions = []string{"us-east-1", "us-west-1", "us-east-2"}
|
|
|
|
|
2019-02-05 14:18:44 -05:00
|
|
|
c.SnapshotAccountIDs = []string{"foo", "bar"}
|
2019-01-29 12:27:42 -05:00
|
|
|
c.OMIRegions = []string{"us-east-1", "us-west-1"}
|
2019-02-05 14:18:44 -05:00
|
|
|
|
2019-01-29 12:27:42 -05:00
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatal("should have error b/c theres a region in in omi_regions that isn't in the key map")
|
|
|
|
}
|
|
|
|
|
|
|
|
// allow rawregion to exist in omi_regions list.
|
|
|
|
accessConf = getFakeAccessConfig("us-east-1")
|
|
|
|
c.OMIRegions = []string{"us-east-1", "us-west-1", "us-east-2"}
|
2019-02-05 14:18:44 -05:00
|
|
|
|
2019-01-29 12:27:42 -05:00
|
|
|
if errs = c.prepareRegions(accessConf); len(errs) > 0 {
|
|
|
|
t.Fatal("should allow user to have the raw region in omi_regions")
|
|
|
|
}
|
|
|
|
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestOMINameValidation(t *testing.T) {
|
|
|
|
c := testOMIConfig()
|
|
|
|
|
|
|
|
accessConf := testAccessConfig()
|
|
|
|
|
|
|
|
c.OMIName = "aa"
|
|
|
|
if err := c.Prepare(accessConf, nil); err == nil {
|
|
|
|
t.Fatal("shouldn't be able to have an omi name with less than 3 characters")
|
|
|
|
}
|
|
|
|
|
|
|
|
var longOmiName string
|
|
|
|
for i := 0; i < 129; i++ {
|
|
|
|
longOmiName += "a"
|
|
|
|
}
|
|
|
|
c.OMIName = longOmiName
|
|
|
|
if err := c.Prepare(accessConf, nil); err == nil {
|
|
|
|
t.Fatal("shouldn't be able to have an omi name with great than 128 characters")
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIName = "+aaa"
|
|
|
|
if err := c.Prepare(accessConf, nil); err == nil {
|
|
|
|
t.Fatal("shouldn't be able to have an omi name with invalid characters")
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIName = "fooBAR1()[] ./-'@_"
|
|
|
|
if err := c.Prepare(accessConf, nil); err != nil {
|
|
|
|
t.Fatal("should be able to use all of the allowed OMI characters")
|
|
|
|
}
|
|
|
|
|
|
|
|
c.OMIName = `xyz-base-2017-04-05-1934`
|
|
|
|
if err := c.Prepare(accessConf, nil); err != nil {
|
|
|
|
t.Fatalf("expected `xyz-base-2017-04-05-1934` to pass validation.")
|
|
|
|
}
|
|
|
|
|
|
|
|
}
|