bskinn / stdio-mgr

Context manager for mocking/wrapping stdin/stdout/stderr
MIT License
14 stars 4 forks source link

DEV: Remove ._buf on RandomTextIO #72

Closed bskinn closed 5 years ago

bskinn commented 5 years ago

No longer needed now that the callback method for persisting the underlying stream contents is implemented.

The two test asserts for ._buf have to just be removed, since direct access to the underlying buffer is no longer available, and thus there is nothing to compare the detached f to.

Closes #67.

codecov-io commented 5 years ago

Codecov Report

Merging #72 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #72   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           3      3           
  Lines         101    100    -1     
=====================================
- Hits          101    100    -1
Impacted Files Coverage Δ
src/stdio_mgr/stdio_mgr.py 100% <100%> (ø) :arrow_up:

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 ec9f50f...e393d66. Read the comment docs.