minor: remove dead_code annotations where possible (#103)

Author: saghm

src/client/auth/mod.rs

@@ -7,7 +7,7 @@ mod test;
 
 use std::{borrow::Cow, str::FromStr};
 
-use bson::{Bson, Document};
+use bson::Document;
 use rand::Rng;
 use typed_builder::TypedBuilder;
 
@@ -198,8 +198,10 @@ pub struct Credential {
 }
 
 impl Credential {
-    #[allow(dead_code)]
+    #[cfg(test)]
     pub(crate) fn into_document(mut self) -> Document {
+        use bson::Bson;
+
         let mut doc = Document::new();
 
         if let Some(s) = self.username.take() {

src/client/executor.rs

@@ -48,7 +48,6 @@ impl Client {
     /// Execute the given operation, optionally specifying a connection used to do so.
     /// If no connection is provided, server selection will performed using the criteria specified
     /// on the operation, if any.
-    #[allow(dead_code)]
     pub(crate) fn execute_operation<T: Operation>(
         &self,
         op: &T,

src/client/mod.rs

@@ -166,17 +166,14 @@ impl Client {
         }
     }
 
-    #[allow(dead_code)]
     pub(crate) fn send_command_started_event(&self, event: CommandStartedEvent) {
         self.emit_command_event(|handler| handler.handle_command_started_event(event.clone()));
     }
 
-    #[allow(dead_code)]
     pub(crate) fn send_command_succeeded_event(&self, event: CommandSucceededEvent) {
         self.emit_command_event(|handler| handler.handle_command_succeeded_event(event.clone()));
     }
 
-    #[allow(dead_code)]
     pub(crate) fn send_command_failed_event(&self, event: CommandFailedEvent) {
         self.emit_command_event(|handler| handler.handle_command_failed_event(event.clone()));
     }

src/operation/count/mod.rs

@@ -22,7 +22,7 @@ impl Count {
         Count { ns, options }
     }
 
-    #[allow(dead_code)]
+    #[cfg(test)]
     pub(crate) fn empty() -> Self {
         Count {
             ns: Namespace {

src/operation/create/mod.rs

@@ -18,7 +18,7 @@ pub(crate) struct Create {
 }
 
 impl Create {
-    #[allow(dead_code)]
+    #[cfg(test)]
     fn empty() -> Self {
         Self::new(
             Namespace {

src/operation/delete/mod.rs

@@ -23,7 +23,7 @@ pub(crate) struct Delete {
 }
 
 impl Delete {
-    #[allow(dead_code)]
+    #[cfg(test)]
     fn empty() -> Self {
         Self::new(
             Namespace {

src/operation/distinct/mod.rs

@@ -34,7 +34,7 @@ impl Distinct {
         }
     }
 
-    #[allow(dead_code)]
+    #[cfg(test)]
     pub(crate) fn empty() -> Self {
         Distinct {
             ns: Namespace {

src/operation/drop_collection/mod.rs

@@ -22,7 +22,7 @@ impl DropCollection {
         DropCollection { ns, options }
     }
 
-    #[allow(dead_code)]
+    #[cfg(test)]
     fn empty() -> Self {
         Self::new(
             Namespace {

src/operation/drop_database/mod.rs

@@ -17,7 +17,7 @@ pub(crate) struct DropDatabase {
 }
 
 impl DropDatabase {
-    #[allow(dead_code)]
+    #[cfg(test)]
     fn empty() -> Self {
         Self::new(String::new(), None)
     }

src/operation/find/mod.rs

@@ -20,7 +20,7 @@ pub(crate) struct Find {
 }
 
 impl Find {
-    #[allow(dead_code)]
+    #[cfg(test)]
     fn empty() -> Self {
         Self::new(
             Namespace {