Webhook unit test panics with invalid memory address about argo-events HOT 2 CLOSED

Looks like the unit tests start a webserver at a fixed address of 9000. There might be a problem if two unit tests run concurrently that use the same port.

Might want to try randomizing the port in signal_test.go:

	web := New()
	signal := v1alpha1.Signal{
		Webhook: &v1alpha1.WebhookSignal{
			Port:     common.WebhookServiceTargetPort,  <<<<--------------------  HERE ----------
			Endpoint: endpoint,
			Method:   httpMethod,
		},
	}
	events, err := web.Start(&signal)

from argo-events.

Thanks for creating this issue, I was noticing the same patterns in some of the tests. I think the port may be the problem. Why would the tests would be running concurrently since the t.Parallel() is not set for these tests?

Also do you think we need to randomize here, why not just increment the port for each test post, put, and delete?

from argo-events.