review-tools

Code review comment for ~jslarraz/review-tools:work-on-test-cases

  1. Git
  2. lp:~jslarraz/review-tools
  3. work-on-test-cases
  4. Merge into master
Revision history for this message
Jorge Sancho Larraz (jslarraz) wrote (last edit ):

Move check for container format to BaseContainer initialization and properly mock required functions: https://git.launchpad.net/~jslarraz/review-tools/commit/?id=9477ee6500544e63ed46ea19ab1486c8f4cf268c

Remove unneeded exception handling on error cases from tests classes (makes code cleaner and improves coverage): https://git.launchpad.net/~jslarraz/review-tools/commit/?id=acb74aab8dab0c2d0eedb354adda7bb0ea10e295

Add missing tests case for missing extension on container file name and mock _unpack_dir: https://git.launchpad.net/~jslarraz/review-tools/commit/?id=037aefd2f9dadec3b3734b1b2f368df8cba569b2

« Back to merge proposal