fix: handle string::npos case in a cleaner way and fix test

Author: gagik

packages/machine-id/binding.cc

@@ -56,9 +56,12 @@ namespace
     {
       return str;
     }
-
     std::string result = str;
-    result.erase(result.find_last_not_of(" \n\r\t") + 1);
+    size_t from_right = result.find_last_not_of(" \n\r\t");
+    if (from_right != std::string::npos)
+    {
+      result.erase(from_right + 1);
+    }
     result.erase(0, result.find_first_not_of(" \n\r\t"));
     return result;
   }

packages/machine-id/src/index.spec.ts

@@ -17,7 +17,7 @@ describe('machine-id', function () {
     let id: string;
 
     beforeEach(function () {
-      const deviceId = getMachineId();
+      const deviceId = getMachineId({ raw: true });
       assert(deviceId);
       id = deviceId;
     });