// Copyright 2023 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 indoc::indoc;use regex::Regex; use crate::common::TestEnvironment; #[test]fn test_snapshot_large_file() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); // test a small file using raw-integer-literal syntax, which is interpreted // in bytes test_env.add_config(r#"snapshot.max-new-file-size = 10"#); work_dir.write_file("empty", ""); work_dir.write_file("large", "a lot of text"); let output = work_dir.run_jj(["file", "list"]); insta::assert_snapshot!(output, @" empty [EOF] ------- stderr ------- Warning: Refused to snapshot some files: large: 13.0B (13 bytes); the maximum size allowed is 10.0B (10 bytes) Hint: This is to prevent large files from being added by accident. To fix this: * Add the file(s) to `.gitignore` * Run `jj config set --repo snapshot.max-new-file-size 13` This will increase the maximum file size allowed for new files, in this repository only. * Run `jj --config snapshot.max-new-file-size=13 status` This will increase the maximum file size allowed for new files, for this command only. [EOF] "); // test with a larger file using 'KB' human-readable syntax test_env.add_config(r#"snapshot.max-new-file-size = "10KB""#); let big_string = vec![0; 1024 * 11]; work_dir.write_file("large", &big_string); let output = work_dir.run_jj(["file", "list"]); insta::assert_snapshot!(output, @" empty [EOF] ------- stderr ------- Warning: Refused to snapshot some files: large: 11.0KiB (11264 bytes); the maximum size allowed is 10.0KiB (10240 bytes) Hint: This is to prevent large files from being added by accident. To fix this: * Add the file(s) to `.gitignore` * Run `jj config set --repo snapshot.max-new-file-size 11264` This will increase the maximum file size allowed for new files, in this repository only. * Run `jj --config snapshot.max-new-file-size=11264 status` This will increase the maximum file size allowed for new files, for this command only. [EOF] "); // test with file track for hint formatting, both files should appear in // warnings even though they were snapshotted separately work_dir.write_file("large 2", big_string); let output = work_dir.run_jj([ "file", "--config=snapshot.auto-track='large'", "track", "large 2", ]); insta::assert_snapshot!(output, @" ------- stderr ------- Warning: Refused to snapshot some files: large: 11.0KiB (11264 bytes); the maximum size allowed is 10.0KiB (10240 bytes) large 2: 11.0KiB (11264 bytes); the maximum size allowed is 10.0KiB (10240 bytes) Hint: This is to prevent large files from being added by accident. To fix this: * Add the file(s) to `.gitignore` * Run `jj config set --repo snapshot.max-new-file-size 11264` This will increase the maximum file size allowed for new files, in this repository only. * Run `jj --config snapshot.max-new-file-size=11264 file track large 'large 2'` This will increase the maximum file size allowed for new files, for this command only. * Run `jj file track --include-ignored large 'large 2'` This will track the file(s) regardless of size. [EOF] "); // test invalid configuration let output = work_dir.run_jj(["file", "list", "--config=snapshot.max-new-file-size=[]"]); insta::assert_snapshot!(output, @" ------- stderr ------- Config error: Invalid type or value for snapshot.max-new-file-size Caused by: Expected a positive integer or a string in '<number><unit>' form For help, see https://docs.jj-vcs.dev/latest/config/ or use `jj help -k config`. [EOF] [exit status: 1] "); // No error if we disable auto-tracking of the path let output = work_dir.run_jj(["file", "list", "--config=snapshot.auto-track='none()'"]); insta::assert_snapshot!(output, @" empty [EOF] "); // max-new-file-size=0 means no limit let output = work_dir.run_jj(["file", "list", "--config=snapshot.max-new-file-size=0"]); insta::assert_snapshot!(output, @" empty large large 2 [EOF] ");} #[test]fn test_snapshot_large_file_restore() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); test_env.add_config("snapshot.max-new-file-size = 10"); work_dir.run_jj(["describe", "-mcommitted"]).success(); work_dir.write_file("file", "small"); // Write a large file in the working copy, restore it from a commit. The // working-copy content shouldn't be overwritten. work_dir.run_jj(["new", "root()"]).success(); work_dir.write_file("file", "a lot of text"); let output = work_dir.run_jj(["restore", "--from=subject(committed)"]); insta::assert_snapshot!(output, @" ------- stderr ------- Warning: Refused to snapshot some files: file: 13.0B (13 bytes); the maximum size allowed is 10.0B (10 bytes) Hint: This is to prevent large files from being added by accident. To fix this: * Add the file(s) to `.gitignore` * Run `jj config set --repo snapshot.max-new-file-size 13` This will increase the maximum file size allowed for new files, in this repository only. * Run `jj --config snapshot.max-new-file-size=13 status` This will increase the maximum file size allowed for new files, for this command only. Working copy (@) now at: kkmpptxz 119f5156 (no description set) Parent commit (@-) : zzzzzzzz 00000000 (empty) (no description set) Added 1 files, modified 0 files, removed 0 files Warning: 1 of those updates were skipped because there were conflicting changes in the working copy. Hint: Inspect the changes compared to the intended target with `jj diff --from 119f5156d330`. Discard the conflicting changes with `jj restore --from 119f5156d330`. [EOF] "); insta::assert_snapshot!(work_dir.read_file("file"), @"a lot of text"); // However, the next command will snapshot the large file because it is now // tracked. TODO: Should we remember the untracked state? let output = work_dir.run_jj(["status"]); insta::assert_snapshot!(output, @" Working copy changes: A file Working copy (@) : kkmpptxz 09eba65e (no description set) Parent commit (@-): zzzzzzzz 00000000 (empty) (no description set) [EOF] ");} #[test]fn test_materialize_and_snapshot_different_conflict_markers() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); // Configure to use Git-style conflict markers test_env.add_config(r#"ui.conflict-marker-style = "git""#); // Create a conflict in the working copy work_dir.write_file( "file", indoc! {" line 1 line 2 line 3 "}, ); work_dir.run_jj(["commit", "-m", "base"]).success(); work_dir.write_file( "file", indoc! {" line 1 line 2 - a line 3 "}, ); work_dir.run_jj(["commit", "-m", "side-a"]).success(); work_dir .run_jj(["new", "subject(base)", "-m", "side-b"]) .success(); work_dir.write_file( "file", indoc! {" line 1 line 2 - b line 3 - b "}, ); work_dir .run_jj(["new", "subject(side-a)", "subject(side-b)"]) .success(); // File should have Git-style conflict markers insta::assert_snapshot!(work_dir.read_file("file"), @r#" line 1 <<<<<<< rlvkpnrz df1cdd77 "side-a" line 2 - a line 3 ||||||| qpvuntsm 2205b3ac "base" line 2 line 3 ======= line 2 - b line 3 - b >>>>>>> zsuskuln 68dcce1b "side-b" "#); // Configure to use JJ-style "snapshot" conflict markers test_env.add_config(r#"ui.conflict-marker-style = "snapshot""#); // Update the conflict, still using Git-style conflict markers work_dir.write_file( "file", indoc! {" line 1 <<<<<<< line 2 - a line 3 - a ||||||| line 2 line 3 ======= line 2 - b line 3 - b >>>>>>> "}, ); // Git-style markers should be parsed, then rendered with new config insta::assert_snapshot!(work_dir.run_jj(["diff", "--git"]), @r#" diff --git a/file b/file --- a/file +++ b/file @@ -2,7 +2,7 @@ <<<<<<< conflict 1 of 1 +++++++ rlvkpnrz df1cdd77 "side-a" line 2 - a -line 3 +line 3 - a ------- qpvuntsm 2205b3ac "base" line 2 line 3 [EOF] "#);} #[test]fn test_snapshot_invalid_ignore_pattern() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); // Test invalid pattern in .gitignore work_dir.write_file(".gitignore", " []\n"); insta::assert_snapshot!(work_dir.run_jj(["st"]), @" Working copy changes: A .gitignore Working copy (@) : qpvuntsm c9cf4826 (no description set) Parent commit (@-): zzzzzzzz 00000000 (empty) (no description set) [EOF] "); // Test invalid UTF-8 in .gitignore work_dir.write_file(".gitignore", b"\xff\n"); insta::assert_snapshot!(work_dir.run_jj(["st"]), @" ------- stderr ------- Internal error: Failed to snapshot the working copy Caused by: 1: Invalid UTF-8 for ignore pattern in $TEST_ENV/repo/.gitignore on line #1: � 2: invalid utf-8 sequence of 1 bytes from index 0 [EOF] [exit status: 255] ");} #[test]fn test_conflict_marker_length_stored_in_working_copy() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); // Create a conflict in the working copy with long markers on one side work_dir.write_file( "file", indoc! {" line 1 line 2 line 3 "}, ); work_dir.run_jj(["commit", "-m", "base"]).success(); work_dir.write_file( "file", indoc! {" line 1 line 2 - left line 3 - left "}, ); work_dir.run_jj(["commit", "-m", "side-a"]).success(); work_dir .run_jj(["new", "subject(base)", "-m", "side-b"]) .success(); work_dir.write_file( "file", indoc! {" line 1 ======= fake marker line 2 - right ======= fake marker line 3 "}, ); work_dir .run_jj(["new", "subject(side-a)", "subject(side-b)"]) .success(); // File should be materialized with long conflict markers insta::assert_snapshot!(work_dir.read_file("file"), @r#" line 1 <<<<<<<<<<< conflict 1 of 1 %%%%%%%%%%% diff from: qpvuntsm 2205b3ac "base" \\\\\\\\\\\ to: rlvkpnrz ccf9527c "side-a" -line 2 -line 3 +line 2 - left +line 3 - left +++++++++++ zsuskuln d7acaf48 "side-b" ======= fake marker line 2 - right ======= fake marker line 3 >>>>>>>>>>> conflict 1 of 1 ends "#); // The timestamps in the `jj debug local-working-copy` output change, so we want // to remove them before asserting the snapshot let timestamp_regex = Regex::new(r"\b\d{10,}\b").unwrap(); let redact_output = |output: String| { let output = timestamp_regex.replace_all(&output, "<timestamp>"); output.into_owned() }; // Working copy should contain conflict marker length let output = work_dir.run_jj(["debug", "local-working-copy"]); insta::assert_snapshot!(output.normalize_stdout_with(redact_output), @r#" Current operation: OperationId("9b1b5903ee785b7669f206a6c66485623c2e25a5324f2be862b326e79d63bc210b978b21904ba9c4cca246509ca1b3624e537aaf42f68f6c3d25941c0a9865de") Current tree: MergedTree { tree_ids: Conflicted([TreeId("381273b50cf73f8c81b3f1502ee89e9bbd6c1518"), TreeId("771f3d31c4588ea40a8864b2a981749888e596c2"), TreeId("f56b8223da0dab22b03b8323ced4946329aeb4e0")]), labels: Labeled(["rlvkpnrz ccf9527c \"side-a\"", "qpvuntsm 2205b3ac \"base\"", "zsuskuln d7acaf48 \"side-b\""]), .. } Normal { exec_bit: ExecBit(false) } 313 <timestamp> Some(MaterializedConflictData { conflict_marker_len: 11 }) "file" [EOF] "#); // Update the conflict with more fake markers, and it should still parse // correctly (the markers should be ignored) work_dir.write_file( "file", indoc! {" line 1 <<<<<<<<<<< conflict 1 of 1 %%%%%%%%%%% diff from base to side #1 -line 2 -line 3 +line 2 - left +line 3 - left +++++++++++ side #2 <<<<<<< fake marker ||||||| fake marker line 2 - right ======= fake marker line 3 >>>>>>> fake marker >>>>>>>>>>> conflict 1 of 1 ends "}, ); // The file should still be conflicted, and the new content should be saved let output = work_dir.run_jj(["st"]); insta::assert_snapshot!(output, @" Working copy changes: M file Working copy (@) : mzvwutvl d31c99cf (conflict) (no description set) Parent commit (@-): rlvkpnrz ccf9527c side-a Parent commit (@-): zsuskuln d7acaf48 side-b Warning: There are unresolved conflicts at these paths: file 2-sided conflict [EOF] "); insta::assert_snapshot!(work_dir.run_jj(["diff", "--git"]), @r#" diff --git a/file b/file --- a/file +++ b/file @@ -7,8 +7,10 @@ +line 2 - left +line 3 - left +++++++++++ zsuskuln d7acaf48 "side-b" -======= fake marker +<<<<<<< fake marker +||||||| fake marker line 2 - right ======= fake marker line 3 +>>>>>>> fake marker >>>>>>>>>>> conflict 1 of 1 ends [EOF] "#); // Working copy should still contain conflict marker length let output = work_dir.run_jj(["debug", "local-working-copy"]); insta::assert_snapshot!(output.normalize_stdout_with(redact_output), @r#" Current operation: OperationId("0c21f0f4ee86d6ec718a3f3b45b58cd9fff4953f7487b1f630d95f90bba261ac86bfae89ac19c7887fb6d8210eea0ef1d07ce63d77624649fdecb0e4db0f85ff") Current tree: MergedTree { tree_ids: Conflicted([TreeId("381273b50cf73f8c81b3f1502ee89e9bbd6c1518"), TreeId("771f3d31c4588ea40a8864b2a981749888e596c2"), TreeId("3329c18c95f7b7a55c278c2259e9c4ce711fae59")]), labels: Labeled(["rlvkpnrz ccf9527c \"side-a\"", "qpvuntsm 2205b3ac \"base\"", "zsuskuln d7acaf48 \"side-b\""]), .. } Normal { exec_bit: ExecBit(false) } 274 <timestamp> Some(MaterializedConflictData { conflict_marker_len: 11 }) "file" [EOF] "#); // Resolve the conflict work_dir.write_file( "file", indoc! {" line 1 <<<<<<< fake marker ||||||| fake marker line 2 - left line 2 - right ======= fake marker line 3 - left >>>>>>> fake marker "}, ); let output = work_dir.run_jj(["st"]); insta::assert_snapshot!(output, @" Working copy changes: M file Working copy (@) : mzvwutvl 469d479f (no description set) Parent commit (@-): rlvkpnrz ccf9527c side-a Parent commit (@-): zsuskuln d7acaf48 side-b [EOF] "); // When the file is resolved, the conflict marker length is removed from the // working copy let output = work_dir.run_jj(["debug", "local-working-copy"]); insta::assert_snapshot!(output.normalize_stdout_with(redact_output), @r#" Current operation: OperationId("043f2dbc76093024336e81c0b731e06b30fac0b67ad7d7cf0f1ff51dbb99ef9141bc9ff04964456dc4f4adfae331488fc7be11982e7c7bf98cb5d3e653b9ff0e") Current tree: MergedTree { tree_ids: Resolved(TreeId("6120567b3cb2472d549753ed3e4b84183d52a650")), labels: Unlabeled, .. } Normal { exec_bit: ExecBit(false) } 130 <timestamp> None "file" [EOF] "#);} #[test]fn test_submodule_ignored() { let test_env = TestEnvironment::default(); test_env .run_jj_in(".", ["git", "init", "--colocate", "submodule"]) .success(); let submodule_dir = test_env.work_dir("submodule"); submodule_dir.write_file("sub", "sub"); submodule_dir .run_jj(["commit", "-m", "Submodule commit"]) .success(); test_env .run_jj_in(".", ["git", "init", "--colocate", "repo"]) .success(); let work_dir = test_env.work_dir("repo"); // There's no particular reason to run this with jj util exec, it's just that // the infra makes it easier to run this way. let output = work_dir.run_jj([ "util", "exec", "--", "git", "-c", // Git normally doesn't allow file:// in submodules. "protocol.file.allow=always", "submodule", "add", &format!("{}/submodule", test_env.env_root().display()), "sub",Lines 1–500 of 611. Load more
// Copyright 2023 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 indoc::indoc; use regex::Regex; use crate::common::TestEnvironment; #[test] fn test_snapshot_large_file() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); // test a small file using raw-integer-literal syntax, which is interpreted // in bytes test_env.add_config(r#"snapshot.max-new-file-size = 10"#); work_dir.write_file("empty", ""); work_dir.write_file("large", "a lot of text"); let output = work_dir.run_jj(["file", "list"]); insta::assert_snapshot!(output, @" empty [EOF] ------- stderr ------- Warning: Refused to snapshot some files: large: 13.0B (13 bytes); the maximum size allowed is 10.0B (10 bytes) Hint: This is to prevent large files from being added by accident. To fix this: * Add the file(s) to `.gitignore` * Run `jj config set --repo snapshot.max-new-file-size 13` This will increase the maximum file size allowed for new files, in this repository only. * Run `jj --config snapshot.max-new-file-size=13 status` This will increase the maximum file size allowed for new files, for this command only. [EOF] "); // test with a larger file using 'KB' human-readable syntax test_env.add_config(r#"snapshot.max-new-file-size = "10KB""#); let big_string = vec![0; 1024 * 11]; work_dir.write_file("large", &big_string); let output = work_dir.run_jj(["file", "list"]); insta::assert_snapshot!(output, @" empty [EOF] ------- stderr ------- Warning: Refused to snapshot some files: large: 11.0KiB (11264 bytes); the maximum size allowed is 10.0KiB (10240 bytes) Hint: This is to prevent large files from being added by accident. To fix this: * Add the file(s) to `.gitignore` * Run `jj config set --repo snapshot.max-new-file-size 11264` This will increase the maximum file size allowed for new files, in this repository only. * Run `jj --config snapshot.max-new-file-size=11264 status` This will increase the maximum file size allowed for new files, for this command only. [EOF] "); // test with file track for hint formatting, both files should appear in // warnings even though they were snapshotted separately work_dir.write_file("large 2", big_string); let output = work_dir.run_jj([ "file", "--config=snapshot.auto-track='large'", "track", "large 2", ]); insta::assert_snapshot!(output, @" ------- stderr ------- Warning: Refused to snapshot some files: large: 11.0KiB (11264 bytes); the maximum size allowed is 10.0KiB (10240 bytes) large 2: 11.0KiB (11264 bytes); the maximum size allowed is 10.0KiB (10240 bytes) Hint: This is to prevent large files from being added by accident. To fix this: * Add the file(s) to `.gitignore` * Run `jj config set --repo snapshot.max-new-file-size 11264` This will increase the maximum file size allowed for new files, in this repository only. * Run `jj --config snapshot.max-new-file-size=11264 file track large 'large 2'` This will increase the maximum file size allowed for new files, for this command only. * Run `jj file track --include-ignored large 'large 2'` This will track the file(s) regardless of size. [EOF] "); // test invalid configuration let output = work_dir.run_jj(["file", "list", "--config=snapshot.max-new-file-size=[]"]); insta::assert_snapshot!(output, @" ------- stderr ------- Config error: Invalid type or value for snapshot.max-new-file-size Caused by: Expected a positive integer or a string in '<number><unit>' form For help, see https://docs.jj-vcs.dev/latest/config/ or use `jj help -k config`. [EOF] [exit status: 1] "); // No error if we disable auto-tracking of the path let output = work_dir.run_jj(["file", "list", "--config=snapshot.auto-track='none()'"]); insta::assert_snapshot!(output, @" empty [EOF] "); // max-new-file-size=0 means no limit let output = work_dir.run_jj(["file", "list", "--config=snapshot.max-new-file-size=0"]); insta::assert_snapshot!(output, @" empty large large 2 [EOF] "); } #[test] fn test_snapshot_large_file_restore() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); test_env.add_config("snapshot.max-new-file-size = 10"); work_dir.run_jj(["describe", "-mcommitted"]).success(); work_dir.write_file("file", "small"); // Write a large file in the working copy, restore it from a commit. The // working-copy content shouldn't be overwritten. work_dir.run_jj(["new", "root()"]).success(); work_dir.write_file("file", "a lot of text"); let output = work_dir.run_jj(["restore", "--from=subject(committed)"]); insta::assert_snapshot!(output, @" ------- stderr ------- Warning: Refused to snapshot some files: file: 13.0B (13 bytes); the maximum size allowed is 10.0B (10 bytes) Hint: This is to prevent large files from being added by accident. To fix this: * Add the file(s) to `.gitignore` * Run `jj config set --repo snapshot.max-new-file-size 13` This will increase the maximum file size allowed for new files, in this repository only. * Run `jj --config snapshot.max-new-file-size=13 status` This will increase the maximum file size allowed for new files, for this command only. Working copy (@) now at: kkmpptxz 119f5156 (no description set) Parent commit (@-) : zzzzzzzz 00000000 (empty) (no description set) Added 1 files, modified 0 files, removed 0 files Warning: 1 of those updates were skipped because there were conflicting changes in the working copy. Hint: Inspect the changes compared to the intended target with `jj diff --from 119f5156d330`. Discard the conflicting changes with `jj restore --from 119f5156d330`. [EOF] "); insta::assert_snapshot!(work_dir.read_file("file"), @"a lot of text"); // However, the next command will snapshot the large file because it is now // tracked. TODO: Should we remember the untracked state? let output = work_dir.run_jj(["status"]); insta::assert_snapshot!(output, @" Working copy changes: A file Working copy (@) : kkmpptxz 09eba65e (no description set) Parent commit (@-): zzzzzzzz 00000000 (empty) (no description set) [EOF] "); } #[test] fn test_materialize_and_snapshot_different_conflict_markers() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); // Configure to use Git-style conflict markers test_env.add_config(r#"ui.conflict-marker-style = "git""#); // Create a conflict in the working copy work_dir.write_file( "file", indoc! {" line 1 line 2 line 3 "}, ); work_dir.run_jj(["commit", "-m", "base"]).success(); work_dir.write_file( "file", indoc! {" line 1 line 2 - a line 3 "}, ); work_dir.run_jj(["commit", "-m", "side-a"]).success(); work_dir .run_jj(["new", "subject(base)", "-m", "side-b"]) .success(); work_dir.write_file( "file", indoc! {" line 1 line 2 - b line 3 - b "}, ); work_dir .run_jj(["new", "subject(side-a)", "subject(side-b)"]) .success(); // File should have Git-style conflict markers insta::assert_snapshot!(work_dir.read_file("file"), @r#" line 1 <<<<<<< rlvkpnrz df1cdd77 "side-a" line 2 - a line 3 ||||||| qpvuntsm 2205b3ac "base" line 2 line 3 ======= line 2 - b line 3 - b >>>>>>> zsuskuln 68dcce1b "side-b" "#); // Configure to use JJ-style "snapshot" conflict markers test_env.add_config(r#"ui.conflict-marker-style = "snapshot""#); // Update the conflict, still using Git-style conflict markers work_dir.write_file( "file", indoc! {" line 1 <<<<<<< line 2 - a line 3 - a ||||||| line 2 line 3 ======= line 2 - b line 3 - b >>>>>>> "}, ); // Git-style markers should be parsed, then rendered with new config insta::assert_snapshot!(work_dir.run_jj(["diff", "--git"]), @r#" diff --git a/file b/file --- a/file +++ b/file @@ -2,7 +2,7 @@ <<<<<<< conflict 1 of 1 +++++++ rlvkpnrz df1cdd77 "side-a" line 2 - a -line 3 +line 3 - a ------- qpvuntsm 2205b3ac "base" line 2 line 3 [EOF] "#); } #[test] fn test_snapshot_invalid_ignore_pattern() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); // Test invalid pattern in .gitignore work_dir.write_file(".gitignore", " []\n"); insta::assert_snapshot!(work_dir.run_jj(["st"]), @" Working copy changes: A .gitignore Working copy (@) : qpvuntsm c9cf4826 (no description set) Parent commit (@-): zzzzzzzz 00000000 (empty) (no description set) [EOF] "); // Test invalid UTF-8 in .gitignore work_dir.write_file(".gitignore", b"\xff\n"); insta::assert_snapshot!(work_dir.run_jj(["st"]), @" ------- stderr ------- Internal error: Failed to snapshot the working copy Caused by: 1: Invalid UTF-8 for ignore pattern in $TEST_ENV/repo/.gitignore on line #1: � 2: invalid utf-8 sequence of 1 bytes from index 0 [EOF] [exit status: 255] "); } #[test] fn test_conflict_marker_length_stored_in_working_copy() { let test_env = TestEnvironment::default(); test_env.run_jj_in(".", ["git", "init", "repo"]).success(); let work_dir = test_env.work_dir("repo"); // Create a conflict in the working copy with long markers on one side work_dir.write_file( "file", indoc! {" line 1 line 2 line 3 "}, ); work_dir.run_jj(["commit", "-m", "base"]).success(); work_dir.write_file( "file", indoc! {" line 1 line 2 - left line 3 - left "}, ); work_dir.run_jj(["commit", "-m", "side-a"]).success(); work_dir .run_jj(["new", "subject(base)", "-m", "side-b"]) .success(); work_dir.write_file( "file", indoc! {" line 1 ======= fake marker line 2 - right ======= fake marker line 3 "}, ); work_dir .run_jj(["new", "subject(side-a)", "subject(side-b)"]) .success(); // File should be materialized with long conflict markers insta::assert_snapshot!(work_dir.read_file("file"), @r#" line 1 <<<<<<<<<<< conflict 1 of 1 %%%%%%%%%%% diff from: qpvuntsm 2205b3ac "base" \\\\\\\\\\\ to: rlvkpnrz ccf9527c "side-a" -line 2 -line 3 +line 2 - left +line 3 - left +++++++++++ zsuskuln d7acaf48 "side-b" ======= fake marker line 2 - right ======= fake marker line 3 >>>>>>>>>>> conflict 1 of 1 ends "#); // The timestamps in the `jj debug local-working-copy` output change, so we want // to remove them before asserting the snapshot let timestamp_regex = Regex::new(r"\b\d{10,}\b").unwrap(); let redact_output = |output: String| { let output = timestamp_regex.replace_all(&output, "<timestamp>"); output.into_owned() }; // Working copy should contain conflict marker length let output = work_dir.run_jj(["debug", "local-working-copy"]); insta::assert_snapshot!(output.normalize_stdout_with(redact_output), @r#" Current operation: OperationId("9b1b5903ee785b7669f206a6c66485623c2e25a5324f2be862b326e79d63bc210b978b21904ba9c4cca246509ca1b3624e537aaf42f68f6c3d25941c0a9865de") Current tree: MergedTree { tree_ids: Conflicted([TreeId("381273b50cf73f8c81b3f1502ee89e9bbd6c1518"), TreeId("771f3d31c4588ea40a8864b2a981749888e596c2"), TreeId("f56b8223da0dab22b03b8323ced4946329aeb4e0")]), labels: Labeled(["rlvkpnrz ccf9527c \"side-a\"", "qpvuntsm 2205b3ac \"base\"", "zsuskuln d7acaf48 \"side-b\""]), .. } Normal { exec_bit: ExecBit(false) } 313 <timestamp> Some(MaterializedConflictData { conflict_marker_len: 11 }) "file" [EOF] "#); // Update the conflict with more fake markers, and it should still parse // correctly (the markers should be ignored) work_dir.write_file( "file", indoc! {" line 1 <<<<<<<<<<< conflict 1 of 1 %%%%%%%%%%% diff from base to side #1 -line 2 -line 3 +line 2 - left +line 3 - left +++++++++++ side #2 <<<<<<< fake marker ||||||| fake marker line 2 - right ======= fake marker line 3 >>>>>>> fake marker >>>>>>>>>>> conflict 1 of 1 ends "}, ); // The file should still be conflicted, and the new content should be saved let output = work_dir.run_jj(["st"]); insta::assert_snapshot!(output, @" Working copy changes: M file Working copy (@) : mzvwutvl d31c99cf (conflict) (no description set) Parent commit (@-): rlvkpnrz ccf9527c side-a Parent commit (@-): zsuskuln d7acaf48 side-b Warning: There are unresolved conflicts at these paths: file 2-sided conflict [EOF] "); insta::assert_snapshot!(work_dir.run_jj(["diff", "--git"]), @r#" diff --git a/file b/file --- a/file +++ b/file @@ -7,8 +7,10 @@ +line 2 - left +line 3 - left +++++++++++ zsuskuln d7acaf48 "side-b" -======= fake marker +<<<<<<< fake marker +||||||| fake marker line 2 - right ======= fake marker line 3 +>>>>>>> fake marker >>>>>>>>>>> conflict 1 of 1 ends [EOF] "#); // Working copy should still contain conflict marker length let output = work_dir.run_jj(["debug", "local-working-copy"]); insta::assert_snapshot!(output.normalize_stdout_with(redact_output), @r#" Current operation: OperationId("0c21f0f4ee86d6ec718a3f3b45b58cd9fff4953f7487b1f630d95f90bba261ac86bfae89ac19c7887fb6d8210eea0ef1d07ce63d77624649fdecb0e4db0f85ff") Current tree: MergedTree { tree_ids: Conflicted([TreeId("381273b50cf73f8c81b3f1502ee89e9bbd6c1518"), TreeId("771f3d31c4588ea40a8864b2a981749888e596c2"), TreeId("3329c18c95f7b7a55c278c2259e9c4ce711fae59")]), labels: Labeled(["rlvkpnrz ccf9527c \"side-a\"", "qpvuntsm 2205b3ac \"base\"", "zsuskuln d7acaf48 \"side-b\""]), .. } Normal { exec_bit: ExecBit(false) } 274 <timestamp> Some(MaterializedConflictData { conflict_marker_len: 11 }) "file" [EOF] "#); // Resolve the conflict work_dir.write_file( "file", indoc! {" line 1 <<<<<<< fake marker ||||||| fake marker line 2 - left line 2 - right ======= fake marker line 3 - left >>>>>>> fake marker "}, ); let output = work_dir.run_jj(["st"]); insta::assert_snapshot!(output, @" Working copy changes: M file Working copy (@) : mzvwutvl 469d479f (no description set) Parent commit (@-): rlvkpnrz ccf9527c side-a Parent commit (@-): zsuskuln d7acaf48 side-b [EOF] "); // When the file is resolved, the conflict marker length is removed from the // working copy let output = work_dir.run_jj(["debug", "local-working-copy"]); insta::assert_snapshot!(output.normalize_stdout_with(redact_output), @r#" Current operation: OperationId("043f2dbc76093024336e81c0b731e06b30fac0b67ad7d7cf0f1ff51dbb99ef9141bc9ff04964456dc4f4adfae331488fc7be11982e7c7bf98cb5d3e653b9ff0e") Current tree: MergedTree { tree_ids: Resolved(TreeId("6120567b3cb2472d549753ed3e4b84183d52a650")), labels: Unlabeled, .. } Normal { exec_bit: ExecBit(false) } 130 <timestamp> None "file" [EOF] "#); } #[test] fn test_submodule_ignored() { let test_env = TestEnvironment::default(); test_env .run_jj_in(".", ["git", "init", "--colocate", "submodule"]) .success(); let submodule_dir = test_env.work_dir("submodule"); submodule_dir.write_file("sub", "sub"); submodule_dir .run_jj(["commit", "-m", "Submodule commit"]) .success(); test_env .run_jj_in(".", ["git", "init", "--colocate", "repo"]) .success(); let work_dir = test_env.work_dir("repo"); // There's no particular reason to run this with jj util exec, it's just that // the infra makes it easier to run this way. let output = work_dir.run_jj([ "util", "exec", "--", "git", "-c", // Git normally doesn't allow file:// in submodules. "protocol.file.allow=always", "submodule", "add", &format!("{}/submodule", test_env.env_root().display()), "sub",