Have Travis run (some) tests on stable Rust.

Author: SimonSapin

.travis.yml

@@ -5,11 +5,10 @@ rust:
   - beta
   - stable
 script:
-  - "if [ $TRAVIS_RUST_VERSION != nightly ]; then cargo build; fi"
-  - "if [ $TRAVIS_RUST_VERSION = nightly ]; then cargo test; fi"
-  - "if [ $TRAVIS_RUST_VERSION = nightly ]; then cargo clean; fi"
-  - "if [ $TRAVIS_RUST_VERSION = nightly ]; then cargo test --features log-events; fi"
+  - cargo build
+  - cargo test
+  - cargo test --features log-events
   - "if [ $TRAVIS_RUST_VERSION = nightly ]; then cargo test --features unstable; fi"
-  - "if [ $TRAVIS_RUST_VERSION = nightly ]; then cd examples/summarize-events/ && cargo build; fi"
+  - "cd examples/summarize-events/ && cargo build"
 notifications:
   webhooks: http://build.servo.org:54856/travis

src/atom/mod.rs

@@ -315,15 +315,15 @@ impl Deserialize for Atom {
     }
 }
 
-#[cfg(test)]
+#[cfg(all(test, feature = "unstable"))]
 mod bench;
 
 #[cfg(test)]
 mod tests {
     use std::mem;
     use std::thread;
     use super::{Atom, StringCacheEntry};
-    use string_cache_shared::{Static, Inline, Dynamic, ENTRY_ALIGNMENT, from_packed_dynamic};
+    use string_cache_shared::{Static, Inline, Dynamic, ENTRY_ALIGNMENT};
 
     #[test]
     fn test_as_slice() {
@@ -546,8 +546,10 @@ mod tests {
     /// Atom uses #[unsafe_no_drop_flag] to stay small, so drop() may be called more than once.
     /// In calls after the first one, the atom will be filled with a POST_DROP value.
     /// drop() must be a no-op in this case.
+    #[cfg(feature = "unstable")]
     #[test]
     fn atom_drop_is_idempotent() {
+        use string_cache_shared::from_packed_dynamic;
         unsafe {
             assert_eq!(from_packed_dynamic(mem::POST_DROP_U64), None);
         }

src/lib.rs

@@ -11,11 +11,11 @@
 #![crate_type = "rlib"]
 
 #![cfg_attr(test, deny(warnings))]
-#![cfg_attr(test, feature(test, filling_drop))]
+#![cfg_attr(all(test, feature = "unstable"), feature(test, filling_drop))]
 #![cfg_attr(feature = "unstable", feature(unsafe_no_drop_flag, plugin))]
 #![cfg_attr(feature = "unstable", plugin(string_cache_plugin))]
 
-#[cfg(test)]
+#[cfg(all(test, feature = "unstable"))]
 extern crate test;
 
 #[macro_use]