working copy: rename working_copy_path() to just path()

It seems pretty clear from the context. Turns out we only use the function in a test case. Maybe we don't even need it. It's easy to provide it, though.
jj-vcs · Oct 12, 2023 · 0aa5f1a · 0aa5f1a
1 parent 9e43207
commit 0aa5f1a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/lib/src/local_working_copy.rs b/lib/src/local_working_copy.rs
@@ -1387,7 +1387,7 @@ impl WorkingCopy for LocalWorkingCopy {
         "local"
     }
 
-    fn working_copy_path(&self) -> &Path {
+    fn path(&self) -> &Path {
         &self.working_copy_path
     }
 

diff --git a/lib/src/working_copy.rs b/lib/src/working_copy.rs
@@ -30,7 +30,7 @@ pub trait WorkingCopy {
     fn name(&self) -> &str;
 
     /// The working copy's root directory.
-    fn working_copy_path(&self) -> &Path;
+    fn path(&self) -> &Path;
 
     /// The working copy's workspace ID.
     fn workspace_id(&self) -> &WorkspaceId;

diff --git a/lib/tests/test_local_working_copy_sparse.rs b/lib/tests/test_local_working_copy_sparse.rs
@@ -88,7 +88,7 @@ fn test_sparse_checkout() {
     // Reload the state to check that it was persisted
     let mut wc = LocalWorkingCopy::load(
         repo.store().clone(),
-        wc.working_copy_path().to_path_buf(),
+        wc.path().to_path_buf(),
         wc.state_path().to_path_buf(),
     );
     assert_eq!(
-Original file line number
+Diff line change
@@ Expand Up / @@ -1387,7 +1387,7 @@ impl WorkingCopy for LocalWorkingCopy { @@
             "local"
         }
-        fn working_copy_path(&self) -> &Path {
+        fn path(&self) -> &Path {
             &self.working_copy_path
         }
@@ Expand Down @@