This might be more comprehensible if it were written as single statements (though I can appreciate that that might be a bit less elegant), purely from a "don't make me think about it" point of view. Also, if it's a series of single asserts, it's likely to be simpler to unwind when something goes wrong.
Hi Jeroen,
Excellent branch this; I don't have any questions other than:
112 + self.assertEqual([ ngStatus. SUPERSEDED, ngStatus. SUPERSEDED, ngStatus. PUBLISHED, ngStatus. DELETED, ngStatus. PENDING,
113 + PackagePublishi
114 + PackagePublishi
115 + PackagePublishi
116 + PackagePublishi
117 + PackagePublishi
118 + ],
119 + [pub.status for pub in spphs])
This might be more comprehensible if it were written as single statements (though I can appreciate that that might be a bit less elegant), purely from a "don't make me think about it" point of view. Also, if it's a series of single asserts, it's likely to be simpler to unwind when something goes wrong.