kubernetes-sigs/kustomize
1
0
Fork 0
mirror of https://github.com/kubernetes-sigs/kustomize.git synced 2026-06-11 17:12:51 +00:00
Code Issues Packages Projects Releases 1 Wiki Activity

Drop all calls to IsEmpty.

Browse Source
This commit is contained in:
jregan
2020-08-09 10:39:04 -07:00
parent 2da8959198
commit 31c59bd7f2
14 changed files with 39 additions and 55 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
api/filters/namespace/namespace.go
View File

@@ -112,7 +112,7 @@ func (ns Filter) roleBindingHack(obj *yaml.RNode, meta yaml.ResourceMeta) error
// Lookup the namespace field on all elements. // Lookup the namespace field on all elements.
// We should change the fieldspec so this isn't necessary. // We should change the fieldspec so this isn't necessary.
obj, err := obj.Pipe(yaml.Lookup(subjectsField)) obj, err := obj.Pipe(yaml.Lookup(subjectsField))
if err != nil || yaml.IsEmpty(obj) { if err != nil || yaml.IsMissingOrNull(obj) {
return err return err
} }
@@ -125,7 +125,7 @@ func (ns Filter) roleBindingHack(obj *yaml.RNode, meta yaml.ResourceMeta) error
name, err := o.Pipe( name, err := o.Pipe(
yaml.Lookup("name"), yaml.Match("default"), yaml.Lookup("name"), yaml.Match("default"),
) )
if err != nil || yaml.IsEmpty(name) { if err != nil || yaml.IsMissingOrNull(name) {
return err return err
} }

2
kyaml/fn/runtime/runtimeutil/functiontypes.go
View File

@@ -138,7 +138,7 @@ func getFunctionSpecFromAnnotation(n *yaml.RNode, meta yaml.ResourceMeta) *Funct
} }
} }
n, err := n.Pipe(yaml.Lookup("metadata", "configFn")) n, err := n.Pipe(yaml.Lookup("metadata", "configFn"))
if err != nil || yaml.IsEmpty(n) { if err != nil || yaml.IsMissingOrNull(n) {
return nil return nil
} }
s, err := n.String() s, err := n.String()

2
kyaml/openapi/openapi.go
View File

@@ -36,7 +36,7 @@ type ResourceSchema struct {
} }
// IsEmpty returns true if the ResourceSchema is empty // IsEmpty returns true if the ResourceSchema is empty
func (rs *ResourceSchema) IsEmpty() bool { func (rs *ResourceSchema) IsMissingOrNull() bool {
if rs == nil || rs.Schema == nil { if rs == nil || rs.Schema == nil {
return true return true
} }

8
kyaml/setters2/list.go
View File

@@ -53,7 +53,7 @@ func (l *List) listSetters(object *yaml.RNode, resourcePath string) error {
if err != nil { if err != nil {
return err return err
} }
if yaml.IsEmpty(def) { if yaml.IsMissingOrNull(def) {
return nil return nil
} }
@@ -73,7 +73,7 @@ func (l *List) listSetters(object *yaml.RNode, resourcePath string) error {
if err != nil { if err != nil {
return err return err
} }
if yaml.IsEmpty(setterNode) { if yaml.IsMissingOrNull(setterNode) {
// has the setter prefix, but missing the setter extension // has the setter prefix, but missing the setter extension
return errors.Errorf("missing x-k8s-cli.setter for %s", key) return errors.Errorf("missing x-k8s-cli.setter for %s", key)
} }
@@ -126,7 +126,7 @@ func (l *List) listSubst(object *yaml.RNode) error {
if err != nil { if err != nil {
return err return err
} }
if yaml.IsEmpty(def) { if yaml.IsMissingOrNull(def) {
return nil return nil
} }
@@ -146,7 +146,7 @@ func (l *List) listSubst(object *yaml.RNode) error {
if err != nil { if err != nil {
return err return err
} }
if yaml.IsEmpty(substNode) { if yaml.IsMissingOrNull(substNode) {
// has the substitution prefix, but missing the setter extension // has the substitution prefix, but missing the setter extension
return errors.Errorf("missing x-k8s-cli.substitution for %s", key) return errors.Errorf("missing x-k8s-cli.substitution for %s", key)
} }

2
kyaml/setters2/walk.go
View File

@@ -110,7 +110,7 @@ func getSchema(r *yaml.RNode, s *openapi.ResourceSchema, field string) *openapi.
return s return s
} }
if yaml.IsEmpty(r) { if yaml.IsMissingOrNull(r) {
return nil return nil
} }

4
kyaml/yaml/fns.go
View File

@@ -70,7 +70,7 @@ func (e ElementSetter) Filter(rn *RNode) (*RNode, error) {
newNode := NewRNode(elem) newNode := NewRNode(elem)
// empty elements are not valid -- they at least need an associative key // empty elements are not valid -- they at least need an associative key
if IsEmpty(newNode) || IsEmptyMap(newNode) { if IsMissingOrNull(newNode) || IsEmptyMap(newNode) {
continue continue
} }
@@ -549,7 +549,7 @@ func IsFoundOrError(rn *RNode, err error) bool {
func ErrorIfAnyInvalidAndNonNull(kind yaml.Kind, rn ...*RNode) error { func ErrorIfAnyInvalidAndNonNull(kind yaml.Kind, rn ...*RNode) error {
for i := range rn { for i := range rn {
if IsEmpty(rn[i]) { if IsMissingOrNull(rn[i]) {
continue continue
} }
if err := ErrorIfInvalid(rn[i], kind); err != nil { if err := ErrorIfInvalid(rn[i], kind); err != nil {

4
kyaml/yaml/merge2/merge2.go
View File

@@ -52,7 +52,7 @@ func (m Merger) VisitMap(nodes walk.Sources, s *openapi.ResourceSchema) (*yaml.R
if err := m.SetStyle(nodes); err != nil { if err := m.SetStyle(nodes); err != nil {
return nil, err return nil, err
} }
if yaml.IsEmpty(nodes.Dest()) { if yaml.IsMissingOrNull(nodes.Dest()) {
// Add // Add
return nodes.Origin(), nil return nodes.Origin(), nil
} }
@@ -108,7 +108,7 @@ func (m Merger) VisitList(nodes walk.Sources, s *openapi.ResourceSchema, kind wa
} }
// Add // Add
if yaml.IsEmpty(nodes.Dest()) { if yaml.IsMissingOrNull(nodes.Dest()) {
return nodes.Origin(), nil return nodes.Origin(), nil
} }
// Clear // Clear

12
kyaml/yaml/merge3/visitor.go
View File

@@ -39,11 +39,11 @@ func (m Visitor) VisitMap(nodes walk.Sources, s *openapi.ResourceSchema) (*yaml.
} }
func (m Visitor) visitAList(nodes walk.Sources, _ *openapi.ResourceSchema) (*yaml.RNode, error) { func (m Visitor) visitAList(nodes walk.Sources, _ *openapi.ResourceSchema) (*yaml.RNode, error) {
if yaml.IsEmpty(nodes.Updated()) && !yaml.IsEmpty(nodes.Origin()) { if yaml.IsMissingOrNull(nodes.Updated()) && !yaml.IsMissingOrNull(nodes.Origin()) {
// implicitly cleared from update -- element was deleted // implicitly cleared from update -- element was deleted
return walk.ClearNode, nil return walk.ClearNode, nil
} }
if yaml.IsEmpty(nodes.Dest()) { if yaml.IsMissingOrNull(nodes.Dest()) {
// not cleared, but missing from the dest // not cleared, but missing from the dest
// initialize a new value that can be recursively merged // initialize a new value that can be recursively merged
return yaml.NewRNode(&yaml.Node{Kind: yaml.SequenceNode}), nil return yaml.NewRNode(&yaml.Node{Kind: yaml.SequenceNode}), nil
@@ -58,11 +58,11 @@ func (m Visitor) VisitScalar(nodes walk.Sources, s *openapi.ResourceSchema) (*ya
// explicitly cleared from either dest or update // explicitly cleared from either dest or update
return nil, nil return nil, nil
} }
if yaml.IsEmpty(nodes.Updated()) != yaml.IsEmpty(nodes.Origin()) { if yaml.IsMissingOrNull(nodes.Updated()) != yaml.IsMissingOrNull(nodes.Origin()) {
// value added or removed in update // value added or removed in update
return nodes.Updated(), nil return nodes.Updated(), nil
} }
if yaml.IsEmpty(nodes.Updated()) && yaml.IsEmpty(nodes.Origin()) { if yaml.IsMissingOrNull(nodes.Updated()) && yaml.IsMissingOrNull(nodes.Origin()) {
// value added or removed in update // value added or removed in update
return nodes.Dest(), nil return nodes.Dest(), nil
} }
@@ -96,11 +96,11 @@ func (m Visitor) visitNAList(nodes walk.Sources) (*yaml.RNode, error) {
return walk.ClearNode, nil return walk.ClearNode, nil
} }
if yaml.IsEmpty(nodes.Updated()) != yaml.IsEmpty(nodes.Origin()) { if yaml.IsMissingOrNull(nodes.Updated()) != yaml.IsMissingOrNull(nodes.Origin()) {
// value added or removed in update // value added or removed in update
return nodes.Updated(), nil return nodes.Updated(), nil
} }
if yaml.IsEmpty(nodes.Updated()) && yaml.IsEmpty(nodes.Origin()) { if yaml.IsMissingOrNull(nodes.Updated()) && yaml.IsMissingOrNull(nodes.Origin()) {
// value not present in source or dest // value not present in source or dest
return nodes.Dest(), nil return nodes.Dest(), nil
} }

2
kyaml/yaml/schema/schema.go
View File

@@ -22,7 +22,7 @@ func IsAssociative(schema *openapi.ResourceSchema, nodes []*yaml.RNode, infer bo
} }
for i := range nodes { for i := range nodes {
node := nodes[i] node := nodes[i]
if yaml.IsEmpty(node) { if yaml.IsMissingOrNull(node) {
continue continue
} }
if node.IsAssociative() { if node.IsAssociative() {

11
kyaml/yaml/types.go
View File

@@ -33,19 +33,20 @@ func MakeNullNode() *RNode {
return NewRNode(&Node{Tag: NodeTagNull}) return NewRNode(&Node{Tag: NodeTagNull})
} }
// IsMissingOrNull returns true if the RNode is nil or contains and explicitly null value. // IsMissingOrNull is true if the RNode is nil or explicitly tagged null.
// TODO: make this a method on RNode.
func IsMissingOrNull(node *RNode) bool { func IsMissingOrNull(node *RNode) bool {
return IsNil(node) || node.YNode().Tag == NodeTagNull return IsNil(node) || node.YNode().Tag == NodeTagNull
} }
// IsEmpty returns true if the RNode is MissingOrNull // Deprecated. Use IsMissingOrNull instead.
func IsEmpty(node *RNode) bool { func IsEmpty(node *RNode) bool {
return IsMissingOrNull(node) return IsMissingOrNull(node)
} }
// IsEmptyMap returns true if the RNode is an empty node or an empty map // IsEmptyMap returns true if the RNode is an empty node or an empty map
func IsEmptyMap(node *RNode) bool { func IsEmptyMap(node *RNode) bool {
return IsEmpty(node) || IsYNodeEmptyMap(node.YNode()) return IsMissingOrNull(node) || IsYNodeEmptyMap(node.YNode())
} }
// IsNil return true if the node is nil, or its underlying YNode is nil. // IsNil return true if the node is nil, or its underlying YNode is nil.
@@ -86,7 +87,7 @@ func IsYNodeString(n *yaml.Node) bool {
// GetValue returns underlying yaml.Node Value field // GetValue returns underlying yaml.Node Value field
func GetValue(node *RNode) string { func GetValue(node *RNode) string {
if IsEmpty(node) { if IsMissingOrNull(node) {
return "" return ""
} }
return node.YNode().Value return node.YNode().Value
@@ -378,7 +379,7 @@ const (
// GetMeta returns the ResourceMeta for an RNode // GetMeta returns the ResourceMeta for an RNode
func (rn *RNode) GetMeta() (ResourceMeta, error) { func (rn *RNode) GetMeta() (ResourceMeta, error) {
if IsEmpty(rn) { if IsMissingOrNull(rn) {
return ResourceMeta{}, nil return ResourceMeta{}, nil
} }
missingMeta := true missingMeta := true

25
kyaml/yaml/types_test.go
View File

@@ -183,42 +183,25 @@ func TestIsMissingOrNull(t *testing.T) {
if !IsMissingOrNull(MakeNullNode()) { if !IsMissingOrNull(MakeNullNode()) {
t.Fatalf("input: with NullNodeTag") t.Fatalf("input: with NullNodeTag")
} }
}
func TestIsEmpty(t *testing.T) {
if !IsEmpty(nil) {
t.Fatalf("input: nil")
}
// missing value or null value
if !IsEmpty(NewRNode(nil)) {
t.Fatalf("input: nil value")
}
// not array or map
if IsEmpty(NewScalarRNode("foo")) {
t.Fatalf("input: not array or map")
}
}
func TestIsEmpty_Arrays(t *testing.T) {
node := NewListRNode() node := NewListRNode()
// empty array. empty array is not expected as empty // empty array. empty array is not expected as empty
if IsEmpty(node) { if IsMissingOrNull(node) {
t.Fatalf("input: empty array") t.Fatalf("input: empty array")
} }
// array with 1 item // array with 1 item
node = NewListRNode("foo") node = NewListRNode("foo")
if IsEmpty(node) { if IsMissingOrNull(node) {
t.Fatalf("input: array with 1 item") t.Fatalf("input: array with 1 item")
} }
// delete the item in array // delete the item in array
node.value.Content = nil node.value.Content = nil
if IsEmpty(node) { if IsMissingOrNull(node) {
t.Fatalf("input: empty array") t.Fatalf("input: empty array")
} }
} }
func TestIsEmpty_Maps(t *testing.T) { func TestIsEmptyMap(t *testing.T) {
node := NewMapRNode(nil) node := NewMapRNode(nil)
// empty map // empty map
if !IsEmptyMap(node) { if !IsEmptyMap(node) {

8
kyaml/yaml/walk/associative_sequence.go
View File

@@ -51,7 +51,7 @@ func (l *Walker) walkAssociativeSequence() (*yaml.RNode, error) {
if err != nil { if err != nil {
return nil, err return nil, err
} }
if yaml.IsEmpty(val) || yaml.IsEmptyMap(val) { if yaml.IsMissingOrNull(val) || yaml.IsEmptyMap(val) {
_, err = dest.Pipe(yaml.ElementSetter{Key: key, Value: value}) _, err = dest.Pipe(yaml.ElementSetter{Key: key, Value: value})
if err != nil { if err != nil {
return nil, err return nil, err
@@ -74,7 +74,7 @@ func (l *Walker) walkAssociativeSequence() (*yaml.RNode, error) {
} }
} }
// field is empty // field is empty
if yaml.IsEmpty(dest) { if yaml.IsMissingOrNull(dest) {
return nil, nil return nil, nil
} }
return dest, nil return dest, nil
@@ -97,7 +97,7 @@ func (l *Walker) walkAssociativeSequence() (*yaml.RNode, error) {
if err != nil { if err != nil {
return nil, err return nil, err
} }
if yaml.IsEmpty(val) { if yaml.IsMissingOrNull(val) {
_, err = dest.Pipe(yaml.ElementSetter{Key: key, Value: value}) _, err = dest.Pipe(yaml.ElementSetter{Key: key, Value: value})
if err != nil { if err != nil {
return nil, err return nil, err
@@ -121,7 +121,7 @@ func (l *Walker) walkAssociativeSequence() (*yaml.RNode, error) {
} }
// field is empty // field is empty
if yaml.IsEmpty(dest) { if yaml.IsMissingOrNull(dest) {
return nil, nil return nil, nil
} }
return dest, nil return dest, nil

4
kyaml/yaml/walk/map.go
View File

@@ -39,7 +39,7 @@ func (l Walker) walkMap() (*yaml.RNode, error) {
continue continue
} }
field := l.Sources[i].Field(key) field := l.Sources[i].Field(key)
if field == nil || yaml.IsEmpty(field.Key) { if field == nil || yaml.IsMissingOrNull(field.Key) {
keys = append(keys, nil) keys = append(keys, nil)
continue continue
} }
@@ -148,7 +148,7 @@ func (l Walker) fieldValue(fieldName string) ([]*yaml.RNode, *openapi.ResourceSc
field := l.Sources[i].Field(fieldName) field := l.Sources[i].Field(fieldName)
f, s := l.valueIfPresent(field) f, s := l.valueIfPresent(field)
fields = append(fields, f) fields = append(fields, f)
if sch == nil && !s.IsEmpty() { if sch == nil && !s.IsMissingOrNull() {
sch = s sch = s
} }
} }

6
kyaml/yaml/walk/walk.go
View File

@@ -40,7 +40,7 @@ type Walker struct {
func (l Walker) Kind() yaml.Kind { func (l Walker) Kind() yaml.Kind {
for _, s := range l.Sources { for _, s := range l.Sources {
if !yaml.IsEmpty(s) { if !yaml.IsMissingOrNull(s) {
return s.YNode().Kind return s.YNode().Kind
} }
} }
@@ -83,7 +83,7 @@ func (l Walker) Walk() (*yaml.RNode, error) {
func (l Walker) GetSchema() *openapi.ResourceSchema { func (l Walker) GetSchema() *openapi.ResourceSchema {
for i := range l.Sources { for i := range l.Sources {
r := l.Sources[i] r := l.Sources[i]
if yaml.IsEmpty(r) { if yaml.IsMissingOrNull(r) {
continue continue
} }
@@ -106,7 +106,7 @@ func (l Walker) GetSchema() *openapi.ResourceSchema {
} }
for i := range l.Sources { for i := range l.Sources {
r := l.Sources[i] r := l.Sources[i]
if yaml.IsEmpty(r) { if yaml.IsMissingOrNull(r) {
continue continue
} }