Skip to content

Commit 5e71f4c

Browse files
committed
Fix clippy
1 parent 1db824f commit 5e71f4c

File tree

1 file changed

+43
-13
lines changed

1 file changed

+43
-13
lines changed

crates/markdown_preview/src/markdown_parser.rs

Lines changed: 43 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -938,6 +938,7 @@ mod tests {
938938
HighlightId, Language, LanguageConfig, LanguageMatcher, LanguageRegistry, tree_sitter_rust,
939939
};
940940
use pretty_assertions::assert_eq;
941+
use ui::Pixels;
941942

942943
async fn parse(input: &str) -> ParsedMarkdown {
943944
parse_markdown(input, None, None).await
@@ -1207,32 +1208,54 @@ mod tests {
12071208
// Test percentage values
12081209
assert_eq!(
12091210
MarkdownParser::parse_length("50%"),
1210-
Some(relative(0.5).into())
1211+
Some(DefiniteLength::Fraction(0.5))
12111212
);
12121213
assert_eq!(
12131214
MarkdownParser::parse_length("100%"),
1214-
Some(relative(1.0).into())
1215+
Some(DefiniteLength::Fraction(1.0))
12151216
);
12161217
assert_eq!(
12171218
MarkdownParser::parse_length("25%"),
1218-
Some(relative(0.25).into())
1219+
Some(DefiniteLength::Fraction(0.25))
12191220
);
12201221
assert_eq!(
12211222
MarkdownParser::parse_length("0%"),
1222-
Some(relative(0.0).into())
1223+
Some(DefiniteLength::Fraction(0.0))
12231224
);
12241225

12251226
// Test pixel values
12261227
assert_eq!(
12271228
MarkdownParser::parse_length("100px"),
1228-
Some(px(100.0).into())
1229+
Some(DefiniteLength::Absolute(AbsoluteLength::Pixels(Pixels(
1230+
100.0
1231+
))))
1232+
);
1233+
assert_eq!(
1234+
MarkdownParser::parse_length("50px"),
1235+
Some(DefiniteLength::Absolute(AbsoluteLength::Pixels(Pixels(
1236+
50.0
1237+
))))
1238+
);
1239+
assert_eq!(
1240+
MarkdownParser::parse_length("0px"),
1241+
Some(DefiniteLength::Absolute(AbsoluteLength::Pixels(Pixels(
1242+
0.0
1243+
))))
12291244
);
1230-
assert_eq!(MarkdownParser::parse_length("50px"), Some(px(50.0).into()));
1231-
assert_eq!(MarkdownParser::parse_length("0px"), Some(px(0.0).into()));
12321245

12331246
// Test values without units (should be treated as pixels)
1234-
assert_eq!(MarkdownParser::parse_length("100"), Some(px(100.0).into()));
1235-
assert_eq!(MarkdownParser::parse_length("42"), Some(px(42.0).into()));
1247+
assert_eq!(
1248+
MarkdownParser::parse_length("100"),
1249+
Some(DefiniteLength::Absolute(AbsoluteLength::Pixels(Pixels(
1250+
100.0
1251+
))))
1252+
);
1253+
assert_eq!(
1254+
MarkdownParser::parse_length("42"),
1255+
Some(DefiniteLength::Absolute(AbsoluteLength::Pixels(Pixels(
1256+
42.0
1257+
))))
1258+
);
12361259

12371260
// Test invalid values
12381261
assert_eq!(MarkdownParser::parse_length("invalid"), None);
@@ -1245,13 +1268,20 @@ mod tests {
12451268
// Test decimal values
12461269
assert_eq!(
12471270
MarkdownParser::parse_length("50.5%"),
1248-
Some(relative(0.505).into())
1271+
Some(DefiniteLength::Fraction(0.505))
12491272
);
12501273
assert_eq!(
12511274
MarkdownParser::parse_length("100.25px"),
1252-
Some(px(100.25).into())
1275+
Some(DefiniteLength::Absolute(AbsoluteLength::Pixels(Pixels(
1276+
100.25
1277+
))))
1278+
);
1279+
assert_eq!(
1280+
MarkdownParser::parse_length("42.0"),
1281+
Some(DefiniteLength::Absolute(AbsoluteLength::Pixels(Pixels(
1282+
42.0
1283+
))))
12531284
);
1254-
assert_eq!(MarkdownParser::parse_length("42.0"), Some(px(42.0).into()));
12551285
}
12561286

12571287
#[gpui::test]
@@ -1331,7 +1361,7 @@ mod tests {
13311361
assert_eq!(
13321362
image.clone(),
13331363
Image {
1334-
source_range: 0..65,
1364+
source_range: 0..75,
13351365
link: Link::Web {
13361366
url: "http://example.com/foo.png".to_string(),
13371367
},

0 commit comments

Comments
 (0)