Skip to content

Commit b7ff05b

Browse files
committed
Adjust tests for new vesa mode detection
1 parent 7f7fec7 commit b7ff05b

File tree

2 files changed

+54
-10
lines changed

2 files changed

+54
-10
lines changed

tests/test_kernels/default_settings/src/bin/check_boot_info.rs

Lines changed: 27 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,34 @@ fn kernel_main(boot_info: &'static mut BootInfo) -> ! {
1414
// check framebuffer
1515
let framebuffer = boot_info.framebuffer.as_ref().unwrap();
1616
assert_eq!(framebuffer.info().byte_len, framebuffer.buffer().len());
17-
assert_eq!(framebuffer.info().horizontal_resolution, 1024);
18-
assert_eq!(framebuffer.info().vertical_resolution, 768);
19-
assert_eq!(framebuffer.info().bytes_per_pixel, 3);
20-
assert_eq!(framebuffer.info().stride, 1024);
17+
if ![640, 1024].contains(&framebuffer.info().horizontal_resolution) {
18+
panic!(
19+
"unexpected horizontal_resolution `{}`",
20+
framebuffer.info().horizontal_resolution
21+
);
22+
}
23+
if ![480, 768].contains(&framebuffer.info().vertical_resolution) {
24+
panic!(
25+
"unexpected vertical_resolution `{}`",
26+
framebuffer.info().vertical_resolution
27+
);
28+
}
29+
if ![3, 4].contains(&framebuffer.info().bytes_per_pixel) {
30+
panic!(
31+
"unexpected bytes_per_pixel `{}`",
32+
framebuffer.info().bytes_per_pixel
33+
);
34+
}
35+
if ![640, 1024].contains(&framebuffer.info().stride) {
36+
panic!("unexpected stride `{}`", framebuffer.info().stride);
37+
}
2138
assert_eq!(framebuffer.info().pixel_format, PixelFormat::RGB);
22-
assert_eq!(framebuffer.buffer().len(), 1024 * 768 * 3);
39+
assert_eq!(
40+
framebuffer.buffer().len(),
41+
framebuffer.info().stride
42+
* framebuffer.info().vertical_resolution
43+
* framebuffer.info().bytes_per_pixel
44+
);
2345

2446
// check defaults for optional features
2547
assert_eq!(boot_info.physical_memory_offset.into_option(), None);

tests/test_kernels/map_phys_mem/src/bin/check_boot_info.rs

Lines changed: 27 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,34 @@ fn kernel_main(boot_info: &'static mut BootInfo) -> ! {
1414
// check framebuffer
1515
let framebuffer = boot_info.framebuffer.as_ref().unwrap();
1616
assert_eq!(framebuffer.info().byte_len, framebuffer.buffer().len());
17-
assert_eq!(framebuffer.info().horizontal_resolution, 1024);
18-
assert_eq!(framebuffer.info().vertical_resolution, 768);
19-
assert_eq!(framebuffer.info().bytes_per_pixel, 3);
20-
assert_eq!(framebuffer.info().stride, 1024);
17+
if ![640, 1024].contains(&framebuffer.info().horizontal_resolution) {
18+
panic!(
19+
"unexpected horizontal_resolution `{}`",
20+
framebuffer.info().horizontal_resolution
21+
);
22+
}
23+
if ![480, 768].contains(&framebuffer.info().vertical_resolution) {
24+
panic!(
25+
"unexpected vertical_resolution `{}`",
26+
framebuffer.info().vertical_resolution
27+
);
28+
}
29+
if ![3, 4].contains(&framebuffer.info().bytes_per_pixel) {
30+
panic!(
31+
"unexpected bytes_per_pixel `{}`",
32+
framebuffer.info().bytes_per_pixel
33+
);
34+
}
35+
if ![640, 1024].contains(&framebuffer.info().stride) {
36+
panic!("unexpected stride `{}`", framebuffer.info().stride);
37+
}
2138
assert_eq!(framebuffer.info().pixel_format, PixelFormat::RGB);
22-
assert_eq!(framebuffer.buffer().len(), 1024 * 768 * 3);
39+
assert_eq!(
40+
framebuffer.buffer().len(),
41+
framebuffer.info().stride
42+
* framebuffer.info().vertical_resolution
43+
* framebuffer.info().bytes_per_pixel
44+
);
2345

2446
// check defaults for optional features
2547
assert!(matches!(

0 commit comments

Comments
 (0)