fix tests

Author: pietroalbini

highfive/newpr.py

@@ -55,11 +55,11 @@ def __init__(self, payload, config, config_dir=None):
 
         self.repo_config = self.load_repo_config(config_dir)
 
-    def load_repo_config(self, config_dir):
+    def load_repo_config(self, config_dir=None):
         '''Load the repository configuration.'''
         (org, repo) = self.payload['repository', 'full_name'].split('/')
         try:
-            return self._load_json_file(config_dir, os.path.join(org, repo) + '.json')
+            return self._load_json_file(os.path.join(org, repo) + '.json', config_dir)
         except IOError:
             raise UnsupportedRepoError
 
@@ -75,7 +75,7 @@ def run(self, event):
         else:
             return 'Unsupported webhook event.\n'
 
-    def _load_json_file(self, config_dir, name):
+    def _load_json_file(self, name, config_dir=None):
         if not config_dir:
             config_dir = os.path.join(os.path.dirname(__file__), 'configs')
 

highfive/tests/test_newpr.py

@@ -84,7 +84,7 @@ def test_load_repo_config_supported(self, mock_load_json_file):
         m.stop_patchers()
         assert m.handler.load_repo_config() == {'a': 'config!'}
         mock_load_json_file.assert_called_once_with(
-            os.path.join('foo', 'blah.json')
+            os.path.join('foo', 'blah.json'), None,
         )
 
     @mock.patch('highfive.newpr.HighfiveHandler._load_json_file')
@@ -99,7 +99,7 @@ def test_load_repo_config_unsupported(self, mock_load_json_file):
         with pytest.raises(newpr.UnsupportedRepoError):
             m.handler.load_repo_config()
         mock_load_json_file.assert_called_once_with(
-            os.path.join('foo', 'blah.json')
+            os.path.join('foo', 'blah.json'), None
         )
 
 class TestNewPRGeneral(TestNewPR):