search

fairdk / lcrs-embedded

LCRS embedded components - "slave" + embedded linux kernel
https://lcrs.fairdanmark.dk/
GNU General Public License v3.0
1 stars 3 forks source link

Models were not copying immutable types when instantiating #2

Closed benjaoming closed 7 years ago

benjaoming commented 7 years ago

Starting to work from my own repo and doing everything through PRs :)

This was the error that caused test breakage. Sorry about that!!

Added some tests to test_protocol to make it more evident why it happened.

codecov-io commented 7 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@22f8283). Click here to learn what that means. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   95.44%           
=========================================
  Files             ?       19           
  Lines             ?      768           
  Branches          ?        0           
=========================================
  Hits              ?      733           
  Misses            ?       35           
  Partials          ?        0
Impacted Files Coverage Δ
lcrs_embedded/system/dmesg.py 95.12% <ø> (ø)
lcrs_embedded/utils/decorators.py 90.41% <ø> (ø)
lcrs_embedded/utils/models.py 100% <100%> (ø)

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 22f8283...4f9a226. Read the comment docs.