summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNiklas Anderson <nkanders@gmail.com>2019-07-26 16:44:10 -0700
committerNiklas Anderson <nkanders@gmail.com>2019-07-26 16:58:12 -0700
commita750e4a1a3006227292bb17d57d78ce84da6bfc6 (patch)
tree46e62337487349a218a775ee9242cab6d5c7026b
parent1c789dda08a57e1a3292d42e2d63f2c4d631057d (diff)
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.
-rw-r--r--exercises/error_handling/option1.rs11
-rw-r--r--info.toml2
2 files changed, 11 insertions, 2 deletions
diff --git a/exercises/error_handling/option1.rs b/exercises/error_handling/option1.rs
index 13fc720..d96a871 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 6c4f639..d7f86f9 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"