fix(test): fix api tests to consume refstr

qri-io · Feb 16, 2021 · e03ef3a · e03ef3a
1 parent 13438b5
commit e03ef3a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/api/api_test.go b/api/api_test.go
@@ -100,6 +100,7 @@ func runHandlerTestCases(t *testing.T, name string, h http.HandlerFunc, cases []
 		if c.muxVars != nil {
 			req = mux.SetURLVars(req, *c.muxVars)
 		}
+		setRefStringFromMuxVars(req)
 		if jsonHeader {
 			req.Header.Set("Content-Type", "application/json")
 		}

diff --git a/lib/datasets.go b/lib/datasets.go
@@ -280,7 +280,9 @@ func (p *GetParams) UnmarshalFromRequest(r *http.Request) error {
 	}
 
 	params := *p
-	params.Refstr = r.FormValue("refstr")
+	if params.Refstr == "" {
+		params.Refstr = r.FormValue("refstr")
+	}
 
 	ref, err := dsref.Parse(params.Refstr)
 	if err != nil {