Fix acceptance test

test/acceptance.go

@@ -52,11 +52,7 @@ func NewAcceptanceTest(t *testing.T, opts *AcceptanceOpts) *AcceptanceTest {
 	return test
 }
 
-var freeAdresses []string
-
 func freeAddress() string {
-	if len(freeAdresses) == 0 {
-		for i := 0; i < 100; i++ {
 	// Let the OS allocate a free address, close it and hope
 	// it is still free when starting Alertmanager.
 	l, err := net.Listen("tcp", ":0")
@@ -65,14 +61,7 @@ func freeAddress() string {
 	}
 	defer l.Close()
 
-			freeAdresses = append(freeAdresses, l.Addr().String())
-		}
-	}
-
-	next := freeAdresses[0]
-	freeAdresses = freeAdresses[1:]
-
-	return next
+	return l.Addr().String()
 }
 
 // Alertmanager returns a new structure that allows starting an instance
@@ -196,7 +185,7 @@ func (am *Alertmanager) Push(at float64, alerts ...*TestAlert) {
 			return
 		}
 
-		resp, err := http.Post(fmt.Sprintf("http://%s/api/alerts", am.addr), "application/json", &buf)
+		resp, err := http.Post(fmt.Sprintf("http://%s/api/v1/alerts", am.addr), "application/json", &buf)
 		if err != nil {
 			am.t.Error(err)
 			return
@@ -253,11 +242,11 @@ func (am *Alertmanager) runActions() {
 		wg.Add(len(fs))
 
 		for _, f := range fs {
-			go func() {
+			go func(f func()) {
 				time.Sleep(ts.Sub(time.Now()))
 				f()
 				wg.Done()
-			}()
+			}(f)
 		}
 	}
 

test/acceptance/simple_test.go

@@ -76,7 +76,7 @@ func TestSilencing(t *testing.T) {
 
 	at := NewAcceptanceTest(t, &AcceptanceOpts{
 		Tolerance: 150 * time.Millisecond,
-		Config:    somethingConfig,
+		Config:    silenceConfig,
 	})
 
 	am := at.Alertmanager()
@@ -86,7 +86,7 @@ func TestSilencing(t *testing.T) {
 
 	// No repeat interval is configured. Thus, we receive an alert
 	// notification every second.
-	am.Push(At(1), Alert("alertname", "test122").Active(1))
+	am.Push(At(1), Alert("alertname", "test1").Active(1))
 	am.Push(At(1), Alert("alertname", "test2").Active(1))
 
 	co.Want(Between(2, 2.5),
@@ -111,7 +111,7 @@ func TestSilencing(t *testing.T) {
 	)
 
 	// Start the flow as defined above and run the checks afterwards.
-	// at.Run()
+	at.Run()
 }
 
 var batchConfig = `