test: Add port validation check after openURL

Author: joaomariolago

src/hal/link/desktop/abstract-link.cpp

@@ -17,6 +17,7 @@ const char* AbstractLink::_urlStringRegex = R"((?P<type>udp|serial):(?P<host>.*)
 std::shared_ptr<AbstractLink> AbstractLink::openUrl(const std::string& url)
 {
     if (url.empty()) {
+        std::cerr << "Empty URL was provided when trying to open a link" << std::endl;
         return {};
     }
 
@@ -30,6 +31,7 @@ std::shared_ptr<AbstractLink> AbstractLink::openUrl(const std::string& url)
     } urlStruct;
 
     if (!regex_search(url, match, regex)) {
+        std::cerr << "Invalid URL provided, should be in format `type:host:config`" << std::endl;
         return {};
     }
 

test/test-device-ping1d.cpp

@@ -18,6 +18,10 @@ int main(int argc, char* argv[])
     }
 
     auto port = AbstractLink::openUrl(CommandLine::self()->connectionString);
+    if (!port) {
+        std::cerr << "Failed to open communication link with device" << std::endl;
+        return -1;
+    }
     Ping1d device = Ping1d(*port.get());
 
     // Basic information

test/test-device-ping360.cpp

@@ -18,6 +18,10 @@ int main(int argc, char* argv[])
     }
 
     auto port = AbstractLink::openUrl(CommandLine::self()->connectionString);
+    if (!port) {
+        std::cerr << "Failed to open communication link with device" << std::endl;
+        return -1;
+    }
     Ping360 device = Ping360(*port.get());
 
     // Basic information

test/test-device.cpp

@@ -19,6 +19,10 @@ int main(int argc, char* argv[])
     }
 
     auto port = AbstractLink::openUrl(CommandLine::self()->connectionString);
+    if (!port) {
+        std::cerr << "Failed to open communication link with device" << std::endl;
+        return -1;
+    }
     PingDevice device = PingDevice(*port.get());
 
     // Basic information