Fix default loading of config

Author: Muscaw

gitea_github_sync/github.py

@@ -2,7 +2,7 @@
 
 from dataclasses import dataclass
 from enum import Flag, auto
-from typing import List
+from typing import List, Optional
 
 from github import Github
 
@@ -36,7 +36,9 @@ def get_repo_name(self) -> str:
         return self.full_repo_name.split("/")[1]
 
 
-def get_github(conf: config.Config = config.load_config()) -> Github:
+def get_github(conf: Optional[config.Config] = None) -> Github:
+    if conf is None:
+        conf = config.load_config()
     return Github(login_or_token=conf.github_token)
 
 

tests/test_github.py

@@ -71,13 +71,17 @@ def test_github(mock_github: MagicMock, conf_fixture: Config) -> None:
     mock_github.assert_called_once_with(conf_fixture.github_token)
 
 
+@patch("gitea_github_sync.github.config.load_config", autospec=True)
 @patch("gitea_github_sync.github.Github", autospec=True)
-def test_github_default_value(mock_github: MagicMock, conf_fixture: Config) -> None:
-    get_github.__defaults__ = (conf_fixture,)
+def test_github_default_value(
+    mock_github: MagicMock, mock_load_config: MagicMock, conf_fixture: Config
+) -> None:
+    mock_load_config.return_value = conf_fixture
     gh = get_github()
 
     assert gh == mock_github.return_value
     mock_github.assert_called_once_with(conf_fixture.github_token)
+    mock_load_config.assert_called_once()
 
 
 @dataclass(frozen=True)