Added Rename and Remove methods to facebox

Author: dahernan

facebox/facebox_teach.go

@@ -95,6 +95,78 @@ func (c *Client) TeachURL(imageURL *url.URL, id, name string) error {
 	return nil
 }
 
+// Remove makes facebox to forget a face
+func (c *Client) Remove(id string) error {
+	if id == "" {
+		return errors.New("id can not be empty")
+	}
+	u, err := url.Parse(c.addr + "/facebox/teach")
+	if err != nil {
+		return err
+	}
+	if !u.IsAbs() {
+		return errors.New("box address must be absolute")
+	}
+
+	q := u.Query()
+	u.Path = u.Path + "/" + id
+	u.RawQuery = q.Encode()
+	req, err := http.NewRequest("DELETE", u.String(), nil)
+	if err != nil {
+		return err
+	}
+	req.Header.Set("Accept", "application/json; charset=utf-8")
+	resp, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return err
+	}
+	defer resp.Body.Close()
+	err = c.parseResponse(resp.Body)
+	if err != nil {
+		return err
+	}
+	return nil
+}
+
+// Rename allows to change the name for a given face
+func (c *Client) Rename(id, name string) error {
+	if id == "" {
+		return errors.New("id can not be empty")
+	}
+	if name == "" {
+		return errors.New("name can not be empty")
+	}
+	u, err := url.Parse(c.addr + "/facebox/teach")
+	if err != nil {
+		return err
+	}
+	if !u.IsAbs() {
+		return errors.New("box address must be absolute")
+	}
+	form := url.Values{}
+	form.Set("name", name)
+
+	q := u.Query()
+	u.Path = u.Path + "/" + id
+	u.RawQuery = q.Encode()
+	req, err := http.NewRequest("PATCH", u.String(), strings.NewReader(form.Encode()))
+	if err != nil {
+		return err
+	}
+	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
+	req.Header.Set("Accept", "application/json; charset=utf-8")
+	resp, err := c.HTTPClient.Do(req)
+	if err != nil {
+		return err
+	}
+	defer resp.Body.Close()
+	err = c.parseResponse(resp.Body)
+	if err != nil {
+		return err
+	}
+	return nil
+}
+
 func (c *Client) parseResponse(r io.Reader) error {
 	var response struct {
 		Success bool

facebox/facebox_teach_test.go

@@ -108,3 +108,40 @@ func TestTeachImageError(t *testing.T) {
 	is.Equal(err.Error(), "facebox: something went wrong")
 
 }
+
+func TestRemove(t *testing.T) {
+	is := is.New(t)
+
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		is.Equal(r.URL.Path, "/facebox/teach/john1.jpg")
+		is.Equal(r.Header.Get("Accept"), "application/json; charset=utf-8")
+		is.Equal(r.Method, "DELETE")
+		io.WriteString(w, `{
+			"success": true
+		}`)
+	}))
+	defer srv.Close()
+
+	fb := facebox.New(srv.URL)
+	err := fb.Remove("john1.jpg")
+	is.NoErr(err)
+}
+
+func TestRename(t *testing.T) {
+	is := is.New(t)
+
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		is.Equal(r.URL.Path, "/facebox/teach/john1.jpg")
+		is.Equal(r.Header.Get("Accept"), "application/json; charset=utf-8")
+		is.Equal(r.Method, "PATCH")
+		is.Equal(r.FormValue("name"), "Sir John L")
+		io.WriteString(w, `{
+			"success": true
+		}`)
+	}))
+	defer srv.Close()
+
+	fb := facebox.New(srv.URL)
+	err := fb.Rename("john1.jpg", "Sir John L")
+	is.NoErr(err)
+}