fix: remove conflicted and redundant error check for ParamsOneOf (#84)

Change-Id: I5b904255a6ba4d2094b530fa94318d6645744232

schema/tool.go

@@ -17,8 +17,6 @@
 package schema
 
 import (
-	"fmt"
-
 	"github.com/getkin/kin-openapi/openapi3"
 )
 
@@ -119,19 +117,6 @@ func (p *ParamsOneOf) ToOpenAPIV3() (*openapi3.Schema, error) {
 		return nil, nil
 	}
 
-	var (
-		useParameterInfo = p.params != nil
-		useOpenAPIV3     = p.openAPIV3 != nil
-	)
-
-	if !useParameterInfo && !useOpenAPIV3 {
-		return nil, fmt.Errorf("ParamsOneOf needs to have at least one method to describe the parameters")
-	}
-
-	if useParameterInfo && useOpenAPIV3 {
-		return nil, fmt.Errorf("ParamsOneOf can only have one method to describe the parameters, but not multiple methods")
-	}
-
 	if p.params != nil {
 		sc := &openapi3.Schema{
 			Properties: make(map[string]*openapi3.SchemaRef, len(p.params)),

schema/tool_test.go

@@ -31,20 +31,6 @@ func TestParamsOneOfToJSONSchema(t *testing.T) {
 			err       error
 		)
 
-		convey.Convey("user provides no option if ParamsOneOf", func() {
-			_, err = oneOf.ToOpenAPIV3()
-			convey.So(err, convey.ShouldNotBeNil)
-			convey.So(err.Error(), convey.ShouldContainSubstring, "ParamsOneOf needs to have at least one method to describe the parameters")
-		})
-
-		convey.Convey("user provides multiple options in ParamsOneOf", func() {
-			oneOf.params = make(map[string]*ParameterInfo)
-			oneOf.openAPIV3 = &openapi3.Schema{}
-			_, err = oneOf.ToOpenAPIV3()
-			convey.So(err, convey.ShouldNotBeNil)
-			convey.So(err.Error(), convey.ShouldContainSubstring, "ParamsOneOf can only have one method to describe the parameters, but not multiple methods")
-		})
-
 		convey.Convey("user provides openAPIV3.0 json schema directly, use what the user provides", func() {
 			oneOf.openAPIV3 = &openapi3.Schema{
 				Type:        openapi3.TypeString,