MartijnBraam / python-isc-dhcp-leases / 114

99%

Pull #25

travis-ci

Handle Lease functionalities when 'starts' is missing

The existing code base expected each lease entry will have 'starts'
property and the code fails at:
https://github.com/MartijnBraam/python-isc-dhcp-leases/blob/master/isc_dhcp_leases/iscdhcpleases.py#L212
self.start = parse_time(properties['starts'])

But lease entry could be without 'starts', like in below 3 examples:
lease 10.37.178.69 {
  tstp 5 2018/03/30 18:24:01;
  tsfp 5 2018/03/30 18:24:01;
  atsfp 5 2018/03/30 18:24:01;
  cltt 5 2018/03/30 17:54:01;
  binding state free;
  hardware ethernet 8c:dc:d4:b2:ea:fc;
  set vendorclass = "PXEClient:Arch:00000:UNDI:002001";
  set ClientIP = "10.37.178.69";
  set ClientMac = "8c:dc:d4:b2:ea:fc";
  set Vendorid = "PXEClient";
}
lease 10.37.178.68 {
  tstp 4 2018/03/29 22:08:05;
  tsfp 4 2018/03/29 22:08:05;
  atsfp 4 2018/03/29 22:08:05;
  cltt 4 2018/03/29 21:38:05;
  binding state free;
  hardware ethernet 8c:dc:d4:b2:d0:a8;
  set vendorclass = "d-i";
}
lease 10.37.178.67 {
  tstp 4 2018/03/29 20:22:16;
  tsfp 4 2018/03/29 20:22:16;
  atsfp 4 2018/03/29 20:22:16;
  binding state free;
  hardware ethernet 8c:dc:d4:b2:d0:a8;
}

This patch fixes the assumption.

29 of 29 new or added lines in 2 files covered. (100.0%)

421 of 421 relevant lines covered (100.0%)

4.0 hits per line