Closed sonelu closed 4 years ago
Merging #27 into master will increase coverage by
2.66%
. The diff coverage is82.56%
.
@@ Coverage Diff @@
## master #27 +/- ##
==========================================
+ Coverage 86.77% 89.43% +2.66%
==========================================
Files 21 23 +2
Lines 1308 1467 +159
Branches 181 189 +8
==========================================
+ Hits 1135 1312 +177
+ Misses 134 117 -17
+ Partials 39 38 -1
Impacted Files | Coverage Δ | |
---|---|---|
roboglia/base/register.py | 100.00% <ø> (ø) |
|
roboglia/dynamixel/bus.py | 86.36% <52.63%> (-0.28%) |
:arrow_down: |
roboglia/i2c/sync.py | 74.41% <74.41%> (ø) |
|
roboglia/base/sync.py | 82.02% <82.35%> (+4.99%) |
:arrow_up: |
roboglia/i2c/bus.py | 86.13% <85.27%> (+86.13%) |
:arrow_up: |
roboglia/base/robot.py | 98.70% <93.75%> (+1.28%) |
:arrow_up: |
roboglia/base/bus.py | 91.66% <95.65%> (-0.65%) |
:arrow_down: |
roboglia/base/__init__.py | 100.00% <100.00%> (ø) |
|
roboglia/base/device.py | 100.00% <100.00%> (+3.27%) |
:arrow_up: |
roboglia/base/thread.py | 86.29% <100.00%> (+0.92%) |
:arrow_up: |
... and 10 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 4832c5b...5c1ca33. Read the comment docs.
Includes:
Tests updated to cover the additional function.
Also the SharedBus is now simplified and no longer using multiple inheritance. It is possible to switch from the robot definition to a mock bus by using 'mock: True' for DynamixeBus and I2CBus (or the Shared equivalents).