Remove unstable feature

Currently it is not possible to run the test suite with `cargo test
--all-features`, this is because we use a feature called `unstable` to
enable the unstable `test` crate that is used for benchmarking.

There is another way to conditionally enable the test crate and guard
the benchmark code. We can use a custom configuration option `bench`
and then set it using `RUSTFLAGS='--cfg=bench`.

Author: tcharding

Cargo.toml

@@ -17,7 +17,6 @@ no-std = ["hashbrown", "bitcoin/no-std"]
 compiler = []
 trace = []
 
-unstable = []
 serde = ["actual-serde", "bitcoin/serde"]
 rand = ["bitcoin/rand"]
 

contrib/test.sh

@@ -10,16 +10,20 @@ cargo update -p serde_derive --precise 1.0.142
 cargo --version
 rustc --version
 
-# Work out if we are using a nightly toolchain.
+# Cache the toolchain we are using.
+NIGHTLY=false
+if cargo --version | grep nightly; then
+    NIGHTLY=true
+fi
 MSRV=false
 if cargo --version | grep "1\.41\.0"; then
     MSRV=true
 fi
 
+# Do pinning based on toolchain.
 if cargo --version | grep "1\.47\.0"; then
     cargo update -p once_cell --precise 1.13.1
 fi
-
 # form_urlencoded 1.1.0 breaks MSRV.
 if [ "$MSRV" = true ]; then
     cargo update -p url --precise 2.2.2
@@ -88,10 +92,18 @@ then
   done
 fi
 
-# Bench if told to (this only works with the nightly toolchain)
+# Bench if told to, only works with non-stable toolchain (nightly, beta).
 if [ "$DO_BENCH" = true ]
 then
-    cargo bench --features="unstable compiler"
+    if [ "$NIGHTLY" = false ]; then
+        if [ -n "$RUSTUP_TOOLCHAIN" ]; then
+            echo "RUSTUP_TOOLCHAIN is set to a non-nightly toolchain but DO_BENCH requires a nightly toolchain"
+        else
+            echo "DO_BENCH requires a nightly toolchain"
+        fi
+        exit 1
+    fi
+    RUSTFLAGS='--cfg=bench' cargo bench
 fi
 
 # Build the docs if told to (this only works with the nightly toolchain)

src/lib.rs

@@ -79,7 +79,8 @@
 //!
 
 #![cfg_attr(all(not(feature = "std"), not(test)), no_std)]
-#![cfg_attr(all(test, feature = "unstable"), feature(test))]
+// Experimental features we need.
+#![cfg_attr(bench, feature(test))]
 // Coding conventions
 #![deny(unsafe_code)]
 #![deny(non_upper_case_globals)]
@@ -109,7 +110,8 @@ extern crate core;
 
 #[cfg(feature = "serde")]
 pub use actual_serde as serde;
-#[cfg(all(test, feature = "unstable"))]
+
+#[cfg(bench)]
 extern crate test;
 
 #[macro_use]

src/policy/compiler.rs

@@ -1613,7 +1613,7 @@ mod tests {
     }
 }
 
-#[cfg(all(test, feature = "unstable"))]
+#[cfg(bench)]
 mod benches {
     use std::str::FromStr;
 

src/policy/mod.rs

@@ -553,7 +553,7 @@ mod tests {
     }
 }
 
-#[cfg(all(test, feature = "compiler", feature = "unstable"))]
+#[cfg(bench)]
 mod benches {
     use core::str::FromStr;