facebook/sapling
1
1
Fork 0
mirror of https://github.com/facebook/sapling.git synced 2024-12-28 15:44:27 +03:00
Code Issues Projects Releases Wiki Activity

dag: test high-level segments building for non-master

Browse Source 
Summary:
I noticed that high-level segments are somehow not built for non-master vertexes.
Add a test to demonstrate the issue.

Reviewed By: DurhamG, sfilipco

Differential Revision: D23095466

fbshipit-source-id: c5a6da14bdfabcf7c432f6c6dfe096c71cc10ee9
This commit is contained in:
Jun Wu 2020-08-20 17:33:07 -07:00 committed by Jun Wu
parent 23074edd9b
commit e11f36e96b
1 changed files with 37 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

39
eden/scm/lib/dag/src/tests.rs
View File

@ -426,6 +426,37 @@ Lv3: R0-11[]"#
);
}
#[test]
fn test_segment_non_master() {
let ascii = r#"
a----b----c----d----e----f----g----------h----i
\ \ /
h---i---j---k l---m---n---o
\ \
-----------------p---q"#;
let built = build_segments(ascii, "i q", 3);
assert_eq!(
built.ascii[0],
r#"
N0---N1---N2---N3---N4---N5---N6---------N11--N12
\ \ /
N11-N12-j---k N7--N8--N9--N10
\ \
-----------------p---q
Lv0: RN0-N6[] N7-N10[N5] N11-N12[N0, N6, N10]"#
);
assert_eq!(
built.ascii[1],
r#"
N0---N1---N2---N3---N4---N5---N6---------N11--N12
\ \ /
N11-N12-N13-k N7--N8--N9--N10
\ \
-----------------N14-N15
Lv0: RN0-N6[] N7-N10[N5] N11-N12[N0, N6, N10] N13-N13[N12] N14-N15[N13, N8]"#
);
}
#[test]
fn test_segment_examples() {
assert_eq!(
@ -1091,8 +1122,12 @@ impl IdMap {
if let Ok(Some(name)) = self.find_name_by_id(id) {
let name = String::from_utf8(name.to_vec()).unwrap();
let id_str = format!("{:01$}", id, name.len());
if name.len() + 1 == id_str.len() {
// Try to replace while maintaining width
// Try to replace while maintaining width
if name.len() + 2 == id_str.len() {
result = result
.replace(&format!("{}--", name), &id_str)
.replace(&format!("{} ", name), &id_str);
} else if name.len() + 1 == id_str.len() {
result = result
.replace(&format!("{}-", name), &id_str)
.replace(&format!("{} ", name), &id_str);