From 45572a28635e63559dcc3b60c30c6ce09acb0a90 Mon Sep 17 00:00:00 2001
From: "Santiago M. Mola" <santi@mola.io>
Date: Thu, 8 Dec 2016 12:32:05 +0100
Subject: remote: fix Connect, returned nil error on failure. (#167)

---
 remote.go      | 2 +-
 remote_test.go | 7 +++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/remote.go b/remote.go
index 2c8a731..3508ed9 100644
--- a/remote.go
+++ b/remote.go
@@ -51,7 +51,7 @@ func (r *Remote) Connect() error {
 	var err error
 	r.fetchSession, err = r.client.NewFetchPackSession(r.endpoint)
 	if err != nil {
-		return nil
+		return err
 	}
 
 	return r.retrieveAdvertisedReferences()
diff --git a/remote_test.go b/remote_test.go
index acf646f..c3e86df 100644
--- a/remote_test.go
+++ b/remote_test.go
@@ -44,6 +44,13 @@ func (s *RemoteSuite) TestnewRemoteInvalidEndpoint(c *C) {
 	c.Assert(err, NotNil)
 }
 
+func (s *RemoteSuite) TestnewRemoteNonExistentEndpoint(c *C) {
+	r := newRemote(nil, nil, &config.RemoteConfig{Name: "foo", URL: "ssh://non-existent/foo.git"})
+
+	err := r.Connect()
+	c.Assert(err, NotNil)
+}
+
 func (s *RemoteSuite) TestnewRemoteInvalidSchemaEndpoint(c *C) {
 	r := newRemote(nil, nil, &config.RemoteConfig{Name: "foo", URL: "qux://foo"})
 
-- 
cgit