Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Catch SlipLib Exception Per #62 #63

Merged
merged 7 commits into from
Mar 9, 2018
Merged

Conversation

kb1lqc
Copy link
Member

@kb1lqc kb1lqc commented Mar 9, 2018

This PR pulls in IT #62 which fixes an exception that was uncaught and found during Hardware In The Loop Testing.

  • Adds an exception for sliplib.slip.ProtocolError to receive()
  • Also adds in a few reminders for unit testing.

kb1lqc and others added 6 commits March 2, 2018 00:48
Pull in Lated Develop Branch For PyPi Installation Fix
Pypi doesn't like v0.0.2a and thinks it's older than v0.0.2 so I need to
move to v0.0.3
Changed to v0.0.3 for pypi packaging version operation
Updated to latest pytest 3.4.1 and revved to v0.0.4
…otocolError

* It was observed in use that an exception while decoding packets which
had SLIP framing errors was not being caught. This commit adds an
exception to do so.
* Also added some TODO notes for unit testing slip code.
@kb1lqc kb1lqc self-assigned this Mar 9, 2018
@coveralls
Copy link

coveralls commented Mar 9, 2018

Pull Request Test Coverage Report for Build 58

  • 2 of 6 (33.33%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.8%) to 67.045%

Changes Missing Coverage Covered Lines Changed/Added Lines %
faradayio/faraday.py 2 6 33.33%
Totals Coverage Status
Change from base Build 48: -2.8%
Covered Lines: 59
Relevant Lines: 88

💛 - Coveralls

@kb1lqc kb1lqc merged commit dcb270a into FaradayRF:develop Mar 9, 2018
@kb1lqc kb1lqc deleted the issue62 branch March 9, 2018 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants