Skip to content

Drop dependency on six and mock - #738

Open
tiltingpenguin wants to merge 10 commits into
python-escpos:masterfrom
tiltingpenguin:master
Open

Drop dependency on six and mock#738
tiltingpenguin wants to merge 10 commits into
python-escpos:masterfrom
tiltingpenguin:master

Conversation

@tiltingpenguin

Copy link
Copy Markdown

Description

Drop usage of six and switch mock over to the standardlibrary implementation

Tested with

Only ran the testsuite

@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.88889% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.04%. Comparing base (56058d8) to head (0fd69f3).

Files with missing lines Patch % Lines
src/escpos/escpos.py 87.50% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #738      +/-   ##
==========================================
- Coverage   79.07%   73.04%   -6.04%     
==========================================
  Files          21       21              
  Lines        1754     1751       -3     
  Branches      243      243              
==========================================
- Hits         1387     1279     -108     
- Misses        286      403     +117     
+ Partials       81       69      -12     
Flag Coverage Δ
unittests 73.04% <88.88%> (-6.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/escpos/constants.py 100.00% <100.00%> (ø)
src/escpos/magicencode.py 90.47% <ø> (-0.08%) ⬇️
src/escpos/escpos.py 78.37% <87.50%> (-0.05%) ⬇️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread src/escpos/escpos.py
Comment thread src/escpos/escpos.py Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants