I see the new VCSHandler classes, but I don't see where they are used. Is this supposed to be followed by more merge requests that replace fetch_cmd with fetch_url and fetch_type?
Aside from this, I think my only comment is that you can replace None checks like this one:
self.assertTrue(bzr is not None)
@Joe
I see the new VCSHandler classes, but I don't see where they are used. Is this supposed to be followed by more merge requests that replace fetch_cmd with fetch_url and fetch_type?
Aside from this, I think my only comment is that you can replace None checks like this one:
self.assertTrue(bzr is not None)
with something like: tNone(bzr)
self.assertIsNo