Jiri Daněk created PROTON-1675:
----------------------------------

             Summary: [cpp] Attempt to listen on invalid URL will not throw 
exception
                 Key: PROTON-1675
                 URL: https://issues.apache.org/jira/browse/PROTON-1675
             Project: Qpid Proton
          Issue Type: Bug
          Components: cpp-binding
    Affects Versions: proton-j-0.18.0
            Reporter: Jiri Daněk
            Assignee: Cliff Jansen
            Priority: Major


The test {{container_test.cpp/test_container_bad_address()}} is not checking 
that expected exceptions are actually thrown. If I add these checks, the test 
starts failing.

{noformat}
diff --git a/proton-c/bindings/cpp/src/container_test.cpp 
b/proton-c/bindings/cpp/src/container_test.cpp
index eb05ddbd..d1dc6628 100644
--- a/proton-c/bindings/cpp/src/container_test.cpp
+++ b/proton-c/bindings/cpp/src/container_test.cpp
@@ -176,12 +176,18 @@ int test_container_bad_address() {
 
     proton::container c;
     // Default fixed-option listener. Valgrind for leaks.
-    try { c.listen("999.666.999.666:0"); } catch (const proton::error&) {}
+    try {
+        c.listen("999.666.999.666:0");
+        FAIL("exception expected");
+    } catch (const proton::error&) {}
     c.run();
     // Dummy listener.
     test_listen_handler l;
     test_handler h2("999.999.999.666", proton::connection_options());
-    try { c.listen("999.666.999.666:0", l); } catch (const proton::error&) {}
+    try {
+        c.listen("999.666.999.666:0", l);
+        FAIL("exception expected");
+    } catch (const proton::error&) {}
     c.run();
     ASSERT(!l.on_open_);
     ASSERT(!l.on_accept_);
{noformat}

No exception is ever thrown: invalid IP, port is already occupied, port is < 
1025 and I am not root... never.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to