Report span of test when should_panic test failed

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

Author: xizheyin

library/test/src/test_result.rs

@@ -45,6 +45,12 @@ pub(crate) fn calc_result<'a>(
     time_opts: Option<&time::TestTimeOptions>,
     exec_time: Option<&time::TestExecTime>,
 ) -> TestResult {
+    let fn_location = if !desc.source_file.is_empty() {
+        &format!(" at {}:{}:{}", desc.source_file, desc.start_line, desc.start_col)
+    } else {
+        ""
+    };
+
     let result = match (&desc.should_panic, task_result) {
         (&ShouldPanic::No, Ok(())) | (&ShouldPanic::Yes, Err(_)) => TestResult::TrOk,
         (&ShouldPanic::YesWithMessage(msg), Err(err)) => {
@@ -72,7 +78,7 @@ pub(crate) fn calc_result<'a>(
             }
         }
         (&ShouldPanic::Yes, Ok(())) | (&ShouldPanic::YesWithMessage(_), Ok(())) => {
-            TestResult::TrFailedMsg("test did not panic as expected".to_string())
+            TestResult::TrFailedMsg(format!("test did not panic as expected{}", fn_location))
         }
         _ => TestResult::TrFailed,
     };

tests/rustdoc-ui/doctest/failed-doctest-should-panic.stdout

@@ -5,7 +5,7 @@ test $DIR/failed-doctest-should-panic.rs - Foo (line 10) - should panic ... FAIL
 failures:
 
 ---- $DIR/failed-doctest-should-panic.rs - Foo (line 10) stdout ----
-note: test did not panic as expected
+note: test did not panic as expected at $DIR/failed-doctest-should-panic.rs:10:0
 
 failures:
     $DIR/failed-doctest-should-panic.rs - Foo (line 10)

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

@@ -0,0 +1,17 @@
+//@ run-fail
+//@ check-run-results
+//@ compile-flags: --test
+//@ exec-env: RUST_BACKTRACE=0
+//@ run-flags: --test-threads=1
+
+#[test]
+#[should_panic]
+fn should_panic_with_any_message_does_not_panic() {
+    // DON'T PANIC
+}
+
+#[test]
+#[should_panic = "message"]
+fn should_panic_with_message_does_not_panic() {
+    // DON'T PANIC
+}

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

@@ -0,0 +1,18 @@
+
+running 2 tests
+test should_panic_with_any_message_does_not_panic - should panic ... FAILED
+test should_panic_with_message_does_not_panic - should panic ... FAILED
+
+failures:
+
+---- should_panic_with_any_message_does_not_panic stdout ----
+note: test did not panic as expected at $DIR/test-should-panic-failed-show-span.rs:9:4
+---- should_panic_with_message_does_not_panic stdout ----
+note: test did not panic as expected at $DIR/test-should-panic-failed-show-span.rs:15:4
+
+failures:
+    should_panic_with_any_message_does_not_panic
+    should_panic_with_message_does_not_panic
+
+test result: FAILED. 0 passed; 2 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
+