Auto merge of #198 - nkanderson:160_options1-add-test, r=komaeda

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.
This commit is contained in:
bors 2019-07-27 10:32:47 +00:00
commit 1890b0019d
2 changed files with 11 additions and 2 deletions

View File

@ -4,7 +4,7 @@
// on `None`. Handle this in a more graceful way than calling `unwrap`! // on `None`. Handle this in a more graceful way than calling `unwrap`!
// Scroll down for hints :) // Scroll down for hints :)
fn main() { pub fn pop_too_much() -> bool {
let mut list = vec![3]; let mut list = vec![3];
let last = list.pop().unwrap(); let last = list.pop().unwrap();
@ -15,9 +15,18 @@ fn main() {
"The second-to-last item in the list is {:?}", "The second-to-last item in the list is {:?}",
second_to_last second_to_last
); );
true
} }
#[cfg(test)]
mod tests {
use super::*;
#[test]
fn should_not_panic() {
assert!(pop_too_much(), true);
}
}

View File

@ -192,7 +192,7 @@ mode = "test"
[[exercises]] [[exercises]]
path = "exercises/error_handling/option1.rs" path = "exercises/error_handling/option1.rs"
mode = "compile" mode = "test"
[[exercises]] [[exercises]]
path = "exercises/error_handling/result1.rs" path = "exercises/error_handling/result1.rs"