mirror of
https://github.com/martinvonz/jj.git
synced 2024-10-23 06:57:32 +00:00
8d090628c3
We had both `repo()` and `mut_repo()` on `Transaction` and I think it was easy to get confused and think that the former returned a `&ReadonlyRepo` but both of them actually return a reference to `MutableRepo` (the latter obviously returns a mutable reference). I hope that renaming to the more idiomatic `repo_mut()` will help clarify. We could instead have renamed them to `mut_repo()` and `mut_repo_mut()` but that seemed unnecessarily long. It would better match the `mut_repo` variables we typically use, though.
55 lines
1.8 KiB
Rust
55 lines
1.8 KiB
Rust
// Copyright 2021 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 jj_lib::repo::RepoLoader;
|
|
use testutils::write_random_commit;
|
|
use testutils::TestRepo;
|
|
|
|
#[test]
|
|
fn test_load_at_operation() {
|
|
let settings = testutils::user_settings();
|
|
let test_repo = TestRepo::init();
|
|
let repo = &test_repo.repo;
|
|
|
|
let mut tx = repo.start_transaction(&settings);
|
|
let commit = write_random_commit(tx.repo_mut(), &settings);
|
|
let repo = tx.commit("add commit");
|
|
|
|
let mut tx = repo.start_transaction(&settings);
|
|
tx.repo_mut().remove_head(commit.id());
|
|
tx.commit("remove commit");
|
|
|
|
// If we load the repo at head, we should not see the commit since it was
|
|
// removed
|
|
let loader = RepoLoader::init(
|
|
&settings,
|
|
repo.repo_path(),
|
|
&TestRepo::default_store_factories(),
|
|
)
|
|
.unwrap();
|
|
let head_repo = loader.load_at_head(&settings).unwrap();
|
|
assert!(!head_repo.view().heads().contains(commit.id()));
|
|
|
|
// If we load the repo at the previous operation, we should see the commit since
|
|
// it has not been removed yet
|
|
let loader = RepoLoader::init(
|
|
&settings,
|
|
repo.repo_path(),
|
|
&TestRepo::default_store_factories(),
|
|
)
|
|
.unwrap();
|
|
let old_repo = loader.load_at(repo.operation()).unwrap();
|
|
assert!(old_repo.view().heads().contains(commit.id()));
|
|
}
|