Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ee7aac10 authored by William Escande's avatar William Escande
Browse files

Fix delete_old_snooz_log_files 

Sometimes the file wasn't delete because the StopAll was ran after the
bindOnce

Test: atest bluetooth_test_gd_unit
Bug: 269535256
Change-Id: Iaaa58dfcb0ad7cf8c853f91946dc7a355c7ef7e0
parent d241f526

system/gd/hal/snoop_logger_test.cc

+13 −0
Original line number Diff line number Diff line
@@ -420,6 +420,15 @@ TEST_F(SnoopLoggerModuleTest, snoop_log_persists) { 
  ASSERT_TRUE(std::filesystem::exists(temp_snooz_log_));

}



void sync_handler(bluetooth::os::Handler* handler) {

  std::promise<void> promise;

  auto future = promise.get_future();

  handler->Post(bluetooth::common::BindOnce(

      &std::promise<void>::set_value, bluetooth::common::Unretained(&promise)));

  auto future_status = future.wait_for(std::chrono::seconds(1));

  ASSERT_EQ(future_status, std::future_status::ready);

}



TEST_F(SnoopLoggerModuleTest, delete_old_snooz_log_files) {

  // Actual test

  auto* snoop_logger = new TestSnoopLoggerModule(
@@ -438,9 +447,13 @@ TEST_F(SnoopLoggerModuleTest, delete_old_snooz_log_files) { 
  handler->Post(bluetooth::common::BindOnce(fake_timerfd_advance, 10));

  ASSERT_TRUE(std::filesystem::exists(temp_snooz_log_));

  handler->Post(bluetooth::common::BindOnce(fake_timerfd_advance, 15));

  sync_handler(handler);

  handler->Post(bluetooth::common::BindOnce(

      [](std::filesystem::path path) { ASSERT_FALSE(std::filesystem::exists(path)); }, temp_snooz_log_));

  sync_handler(handler);



  test_registry->StopAll();

  ASSERT_FALSE(std::filesystem::exists(temp_snooz_log_));

}



TEST_F(SnoopLoggerModuleTest, rotate_file_at_new_session_test) {