commit d50f3d9292facf294e21eaca15275fa251d3b523
author Harvey Yang <chenghaoyang@chromium.org> Wed Dec 16 11:29:35 2020 +0800
committer Commit Bot <commit-bot@chromium.org> Wed Dec 16 19:38:07 2020 +0000
tree 79b0b3b1fd735f2c52f4b44e254fa40fea4f4cfa
parent 59aefc9f36fbc6b2bff95c7d25d5f5a42ac01511

power: Fix Fuzzer test with AmbientLightSensorDelegate

Fixes <6a598f040785> (power: Modify AmbientLightSensor and Add
AmbientLightSensorDelegate)

This commit updates AmbientLightSensorFuzzer with
AmbientLightSensorDelegate.

BUG=chromium:1158423
TEST=builds and run fuzzer test: powerd_als_fuzzer

Change-Id: I4fc9d7e5805ca93a9b6ba9cec2e71c071b91d8ea
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2593499
Reviewed-by: Cheng-Hao Yang <chenghaoyang@chromium.org>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>

power_manager/powerd/system/fuzzers/powerd_als_fuzzer.cc

diff --git a/power_manager/powerd/system/fuzzers/powerd_als_fuzzer.cc b/power_manager/powerd/system/fuzzers/powerd_als_fuzzer.cc
index e9d1e4b..b826ac9 100644
--- a/power_manager/powerd/system/fuzzers/powerd_als_fuzzer.cc
+++ b/power_manager/powerd/system/fuzzers/powerd_als_fuzzer.cc
@@ -3,6 +3,7 @@
 // found in the LICENSE file.
 
 #include "power_manager/powerd/system/ambient_light_sensor.h"
+#include "power_manager/powerd/system/ambient_light_sensor_file.h"
 
 #include <base/files/file_path.h>
 #include <base/files/file_util.h>
@@ -57,11 +58,18 @@
     base::FilePath loc0_file_ = device0_dir.Append("location");
     CHECK(brillo::WriteStringToFile(loc0_file_, "lid"));
 
-    sensor_ = std::make_unique<system::AmbientLightSensor>(SensorLocation::LID);
-    sensor_->set_device_list_path_for_testing(temp_dir_.GetPath());
+    sensor_ = std::make_unique<system::AmbientLightSensor>();
+
+    auto als = std::make_unique<system::AmbientLightSensorFile>(
+        SensorLocation::LID, false);
+    als_ = als.get();
+
+    sensor_->SetDelegate(std::move(als));
+    als_->set_device_list_path_for_testing(temp_dir_.GetPath());
   }
 
   std::unique_ptr<AmbientLightSensor> sensor_;
+  AmbientLightSensorFile* als_;
 
  protected:
   base::ScopedTempDir temp_dir_;
@@ -84,7 +92,7 @@
 
   base::TestMockTimeTaskRunner::ScopedContext scoped_context(task_runner.get());
   als_fuzzer->SetUp(fuzz_dp, is_color);
-  als_fuzzer->sensor_->Init(false /* read immediately */);
+  als_fuzzer->als_->Init(false /* read immediately */);
 
   // Move time ahead enough so that async file reads occur.
   task_runner->FastForwardBy(base::TimeDelta::FromMilliseconds(4000));