Rule parsing error.
This commit is contained in:
parent
31f92001e2
commit
ec86149b1e
|
@ -269,6 +269,9 @@ func (r *Rules) Parse(expression string) (*mux.Route, error) {
|
||||||
if r.err != nil {
|
if r.err != nil {
|
||||||
return r.err
|
return r.err
|
||||||
}
|
}
|
||||||
|
if resultRoute == nil {
|
||||||
|
return fmt.Errorf("invalid expression: %s", expression)
|
||||||
|
}
|
||||||
if resultRoute.GetError() != nil {
|
if resultRoute.GetError() != nil {
|
||||||
return resultRoute.GetError()
|
return resultRoute.GetError()
|
||||||
}
|
}
|
||||||
|
|
|
@ -218,11 +218,17 @@ func TestHostRegexp(t *testing.T) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
type fakeHandler struct {
|
func TestParseInvalidSyntax(t *testing.T) {
|
||||||
name string
|
router := mux.NewRouter()
|
||||||
}
|
router.StrictSlash(true)
|
||||||
|
|
||||||
func (h *fakeHandler) ServeHTTP(http.ResponseWriter, *http.Request) {}
|
rules := &Rules{Route: &types.ServerRoute{Route: router.NewRoute()}}
|
||||||
|
expression01 := "Path: /path1;Query:param_one=true, /path2"
|
||||||
|
|
||||||
|
routeFoo, err := rules.Parse(expression01)
|
||||||
|
require.Error(t, err)
|
||||||
|
assert.Nil(t, routeFoo)
|
||||||
|
}
|
||||||
|
|
||||||
func TestPathPrefix(t *testing.T) {
|
func TestPathPrefix(t *testing.T) {
|
||||||
testCases := []struct {
|
testCases := []struct {
|
||||||
|
@ -287,3 +293,9 @@ func TestPathPrefix(t *testing.T) {
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
type fakeHandler struct {
|
||||||
|
name string
|
||||||
|
}
|
||||||
|
|
||||||
|
func (h *fakeHandler) ServeHTTP(http.ResponseWriter, *http.Request) {}
|
||||||
|
|
Loading…
Reference in a new issue