Skip to content

Commit e5d4a24

Browse files
beorn7stuartnelson3
authored andcommitted
Adjust TestRegisterWithOrGet to the present (#477)
Signed-off-by: beorn7 <[email protected]>
1 parent 93edea3 commit e5d4a24

File tree

1 file changed

+5
-11
lines changed

1 file changed

+5
-11
lines changed

prometheus/registry_test.go

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -738,14 +738,8 @@ func BenchmarkHandler(b *testing.B) {
738738
}
739739
}
740740

741-
func TestRegisterWithOrGet(t *testing.T) {
742-
// Replace the default registerer just to be sure. This is bad, but this
743-
// whole test will go away once RegisterOrGet is removed.
744-
oldRegisterer := prometheus.DefaultRegisterer
745-
defer func() {
746-
prometheus.DefaultRegisterer = oldRegisterer
747-
}()
748-
prometheus.DefaultRegisterer = prometheus.NewRegistry()
741+
func TestAlreadyRegistered(t *testing.T) {
742+
reg := prometheus.NewRegistry()
749743
original := prometheus.NewCounterVec(
750744
prometheus.CounterOpts{
751745
Name: "test",
@@ -761,11 +755,11 @@ func TestRegisterWithOrGet(t *testing.T) {
761755
[]string{"foo", "bar"},
762756
)
763757
var err error
764-
if err = prometheus.Register(original); err != nil {
758+
if err = reg.Register(original); err != nil {
765759
t.Fatal(err)
766760
}
767-
if err = prometheus.Register(equalButNotSame); err == nil {
768-
t.Fatal("expected error when registringe equal collector")
761+
if err = reg.Register(equalButNotSame); err == nil {
762+
t.Fatal("expected error when registering equal collector")
769763
}
770764
if are, ok := err.(prometheus.AlreadyRegisteredError); ok {
771765
if are.ExistingCollector != original {

0 commit comments

Comments
 (0)