converted all ftests to use ignore annotation

CHANGELOG.md

@@ -13,6 +13,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - Removed many of the unwraps in named binary
 - Reworked all errors to use error-chain
 - Adjusted interface for Signer to use duration
+- All `#[cfg(ftest)]` tests now `#[ignore]`
 
 ### Fixed
 - TXT record case sensitivity

Cargo.toml

@@ -35,9 +35,6 @@ license = "MIT OR Apache-2.0"
 build = "build.rs"
 
 [features]
-# run functional tests with tests
-ftest = []
-
 
 [lib]
 name = "trust_dns"

README.md

@@ -149,7 +149,7 @@ presume that the trust-dns repos have already been synced to the local system:
     and also make some remote requests to verify compatibility with other DNS
     systems. These can not currently be run on Travis for example.
 
-        $ cargo test --features=ftest
+        $ cargo test -- --ignored
 
 -   Benchmarks
 

src/client/client.rs

@@ -1040,9 +1040,7 @@ mod test {
   use ::op::ResponseCode;
   use ::rr::{DNSClass, Record, RecordType, domain, RData};
   use ::rr::dnssec::{Algorithm, Signer, TrustAnchor};
-  #[cfg(feature = "ftest")]
   use ::tcp::TcpClientConnection;
-  #[cfg(feature = "ftest")]
   use ::udp::UdpClientConnection;
 
   #[test]
@@ -1057,7 +1055,7 @@ mod test {
   }
 
   #[test]
-  #[cfg(feature = "ftest")]
+  #[ignore]
   fn test_query_udp() {
     let addr: SocketAddr = ("8.8.8.8",53).to_socket_addrs().unwrap().next().unwrap();
     let conn = UdpClientConnection::new(addr).unwrap();
@@ -1067,7 +1065,7 @@ mod test {
   }
 
   #[test]
-  #[cfg(feature = "ftest")]
+  #[ignore]
   fn test_query_tcp() {
     let addr: SocketAddr = ("8.8.8.8",53).to_socket_addrs().unwrap().next().unwrap();
     let conn = TcpClientConnection::new(addr).unwrap();
@@ -1122,7 +1120,7 @@ mod test {
   }
 
   #[test]
-  #[cfg(feature = "ftest")]
+  #[ignore]
   fn test_secure_query_example_udp() {
     let addr: SocketAddr = ("8.8.8.8",53).to_socket_addrs().unwrap().next().unwrap();
     let conn = UdpClientConnection::new(addr).unwrap();
@@ -1132,7 +1130,7 @@ mod test {
   }
 
   #[test]
-  #[cfg(feature = "ftest")]
+  #[ignore]
   fn test_secure_query_example_tcp() {
     let addr: SocketAddr = ("8.8.8.8",53).to_socket_addrs().unwrap().next().unwrap();
     let conn = TcpClientConnection::new(addr).unwrap();
@@ -1188,7 +1186,7 @@ mod test {
   }
 
   #[test]
-  #[cfg(feature = "ftest")]
+  #[ignore]
   fn test_nsec_query_example_udp() {
     let addr: SocketAddr = ("8.8.8.8",53).to_socket_addrs().unwrap().next().unwrap();
     let conn = UdpClientConnection::new(addr).unwrap();
@@ -1197,7 +1195,7 @@ mod test {
   }
 
   #[test]
-  #[cfg(feature = "ftest")]
+  #[ignore]
   fn test_nsec_query_example_tcp() {
     let addr: SocketAddr = ("8.8.8.8",53).to_socket_addrs().unwrap().next().unwrap();
     let conn = TcpClientConnection::new(addr).unwrap();
@@ -1219,7 +1217,7 @@ mod test {
 
 
   #[test]
-  #[cfg(feature = "ftest")]
+  #[ignore]
   fn test_nsec_query_type() {
     let name = domain::Name::with_labels(vec!["www".to_string(), "example".to_string(), "com".to_string()]);
 
@@ -1240,7 +1238,7 @@ mod test {
   //
   // TODO these NSEC3 tests don't work, it seems that the zone is not signed properly.
   // #[test]
-  // #[cfg(feature = "ftest")]
+  // #[ignore]
   // fn test_nsec3_sdsmt() {
   //   let addr: SocketAddr = ("75.75.75.75",53).to_socket_addrs().unwrap().next().unwrap();
   //   let conn = TcpClientConnection::new(addr).unwrap();
@@ -1257,7 +1255,7 @@ mod test {
   // TODO: disabled until I decide what to do with NSEC3 see issue #10
   //
   // #[test]
-  // #[cfg(feature = "ftest")]
+  // #[ignore]
   // fn test_nsec3_sdsmt_type() {
   //   let addr: SocketAddr = ("75.75.75.75",53).to_socket_addrs().unwrap().next().unwrap();
   //   let conn = TcpClientConnection::new(addr).unwrap();

src/server/server.rs

@@ -336,9 +336,7 @@ mod server_tests {
   use ::op::*;
   use ::client::{Client, ClientConnection};
   use ::udp::UdpClientConnection;
-  #[cfg(feature = "ftest")]
   use ::tcp::TcpClientConnection;
-  #[cfg(feature = "ftest")]
   use mio::tcp::TcpListener;
 
   #[test]
@@ -362,7 +360,7 @@ mod server_tests {
   }
 
   #[test]
-  #[cfg(feature = "ftest")]
+  #[ignore]
   fn test_server_www_tcp() {
     use mio::tcp::TcpListener;
 
@@ -433,7 +431,6 @@ mod server_tests {
     server.listen().unwrap();
   }
 
-  #[cfg(feature = "ftest")]
   fn server_thread_tcp(tcp_listener: TcpListener) {
     let catalog = new_catalog();
     let mut server = Server::new(catalog);