From a750e4a1a3006227292bb17d57d78ce84da6bfc6 Mon Sep 17 00:00:00 2001 From: Niklas Anderson Date: Fri, 26 Jul 2019 16:44:10 -0700 Subject: [PATCH] fix(option1): Add test for prematurely passing exercise Fixes the bug referenced in #160, but does not address the larger feature work referenced by the issue. --- exercises/error_handling/option1.rs | 11 ++++++++++- info.toml | 2 +- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/exercises/error_handling/option1.rs b/exercises/error_handling/option1.rs index 13fc7202..d96a8710 100644 --- a/exercises/error_handling/option1.rs +++ b/exercises/error_handling/option1.rs @@ -4,7 +4,7 @@ // on `None`. Handle this in a more graceful way than calling `unwrap`! // Scroll down for hints :) -fn main() { +pub fn pop_too_much() -> bool { let mut list = vec![3]; let last = list.pop().unwrap(); @@ -15,9 +15,18 @@ fn main() { "The second-to-last item in the list is {:?}", second_to_last ); + true } +#[cfg(test)] +mod tests { + use super::*; + #[test] + fn should_not_panic() { + assert!(pop_too_much(), true); + } +} diff --git a/info.toml b/info.toml index 6c4f6398..d7f86f9b 100644 --- a/info.toml +++ b/info.toml @@ -192,7 +192,7 @@ mode = "test" [[exercises]] path = "exercises/error_handling/option1.rs" -mode = "compile" +mode = "test" [[exercises]] path = "exercises/error_handling/result1.rs"