Fixing request for merges testing issues

- made request_fetch use read only remotes so it does not break in travis
- upated cassette for request list so namespace and repo path are differenciated

Author: guyzmo

git_repo/services/ext/github.py

@@ -85,7 +85,7 @@ def request_list(self, user, repo):
         for pull in repository.iter_pulls():
             yield ( pull.number, pull.title, pull.links['issue'] )
 
-    def request_fetch(self, user, repo, request, pull=False): #pragma: no cover
+    def request_fetch(self, user, repo, request, pull=False):
         if pull:
             raise NotImplementedError('Pull operation on requests for merge are not yet supported')
         log.info('remotes: {}'.format(self.repository.remotes))

tests/helpers.py

@@ -450,7 +450,7 @@ def action_request_list(self, cassette_name, namespace, repository, rq_list_data
     def action_request_fetch(self, cassette_name, namespace, repository, request, pull=False):
         with self.recorder.use_cassette('_'.join(['test', self.service.name, cassette_name])):
             self.service.connect()
-            self.service.clone(namespace, repository)
+            self.service.clone(namespace, repository, rw=False)
             self.service.request_fetch(repository, namespace, request)
             assert self.repository.branches[-1].name == 'request-{}'.format(request)