Comment 2 for bug 450629

Revision history for this message
Thomas Herve (therve) wrote :

[1]
+ def completed(result):
+ self.fail()
+
+ def aborted(failure):
+ self.assertEquals(fetched_urls, ["http://right/", "http://wrong/"])
+
+ d.addCallback(completed)
+ d.addErrback(aborted)
+ return d

You should use assertFailure in this case: you can remove the self.fail(), and add aborted as a callback.

[2]
+ L{fetch_to_files} fetches a list of URLs and save they're content

Typo: their content

[3] Pyflakes:
landscape/package/tests/test_taskhandler.py:7: 'CommandError' imported but unused

+1!