Resolve new nightly clippy lints (#409)

Author: jtescher

opentelemetry/src/sdk/metrics/aggregators/ddsketch.rs

@@ -878,7 +878,7 @@ mod tests {
             store.add(i)
         }
         assert_eq!(store.count, 1400);
-        assert_eq!(store.bins.len(), 200 as usize);
+        assert_eq!(store.bins.len(), 200);
     }
 
     /// Test to see if copy_from_slice will panic because the range size is different in left and right

opentelemetry/src/sdk/trace/span_processor.rs

@@ -642,8 +642,10 @@ mod tests {
     #[tokio::test]
     async fn test_batch_span_processor() {
         let (exporter, mut export_receiver, _shutdown_receiver) = new_tokio_test_exporter();
-        let mut config = BatchConfig::default();
-        config.scheduled_delay = Duration::from_secs(60 * 60 * 24); // set the tick to 24 hours so we know the span must be exported via force_flush
+        let config = BatchConfig {
+            scheduled_delay: Duration::from_secs(60 * 60 * 24), // set the tick to 24 hours so we know the span must be exported via force_flush
+            ..Default::default()
+        };
         let spawn = |fut| tokio::task::spawn_blocking(|| futures::executor::block_on(fut));
         let mut processor = BatchSpanProcessor::new(
             Box::new(exporter),
@@ -740,9 +742,11 @@ mod tests {
     // otherwise the exporter should be able to export within time out duration.
     #[cfg(feature = "async-std")]
     async fn timeout_test_std_async(time_out: bool) {
-        let mut config = BatchConfig::default();
-        config.max_export_timeout = time::Duration::from_millis(if time_out { 5 } else { 60 });
-        config.scheduled_delay = Duration::from_secs(60 * 60 * 24); // set the tick to 24 hours so we know the span must be exported via force_flush
+        let config = BatchConfig {
+            max_export_timeout: time::Duration::from_millis(if time_out { 5 } else { 60 }),
+            scheduled_delay: Duration::from_secs(60 * 60 * 24), // set the tick to 24 hours so we know the span must be exported via force_flush
+            ..Default::default()
+        };
         let exporter = BlockingExporter {
             delay_for: time::Duration::from_millis(if !time_out { 5 } else { 60 }),
             delay_fn: async_std::task::sleep,
@@ -768,9 +772,11 @@ mod tests {
     // If the time_out is true, then the result suppose to ended with timeout.
     // otherwise the exporter should be able to export within time out duration.
     async fn timeout_test_tokio(time_out: bool) {
-        let mut config = BatchConfig::default();
-        config.max_export_timeout = time::Duration::from_millis(if time_out { 5 } else { 60 });
-        config.scheduled_delay = Duration::from_secs(60 * 60 * 24); // set the tick to 24 hours so we know the span must be exported via force_flush
+        let config = BatchConfig {
+            max_export_timeout: time::Duration::from_millis(if time_out { 5 } else { 60 }),
+            scheduled_delay: Duration::from_secs(60 * 60 * 24), // set the tick to 24 hours so we know the span must be exported via force_flush,
+            ..Default::default()
+        };
         let exporter = BlockingExporter {
             delay_for: time::Duration::from_millis(if !time_out { 5 } else { 60 }),
             delay_fn: tokio::time::delay_for,

opentelemetry/src/sdk/trace/tracer.rs

@@ -352,14 +352,16 @@ mod tests {
             .build();
         let tracer = tracer_provider.get_tracer("test", None);
         let trace_state = TraceState::from_key_value(vec![("foo", "bar")]).unwrap();
-        let mut span_builder = SpanBuilder::default();
-        span_builder.parent_context = Some(Context::current_with_span(TestSpan(SpanContext::new(
-            TraceId::from_u128(128),
-            SpanId::from_u64(64),
-            TRACE_FLAG_SAMPLED,
-            true,
-            trace_state,
-        ))));
+        let span_builder = SpanBuilder {
+            parent_context: Some(Context::current_with_span(TestSpan(SpanContext::new(
+                TraceId::from_u128(128),
+                SpanId::from_u64(64),
+                TRACE_FLAG_SAMPLED,
+                true,
+                trace_state,
+            )))),
+            ..Default::default()
+        };
 
         // Test sampler should change trace state
         let span = tracer.build(span_builder);