dhcpd: add todo
This commit is contained in:
parent
638c4ce2af
commit
9a77f79869
|
@ -33,6 +33,8 @@ func normalizeIP(ip net.IP) net.IP {
|
|||
}
|
||||
|
||||
// Load lease table from DB
|
||||
//
|
||||
// TODO(s.chzhen): Decrease complexity.
|
||||
func (s *server) dbLoad() (err error) {
|
||||
dynLeases := []*Lease{}
|
||||
staticLeases := []*Lease{}
|
||||
|
|
|
@ -73,13 +73,14 @@ func TestServer_handleDHCPStatus(t *testing.T) {
|
|||
return resp
|
||||
}
|
||||
|
||||
t.Run("status", func(t *testing.T) {
|
||||
ok := t.Run("status", func(t *testing.T) {
|
||||
resp := defaultResponse()
|
||||
|
||||
checkStatus(t, resp)
|
||||
})
|
||||
require.True(t, ok)
|
||||
|
||||
t.Run("add_static_lease", func(t *testing.T) {
|
||||
ok = t.Run("add_static_lease", func(t *testing.T) {
|
||||
w := httptest.NewRecorder()
|
||||
|
||||
b := &bytes.Buffer{}
|
||||
|
@ -98,8 +99,9 @@ func TestServer_handleDHCPStatus(t *testing.T) {
|
|||
|
||||
checkStatus(t, resp)
|
||||
})
|
||||
require.True(t, ok)
|
||||
|
||||
t.Run("add_invalid_lease", func(t *testing.T) {
|
||||
ok = t.Run("add_invalid_lease", func(t *testing.T) {
|
||||
w := httptest.NewRecorder()
|
||||
|
||||
b := &bytes.Buffer{}
|
||||
|
@ -114,8 +116,9 @@ func TestServer_handleDHCPStatus(t *testing.T) {
|
|||
s.handleDHCPAddStaticLease(w, r)
|
||||
assert.Equal(t, http.StatusBadRequest, w.Code)
|
||||
})
|
||||
require.True(t, ok)
|
||||
|
||||
t.Run("remove_static_lease", func(t *testing.T) {
|
||||
ok = t.Run("remove_static_lease", func(t *testing.T) {
|
||||
w := httptest.NewRecorder()
|
||||
|
||||
b := &bytes.Buffer{}
|
||||
|
@ -133,8 +136,9 @@ func TestServer_handleDHCPStatus(t *testing.T) {
|
|||
|
||||
checkStatus(t, resp)
|
||||
})
|
||||
require.True(t, ok)
|
||||
|
||||
t.Run("set_config", func(t *testing.T) {
|
||||
ok = t.Run("set_config", func(t *testing.T) {
|
||||
w := httptest.NewRecorder()
|
||||
|
||||
resp := defaultResponse()
|
||||
|
@ -153,4 +157,5 @@ func TestServer_handleDHCPStatus(t *testing.T) {
|
|||
|
||||
checkStatus(t, resp)
|
||||
})
|
||||
require.True(t, ok)
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue