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

Hardware team - Expand LAN Functionality and add automated testing #154

Merged
merged 36 commits into from Mar 26, 2020

Conversation

@ab7289
Copy link
Contributor

ab7289 commented Mar 23, 2020

This PR expands the existing LAN server/client functionality as well as begins to add in automated tests for the hardware portions.

The overall project code coverage total drops in this PR, but that is due to the fact that the hardware code was not being included in the code coverage calculation previously. We are continuously adding tests to re-expand the code coverage.

Additionally this PR moves the flake8 and coverage configurations to config files (.flake8 and .coveragerc, respectively) so that developers do not need to remember the exact flags needed to pass into flake8 or python test. The whole team will now be testing with the same configuration, reducing the chances that bugs slip through due to inconsistencies between local tests and travis tests.

PrabhanshuAttri and others added 30 commits Mar 18, 2020
…whether we're running against django or hardware
Added LAN client and logging to LAN Server
ab7289 and others added 5 commits Mar 21, 2020
…tatements in get_port
Feature/hardware tests
Sync `hardware-team` with `master`
@ab7289 ab7289 merged commit 432dfd1 into master Mar 26, 2020
2 of 3 checks passed
2 of 3 checks passed
coverage/coveralls Coverage decreased (-14.6%) to 84.192%
Details
Travis CI - Branch Build Passed
Details
Travis CI - Pull Request Build Passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.