Merge pull request #355 from pivotal-cf/feat/world-class-dev-experien…

…ce--use-hello-tile

test(commands/test_tile): use hello tile instead of tas for unit tests

.gitignore

@@ -13,3 +13,5 @@ env.yml
 *.key
 *.pem
 .vscode
+
+vendor

.gitmodules

@@ -1,3 +1,6 @@
 [submodule "internal/acceptance/workflows/hello-tile"]
 	path = internal/acceptance/workflows/hello-tile
 	url = [email protected]:crhntr/hello-tile
+[submodule "internal/commands/testdata/test_tile/hello-tile"]
+	path = internal/commands/testdata/test_tile/hello-tile
+	url = [email protected]:crhntr/hello-tile.git

internal/commands/fakes/conn.go

@@ -0,0 +1,44 @@
+package fakes
+
+import (
+	"io"
+	"net"
+	"time"
+)
+
+type Conn struct {
+	R io.Reader
+	W io.Writer
+}
+
+func (c *Conn) LocalAddr() net.Addr {
+	return nil
+}
+
+func (c *Conn) RemoteAddr() net.Addr {
+	return &net.UnixAddr{Name: "test", Net: "test"}
+}
+
+func (c *Conn) SetDeadline(t time.Time) error {
+	return nil
+}
+
+func (c *Conn) SetReadDeadline(t time.Time) error {
+	return nil
+}
+
+func (c *Conn) SetWriteDeadline(t time.Time) error {
+	return nil
+}
+
+func (c *Conn) Read(p []byte) (int, error) {
+	return c.R.Read(p)
+}
+
+func (c *Conn) Write(p []byte) (int, error) {
+	return c.W.Write(p)
+}
+
+func (c *Conn) Close() error {
+	return nil
+}

internal/commands/test_tile.go

@@ -94,7 +94,7 @@ func (u ManifestTest) Execute(args []string) error {
 	}
 
 	session, _ := mobySession.NewSession(u.ctx, "waypoint", "")
-	defer session.Close()
+	defer closeAndIgnoreError(session)
 	session.Allow(sshp)
 	dialSession := func(ctx context.Context, proto string, meta map[string][]string) (net.Conn, error) {
 		return u.mobi.DialHijack(ctx, "/session", proto, meta)