forked from Jonathan-Rosenberg/oasdiff
-
Notifications
You must be signed in to change notification settings - Fork 1
/
check-response-optional-property-updated_test.go
103 lines (91 loc) · 4.5 KB
/
check-response-optional-property-updated_test.go
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
package checker_test
import (
"testing"
"github.com/stretchr/testify/require"
"github.com/treeverse/oasdiff/checker"
"github.com/treeverse/oasdiff/diff"
)
// CL: removing an optional write-only property from a response
func TestResponseOptionalPropertyUpdatedCheck(t *testing.T) {
s1, err := open("../data/checker/response_optional_property_removed_base.yaml")
require.NoError(t, err)
s2, err := open("../data/checker/response_optional_property_removed_revision.yaml")
require.NoError(t, err)
d, osm, err := diff.GetWithOperationsSourcesMap(getConfig(), s1, s2)
require.NoError(t, err)
errs := checker.CheckBackwardCompatibilityUntilLevel(singleCheckConfig(checker.ResponseOptionalPropertyUpdatedCheck), d, osm, checker.INFO)
require.Len(t, errs, 1)
require.Equal(t, checker.ApiChange{
Id: "response-optional-property-removed",
Text: "removed the optional property 'data/id' from the response with the '200' status",
Comment: "",
Level: checker.WARN,
Operation: "POST",
Path: "/api/v1.0/groups",
Source: "../data/checker/response_optional_property_removed_revision.yaml",
OperationId: "createOneGroup",
}, errs[0])
}
// CL: adding an optional write-only property to a response
func TestResponseOptionalPropertyAddedCheck(t *testing.T) {
s1, err := open("../data/checker/response_optional_property_removed_revision.yaml")
require.NoError(t, err)
s2, err := open("../data/checker/response_optional_property_removed_base.yaml")
require.NoError(t, err)
d, osm, err := diff.GetWithOperationsSourcesMap(getConfig(), s1, s2)
require.NoError(t, err)
errs := checker.CheckBackwardCompatibilityUntilLevel(singleCheckConfig(checker.ResponseOptionalPropertyUpdatedCheck), d, osm, checker.INFO)
require.Len(t, errs, 1)
require.Equal(t, checker.ApiChange{
Id: "response-optional-property-added",
Text: "added the optional property 'data/id' to the response with the '200' status",
Comment: "",
Level: checker.INFO,
Operation: "POST",
Path: "/api/v1.0/groups",
Source: "../data/checker/response_optional_property_removed_base.yaml",
OperationId: "createOneGroup",
}, errs[0])
}
// CL: removing an optional write-only property from a response
func TestResponseOptionalWriteOnlyPropertyRemovedCheck(t *testing.T) {
s1, err := open("../data/checker/response_optional_property_removed_base.yaml")
require.NoError(t, err)
s2, err := open("../data/checker/response_optional_property_removed_revision.yaml")
require.NoError(t, err)
s1.Spec.Paths["/api/v1.0/groups"].Post.Responses["200"].Value.Content["application/json"].Schema.Value.Properties["data"].Value.Properties["id"].Value.WriteOnly = true
d, osm, err := diff.GetWithOperationsSourcesMap(getConfig(), s1, s2)
require.NoError(t, err)
errs := checker.CheckBackwardCompatibilityUntilLevel(singleCheckConfig(checker.ResponseOptionalPropertyUpdatedCheck), d, osm, checker.INFO)
require.Len(t, errs, 1)
require.Equal(t, checker.ApiChange{
Id: "response-optional-write-only-property-removed",
Text: "removed the optional write-only property 'data/id' from the response with the '200' status",
Level: checker.INFO,
Operation: "POST",
Path: "/api/v1.0/groups",
Source: "../data/checker/response_optional_property_removed_revision.yaml",
OperationId: "createOneGroup",
}, errs[0])
}
// CL: adding an optional write-only property to a response
func TestResponseOptionalWriteOnlyPropertyAddedCheck(t *testing.T) {
s1, err := open("../data/checker/response_optional_property_removed_revision.yaml")
require.NoError(t, err)
s2, err := open("../data/checker/response_optional_property_removed_base.yaml")
require.NoError(t, err)
s2.Spec.Paths["/api/v1.0/groups"].Post.Responses["200"].Value.Content["application/json"].Schema.Value.Properties["data"].Value.Properties["id"].Value.WriteOnly = true
d, osm, err := diff.GetWithOperationsSourcesMap(getConfig(), s1, s2)
require.NoError(t, err)
errs := checker.CheckBackwardCompatibilityUntilLevel(singleCheckConfig(checker.ResponseOptionalPropertyUpdatedCheck), d, osm, checker.INFO)
require.Len(t, errs, 1)
require.Equal(t, checker.ApiChange{
Id: "response-optional-write-only-property-added",
Text: "added the optional write-only property 'data/id' to the response with the '200' status",
Level: checker.INFO,
Operation: "POST",
Path: "/api/v1.0/groups",
Source: "../data/checker/response_optional_property_removed_base.yaml",
OperationId: "createOneGroup",
}, errs[0])
}