OK NEW SERVER PASSES TESTS WOO

spike/review/reinvent/ezmded/server/server.go

@@ -212,7 +212,6 @@ func (server *Server) apiV0FilesHandler(w http.ResponseWriter, r *http.Request)
 	case http.MethodDelete:
 		return server.apiV0FilesIDDelHandler(w, r)
 	}
-	log.Printf("no method for %s: %s", r.Method, r.URL)
 	http.NotFound(w, r)
 	return nil
 }
@@ -238,7 +237,14 @@ func (server *Server) apiV0FilesPostHandler(w http.ResponseWriter, r *http.Reque
 
 	pid := server.fileId(r)
 	id := append(pid, strings.Split(uuid.New().String(), "-")[0])
-	return server.tree().Put(id, Leaf{Title: r.Header.Get("Title"), Content: string(b)})
+	if err := server.tree().Put(id, Leaf{Title: r.Header.Get("Title"), Content: string(b)}); err != nil {
+		return err
+	}
+	return json.NewEncoder(w).Encode(map[string]map[string]string{
+		"data": map[string]string{
+			"filePath": path.Join("/api/v0/files/", server.urlFileId(id)),
+		},
+	})
 }
 
 func (server *Server) apiV0FilesIDGetHandler(w http.ResponseWriter, r *http.Request) error {
@@ -282,7 +288,13 @@ func (server *Server) urlFileId(id []string) string {
 }
 
 func (server *Server) fileId(r *http.Request) []string {
-	return strings.Split(strings.TrimPrefix(r.URL.Path, "/api/v0/files/"), "/")
+	return strings.Split(
+		strings.TrimPrefix(
+			strings.Trim(r.URL.Path, "/"),
+			"api/v0/files/",
+		),
+		"/",
+	)
 }
 
 func (server *Server) apiV0FilesIDPutHandler(w http.ResponseWriter, r *http.Request) error {
@@ -302,7 +314,14 @@ func (server *Server) apiV0FilesIDPutHandler(w http.ResponseWriter, r *http.Requ
 	leaf.Title = r.Header.Get("Title")
 	leaf.Deleted = false
 
-	return server.tree().Put(id, leaf)
+	if err := server.tree().Put(id, leaf); err != nil {
+		return err
+	}
+	return json.NewEncoder(w).Encode(map[string]map[string]string{
+		"data": map[string]string{
+			"filePath": path.Join("/api/v0/files/", server.urlFileId(id)),
+		},
+	})
 }
 
 func (server *Server) apiV0SearchHandler(w http.ResponseWriter, r *http.Request) error {

spike/review/reinvent/ezmded/server/server_test.go

@@ -1,8 +1,11 @@
 package main
 
 import (
+	"bytes"
+	"encoding/json"
 	"net/http"
 	"net/http/httptest"
+	"net/url"
 	"path"
 	"strings"
 	"testing"
@@ -104,20 +107,18 @@ func TestServerRoutes(t *testing.T) {
 	}
 }
 
-/*
 func TestServerPutTreeGetFile(t *testing.T) {
 	server := NewServer(t.TempDir())
 	if err := server.Routes(); err != nil {
 		t.Fatal(err)
 	}
-	server.tree().Put("my pid", Branch{})
+	server.tree().Put([]string{"my pid"}, Leaf{})
 	var id string
 	t.Run("put to create an id", func(t *testing.T) {
-		r := httptest.NewRequest(http.MethodPut, "/my-put-id", strings.NewReader("body"))
+		r := httptest.NewRequest(http.MethodPut, "/my%20pid/my-put-id", strings.NewReader("body"))
 		r.Header.Set("Title", "my put title")
-		r.Header.Set("PID", "my pid")
 		w := httptest.NewRecorder()
-		if err := server.apiV0FilesIDHandler(w, r); err != nil {
+		if err := server.apiV0FilesIDPutHandler(w, r); err != nil {
 			t.Fatal(err)
 		}
 		t.Logf("%s", w.Body.Bytes())
@@ -137,9 +138,8 @@ func TestServerPutTreeGetFile(t *testing.T) {
 		}
 	})
 	t.Run("post", func(t *testing.T) {
-		r := httptest.NewRequest(http.MethodPost, "/", strings.NewReader("body"))
+		r := httptest.NewRequest(http.MethodPost, "/my%20pid", strings.NewReader("body"))
 		r.Header.Set("Title", "my title")
-		r.Header.Set("PID", "my pid")
 		w := httptest.NewRecorder()
 		if err := server.apiV0FilesHandler(w, r); err != nil {
 			t.Fatal(err)
@@ -156,7 +156,7 @@ func TestServerPutTreeGetFile(t *testing.T) {
 		if err := json.NewDecoder(w.Body).Decode(&resp); err != nil {
 			t.Fatal(err)
 		}
-		id = path.Base(resp.Data.FilePath)
+		id = strings.TrimPrefix(resp.Data.FilePath, "/api/v0/files/")
 	})
 	t.Run("tree", func(t *testing.T) {
 		r := httptest.NewRequest(http.MethodGet, "/", nil)
@@ -168,15 +168,15 @@ func TestServerPutTreeGetFile(t *testing.T) {
 		if w.Code != http.StatusOK {
 			t.Fatal(w)
 		}
-		if !bytes.Contains(w.Body.Bytes(), []byte(`{"Title":"my title","Deleted":false,"Updated":"`)) {
+		if !bytes.Contains(w.Body.Bytes(), []byte(`{"Title":"my title","Deleted":false,"Content":"`)) {
 			t.Fatal(w)
 		}
 	})
 	t.Run("get", func(t *testing.T) {
-		r := httptest.NewRequest(http.MethodGet, "/"+id, nil)
+		r := httptest.NewRequest(http.MethodGet, "/"+url.PathEscape(id), nil)
 		t.Logf("%s", r.URL.String())
 		w := httptest.NewRecorder()
-		if err := server.apiV0FilesIDHandler(w, r); err != nil {
+		if err := server.apiV0FilesIDGetHandler(w, r); err != nil {
 			t.Fatal(err)
 		}
 		t.Logf("%s", w.Body.Bytes())
@@ -188,9 +188,6 @@ func TestServerPutTreeGetFile(t *testing.T) {
 		}
 		if title := w.Header().Get("Title"); title != "my title" {
 			t.Fatal(title)
-		} else if pid := w.Header().Get("PID"); pid != "my pid" {
-			t.Fatal(pid)
 		}
 	})
 }
-*/