summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Davison <dandavison7@gmail.com>2020-12-27 19:09:42 +0000
committerDan Davison <dandavison7@gmail.com>2020-12-27 21:51:28 +0000
commit0a57ed963541edb62f90882ba18acc2427d87f08 (patch)
tree0a8d6b75795f7f244a30a5de465460a2c6f5df7e
parent12996f1824c78d67b22ea2e92cd1ff84f00e274a (diff)
Fix tests
-rw-r--r--src/hunk_header.rs3
-rw-r--r--src/tests/test_example_diffs.rs30
2 files changed, 14 insertions, 19 deletions
diff --git a/src/hunk_header.rs b/src/hunk_header.rs
index b335b097..7773c1df 100644
--- a/src/hunk_header.rs
+++ b/src/hunk_header.rs
@@ -96,7 +96,8 @@ fn _write_hunk_header(
);
have_hunk_header = true;
};
- if !config.line_numbers
+ if false
+ && !config.line_numbers
&& config.line_numbers_show_first_line_number
&& !config.hunk_header_style.is_raw
&& !config.color_only
diff --git a/src/tests/test_example_diffs.rs b/src/tests/test_example_diffs.rs
index 94b18f11..28186122 100644
--- a/src/tests/test_example_diffs.rs
+++ b/src/tests/test_example_diffs.rs
@@ -74,7 +74,7 @@ mod tests {
let config = integration_test_utils::make_config_from_args(&[]);
let output = integration_test_utils::get_line_of_code_from_delta(
&ADDED_FILE_INPUT,
- 12,
+ 11,
"class X:",
&config,
);
@@ -88,7 +88,7 @@ mod tests {
let config = integration_test_utils::make_config_from_args(&[]);
let input = ADDED_FILE_INPUT.replace("a.py", "a");
let output =
- integration_test_utils::get_line_of_code_from_delta(&input, 12, "class X:", &config);
+ integration_test_utils::get_line_of_code_from_delta(&input, 11, "class X:", &config);
ansi_test_utils::assert_has_color_other_than_plus_color(&output, &config);
}
@@ -104,7 +104,7 @@ mod tests {
]);
let input = ADDED_FILE_INPUT.replace("a.py", "a");
let output =
- integration_test_utils::get_line_of_code_from_delta(&input, 12, "class X:", &config);
+ integration_test_utils::get_line_of_code_from_delta(&input, 11, "class X:", &config);
ansi_test_utils::assert_has_plus_color_only(&output, &config);
}
@@ -117,14 +117,10 @@ mod tests {
// Header
assert_eq!(lines.nth(1).unwrap(), "comparing: one.rs ⟶ src/two.rs");
- // Line
- assert_eq!(lines.nth(2).unwrap(), "5");
// Change
- assert_eq!(lines.nth(2).unwrap(), "println!(\"Hello ruster\");");
- // Next chunk
- assert_eq!(lines.nth(2).unwrap(), "43");
+ assert_eq!(lines.nth(4).unwrap(), "println!(\"Hello ruster\");");
// Unchanged in second chunk
- assert_eq!(lines.nth(2).unwrap(), "Unchanged");
+ assert_eq!(lines.nth(4).unwrap(), "Unchanged");
}
#[test]
@@ -139,10 +135,8 @@ mod tests {
lines.nth(1).unwrap(),
"comparing: a/different ⟶ b/different"
);
- // Line number
- assert_eq!(lines.nth(2).unwrap(), "1");
// Change
- assert_eq!(lines.nth(2).unwrap(), "This is different from b");
+ assert_eq!(lines.nth(4).unwrap(), "This is different from b");
// File uniqueness
assert_eq!(lines.nth(2).unwrap(), "Only in a/: just_a");
// FileMeta divider
@@ -196,7 +190,7 @@ mod tests {
let output = integration_test_utils::run_delta(DIFF_WITH_MERGE_CONFLICT, &config);
// TODO: The + in the first column is being removed.
assert!(strip_ansi_codes(&output).contains("+>>>>>>> Stashed changes"));
- assert_eq!(output.lines().count(), 46);
+ assert_eq!(output.lines().count(), 45);
}
#[test]
@@ -1370,7 +1364,7 @@ impl<'a> Alignment<'a> { │
empty_line_marker_style,
]);
let output = integration_test_utils::run_delta(example_diff, &config);
- let line = output.lines().nth(6).unwrap();
+ let line = output.lines().nth(5).unwrap();
if base_style_has_background_color {
let style = style::Style::from_str(base_style, None, None, true, false);
assert_eq!(
@@ -1400,11 +1394,11 @@ impl<'a> Alignment<'a> { │
whitespace_error_style,
]);
let output = integration_test_utils::run_delta(DIFF_WITH_WHITESPACE_ERROR, &config);
- ansi_test_utils::assert_line_has_style(&output, 6, " ", whitespace_error_style, &config);
+ ansi_test_utils::assert_line_has_style(&output, 5, " ", whitespace_error_style, &config);
let output = integration_test_utils::run_delta(DIFF_WITH_REMOVED_WHITESPACE_ERROR, &config);
ansi_test_utils::assert_line_does_not_have_style(
&output,
- 6,
+ 5,
" ",
whitespace_error_style,
&config,
@@ -1421,7 +1415,7 @@ impl<'a> Alignment<'a> { │
plus_style,
]);
let output = integration_test_utils::run_delta(DIFF_WITH_ADDED_EMPTY_LINE, &config);
- ansi_test_utils::assert_line_has_style(&output, 6, "", plus_style, &config)
+ ansi_test_utils::assert_line_has_style(&output, 5, "", plus_style, &config)
}
#[test]
@@ -1434,7 +1428,7 @@ impl<'a> Alignment<'a> { │
plus_style,
]);
let output = integration_test_utils::run_delta(DIFF_WITH_SINGLE_CHARACTER_LINE, &config);
- ansi_test_utils::assert_line_has_style(&output, 12, "+}", plus_style, &config)
+ ansi_test_utils::assert_line_has_style(&output, 11, "+}", plus_style, &config)
}
#[test]