diff options
| author | Chirantan Ekbote <chirantan@chromium.org> | 2019-03-12 11:44:58 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2019-03-13 21:05:14 -0700 |
| commit | e00faed85c496403b6bf4fa08292ee90291e9c1d (patch) | |
| tree | 18ce07c2cd4d06c5c7f309f407bbdcd1817335c9 | |
| parent | 2727413c1064eef86056b0f30126622dcbef66e0 (diff) | |
| download | crosvm-e00faed85c496403b6bf4fa08292ee90291e9c1d.tar crosvm-e00faed85c496403b6bf4fa08292ee90291e9c1d.tar.gz crosvm-e00faed85c496403b6bf4fa08292ee90291e9c1d.tar.bz2 crosvm-e00faed85c496403b6bf4fa08292ee90291e9c1d.tar.lz crosvm-e00faed85c496403b6bf4fa08292ee90291e9c1d.tar.xz crosvm-e00faed85c496403b6bf4fa08292ee90291e9c1d.tar.zst crosvm-e00faed85c496403b6bf4fa08292ee90291e9c1d.zip | |
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 <chromiumos-cl-exonerator@appspot.gserviceaccount.com> Tested-by: kokoro <noreply+kokoro@google.com> Reviewed-by: Jeffrey Kardatzke <jkardatzke@google.com> Reviewed-by: Zach Reizner <zachr@chromium.org> Reviewed-by: Dylan Reid <dgreid@chromium.org>
| -rw-r--r-- | devices/src/register_space/register_space.rs | 1 |
1 files changed, 1 insertions, 0 deletions
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!( |