From 899ee4e2c4a6d3abe0b236195ee0aa690cb9adac Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Tue, 19 Jun 2018 11:48:33 +0200 Subject: [PATCH] [IMP] runbot_merge: error-reporting during PR sync --- runbot_merge/github.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/runbot_merge/github.py b/runbot_merge/github.py index c499c7b6..e4fdb934 100644 --- a/runbot_merge/github.py +++ b/runbot_merge/github.py @@ -4,6 +4,7 @@ import logging import requests +from odoo.exceptions import UserError from . import exceptions _logger = logging.getLogger(__name__) @@ -115,14 +116,17 @@ class GH(object): cursor = None owner, name = self._repo.split('/') while True: - response = self._session.post('{}/graphql'.format(self._url), json={ + r = self._session.post('{}/graphql'.format(self._url), json={ 'query': PR_QUERY, 'variables': { 'owner': owner, 'name': name, 'cursor': cursor, } - }).json() + }) + response = r.json() + if 'data' not in response: + raise UserError('\n'.join(e['message'] for e in response.get('errors', map(str, [r.status_code, r.reason, r.text])))) result = response['data']['repository']['pullRequests'] for pr in result['nodes']: