Merge pull request #1 from fulldump/fix/routing-base-case

fulldump · web-flow · commit 37652c1c91d7 · 2018-09-28T18:55:12.000+02:00
fix: Routing base case
diff --git a/resource.go b/resource.go
@@ -101,11 +101,11 @@ func (r *R) resourceParts(parts []string) *R {
 // Resource defines a new resource below current resource
 func (r *R) Resource(locator string) *R {
 
-	locator = strings.TrimPrefix(locator, "/")
 	if locator == "" {
 		return r
 	}
 
+	locator = strings.TrimPrefix(locator, "/")
 	parts := strings.Split(locator, "/")
 
 	return r.resourceParts(parts)
diff --git a/resource_test.go b/resource_test.go
@@ -36,7 +36,7 @@ func TestR_Resource_BaseCaseSlash(t *testing.T) {
 	root := NewResource()
 	r := root.Resource("/")
 
-	AssertEqual(t, r, root)
+	AssertEqual(t, r, root.children[0])
 }
 
 func TestR_Resource_NextCase(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ func TestR_Resource_BaseCaseSlash(t *testing.T) {`
`36`	`36`	`root := NewResource()`
`37`	`37`	`r := root.Resource("/")`
`38`	`38`
`39`		`- AssertEqual(t, r, root)`
	`39`	`+ AssertEqual(t, r, root.children[0])`
`40`	`40`	`}`
`41`	`41`
`42`	`42`	`func TestR_Resource_NextCase(t *testing.T) {`