Was changing
lib/lp/soyuz/scripts/tests/upload_test_files/drdsl_1.2.0.orig.tar.gz intentional ?
The class definition of PublishingSetTests needs to be preceded by two empty lines.
There seems to be quite a bit of duplication in the various methods in PublishingSetTests, perhaps some of that code could be moved into a setUp() method.
I'm bothered by the size of the test factory and its lack of tests. Seeing it extended further worries me a bit, though I don't have any easy alternatives.
Hi James,
Thanks for the cleanups.
Was changing soyuz/scripts/ tests/upload_ test_files/ drdsl_1. 2.0.orig. tar.gz intentional ?
lib/lp/
The class definition of PublishingSetTests needs to be preceded by two empty lines.
There seems to be quite a bit of duplication in the various methods in PublishingSetTests, perhaps some of that code could be moved into a setUp() method.
I'm bothered by the size of the test factory and its lack of tests. Seeing it extended further worries me a bit, though I don't have any easy alternatives.