napalm-automation / napalm-nxos

Apache License 2.0
9 stars 21 forks source link

Adding unit tests #107

Closed ktbyers closed 7 years ago

ktbyers commented 7 years ago

get_facts should pass the new unit tests

ktbyers commented 7 years ago

There might be a bug in this:

def _compute_timestamp(stupid_cisco_output):

I replaced it with what I used for Cisco IOS. I was getting a strange result from it (though need to look at it more closely). Should replace in nxos API code also (if it is broken).