cros: Added a fake fingerprint storage class.

chromeos/dbus/biod/fake_biod_client.h

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROMEOS_DBUS_BIOD_FAKE_BIOD_CLIENT_H_
 #define CHROMEOS_DBUS_BIOD_FAKE_BIOD_CLIENT_H_
 
+#include <map>
+#include <string>
+
 #include "base/macros.h"
 #include "base/observer_list.h"
 #include "chromeos/chromeos_export.h"
 #include "chromeos/dbus/biod/biod_client.h"
+#include "dbus/object_path.h"
 
 namespace dbus {
 class Bus;
-class ObjectPath;
 }  // namespace dbus
 
 namespace chromeos {
 
-// A fake implementation of BiodClient.
+class FakeBiodClientTest;
+
+// A fake implementation of BiodClient. It emulates the real Biod daemon by
+// providing the same API and storing fingerprints locally. A fingerprint is
+// represented by a vector of strings. During enrollment, fake enrollments are
+// sent as strings. If they are successful they get push backed to the current
+// fingerprint, until a completed enroll scan is sent. An attempt scan is also
+// sent with a string. If that string matches any string in the stored
+// fingerprint vector, it is considered a match.
 class CHROMEOS_EXPORT FakeBiodClient : public BiodClient {
  public:
+  // The current session of fingerprint storage. The session determines which
+  // events will be sent from user finger touches.
+  enum class FingerprintSession {

[Daniel Erat, 2017/04/07 23:40:56]

move this to the private section

[sammiequon, 2017/04/08 00:43:32]

Done.

+    NONE,
+    ENROLL,
+    AUTH,
+  };
+
   FakeBiodClient();
   ~FakeBiodClient() override;
 
   // Emulates the biod daemon by sending events which the daemon normally sends.
   // Notifies |observers_| about various events. These will be used in tests.
-  void SendEnrollScanDone(biod::ScanResult type_result, bool is_complete);
-  void SendAuthScanDone(biod::ScanResult type_result,
-                        const AuthScanMatches& matches);
+
+  // Emulates a scan that occurs during enrolling a new fingerprint.
+  // |fingerprint| is the fake data of the finger as a string. If |is_complete|
+  // is true the enroll session is finished, and the record is stored.
+  void SendEnrollScanDone(const std::string& fingerprint,
+                          biod::ScanResult type_result,
+                          bool is_complete);
+  // Emulates a scan that occurs during a authentication session. |fingerprint|
+  // is a string which represents the finger, and will be compared with all the
+  // stored fingerprints.
+  void SendAuthScanDone(const std::string& fingerprint,
+                        biod::ScanResult type_result);
   void SendSessionFailed();
 
   // BiodClient:
   void Init(dbus::Bus* bus) override;
   void AddObserver(Observer* observer) override;
   void RemoveObserver(Observer* observer) override;
   bool HasObserver(const Observer* observer) const override;
   void StartEnrollSession(const std::string& user_id,
                           const std::string& label,
                           const ObjectPathCallback& callback) override;
   void GetRecordsForUser(const std::string& user_id,
                          const UserRecordsCallback& callback) override;
   void DestroyAllRecords() override;
   void StartAuthSession(const ObjectPathCallback& callback) override;
   void RequestType(const BiometricTypeCallback& callback) override;
   void CancelEnrollSession(
       const dbus::ObjectPath& enroll_session_path) override;
   void EndAuthSession(const dbus::ObjectPath& auth_session_path) override;
   void SetRecordLabel(const dbus::ObjectPath& record_path,
                       const std::string& label) override;
   void RemoveRecord(const dbus::ObjectPath& record_path) override;
   void RequestRecordLabel(const dbus::ObjectPath& record_path,
                           const LabelCallback& callback) override;
 
+  // Clears all stored and current records from the fake storage.
+  void Reset();

[Daniel Erat, 2017/04/07 23:40:56]

nit: move this above overrides

[sammiequon, 2017/04/08 00:43:32]

Done.

+
  private:
+  friend FakeBiodClientTest;

[Daniel Erat, 2017/04/07 23:40:56]

consider not listing the test class as a friend, since it often encourages tests
to become overly reliant on the implementation. since this is a fake, you can
probably just add accessors for any members that need to be accessed, right?

[sammiequon, 2017/04/08 00:43:32]

Acknowledged.

+
+  // FakeRecord is the definition of a fake stored fingerprint template.
+  struct FakeRecord {
+    FakeRecord();
+    ~FakeRecord();
+
+    std::string user_id;
+    std::string label;
+    // A fake fingerprint is a vector which consists of all the strings which
+    // were "pressed" during the enroll session.
+    std::vector<std::string> fake_fingerprint;
+  };
+
+  // The stored fingerprints.
+  std::map<dbus::ObjectPath, std::unique_ptr<FakeRecord>> records_;
+
+  // Current record in process of getting enrolled and its path. These are
+  // assigned at the start of an enroll session and freed when the enroll
+  // session is finished or cancelled.
+  dbus::ObjectPath current_record_path_;
+  std::unique_ptr<FakeRecord> current_record_;
+
+  // Unique indentifier that gets updated each time an enroll session is started
+  // to ensure each record is stored at a different path.
+  int next_record_unique_id_ = 1;
+
+  // The current session of the fake storage.
+  FingerprintSession current_session_ = FingerprintSession::NONE;
+
   base::ObserverList<Observer> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(FakeBiodClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_BIOD_FAKE_BIOD_CLIENT_H_