611 lines · 20.9 KB · Rust
RawDownload
// 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.

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t