def build_dependency_graph(self): """Build the dependency graph for the review request. Only review requests in the same repository as this one will be in the graph. A ValueError is raised if the graph would contain cycles. """ def get_url(resource): """Get the URL of the resource.""" if hasattr(resource, 'href'): return resource.href else: return resource.absolute_url # Even with the API cache, we don't want to be making more requests # than necessary. The review request resource will be cached by an # ETag, so there will still be a round trip if we don't cache them # here. review_requests_by_url = {} review_requests_by_url[self.absolute_url] = self def get_review_request_resource(resource): url = get_url(resource) if url not in review_requests_by_url: review_requests_by_url[url] = resource.get(expand='repository') return review_requests_by_url[url] repository = self.get_repository() graph = defaultdict(set) visited = set() unvisited = deque() unvisited.append(self) while unvisited: head = unvisited.popleft() if head in visited: continue visited.add(get_url(head)) for tail in head.depends_on: tail = get_review_request_resource(tail) if path_exists(graph, tail.id, head.id): raise ValueError('Circular dependencies.') # We don't want to include review requests for other # repositories, so we'll stop if we reach one. We also don't # want to re-land submitted review requests. if (repository.id == tail.repository.id and tail.status != 'submitted'): graph[head].add(tail) unvisited.append(tail) graph.default_factory = None return graph
def build_dependency_graph(self): """Build the dependency graph for the review request. Only review requests in the same repository as this one will be in the graph. A ValueError is raised if the graph would contain cycles. """ def get_url(resource): """Get the URL of the resource.""" if hasattr(resource, 'href'): return resource.href else: return resource.absolute_url # Even with the API cache, we don't want to be making more requests # than necessary. The review request resource will be cached by an # ETag, so there will still be a round trip if we don't cache them # here. review_requests_by_url = {} review_requests_by_url[self.absolute_url] = self def get_review_request_resource(resource): url = get_url(resource) if url not in review_requests_by_url: review_requests_by_url[url] = resource.get(expand='repository') return review_requests_by_url[url] repository = self.get_repository() graph = defaultdict(set) visited = set() unvisited = deque() unvisited.append(self) while unvisited: head = unvisited.popleft() if head in visited: continue visited.add(get_url(head)) for tail in head.depends_on: tail = get_review_request_resource(tail) if path_exists(graph, tail.id, head.id): raise ValueError('Circular dependencies.') # We don't want to include review requests for other # repositories, so we'll stop if we reach one. We also don't # want to re-land submitted review requests. if (repository.id == tail.repository.id and tail.status != 'submitted'): graph[head].add(tail) unvisited.append(tail) graph.default_factory = None return graph