feat(#5): remove routes

router.go

@@ -233,14 +233,17 @@ func doMatch(routes []*Route, path string) (url.Values, *Route) {
 
 // Remove matches and removes the matched route from the router stack.
 func (r *Router) Remove(method, path string) bool {
-	// TODO
-	return true
-}
-
-// RemoveRoute removes the given route from the router stack.
-func (r *Router) RemoveRoute(route *Route) bool {
-	// TODO
-	return true
+	routes := r.Routes[method]
+	if routes == nil || len(routes) == 0 {
+		return false
+	}
+	for i, route := range routes {
+		if _, ok := route.Match(path); ok {
+			r.Routes[method] = append(routes[:i], routes[i+1:]...)
+			return true
+		}
+	}
+	return false
 }
 
 // HandleHTTP matches r.URL.Path against its routing table

router_test.go

@@ -11,6 +11,28 @@ import (
 	"testing"
 )
 
+func TestRouterRemoveRoute(t *testing.T) {
+	p := New()
+	p.Get("/foo")
+	p.All("/bar")
+
+	st.Expect(t, len(p.Routes["GET"]), 1)
+	st.Expect(t, len(p.Routes["*"]), 1)
+
+	st.Expect(t, routeExists(p, "*", "/bar"), true)
+	st.Expect(t, routeExists(p, "GET", "/foo"), true)
+
+	st.Expect(t, p.Remove("GET", "/foo"), true)
+	st.Expect(t, len(p.Routes["GET"]), 0)
+	st.Expect(t, routeExists(p, "GET", "/foo"), false)
+
+	st.Expect(t, p.Remove("*", "/bar"), true)
+	st.Expect(t, len(p.Routes["*"]), 0)
+	st.Expect(t, routeExists(p, "*", "/bar"), false)
+
+	st.Expect(t, p.Remove("*", "/baz"), false)
+}
+
 func TestRoutingHit(t *testing.T) {
 	p := New()
 
@@ -184,3 +206,8 @@ func newRequest(method, urlStr string, body io.Reader) *http.Request {
 	}
 	return req
 }
+
+func routeExists(r *Router, method, path string) bool {
+	route, _ := r.FindRoute(method, path)
+	return route != nil
+}