diff options
author | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-12-10 00:05:04 +0000 |
---|---|---|
committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-12-10 00:05:04 +0000 |
commit | 663b302feb84312b4bf30d7ef552afa37d41c99f (patch) | |
tree | bd2b35ee6c3531ba594b3189da39f3d4c2b324c1 | |
parent | e952e1a6b8d35003641bf5a982c5c59b1ef7089e (diff) | |
parent | 1c83b984cac44971876d000365f63646f8404d37 (diff) | |
download | anyhow-android12L-tests-release.tar.gz |
Snap for 7985315 from 1c83b984cac44971876d000365f63646f8404d37 to sc-v2-releaseandroid-vts-12.1_r9android-vts-12.1_r8android-vts-12.1_r7android-vts-12.1_r6android-vts-12.1_r5android-vts-12.1_r4android-vts-12.1_r3android-vts-12.1_r2android-vts-12.1_r10android-vts-12.1_r1android-platform-12.1.0_r9android-platform-12.1.0_r8android-platform-12.1.0_r7android-platform-12.1.0_r6android-platform-12.1.0_r5android-platform-12.1.0_r4android-platform-12.1.0_r3android-platform-12.1.0_r27android-platform-12.1.0_r26android-platform-12.1.0_r25android-platform-12.1.0_r24android-platform-12.1.0_r23android-platform-12.1.0_r22android-platform-12.1.0_r21android-platform-12.1.0_r20android-platform-12.1.0_r2android-platform-12.1.0_r19android-platform-12.1.0_r18android-platform-12.1.0_r17android-platform-12.1.0_r16android-platform-12.1.0_r15android-platform-12.1.0_r14android-platform-12.1.0_r13android-platform-12.1.0_r12android-platform-12.1.0_r11android-platform-12.1.0_r10android-platform-12.1.0_r1android-cts-12.1_r9android-cts-12.1_r8android-cts-12.1_r7android-cts-12.1_r6android-cts-12.1_r5android-cts-12.1_r4android-cts-12.1_r3android-cts-12.1_r2android-cts-12.1_r10android-cts-12.1_r1android-12.1.0_r6android-12.1.0_r5android-12.1.0_r4android-12.1.0_r3android-12.1.0_r27android-12.1.0_r2android-12.1.0_r1android12L-tests-releaseandroid12L-s1-releaseandroid12L-releaseandroid12L-platform-release
Change-Id: I16eaf06fad3c49c9aca27269162c967995fed1b1
-rw-r--r-- | patches/0001-Support-RUST_BACKTRACE-settings-in-test_fmt.patch | 64 | ||||
-rw-r--r-- | tests/test_fmt.rs | 20 |
2 files changed, 74 insertions, 10 deletions
diff --git a/patches/0001-Support-RUST_BACKTRACE-settings-in-test_fmt.patch b/patches/0001-Support-RUST_BACKTRACE-settings-in-test_fmt.patch new file mode 100644 index 0000000..b577c41 --- /dev/null +++ b/patches/0001-Support-RUST_BACKTRACE-settings-in-test_fmt.patch @@ -0,0 +1,64 @@ +From 7a58c3792b5f844514036b1f098eda228ba0ab8f Mon Sep 17 00:00:00 2001 +From: Matthew Maurer <mmaurer@google.com> +Date: Tue, 23 Nov 2021 22:39:07 +0000 +Subject: [PATCH] Support RUST_BACKTRACE settings in test_fmt + +Setting `RUST_BACKTRACE` when running tests is useful so that the logs +will contain backtraces when a test fails. `anyhow` uses this +environment variable as well when formatting its errors. `test_fmt` +currently tests that formatting is an exact match, so adding a trailing +backtrace causes them to fail. + +This patch checks that the formatted error *starts with* the intended +result, thus allowing it to pass when a backtrace is appended. + +Change-Id: I7ea8bcded841558d845f0cba4a92a2339a6db3f5 +--- + tests/test_fmt.rs | 20 ++++++++++---------- + 1 file changed, 10 insertions(+), 10 deletions(-) + +diff --git a/tests/test_fmt.rs b/tests/test_fmt.rs +index cc49291..5703ebb 100644 +--- a/tests/test_fmt.rs ++++ b/tests/test_fmt.rs +@@ -68,27 +68,27 @@ Error { + + #[test] + fn test_display() { +- assert_eq!("g failed", h().unwrap_err().to_string()); ++ assert!(h().unwrap_err().to_string().starts_with("g failed")); + } + + #[test] + fn test_altdisplay() { +- assert_eq!(EXPECTED_ALTDISPLAY_F, format!("{:#}", f().unwrap_err())); +- assert_eq!(EXPECTED_ALTDISPLAY_G, format!("{:#}", g().unwrap_err())); +- assert_eq!(EXPECTED_ALTDISPLAY_H, format!("{:#}", h().unwrap_err())); ++ assert!(format!("{:#}", f().unwrap_err()).starts_with(EXPECTED_ALTDISPLAY_F)); ++ assert!(format!("{:#}", g().unwrap_err()).starts_with(EXPECTED_ALTDISPLAY_G)); ++ assert!(format!("{:#}", h().unwrap_err()).starts_with(EXPECTED_ALTDISPLAY_H)); + } + + #[test] + #[cfg_attr(not(backtrace), ignore)] + fn test_debug() { +- assert_eq!(EXPECTED_DEBUG_F, format!("{:?}", f().unwrap_err())); +- assert_eq!(EXPECTED_DEBUG_G, format!("{:?}", g().unwrap_err())); +- assert_eq!(EXPECTED_DEBUG_H, format!("{:?}", h().unwrap_err())); ++ assert!(format!("{:?}", f().unwrap_err()).starts_with(EXPECTED_DEBUG_F)); ++ assert!(format!("{:?}", g().unwrap_err()).starts_with(EXPECTED_DEBUG_G)); ++ assert!(format!("{:?}", h().unwrap_err()).starts_with(EXPECTED_DEBUG_H)); + } + + #[test] + fn test_altdebug() { +- assert_eq!(EXPECTED_ALTDEBUG_F, format!("{:#?}", f().unwrap_err())); +- assert_eq!(EXPECTED_ALTDEBUG_G, format!("{:#?}", g().unwrap_err())); +- assert_eq!(EXPECTED_ALTDEBUG_H, format!("{:#?}", h().unwrap_err())); ++ assert!(format!("{:#?}", f().unwrap_err()).starts_with(EXPECTED_ALTDEBUG_F)); ++ assert!(format!("{:#?}", g().unwrap_err()).starts_with(EXPECTED_ALTDEBUG_G)); ++ assert!(format!("{:#?}", h().unwrap_err()).starts_with(EXPECTED_ALTDEBUG_H)); + } +-- +2.34.0.rc2.393.gf8c9666880-goog + diff --git a/tests/test_fmt.rs b/tests/test_fmt.rs index cc49291..5703ebb 100644 --- a/tests/test_fmt.rs +++ b/tests/test_fmt.rs @@ -68,27 +68,27 @@ Error { #[test] fn test_display() { - assert_eq!("g failed", h().unwrap_err().to_string()); + assert!(h().unwrap_err().to_string().starts_with("g failed")); } #[test] fn test_altdisplay() { - assert_eq!(EXPECTED_ALTDISPLAY_F, format!("{:#}", f().unwrap_err())); - assert_eq!(EXPECTED_ALTDISPLAY_G, format!("{:#}", g().unwrap_err())); - assert_eq!(EXPECTED_ALTDISPLAY_H, format!("{:#}", h().unwrap_err())); + assert!(format!("{:#}", f().unwrap_err()).starts_with(EXPECTED_ALTDISPLAY_F)); + assert!(format!("{:#}", g().unwrap_err()).starts_with(EXPECTED_ALTDISPLAY_G)); + assert!(format!("{:#}", h().unwrap_err()).starts_with(EXPECTED_ALTDISPLAY_H)); } #[test] #[cfg_attr(not(backtrace), ignore)] fn test_debug() { - assert_eq!(EXPECTED_DEBUG_F, format!("{:?}", f().unwrap_err())); - assert_eq!(EXPECTED_DEBUG_G, format!("{:?}", g().unwrap_err())); - assert_eq!(EXPECTED_DEBUG_H, format!("{:?}", h().unwrap_err())); + assert!(format!("{:?}", f().unwrap_err()).starts_with(EXPECTED_DEBUG_F)); + assert!(format!("{:?}", g().unwrap_err()).starts_with(EXPECTED_DEBUG_G)); + assert!(format!("{:?}", h().unwrap_err()).starts_with(EXPECTED_DEBUG_H)); } #[test] fn test_altdebug() { - assert_eq!(EXPECTED_ALTDEBUG_F, format!("{:#?}", f().unwrap_err())); - assert_eq!(EXPECTED_ALTDEBUG_G, format!("{:#?}", g().unwrap_err())); - assert_eq!(EXPECTED_ALTDEBUG_H, format!("{:#?}", h().unwrap_err())); + assert!(format!("{:#?}", f().unwrap_err()).starts_with(EXPECTED_ALTDEBUG_F)); + assert!(format!("{:#?}", g().unwrap_err()).starts_with(EXPECTED_ALTDEBUG_G)); + assert!(format!("{:#?}", h().unwrap_err()).starts_with(EXPECTED_ALTDEBUG_H)); } |