Add basic test case for #707

cmd/promxy/demo_robust.conf

@@ -6,12 +6,6 @@ global:
   external_labels:
     source: promxy
 
-# Rule files specifies a list of globs. Rules and alerts are read from
-# all matching files.
-rule_files:
-- "*rule"
-
-
 ##
 ### Promxy configuration
 ##

pkg/logging/logging.go

@@ -133,12 +133,14 @@ func NewApacheLoggingHandler(handler http.Handler, logHandlers ...LogRecordHandl
 }
 
 func (h *ApacheLoggingHandler) runHandler(rw http.ResponseWriter, r *http.Request) (err error) {
-	defer func() {
-		if rec := recover(); rec != nil {
-			// Just return a stack trace always
-			err = errors.Wrap(errors.New(string(debug.Stack())), "Error running handler")
-		}
-	}()
+	if false {
+		defer func() {
+			if rec := recover(); rec != nil {
+				// Just return a stack trace always
+				err = errors.Wrap(errors.New(string(debug.Stack())), "Error running handler")
+			}
+		}()
+	}
 	h.handler.ServeHTTP(rw, r)
 	return
 }

pkg/promclient/iterators_test.go

@@ -0,0 +1,35 @@
+package promclient
+
+import (
+	"strconv"
+	"testing"
+
+	"github.com/prometheus/common/model"
+	"github.com/prometheus/prometheus/model/labels"
+)
+
+func TestIterators(t *testing.T) {
+	tests := []struct {
+		val    interface{} // value
+		labels labels.Labels
+	}{
+		{
+			val: &model.Scalar{
+				Value:     0,
+				Timestamp: 0,
+			},
+		},
+	}
+
+	for i, test := range tests {
+		t.Run(strconv.Itoa(i), func(t *testing.T) {
+			it := NewSeriesIterator(test.val)
+
+			labels := it.Labels()
+			if labels.String() != test.labels.String() {
+				t.Fatalf("Mismatch in labels expected=%+v actual=%+v", test.labels, labels)
+			}
+		})
+	}
+
+}