Code review comment for lp:~salgado/launchpad/remove-security-upload-policy

Revision history for this message
Guilherme Salgado (salgado) wrote :

I've managed to get that test passing again, as well as fixing another one that I broke because it was importing its base class from a file that had been removed.

In the latter case I just had to copy the class from the removed file back.

For the former I removed a chunk of the test but to make sure we don't lose coverage I'll write a unit test on my vostok-upload-policy, which adds a test_uploadpolicy.py file. The rest should be self-explanatory.

« Back to merge proposal