Report span of test when should_panic test failed

Signed-off-by: xizheyin <[email protected]>

Author: xizheyin

library/test/src/test_result.rs

@@ -72,7 +72,8 @@ pub(crate) fn calc_result<'a>(
             }
         }
         (&ShouldPanic::Yes, Ok(())) | (&ShouldPanic::YesWithMessage(_), Ok(())) => {
-            TestResult::TrFailedMsg("test did not panic as expected".to_string())
+            let span_str = format!("{}:{}:{}", desc.source_file, desc.start_line, desc.start_col);
+            TestResult::TrFailedMsg(format!("test did not panic as expected at {}", span_str))
         }
         _ => TestResult::TrFailed,
     };

tests/ui/test-should-panic-failed-show-span.rs

@@ -0,0 +1,30 @@
+//@ run-fail
+//@ check-run-results
+//@ compile-flags: --test
+
+
+#[test]
+#[should_panic]
+fn do_panic1() {
+    panic!("Panic!");
+}
+
+#[test]
+#[should_panic = "SubString"]
+fn do_panic2() {
+    // DON'T PANIC
+    panic!("SubString");
+}
+
+#[test]
+#[should_panic]
+fn dont_panic1() {
+    // DON'T PANIC
+}
+
+
+#[test]
+#[should_panic = "SubString"]
+fn dont_panic2() {
+    // DON'T PANIC
+}

tests/ui/test-should-panic-failed-show-span.run.stdout

@@ -0,0 +1,20 @@
+
+running 4 tests
+test dont_panic1 - should panic ... FAILED
+test do_panic2 - should panic ... ok
+test do_panic1 - should panic ... ok
+test dont_panic2 - should panic ... FAILED
+
+failures:
+
+---- dont_panic1 stdout ----
+note: test did not panic as expected at $DIR/test-should-panic-failed-show-span.rs:21:4
+---- dont_panic2 stdout ----
+note: test did not panic as expected at $DIR/test-should-panic-failed-show-span.rs:28:4
+
+failures:
+    dont_panic1
+    dont_panic2
+
+test result: FAILED. 2 passed; 2 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
+