Merge pull request #339 from moorereason/StatusOK
Fix TestWebhook tests
This commit is contained in:
commit
1cf531b1c3
@ -1,8 +1,9 @@
|
|||||||
language: go
|
language: go
|
||||||
|
|
||||||
go:
|
go:
|
||||||
- 1.8.x
|
- 1.11.x
|
||||||
- 1.9.x
|
- 1.12.x
|
||||||
|
- 1.13.x
|
||||||
- tip
|
- tip
|
||||||
|
|
||||||
os:
|
os:
|
||||||
|
@ -76,7 +76,6 @@ func TestWebhook(t *testing.T) {
|
|||||||
|
|
||||||
for _, tt := range hookHandlerTests {
|
for _, tt := range hookHandlerTests {
|
||||||
t.Run(tt.desc, func(t *testing.T) {
|
t.Run(tt.desc, func(t *testing.T) {
|
||||||
|
|
||||||
ip, port := serverAddress(t)
|
ip, port := serverAddress(t)
|
||||||
args := []string{fmt.Sprintf("-hooks=%s", configPath), fmt.Sprintf("-ip=%s", ip), fmt.Sprintf("-port=%s", port), "-verbose"}
|
args := []string{fmt.Sprintf("-hooks=%s", configPath), fmt.Sprintf("-ip=%s", ip), fmt.Sprintf("-port=%s", port), "-verbose"}
|
||||||
|
|
||||||
@ -249,7 +248,7 @@ func serverAddress(t *testing.T) (string, string) {
|
|||||||
func waitForServerReady(t *testing.T, ip, port string) {
|
func waitForServerReady(t *testing.T, ip, port string) {
|
||||||
waitForServer(t,
|
waitForServer(t,
|
||||||
fmt.Sprintf("http://%v:%v/", ip, port),
|
fmt.Sprintf("http://%v:%v/", ip, port),
|
||||||
http.StatusNotFound,
|
http.StatusOK,
|
||||||
5*time.Second)
|
5*time.Second)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user