Cache branch descriptions in the store

Author: kaste

core/git_mixins/branches.py

@@ -139,6 +139,7 @@ def fetch_branch_description_subjects(self):
 
             branch_name, description = match.group(1), match.group(2)
             rv[branch_name] = description
+        store.update_state(self.repo_path, {"descriptions": rv})
         return rv
 
     def _parse_branch_line(self, line):

core/interfaces/branch.py

@@ -166,6 +166,7 @@ def refresh_view_state(self):
             'git_root': self.short_repo_path,
             'branches': state.get("branches", []),
             'recent_commits': state.get("recent_commits", []),
+            'descriptions': state.get("descriptions", {}),
             'show_help': not self.view.settings().get("git_savvy.help_hidden"),
         })
 
@@ -221,11 +222,14 @@ def on_status_update(self, _repo_path, state):
             new_state = {}
         new_state["branches"] = state.get("branches")
         new_state["recent_commits"] = state.get("recent_commits")
+        new_state["descriptions"] = state.get("descriptions")
         self.update_state(new_state, then=self.just_render)
 
     def on_create(self):
         self._unsubscribe = store.subscribe(
-            self.repo_path, {"status", "branches", "recent_commits"}, self.on_status_update
+            self.repo_path,
+            {"status", "branches", "recent_commits", "descriptions"},
+            self.on_status_update
         )
 
     def on_close(self):

core/store.py

@@ -32,6 +32,7 @@
             "short_hash_length": int,
             "stashes": List[Stash],
             "recent_commits": List[Commit],
+            "descriptions": Dict[str, str],
         },
         total=False
     )

tests/test_fetch_branch_descriptions.py

@@ -4,7 +4,7 @@
 from GitSavvy.tests.mockito import when
 from GitSavvy.tests.parameterized import parameterized as p
 
-from GitSavvy.core.git_mixins.branches import BranchesMixin
+from GitSavvy.core.git_command import GitCommand
 
 
 examples = [
@@ -32,6 +32,7 @@
 class TestFetchBranchDescriptions(DeferrableTestCase):
     @p.expand(examples)
     def test_description_subjects(self, git_output, expected):
-        test = BranchesMixin()
+        test = GitCommand()
+        when(test).get_repo_path().thenReturn("probably/here")
         when(test, strict=False).git("config", ...).thenReturn(git_output)
         self.assertEqual(expected, test.fetch_branch_description_subjects())