Maria,

Thank you so much for these :)

> Added some tests for APK comparator, see attached patch.

Unfortunately I'm seeing:

differences = [<Difference zipinfo {} -- zipinfo {} []>, <Difference APK 
metadata -- APK metadata []>, <Difference AndroidManifest.xml -- 
AndroidManifest.xml []>, <Difference original/AndroidManifest.xml -- 
original/AndroidManifest.xml []>]

    @skip_unless_tools_exist('apktool', 'zipinfo')
    def test_android_manifest(differences):
>       assert differences[1].source1 == 'AndroidManifest.xml'
E       assert 'APK metadata' == 'AndroidManifest.xml'
E         - APK metadata
E         + AndroidManifest.xml

Any ideas?

> However, please keep in mind apktool takes a lot of time to run even on toy
> examples

Whilst I am not a fan of it slowing down this much, I think I'd
prefer to see some tests in there (that we can speed up later)
rather than none.


Regards,

-- 
      ,''`.
     : :'  :     Chris Lamb
     `. `'`      la...@debian.org / chris-lamb.co.uk
       `-

_______________________________________________
Reproducible-builds mailing list
Reproducible-builds@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/reproducible-builds

Reply via email to