Make Device::enumerate static

Device::enumerate does not need any instance data, therefore it is made
static.  Note that this not only changes the public API by making the
method static.  We also return all connected Nitrokey devices instead of
only Storage devices.

The NitrokeyManager method list_devices_by_cpuID is changed to check the
device type so that they still only return Storage devices.  The
list_device method now returns both Storage and Pro devices.

1 files changed, 6 insertions, 2 deletions

diff --git a/unittest/test_multiple_devices.cc b/unittest/test_multiple_devices.cc
index f9e9ad2..183af4f 100644
--- a/unittest/test_multiple_devices.cc
+++ b/unittest/test_multiple_devices.cc
@@ -35,9 +35,11 @@ using namespace nitrokey;
 
 TEST_CASE("List devices", "[BASIC]") {
     shared_ptr<Stick20> d = make_shared<Stick20>();
-    auto v = d->enumerate();
+    auto v = Device::enumerate();
     REQUIRE(v.size() > 0);
     for (auto i : v){
+        if (i.m_deviceModel != DeviceModel::STORAGE)
+            continue;
         auto a = i.m_path;
         std::cout << a;
         d->set_path(a);
@@ -54,11 +56,13 @@ TEST_CASE("List devices", "[BASIC]") {
 
 TEST_CASE("Regenerate AES keys", "[BASIC]") {
     shared_ptr<Stick20> d = make_shared<Stick20>();
-    auto v = d->enumerate();
+    auto v = Device::enumerate();
     REQUIRE(v.size() > 0);
 
     std::vector<shared_ptr<Stick20>> devices;
     for (auto i : v){
+        if (i.m_deviceModel != DeviceModel::STORAGE)
+            continue;
         auto a = i.m_path;
         std::cout << a << endl;
         d = make_shared<Stick20>();