From 0aa3a22f71f8fb0adde0d76f1c206b1a97f4537b Mon Sep 17 00:00:00 2001 From: Matt Kulukundis Date: Mon, 15 Jul 2024 12:17:51 -0400 Subject: [PATCH] diff: add unit tests for copy tracking in the git backend --- lib/tests/runner.rs | 1 + lib/tests/test_copy_tracking.rs | 120 ++++++++++++++++++++++++++++++++ 2 files changed, 121 insertions(+) create mode 100644 lib/tests/test_copy_tracking.rs diff --git a/lib/tests/runner.rs b/lib/tests/runner.rs index 03f134bbde..145059b4f8 100644 --- a/lib/tests/runner.rs +++ b/lib/tests/runner.rs @@ -10,6 +10,7 @@ mod test_bad_locking; mod test_commit_builder; mod test_commit_concurrent; mod test_conflicts; +mod test_copy_tracking; mod test_default_revset_graph_iterator; mod test_diff_summary; mod test_git; diff --git a/lib/tests/test_copy_tracking.rs b/lib/tests/test_copy_tracking.rs new file mode 100644 index 0000000000..770deff3a1 --- /dev/null +++ b/lib/tests/test_copy_tracking.rs @@ -0,0 +1,120 @@ +// Copyright 2024 The Jujutsu Authors +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// https://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +use std::collections::HashMap; + +use futures::executor::block_on_stream; +use jj_lib::backend::{Backend, CommitId, CopySource, CopySources}; +use jj_lib::commit::Commit; +use jj_lib::git_backend::GitBackend; +use jj_lib::repo::Repo; +use jj_lib::repo_path::{RepoPath, RepoPathBuf}; +use jj_lib::settings::UserSettings; +use jj_lib::transaction::Transaction; +use testutils::{create_tree, TestRepo, TestRepoBackend}; + +fn get_copy_records( + backend: &GitBackend, + paths: &[RepoPathBuf], + a: &Commit, + b: &Commit, +) -> HashMap> { + let stream = backend + .get_copy_records(paths, &[a.id().clone()], &[b.id().clone()]) + .unwrap(); + let mut res: HashMap> = HashMap::new(); + for copy_record in block_on_stream(stream).filter_map(|r| r.ok()) { + res.insert( + copy_record.target.as_internal_file_string().into(), + match copy_record.sources { + CopySources::Resolved(CopySource { path, .. }) => { + vec![path.as_internal_file_string().into()] + } + CopySources::Conflict(conflicting) => conflicting + .iter() + .map(|s| s.path.as_internal_file_string().into()) + .collect(), + }, + ); + } + res +} + +fn make_commit( + tx: &mut Transaction, + settings: &UserSettings, + parents: Vec, + content: &[(&RepoPath, &str)], +) -> Commit { + let tree = create_tree(tx.base_repo(), content); + tx.mut_repo() + .new_commit(settings, parents, tree.id()) + .write() + .unwrap() +} + +#[test] +fn test_git_detection() { + let settings = testutils::user_settings(); + let test_repo = TestRepo::init_with_backend(TestRepoBackend::Git); + let repo = &test_repo.repo; + + let paths = &[ + RepoPathBuf::from_internal_string("file0"), + RepoPathBuf::from_internal_string("file1"), + RepoPathBuf::from_internal_string("file2"), + ]; + + let mut tx = repo.start_transaction(&settings); + let commit_a = make_commit( + &mut tx, + &settings, + vec![repo.store().root_commit_id().clone()], + &[(&paths[0], "content")], + ); + let commit_b = make_commit( + &mut tx, + &settings, + vec![commit_a.id().clone()], + &[(&paths[1], "content")], + ); + let commit_c = make_commit( + &mut tx, + &settings, + vec![commit_b.id().clone()], + &[(&paths[2], "content")], + ); + + let backend: &GitBackend = repo.store().backend_impl().downcast_ref().unwrap(); + assert_eq!( + get_copy_records(backend, paths, &commit_a, &commit_b), + HashMap::from([("file1".to_string(), vec!["file0".to_string()])]) + ); + assert_eq!( + get_copy_records(backend, paths, &commit_b, &commit_c), + HashMap::from([("file2".to_string(), vec!["file1".to_string()])]) + ); + assert_eq!( + get_copy_records(backend, paths, &commit_a, &commit_c), + HashMap::from([("file2".to_string(), vec!["file0".to_string()])]) + ); + assert_eq!( + get_copy_records(backend, &[], &commit_a, &commit_c), + HashMap::default(), + ); + assert_eq!( + get_copy_records(backend, paths, &commit_c, &commit_c), + HashMap::default(), + ); +}