fix: add testing for sets and some type fixes (#159)

* fix: add testing for sets and some type fixes

* Fix the refreshTTL tests.

* "returns a miss after overriding the client timeout with a short duration" needed
  a sleep to be reliable.
* "returns a miss after the client default when refreshTTL is false" needed to
  create the Set before testing its refresh. The TTL will always be applied when
  creating a new Set.
* Switch those tests to a more compact and complete testing style.

* fix: seed set for collection ttl tests in BeforeEach

* fix: prefer less verbose Expect

---------

Co-authored-by: Michael G. Schwern <schwern@pobox.com>

momento/set_add_elements.go

@@ -33,7 +33,9 @@ type SetAddElementsRequest struct {
 
 func (r *SetAddElementsRequest) cacheName() string { return r.CacheName }
 
-func (r *SetAddElementsRequest) ttl() time.Duration { return r.CollectionTTL.Ttl }
+func (r *SetAddElementsRequest) ttl() time.Duration {
+	return r.CollectionTTL.Ttl
+}
 
 func (r *SetAddElementsRequest) requestName() string { return "SetAddElements" }
 

momento/set_fetch.go

@@ -75,11 +75,11 @@ func (r *SetFetchRequest) makeGrpcRequest(metadata context.Context, client scsDa
 func (r *SetFetchRequest) interpretGrpcResponse() error {
 	switch rtype := r.grpcResponse.Set.(type) {
 	case *pb.XSetFetchResponse_Found:
-		r.response = SetFetchHit{
+		r.response = &SetFetchHit{
 			elements: rtype.Found.Elements,
 		}
 	case *pb.XSetFetchResponse_Missing:
-		r.response = SetFetchMiss{}
+		r.response = &SetFetchMiss{}
 	default:
 		return errUnexpectedGrpcResponse(r, r.grpcResponse)
 	}