fix(test2): Capture names as full paths

Regarding the bug shown visible in the test in the previous commit,
where two passing tests both named `foo` would give output that
only 1 test passed: The test macros should now always give unique
names to all tests, so this is no longer visible when using the macros.
However, unique test names is noo guaranteed when using the imperative
API, so the bug could still happen (just not when using the macros).

Author: skogseth

crates/libtest2/src/macros.rs

@@ -110,7 +110,9 @@ macro_rules! _test_parse {
             fn name(&self) -> &str {
                 $crate::_private::push!(crate::TESTS, _: $crate::_private::DynCase = $crate::_private::DynCase(&$name));
 
-                stringify!($name)
+                const FULL_PATH: &str = concat!(std::module_path!(), "::", stringify!($name));
+                let i = FULL_PATH.find("::").expect("we have inserted this in the line above so it must be there");
+                &FULL_PATH[(i+2)..]
             }
             fn kind(&self) -> $crate::_private::TestKind {
                 Default::default()

crates/libtest2/tests/testsuite/macros.rs

@@ -30,10 +30,10 @@ fn check() {
     let data = str![[r#"
 
 running 2 tests
-test foo ... ok
-test foo ... ok
+test foo              ... ok
+test some_module::foo ... ok
 
-test result: ok. 1 passed; 0 failed; 0 ignored; 0 filtered out; finished in [..]s
+test result: ok. 2 passed; 0 failed; 0 ignored; 0 filtered out; finished in [..]s
 
 
 "#]];