Merge pull request #26 from danielgtaylor/subresource-fix

fix: copy router into sub-resources
danielgtaylor · Feb 23, 2022 · b8b221c · b8b221c
2 parents 58fff38 + 11d7e9f
commit b8b221c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/resource.go b/resource.go
@@ -95,6 +95,7 @@ func (r *Resource) SubResource(path string) *Resource {
 	sub := &Resource{
 		path:         r.path + path,
 		mux:          r.mux.Route(path, nil),
+		router:       r.router,
 		subResources: []*Resource{},
 		operations:   []*Operation{},
 		tags:         append([]string{}, r.tags...),

diff --git a/router_test.go b/router_test.go
@@ -400,3 +400,12 @@ func TestGetOperationDoesNotCrash(t *testing.T) {
 		assert.NotNil(t, info)
 	})
 }
+
+func TestSubResource(t *testing.T) {
+	app := newTestRouter()
+
+	// This should not crash.
+	app.Resource("/").SubResource("/foo").SubResource("/bar").Get("get-bar", "docs", NewResponse(http.StatusOK, "ok")).Run(func(ctx Context) {
+		// Do nothing
+	})
+}