From 5c860fdbf28128e7d11a9dff8b5c30777c2cbfeb Mon Sep 17 00:00:00 2001 From: Tom Clegg Date: Tue, 27 Jun 2017 13:43:25 -0400 Subject: [PATCH] 11901: Test 404 when health checks disabled. Arvados-DCO-1.1-Signed-off-by: Tom Clegg --- services/ws/server_test.go | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/services/ws/server_test.go b/services/ws/server_test.go index da7440d0e0..a0c4c8ccde 100644 --- a/services/ws/server_test.go +++ b/services/ws/server_test.go @@ -65,6 +65,7 @@ func (s *serverSuite) TestBadDB(c *check.C) { func (s *serverSuite) TestHealth(c *check.C) { go s.srv.Run() + defer s.srv.Close() s.srv.WaitReady() for _, token := range []string{"", "foo", s.cfg.ManagementToken} { req, err := http.NewRequest("GET", "http://"+s.srv.listener.Addr().String()+"/_health/ping", nil) @@ -84,3 +85,18 @@ func (s *serverSuite) TestHealth(c *check.C) { } } } + +func (s *serverSuite) TestHealthDisabled(c *check.C) { + s.cfg.ManagementToken = "" + + go s.srv.Run() + defer s.srv.Close() + s.srv.WaitReady() + + req, err := http.NewRequest("GET", "http://"+s.srv.listener.Addr().String()+"/_health/ping", nil) + c.Assert(err, check.IsNil) + req.Header.Add("Authorization", "Bearer "+arvadostest.ManagementToken) + resp, err := http.DefaultClient.Do(req) + c.Check(err, check.IsNil) + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) +} -- 2.30.2