DO NOT MERGE: warnings

.travis.yml

@@ -10,5 +10,9 @@ rust:
   - beta
   - nightly
 matrix:
+  include:
+    - rust: stable
+      env: DENYWARNINGS=1
   allow_failures:
+    - env: DENYWARNINGS=1
     - rust: nightly

_test/check-exercises.sh

@@ -1,9 +1,16 @@
 #!/bin/bash
 
-set -e
+# In DENYWARNINGS mode, do not set -e so that we run all tests.
+# This allows us to see all warnings.
+if [ -z "$DENYWARNINGS" ]; then
+    set -e
+fi
+
 tmp=${TMPDIR:-/tmp/}
 mkdir "${tmp}exercises"
 
+exitcode=0
+
 # An exercise worth testing is defined here as any top level directory with
 # a 'tests' directory
 for exercise in exercises/*/tests; do
@@ -19,22 +26,35 @@ for exercise in exercises/*/tests; do
     (
         cd $workdir
         [ -d src ] || mkdir src
+
         cp example.rs src/lib.rs
 
         # Forcibly strip all "ignore" statements from the testing files
         for test in tests/*.rs; do
             sed -i '/\[ignore\]/d' $test
         done
 
-        # Run the test and get the status
-        cargo test
+        if [ -n "$DENYWARNINGS" ]; then
+            sed -i -e '1i #![deny(warnings)]' src/lib.rs
+
+            # No-run mode so we see no test output.
+            # Quiet mode so we see no compile output
+            # (such as "Compiling"/"Downloading").
+            # Compiler errors will still be shown though.
+            # Both flags are necessary to keep things quiet.
+            cargo test --quiet --no-run
+        else
+            # Run the test and get the status
+            cargo test
+        fi
     )
 
     status=$?
 
     if [ $status -ne 0 ]
     then
-        echo "Failed";
-        return 1;
+        exitcode=1
     fi
 done
+
+exit $exitcode

exercises/tournament/example.rs

@@ -1,3 +1,5 @@
+use std::convert::AsRef;
+
 use std::cmp::Ordering::Equal;
 use std::collections::HashMap;
 use std::collections::hash_map::Entry;

exercises/word-count/example.rs

@@ -1,3 +1,5 @@
+use std::convert::AsRef;
+
 use std::collections::HashMap;
 use std::collections::hash_map::Entry;