Merge branch 'main' of https://github.com/pb33f/libopenapi into v0.13.0

Signed-off-by: quobix <dave@quobix.com>

# Conflicts:
#	what-changed/model/info_test.go
This commit is contained in:
quobix
2023-11-04 11:44:00 -04:00
5 changed files with 126 additions and 7 deletions

View File

@@ -11,8 +11,9 @@ import (
// InfoChanges represents the number of changes to an Info object. Part of an OpenAPI document
type InfoChanges struct {
*PropertyChanges
ContactChanges *ContactChanges `json:"contact,omitempty" yaml:"contact,omitempty"`
LicenseChanges *LicenseChanges `json:"license,omitempty" yaml:"license,omitempty"`
ContactChanges *ContactChanges `json:"contact,omitempty" yaml:"contact,omitempty"`
LicenseChanges *LicenseChanges `json:"license,omitempty" yaml:"license,omitempty"`
ExtensionChanges *ExtensionChanges `json:"extensions,omitempty" yaml:"extensions,omitempty"`
}
// GetAllChanges returns a slice of all changes made between Info objects
@@ -25,6 +26,9 @@ func (i *InfoChanges) GetAllChanges() []*Change {
if i.LicenseChanges != nil {
changes = append(changes, i.LicenseChanges.GetAllChanges()...)
}
if i.ExtensionChanges != nil {
changes = append(changes, i.ExtensionChanges.GetAllChanges()...)
}
return changes
}
@@ -37,6 +41,9 @@ func (i *InfoChanges) TotalChanges() int {
if i.LicenseChanges != nil {
t += i.LicenseChanges.TotalChanges()
}
if i.ExtensionChanges != nil {
t += i.ExtensionChanges.TotalChanges()
}
return t
}
@@ -139,6 +146,10 @@ func CompareInfo(l, r *base.Info) *InfoChanges {
l.License.ValueNode, nil, false, r.License.Value, nil)
}
}
// check extensions.
i.ExtensionChanges = CompareExtensions(l.Extensions, r.Extensions)
i.PropertyChanges = NewPropertyChanges(changes)
if i.TotalChanges() <= 0 {
return nil

View File

@@ -5,12 +5,13 @@ package model
import (
"context"
"testing"
"github.com/pb33f/libopenapi/datamodel/low"
"github.com/pb33f/libopenapi/datamodel/low/base"
"github.com/pb33f/libopenapi/datamodel/low/v3"
v3 "github.com/pb33f/libopenapi/datamodel/low/v3"
"github.com/stretchr/testify/assert"
"gopkg.in/yaml.v3"
"testing"
)
func TestCompareInfo_DescriptionAdded(t *testing.T) {
@@ -375,7 +376,8 @@ contact:
name: buckaroo
email: buckaroo@pb33f.io
license:
name: MIT`
name: MIT
x-extension: extension`
right := `title: a nice spec
termsOfService: https://pb33f.io/terms
@@ -384,7 +386,8 @@ contact:
name: buckaroo
email: buckaroo@pb33f.io
license:
name: MIT`
name: MIT
x-extension: extension`
var lNode, rNode yaml.Node
_ = yaml.Unmarshal([]byte(left), &lNode)
@@ -402,3 +405,97 @@ license:
extChanges := CompareInfo(&lDoc, &rDoc)
assert.Nil(t, extChanges)
}
func TestCompareInfo_ExtensionAdded(t *testing.T) {
left := `title: a nice spec
version: '1.2.3'
`
right := `title: a nice spec
version: '1.2.3'
x-extension: new extension
`
var lNode, rNode yaml.Node
_ = yaml.Unmarshal([]byte(left), &lNode)
_ = yaml.Unmarshal([]byte(right), &rNode)
// create low level objects
var lDoc base.Info
var rDoc base.Info
_ = low.BuildModel(lNode.Content[0], &lDoc)
_ = low.BuildModel(rNode.Content[0], &rDoc)
_ = lDoc.Build(nil, lNode.Content[0], nil)
_ = rDoc.Build(nil, rNode.Content[0], nil)
// compare.
extChanges := CompareInfo(&lDoc, &rDoc)
assert.Equal(t, 1, extChanges.TotalChanges())
assert.Len(t, extChanges.GetAllChanges(), 1)
assert.Equal(t, ObjectAdded, extChanges.ExtensionChanges.Changes[0].ChangeType)
assert.Equal(t, "x-extension", extChanges.ExtensionChanges.Changes[0].Property)
}
func TestCompareInfo_ExtensionRemoved(t *testing.T) {
left := `title: a nice spec
version: '1.2.3'
x-extension: extension
`
right := `title: a nice spec
version: '1.2.3'
`
var lNode, rNode yaml.Node
_ = yaml.Unmarshal([]byte(left), &lNode)
_ = yaml.Unmarshal([]byte(right), &rNode)
// create low level objects
var lDoc base.Info
var rDoc base.Info
_ = low.BuildModel(lNode.Content[0], &lDoc)
_ = low.BuildModel(rNode.Content[0], &rDoc)
_ = lDoc.Build(nil, lNode.Content[0], nil)
_ = rDoc.Build(nil, rNode.Content[0], nil)
// compare.
extChanges := CompareInfo(&lDoc, &rDoc)
assert.Equal(t, 1, extChanges.TotalChanges())
assert.Len(t, extChanges.GetAllChanges(), 1)
assert.Equal(t, ObjectRemoved, extChanges.ExtensionChanges.Changes[0].ChangeType)
assert.Equal(t, "x-extension", extChanges.ExtensionChanges.Changes[0].Property)
}
func TestCompareInfo_ExtensionModified(t *testing.T) {
left := `title: a nice spec
version: '1.2.3'
x-extension: original extension
`
right := `title: a nice spec
version: '1.2.3'
x-extension: new extension
`
var lNode, rNode yaml.Node
_ = yaml.Unmarshal([]byte(left), &lNode)
_ = yaml.Unmarshal([]byte(right), &rNode)
// create low level objects
var lDoc base.Info
var rDoc base.Info
_ = low.BuildModel(lNode.Content[0], &lDoc)
_ = low.BuildModel(rNode.Content[0], &rDoc)
_ = lDoc.Build(nil, lNode.Content[0], nil)
_ = rDoc.Build(nil, rNode.Content[0], nil)
// compare.
extChanges := CompareInfo(&lDoc, &rDoc)
assert.Equal(t, 1, extChanges.TotalChanges())
assert.Len(t, extChanges.GetAllChanges(), 1)
assert.Equal(t, Modified, extChanges.ExtensionChanges.Changes[0].ChangeType)
assert.Equal(t, "x-extension", extChanges.ExtensionChanges.Changes[0].Property)
}