From a607c9eace60abc22ae5c1432b183f8be46500ba Mon Sep 17 00:00:00 2001 From: Xiang Li Date: Fri, 22 Nov 2013 13:27:04 -0800 Subject: [PATCH] fix dep issue --- server/tls_info.go | 4 ++-- tests/functional/single_node_recovery_test.go | 8 +++----- tests/functional/single_node_test.go | 6 +++--- tests/functional/util.go | 2 +- 4 files changed, 9 insertions(+), 11 deletions(-) diff --git a/server/tls_info.go b/server/tls_info.go index 1048f21e7..6b16db013 100644 --- a/server/tls_info.go +++ b/server/tls_info.go @@ -4,7 +4,7 @@ import ( "crypto/tls" "crypto/x509" "encoding/pem" - "errors" + "fmt" "io/ioutil" ) @@ -27,7 +27,7 @@ func (info TLSInfo) Config() (TLSConfig, error) { // Both the key and cert must be present. if info.KeyFile == "" || info.CertFile == "" { - return t, errors.New("KeyFile and CertFile must both be present") + return t, fmt.Errorf("KeyFile and CertFile must both be present[key: %v, cert: %v]", info.KeyFile, info.CertFile) } tlsCert, err := tls.LoadX509KeyPair(info.CertFile, info.KeyFile) diff --git a/tests/functional/single_node_recovery_test.go b/tests/functional/single_node_recovery_test.go index 51b12894e..dde9e16ca 100644 --- a/tests/functional/single_node_recovery_test.go +++ b/tests/functional/single_node_recovery_test.go @@ -50,18 +50,16 @@ func TestSingleNodeRecovery(t *testing.T) { time.Sleep(time.Second) - results, err := c.Get("foo", false) + result, err = c.Get("foo", false) if err != nil { t.Fatal("get fail: " + err.Error()) return } - kv := results.Kvs[0] - - if err != nil || kv.Key != "/foo" || kv.Value != "bar" || kv.TTL > 99 { + if err != nil || result.Key != "/foo" || result.Value != "bar" || result.TTL > 99 { if err != nil { t.Fatal(err) } - t.Fatalf("Recovery Get failed with %s %s %v", kv.Key, kv.Value, kv.TTL) + t.Fatalf("Recovery Get failed with %s %s %v", result.Key, result.Value, result.TTL) } } diff --git a/tests/functional/single_node_test.go b/tests/functional/single_node_test.go index e8ea18af2..87d297eea 100644 --- a/tests/functional/single_node_test.go +++ b/tests/functional/single_node_test.go @@ -51,9 +51,9 @@ func TestSingleNode(t *testing.T) { // Add a test-and-set test // First, we'll test we can change the value if we get it write - result, match, err := c.CompareAndSwap("foo", "foobar", 100, "foo", 0) + result, err = c.CompareAndSwap("foo", "foobar", 100, "bar", 0) - if err != nil || result.Key != "/foo" || result.Value != "foobar" || result.PrevValue != "bar" || result.TTL != 100 || !match { + if err != nil || result.Key != "/foo" || result.Value != "foobar" || result.PrevValue != "bar" || result.TTL != 100 { if err != nil { t.Fatal(err) } @@ -61,7 +61,7 @@ func TestSingleNode(t *testing.T) { } // Next, we'll make sure we can't set it without the correct prior value - _, _, err = c.CompareAndSwap("foo", "foofoo", 100, "bar", 0) + _, err = c.CompareAndSwap("foo", "foofoo", 100, "bar", 0) if err == nil { t.Fatalf("Set 4 expecting error when setting key with incorrect previous value") diff --git a/tests/functional/util.go b/tests/functional/util.go index 791f5045c..eb0173c76 100644 --- a/tests/functional/util.go +++ b/tests/functional/util.go @@ -80,7 +80,7 @@ func CreateCluster(size int, procAttr *os.ProcAttr, ssl bool) ([][]string, []*os sslServer2 := []string{"-peer-ca-file=../../fixtures/ca/ca.crt", "-peer-cert-file=../../fixtures/ca/server2.crt", - "-peer-cert-file=../../fixtures/ca/server2.key.insecure", + "-peer-key-file=../../fixtures/ca/server2.key.insecure", } for i := 0; i < size; i++ {