Modified infinite value of optical powers from -40 to -100 and added another unit test

napalm-automation / napalm-ios

Apache License 2.0

31 stars 40 forks source link

Modified infinite value of optical powers from -40 to -100 and added another unit test #139

Closed kaage closed 7 years ago

kaage commented 7 years ago

This was made because of current infinite value of -40.0 (or 0.0 in some cases) can be completely valid value for optical power. -100.0 is far enough for anyone to mistakenly think as valid value.

coveralls commented 7 years ago

Coverage remained the same at 67.33% when pulling 542c3572ac5d0a269c76895c8c727e0ca7109792 on kaage:develop into c069f5499658b7a8e01fcb7f53136ed9c50a6c48 on napalm-automation:develop.

coveralls commented 7 years ago

Coverage increased (+0.09%) to 67.422% when pulling 515c2d4f0335bf6acf44bd3426075818054f9c21 on kaage:develop into c069f5499658b7a8e01fcb7f53136ed9c50a6c48 on napalm-automation:develop.