PyEasyArchive icon indicating copy to clipboard operation
PyEasyArchive copied to clipboard

Add support for reading and writing encrypted archives with test suites

Open udgover opened this issue 5 years ago • 5 comments

udgover avatar Jun 02 '20 11:06 udgover

Pull Request Test Coverage Report for Build 50

  • 43 of 52 (82.69%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.5%) to 71.443%

Changes Missing Coverage Covered Lines Changed/Added Lines %
libarchive/adapters/archive_read.py 11 14 78.57%
libarchive/adapters/archive_write.py 15 21 71.43%
<!-- Total: 43 52
Totals Coverage Status
Change from base Build 47: 0.5%
Covered Lines: 708
Relevant Lines: 991

💛 - Coveralls

coveralls avatar Jun 02 '20 11:06 coveralls

I found the issue with Python 2.7, it's related to a call to bytes which works perfectly with Python3 but not Python2. Do I need to add version check to call the right function depending on Py2 or Py3? Do you still want to support Python2.7?

Based on your feedback, I'll commit the patch.

udgover avatar Jun 03 '20 09:06 udgover

Sorry for the previous commit which only fixed python2 and not python3. Now all tests are green!

udgover avatar Jun 04 '20 10:06 udgover

Did you have time to check this PR?

udgover avatar Jun 10 '20 15:06 udgover

Hello,

Any news to review this PR?

udgover avatar Sep 22 '20 09:09 udgover