Merge branch 'joamag/audio-tests' into 'master'

Make dmg-audio tests pass

See merge request joamag/boytacean!26

frontends/sdl/README.md

@@ -79,7 +79,7 @@ cargo vcpkg -v build
 It's possible to run the emulator in headless mode using the `--headless` parameter:
 
 ```bash
-cargo run  -- --rom-path ../../res/roms/test/blargg/cpu/cpu_instrs.gb --cycles 100000000  --headless --device stdout --unlimited
+cargo run -- --rom-path ../../res/roms/test/blargg/cpu/cpu_instrs.gb --cycles 100000000 --headless --device stdout --unlimited
 ```
 
 ## Features

frontends/sdl/src/main.rs

@@ -662,6 +662,13 @@ struct Args {
     #[arg(short, long, default_value_t = String::from("printer"), help = "Serial device to be used")]
     device: String,
 
+    #[arg(
+        long,
+        default_value_t = false,
+        help = "If set no boot ROM will be loaded"
+    )]
+    no_boot: bool,
+
     #[arg(long, default_value_t = false, help = "If set no PPU will be used")]
     no_ppu: bool,
 
@@ -740,7 +747,7 @@ fn main() {
     // parses the provided command line arguments and uses them to
     // obtain structured values
     let args = Args::parse();
-    let mode: GameBoyMode = if args.mode == "auto" {
+    let mode = if args.mode == "auto" {
         GameBoyMode::Dmg
     } else {
         GameBoyMode::from_string(&args.mode)
@@ -760,7 +767,7 @@ fn main() {
     game_boy.set_dma_enabled(!args.no_dma);
     game_boy.set_timer_enabled(!args.no_timer);
     game_boy.attach_serial(device);
-    game_boy.load(true);
+    game_boy.load(!args.no_boot);
 
     // prints the current version of the emulator (informational message)
     println!("========= Boytacean =========\n{}", game_boy);

frontends/sdl/src/test.rs

@@ -64,11 +64,37 @@ mod tests {
             Some(50000000),
             TestOptions::default(),
         );
-        compare_images(&result, "res/test/blargg/instr_timing/instr_timing.png");
         let image_result = compare_images(&result, "res/test/blargg/instr_timing/instr_timing.png");
         assert_eq!(image_result, true);
     }
 
+    #[test]
+    fn test_blargg_dmg_sound() {
+        let result: [u8; FRAME_BUFFER_SIZE] = run_image_test(
+            "../../res/roms/test/blargg/dmg_sound/01-registers.gb",
+            Some(50000000),
+            TestOptions::default(),
+        );
+        let image_result = compare_images(&result, "res/test/blargg/dmg_sound/01-registers.png");
+        assert_eq!(image_result, true);
+
+        let result: [u8; FRAME_BUFFER_SIZE] = run_image_test(
+            "../../res/roms/test/blargg/dmg_sound/02-len ctr.gb",
+            Some(50000000),
+            TestOptions::default(),
+        );
+        let image_result = compare_images(&result, "res/test/blargg/dmg_sound/02-len ctr.png");
+        assert_eq!(image_result, true);
+
+        let result: [u8; FRAME_BUFFER_SIZE] = run_image_test(
+            "../../res/roms/test/blargg/dmg_sound/03-trigger.gb",
+            Some(100000000),
+            TestOptions::default(),
+        );
+        let image_result = compare_images(&result, "res/test/blargg/dmg_sound/03-trigger.png");
+        assert_eq!(image_result, true);
+    }
+
     #[test]
     fn test_dmg_acid2() {
         let result: [u8; FRAME_BUFFER_SIZE] = run_image_test(