Implement snapshot::create_tree() with worktree snapshot supports

Author: Byron

crates/but-workspace/src/snapshot.rs

@@ -3,13 +3,14 @@
 ///
 pub mod create_tree {
     use bstr::BString;
+    use but_graph::VirtualBranchesTomlMetadata;
 
     /// A way to determine what should be included in the snapshot when calling [create_tree()](function::create_tree).
-    pub struct State<'a> {
+    pub struct State {
         /// The result of a previous worktree changes call.
         ///
         /// It contains detailed information about the complete set of possible changes to become part of the worktree.
-        pub changes: &'a but_core::WorktreeChanges,
+        pub changes: but_core::WorktreeChanges,
         /// Repository-relative and slash-separated paths that match any change in the  [`changes`](State::changes) field.
         /// **It's an error if there is no match.** as there is not supposed to be a snapshot without a change to the working tree.
         pub selection: Vec<BString>,
@@ -40,6 +41,14 @@ pub mod create_tree {
         pub metadata: Option<gix::ObjectId>,
     }
 
+    /// A utility to more easily use *no* workspace or metadata.
+    pub fn no_workspace_and_meta() -> Option<(
+        &'static but_graph::projection::Workspace<'static>,
+        &'static VirtualBranchesTomlMetadata,
+    )> {
+        None
+    }
+
     pub(super) mod function {
         use super::{Outcome, State};
         use but_core::RefMetadata;

crates/but-workspace/tests/workspace/main.rs

@@ -5,6 +5,7 @@ mod branch_details;
 mod commit_engine;
 mod flatten_diff_specs;
 mod ref_info;
+mod snapshot;
 mod tree_manipulation;
 mod ui;
 

crates/but-workspace/tests/workspace/snapshot.rs

@@ -0,0 +1,26 @@
+mod create_tree {
+    use crate::utils::read_only_in_memory_scenario;
+    use but_graph::VirtualBranchesTomlMetadata;
+    use but_workspace::snapshot;
+
+    #[test]
+    fn worktree_all_filetypes() -> anyhow::Result<()> {
+        let repo = read_only_in_memory_scenario("all-file-types-changed")?;
+        let (head_tree_id, state, no_workspace_and_meta) = args_for_worktree_changes(&repo)?;
+        snapshot::create_tree(head_tree_id, state, no_workspace_and_meta)?;
+        Ok(())
+    }
+
+    fn args_for_worktree_changes(
+        repo: &gix::Repository,
+    ) -> anyhow::Result<(
+        gix::Id<'_>,
+        snapshot::create_tree::State,
+        Option<(
+            &'static but_graph::projection::Workspace<'static>,
+            &'static VirtualBranchesTomlMetadata,
+        )>,
+    )> {
+        todo!()
+    }
+}