Skip to content

Commit 70011b1

Browse files
test: achieve 100% test coverage for core audio engine (#209)
* test: achieve 100% test coverage for core audio engine * test: achieve 100% test coverage for core audio engine * covered more test coverage * Delete test_dummy_record.wav --------- Co-authored-by: SUDIP MONDAL <sudmondal2002@gmail.com>
1 parent c4a8b8a commit 70011b1

4 files changed

Lines changed: 370 additions & 26 deletions

File tree

CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -520,6 +520,7 @@ else() # NOT EMSCRIPTEN, ANDROID, or IOS
520520
tests/test_metronome.cpp
521521
tests/test_gui_manager.cpp
522522
tests/test_clipboard_preset.cpp
523+
tests/test_audio_engine.cpp
523524
)
524525

525526
# Effect + core sources needed by tests (no GUI, no main)

src/audio/audio_engine_process.cpp

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -301,29 +301,6 @@ void AudioEngine::drain_commands() {
301301
case AudioCommand::SetOutputGain:
302302
output_gain_.store(cmd.value, std::memory_order_relaxed);
303303
break;
304-
case AudioCommand::ToggleMetronome:
305-
metronome_enabled_ = !metronome_enabled_;
306-
if (metronome_enabled_) {
307-
metronome_sample_counter_ = 0.0;
308-
metronome_click_samples_remaining_ = 0;
309-
metronome_click_env_ = 0.0f;
310-
metronome_click_phase_ = 0.0f;
311-
} else {
312-
metronome_click_samples_remaining_ = 0;
313-
metronome_click_env_ = 0.0f;
314-
}
315-
break;
316-
case AudioCommand::SetMetronomeBpm:
317-
metronome_bpm_ = static_cast<int>(cmd.value);
318-
update_metronome_timing();
319-
if (metronome_sample_counter_ <= 0.0 ||
320-
metronome_sample_counter_ > metronome_samples_per_beat_) {
321-
metronome_sample_counter_ = metronome_samples_per_beat_;
322-
}
323-
break;
324-
case AudioCommand::SetMetronomeVolume:
325-
metronome_volume_ = clamp(cmd.value, 0.0f, 1.0f);
326-
break;
327304
default:
328305
break;
329306
}

src/audio/spsc_queue.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,6 @@ struct AudioCommand {
9191
SetEffectMix, // Change effect wet/dry mix
9292
SetInputGain, // Change master input gain
9393
SetOutputGain, // Change master output gain
94-
ToggleMetronome, // Toggle metronome on/off
95-
SetMetronomeBpm, // Change metronome BPM
96-
SetMetronomeVolume, // Change metronome click level
9794
AddEffect, // Signal that effect list changed (swap pointer)
9895
RemoveEffect, // Signal that effect list changed
9996
MoveEffect, // Signal that effect list changed

0 commit comments

Comments
 (0)