napalm-automation / napalm-ios

Apache License 2.0
31 stars 40 forks source link

Fixing issue with lldp neighbors detail and padding #84

Closed ktbyers closed 7 years ago

ktbyers commented 7 years ago

The padding function was broken.

Also it is better to just generate an exception in this case (as it indicates something went wrong in the parsing).

Also add support for interface argument per getter definition.

ktbyers commented 7 years ago

See issue #49

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.2%) to 69.282% when pulling 9ceb34003279077383902363ea4efc659f01d661 on ktbyers:devel into 796288d4702c02054f526c964bb827a8c4af4417 on napalm-automation:develop.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.03%) to 69.011% when pulling 4e319f31b5b19378a448aef6b7e9f61c78bca220 on ktbyers:devel into 796288d4702c02054f526c964bb827a8c4af4417 on napalm-automation:develop.

dbarrosop commented 7 years ago

+1