Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add HaveValue matcher #485

Merged
merged 4 commits into from Dec 7, 2021
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
78 changes: 78 additions & 0 deletions matchers/have_value.go
@@ -0,0 +1,78 @@
package matchers

import (
"reflect"

"github.com/onsi/gomega/format"
"github.com/onsi/gomega/types"
)

// HaveValue applies the given matcher to the value of actual, optionally and
// repeatedly dereferencing pointers or taking the concrete value of interfaces.
// Thus, the matcher will always be applied to non-pointer and non-interface
// values only. HaveValue will fail with an error if a pointer or interface is
// nil. It will also fail for more than 31 pointer or interface dereferences to
// guard against mistakenly applying it to arbitrarily deep linked pointers.
//
// HaveValue differs from gstruct.PointTo in that it does not expect actual to
// be a pointer (as gstruct.PointTo does) but instead also accepts non-pointer
// and even interface values.
//
// actual := 42
// Expect(actual).To(HaveValue(42))
// Expect(&actual).To(HaveValue(42))
func HaveValue(matcher types.GomegaMatcher) types.GomegaMatcher {
return &HaveValueMatcher{
Matcher: matcher,
}
}

type HaveValueMatcher struct {
Matcher types.GomegaMatcher // the matcher to apply to the "resolved" actual value.
failure string // failure message, if any.
}

func (m *HaveValueMatcher) Match(actual interface{}) (bool, error) {
val := reflect.ValueOf(actual)
for allowedIndirs := 32; allowedIndirs > 0; allowedIndirs-- {
// return an error if value isn't valid. Please note that we cannot
// check for nil here, as we might not deal with a pointer or interface
// at this point.
if !val.IsValid() {
m.failure = format.Message(
actual, "not to be <nil>")
return false, nil
}
switch val.Kind() {
case reflect.Ptr, reflect.Interface:
// resolve pointers and interfaces to their values, then rinse and
// repeat.
if val.IsNil() {
m.failure = format.Message(
actual, "not to be <nil>")
return false, nil
}
val = val.Elem()
continue
default:
// forward the final value to the specified matcher.
elem := val.Interface()
match, err := m.Matcher.Match(elem)
if !match {
m.failure = m.Matcher.FailureMessage(elem)
}
return match, err
}
}
// too many indirections: extreme star gazing, indeed...?
m.failure = format.Message(actual, "indirecting too many times")
thediveo marked this conversation as resolved.
Show resolved Hide resolved
return false, nil
}

func (m *HaveValueMatcher) FailureMessage(_ interface{}) (message string) {
return m.failure
}

func (m *HaveValueMatcher) NegatedFailureMessage(actual interface{}) (message string) {
return m.Matcher.NegatedFailureMessage(actual)
thediveo marked this conversation as resolved.
Show resolved Hide resolved
}
54 changes: 54 additions & 0 deletions matchers/have_value_test.go
@@ -0,0 +1,54 @@
package matchers_test

import (
. "github.com/onsi/ginkgo/v2"
. "github.com/onsi/gomega"
. "github.com/onsi/gomega/matchers"
)

type I interface {
M()
}

type S struct {
V int
}

func (s S) M() {}

var _ = Describe("HaveValue", func() {

It("should fail when passed nil", func() {
var p *struct{}
m := HaveValue(BeNil())
Expect(m.Match(p)).To(BeFalse())
Expect(m.FailureMessage(p)).To(MatchRegexp("not to be <nil>$"))
})

It("should fail when passed nil indirectly", func() {
var p *struct{}
m := HaveValue(BeNil())
Expect(m.Match(&p)).To(BeFalse())
Expect(m.FailureMessage(&p)).To(MatchRegexp("not to be <nil>$"))
})

It("should unwrap the value pointed to, even repeatedly", func() {
i := 1
Expect(&i).To(HaveValue(Equal(1)))
Expect(&i).NotTo(HaveValue(Equal(2)))

pi := &i
Expect(pi).To(HaveValue(Equal(1)))
Expect(pi).NotTo(HaveValue(Equal(2)))

Expect(&pi).To(HaveValue(Equal(1)))
Expect(&pi).NotTo(HaveValue(Equal(2)))
})

It("should unwrap the value of an interface", func() {
var i I = &S{V: 42}
Expect(i).To(HaveValue(Equal(S{V: 42})))
Expect(i).NotTo(HaveValue(Equal(S{})))
})

})