From e00faed85c496403b6bf4fa08292ee90291e9c1d Mon Sep 17 00:00:00 2001 From: Chirantan Ekbote Date: Tue, 12 Mar 2019 11:44:58 -0700 Subject: register_space: Only run panic test when debug assertions are on The regs_reg_overlap() test is a panic test but the function that it is testing only uses debug_asserts so the test will fail if debug assertions are disabled. Only run the test when debug assertions are enabled. BUG=chromium:940668 TEST=`FEATURES=test USE=-cros-debug emerge-nami crosvm` Change-Id: Ie722cb49908ae4c4a9ecc5f248a6ec25fbcc05c9 Reviewed-on: https://chromium-review.googlesource.com/1518729 Commit-Ready: ChromeOS CL Exonerator Bot Tested-by: kokoro Reviewed-by: Jeffrey Kardatzke Reviewed-by: Zach Reizner Reviewed-by: Dylan Reid --- devices/src/register_space/register_space.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/devices/src/register_space/register_space.rs b/devices/src/register_space/register_space.rs index df99f89..acb0df9 100644 --- a/devices/src/register_space/register_space.rs +++ b/devices/src/register_space/register_space.rs @@ -113,6 +113,7 @@ mod tests { #[test] #[should_panic] + #[cfg(debug_assertions)] fn regs_reg_overlap() { let mut regs = RegisterSpace::new(); regs.add_register(static_register!( -- cgit 1.4.1