clean-up todos

endpoint.go

@@ -29,7 +29,7 @@ func init() {
 }
 
 func serve(w http.ResponseWriter, r *http.Request) error {
-	path := r.URL.Path // @todo: is this already cleaned?
+	path := r.URL.Path
 	isFolder := path[len(path)-1] == '/'
 
 	switch r.Method {
@@ -288,7 +288,6 @@ func PutDocument(w http.ResponseWriter, r *http.Request) error {
 
 	mime := r.Header.Get("Content-Type")
 
-	// @todo: merge Create and Update into one function?
 	if found {
 		fd, err := FS.Create(n.sname)
 		if err != nil {

endpoint_test.go

@@ -685,11 +685,6 @@ func TestPutDocumentUpdatesAncestorETags(t *testing.T) {
 		if e := r.Header.Get("ETag"); e != testDocumentDirETag1 {
 			t.Errorf("got: `%s', want: `%s'", e, testDocumentDirETag1)
 		}
-		// @todo: maybe we want to validate this as well?
-		//bs, err := io.ReadAll(r.Body)
-		//if err != nil {
-		//	t.Error(err)
-		//}
 	}
 
 	// Get root folder ETag
@@ -707,11 +702,6 @@ func TestPutDocumentUpdatesAncestorETags(t *testing.T) {
 		if e := r.Header.Get("ETag"); e != testRootETag1 {
 			t.Errorf("got: `%s', want: `%s'", e, testRootETag1)
 		}
-		// @todo: maybe we want to validate this as well?
-		//bs, err := io.ReadAll(r.Body)
-		//if err != nil {
-		//	t.Error(err)
-		//}
 	}
 
 	// PUT second document
@@ -748,11 +738,6 @@ func TestPutDocumentUpdatesAncestorETags(t *testing.T) {
 		if e := r.Header.Get("ETag"); e != testDocumentDirETag2 {
 			t.Errorf("got: `%s', want: `%s'", e, testDocumentDirETag2)
 		}
-		// @todo: maybe we want to validate this as well?
-		//bs, err := io.ReadAll(r.Body)
-		//if err != nil {
-		//	t.Error(err)
-		//}
 	}
 
 	// Get root folder ETag
@@ -770,11 +755,6 @@ func TestPutDocumentUpdatesAncestorETags(t *testing.T) {
 		if e := r.Header.Get("ETag"); e != testRootETag2 {
 			t.Errorf("got: `%s', want: `%s'", e, testRootETag2)
 		}
-		// @todo: maybe we want to validate this as well?
-		//bs, err := io.ReadAll(r.Body)
-		//if err != nil {
-		//	t.Error(err)
-		//}
 	}
 }