From f7b29483686d0bbd09af5410dd6fe8f695187361 Mon Sep 17 00:00:00 2001
From: Anthony Sottile <asottile@umich.edu>
Date: Wed, 15 Feb 2017 08:57:10 -0800
Subject: [PATCH] Rename some variables to be more like our internal state

---
 pre_commit/repository.py |  8 --------
 pre_commit/store.py      | 20 ++++++++++----------
 tests/repository_test.py |  7 -------
 3 files changed, 10 insertions(+), 25 deletions(-)

diff --git a/pre_commit/repository.py b/pre_commit/repository.py
index a8be521d..8ec6302b 100644
--- a/pre_commit/repository.py
+++ b/pre_commit/repository.py
@@ -51,10 +51,6 @@ class Repository(object):
     def repo_url(self):
         return self.repo_config['repo']
 
-    @cached_property
-    def sha(self):
-        return self.repo_config['sha']
-
     @cached_property
     def languages(self):
         return {
@@ -215,10 +211,6 @@ class LocalRepository(Repository):
     def cmd_runner(self):
         return PrefixedCommandRunner(git.get_root())
 
-    @cached_property
-    def sha(self):
-        raise NotImplementedError
-
     @cached_property
     def manifest(self):
         raise NotImplementedError
diff --git a/pre_commit/store.py b/pre_commit/store.py
index 8e7e9ddf..aecc7dc1 100644
--- a/pre_commit/store.py
+++ b/pre_commit/store.py
@@ -36,14 +36,14 @@ class Store(object):
     get_default_directory = staticmethod(_get_default_directory)
 
     class RepoPathGetter(object):
-        def __init__(self, repo, sha, store):
+        def __init__(self, repo, ref, store):
             self._repo = repo
-            self._sha = sha
+            self._ref = ref
             self._store = store
 
         @cached_property
         def repo_path(self):
-            return self._store.clone(self._repo, self._sha)
+            return self._store.clone(self._repo, self._ref)
 
     def __init__(self, directory=None):
         if directory is None:
@@ -97,15 +97,15 @@ class Store(object):
         self._create()
         self.__created = True
 
-    def clone(self, url, sha):
-        """Clone the given url and checkout the specific sha."""
+    def clone(self, url, ref):
+        """Clone the given url and checkout the specific ref."""
         self.require_created()
 
         # Check if we already exist
         with sqlite3.connect(self.db_path) as db:
             result = db.execute(
                 'SELECT path FROM repos WHERE repo = ? AND ref = ?',
-                [url, sha],
+                [url, ref],
             ).fetchone()
             if result:
                 return result[0]
@@ -118,18 +118,18 @@ class Store(object):
                 'git', 'clone', '--no-checkout', url, dir, env=no_git_env(),
             )
             with cwd(dir):
-                cmd_output('git', 'reset', sha, '--hard', env=no_git_env())
+                cmd_output('git', 'reset', ref, '--hard', env=no_git_env())
 
         # Update our db with the created repo
         with sqlite3.connect(self.db_path) as db:
             db.execute(
                 'INSERT INTO repos (repo, ref, path) VALUES (?, ?, ?)',
-                [url, sha, dir],
+                [url, ref, dir],
             )
         return dir
 
-    def get_repo_path_getter(self, repo, sha):
-        return self.RepoPathGetter(repo, sha, self)
+    def get_repo_path_getter(self, repo, ref):
+        return self.RepoPathGetter(repo, ref, self)
 
     @cached_property
     def cmd_runner(self):
diff --git a/tests/repository_test.py b/tests/repository_test.py
index c0cd572b..984973f5 100644
--- a/tests/repository_test.py
+++ b/tests/repository_test.py
@@ -458,11 +458,6 @@ def test_repo_url(mock_repo_config):
     assert repo.repo_url == 'git@github.com:pre-commit/pre-commit-hooks'
 
 
-def test_sha(mock_repo_config):
-    repo = Repository(mock_repo_config, None)
-    assert repo.sha == '5e713f8878b7d100c0e059f8cc34be4fc2e8f897'
-
-
 @pytest.mark.integration
 def test_languages(tempdir_factory, store):
     path = make_repo(tempdir_factory, 'python_hooks_repo')
@@ -684,8 +679,6 @@ def test_tags_on_repositories(in_tmpdir, tempdir_factory, store):
 def test_local_repository():
     config = config_with_local_hooks()
     local_repo = Repository.create(config, 'dummy')
-    with pytest.raises(NotImplementedError):
-        local_repo.sha
     with pytest.raises(NotImplementedError):
         local_repo.manifest
     assert len(local_repo.hooks) == 1