Added a test around file upload access issues

tests/transfer_test.go

@@ -96,6 +96,7 @@ func ftpDelete(t *testing.T, ftp *goftp.Client, filename string) {
 	}
 }
 
+// TestTransfer validates the upload of file in both active and passive mode
 func TestTransfer(t *testing.T) {
 	s := NewTestServer(true)
 	s.Settings.NonStandardActiveDataPort = true
@@ -137,3 +138,32 @@ func testTransferOnConnection(t *testing.T, server *server.FtpServer, active boo
 		t.Fatal("The two files don't have the same hash:", hashUpload, "!=", hashDownload)
 	}
 }
+
+// TestFailedTransfer validates the handling of failed transfer caused by file access issues
+func TestFailedTransfer(t *testing.T) {
+	s := NewTestServer(true)
+	defer s.Stop()
+
+	conf := goftp.Config{
+		User:     "test",
+		Password: "test",
+	}
+
+	var err error
+	var c *goftp.Client
+
+	if c, err = goftp.DialConfig(conf, s.Listener.Addr().String()); err != nil {
+		t.Fatal("Couldn't connect", err)
+	}
+	defer c.Close()
+
+	// We create a 1KB file and upload it
+	file := createTemporaryFile(t, 1*1024)
+	if err = c.Store("/non/existing/path/file.bin", file); err == nil {
+		t.Fatal("This upload should have failed")
+	}
+
+	if err = c.Store("file.bin", file); err != nil {
+		t.Fatal("This upload should have succeeded", err)
+	}
+}