C++: Improve namespaces/namespaces test

Author: ian-semmle

cpp/ql/test/library-tests/namespaces/namespaces/decls.expected

@@ -1,18 +1,18 @@
-| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | __va_list_tag |
-| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | fp_offset |
-| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | gp_offset |
-| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | operator= |
-| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | operator= |
-| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | overflow_arg_area |
-| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | reg_save_area |
-| file://:0:0:0:0 | B | namespaces.cpp:32:7:32:7 | x |
-| namespaces.cpp:11:13:11:13 | C::D | file://:0:0:0:0 | p#0 |
-| namespaces.cpp:11:13:11:13 | C::D | file://:0:0:0:0 | p#0 |
-| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:13:17:13:17 | f |
-| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:15:12:15:12 | E |
-| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:15:12:15:12 | operator= |
-| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:15:12:15:12 | operator= |
-| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:17:12:17:12 | g |
-| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:17:18:17:18 | p |
-| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:18:12:18:12 | a |
-| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:20:13:20:13 | b |
+| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | __va_list_tag | __va_list_tag |
+| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | fp_offset | __va_list_tag::fp_offset |
+| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | gp_offset | __va_list_tag::gp_offset |
+| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | operator= | __va_list_tag::operator= |
+| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | operator= | __va_list_tag::operator= |
+| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | overflow_arg_area | __va_list_tag::overflow_arg_area |
+| file://:0:0:0:0 | (global namespace) | file://:0:0:0:0 | reg_save_area | __va_list_tag::reg_save_area |
+| file://:0:0:0:0 | B | namespaces.cpp:32:7:32:7 | x | <none> |
+| namespaces.cpp:11:13:11:13 | C::D | file://:0:0:0:0 | p#0 | <none> |
+| namespaces.cpp:11:13:11:13 | C::D | file://:0:0:0:0 | p#0 | <none> |
+| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:13:17:13:17 | f | C::D::f |
+| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:15:12:15:12 | E | C::D::E |
+| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:15:12:15:12 | operator= | C::D::E::operator= |
+| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:15:12:15:12 | operator= | C::D::E::operator= |
+| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:17:12:17:12 | g | C::D::E::g |
+| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:17:18:17:18 | p | <none> |
+| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:18:12:18:12 | a | <none> |
+| namespaces.cpp:11:13:11:13 | C::D | namespaces.cpp:20:13:20:13 | b | <none> |

cpp/ql/test/library-tests/namespaces/namespaces/decls.ql

@@ -1,6 +1,12 @@
 import cpp
 
+string qual(Declaration d) {
+  if exists(d.getQualifiedName())
+  then result = d.getQualifiedName()
+  else result = "<none>"
+}
+
 from Namespace n, Declaration d
 where n = d.getNamespace()
-select n, d
+select n, d, qual(d)