make chrono an optional dependency

Author: mhils

Cargo.toml

@@ -12,11 +12,12 @@ edition = "2021"
 rust-version = "1.70"
 
 [features]
-default = ["uuids"]
+default = ["timestamps", "uuids"]
+timestamps = ["dep:chrono"]
 uuids = ["dep:uuid", "dep:newtype-uuid"]
 
 [dependencies]
-chrono = { version = "0.4.41", default-features = false, features = ["std"] }
+chrono = { version = "0.4.41", default-features = false, features = ["std"], optional = true }
 indexmap = "2.7.1"
 quick-xml = "0.38.1"
 newtype-uuid = { version = "1.2.4", optional = true }

src/report.rs

@@ -2,6 +2,7 @@
 // SPDX-License-Identifier: MIT OR Apache-2.0
 
 use crate::{serialize::serialize_report, SerializeError};
+#[cfg(feature = "timestamps")]
 use chrono::{DateTime, FixedOffset};
 use indexmap::map::IndexMap;
 #[cfg(feature = "uuids")]
@@ -41,6 +42,7 @@ pub struct Report {
     /// The time at which the first test in this report began execution.
     ///
     /// This is not part of the JUnit spec, but may be useful for some tools.
+    #[cfg(feature = "timestamps")]
     pub timestamp: Option<DateTime<FixedOffset>>,
 
     /// The overall time taken by the test suite.
@@ -68,6 +70,7 @@ impl Report {
             name: name.into(),
             #[cfg(feature = "uuids")]
             uuid: None,
+            #[cfg(feature = "timestamps")]
             timestamp: None,
             time: None,
             tests: 0,
@@ -96,6 +99,7 @@ impl Report {
     }
 
     /// Sets the start timestamp for the report.
+    #[cfg(feature = "timestamps")]
     pub fn set_timestamp(&mut self, timestamp: impl Into<DateTime<FixedOffset>>) -> &mut Self {
         self.timestamp = Some(timestamp.into());
         self
@@ -174,6 +178,7 @@ pub struct TestSuite {
     pub failures: usize,
 
     /// The time at which the TestSuite began execution.
+    #[cfg(feature = "timestamps")]
     pub timestamp: Option<DateTime<FixedOffset>>,
 
     /// The overall time taken by the TestSuite.
@@ -201,6 +206,7 @@ impl TestSuite {
         Self {
             name: name.into(),
             time: None,
+            #[cfg(feature = "timestamps")]
             timestamp: None,
             tests: 0,
             disabled: 0,
@@ -215,6 +221,7 @@ impl TestSuite {
     }
 
     /// Sets the start timestamp for the TestSuite.
+    #[cfg(feature = "timestamps")]
     pub fn set_timestamp(&mut self, timestamp: impl Into<DateTime<FixedOffset>>) -> &mut Self {
         self.timestamp = Some(timestamp.into());
         self
@@ -318,6 +325,7 @@ pub struct TestCase {
     /// The time at which this test case began execution.
     ///
     /// This is not part of the JUnit spec, but may be useful for some tools.
+    #[cfg(feature = "timestamps")]
     pub timestamp: Option<DateTime<FixedOffset>>,
 
     /// The time it took to execute this test case.
@@ -346,6 +354,7 @@ impl TestCase {
             name: name.into(),
             classname: None,
             assertions: None,
+            #[cfg(feature = "timestamps")]
             timestamp: None,
             time: None,
             status,
@@ -369,6 +378,7 @@ impl TestCase {
     }
 
     /// Sets the start timestamp for the test case.
+    #[cfg(feature = "timestamps")]
     pub fn set_timestamp(&mut self, timestamp: impl Into<DateTime<FixedOffset>>) -> &mut Self {
         self.timestamp = Some(timestamp.into());
         self
@@ -557,6 +567,7 @@ pub struct TestRerun {
     /// The time at which this rerun began execution.
     ///
     /// This is not part of the JUnit spec, but may be useful for some tools.
+    #[cfg(feature = "timestamps")]
     pub timestamp: Option<DateTime<FixedOffset>>,
 
     /// The time it took to execute this rerun.
@@ -590,6 +601,7 @@ impl TestRerun {
     pub fn new(kind: NonSuccessKind) -> Self {
         TestRerun {
             kind,
+            #[cfg(feature = "timestamps")]
             timestamp: None,
             time: None,
             message: None,
@@ -602,6 +614,7 @@ impl TestRerun {
     }
 
     /// Sets the start timestamp for this rerun.
+    #[cfg(feature = "timestamps")]
     pub fn set_timestamp(&mut self, timestamp: impl Into<DateTime<FixedOffset>>) -> &mut Self {
         self.timestamp = Some(timestamp.into());
         self

src/serialize.rs

@@ -7,6 +7,7 @@ use crate::{
     NonSuccessKind, Property, Report, SerializeError, TestCase, TestCaseStatus, TestRerun,
     TestSuite, XmlString,
 };
+#[cfg(feature = "timestamps")]
 use chrono::{DateTime, FixedOffset};
 use quick_xml::{
     events::{BytesDecl, BytesEnd, BytesStart, BytesText, Event},
@@ -54,6 +55,7 @@ pub(crate) fn serialize_report_impl(
         name,
         #[cfg(feature = "uuids")]
         uuid,
+        #[cfg(feature = "timestamps")]
         timestamp,
         time,
         tests,
@@ -73,6 +75,7 @@ pub(crate) fn serialize_report_impl(
     if let Some(uuid) = uuid {
         testsuites_tag.push_attribute(("uuid", uuid.to_string().as_str()));
     }
+    #[cfg(feature = "timestamps")]
     if let Some(timestamp) = timestamp {
         serialize_timestamp(&mut testsuites_tag, timestamp);
     }
@@ -103,6 +106,7 @@ pub(crate) fn serialize_test_suite(
         errors,
         failures,
         time,
+        #[cfg(feature = "timestamps")]
         timestamp,
         test_cases,
         properties,
@@ -120,6 +124,7 @@ pub(crate) fn serialize_test_suite(
         ("failures", failures.to_string().as_str()),
     ]);
 
+    #[cfg(feature = "timestamps")]
     if let Some(timestamp) = timestamp {
         serialize_timestamp(&mut test_suite_tag, timestamp);
     }
@@ -174,6 +179,7 @@ fn serialize_test_case(
         name,
         classname,
         assertions,
+        #[cfg(feature = "timestamps")]
         timestamp,
         time,
         status,
@@ -192,6 +198,7 @@ fn serialize_test_case(
         testcase_tag.push_attribute(("assertions", format!("{assertions}").as_str()));
     }
 
+    #[cfg(feature = "timestamps")]
     if let Some(timestamp) = timestamp {
         serialize_timestamp(&mut testcase_tag, timestamp);
     }
@@ -308,6 +315,7 @@ fn serialize_rerun(
     writer: &mut Writer<impl io::Write>,
 ) -> quick_xml::Result<()> {
     let TestRerun {
+        #[cfg(feature = "timestamps")]
         timestamp,
         time,
         kind,
@@ -327,6 +335,7 @@ fn serialize_rerun(
     };
 
     let mut tag = BytesStart::new(tag_name);
+    #[cfg(feature = "timestamps")]
     if let Some(timestamp) = timestamp {
         serialize_timestamp(&mut tag, timestamp);
     }
@@ -411,6 +420,7 @@ fn serialize_end_tag(
     writer.write_event(Event::End(end_tag))
 }
 
+#[cfg(feature = "timestamps")]
 fn serialize_timestamp(tag: &mut BytesStart<'_>, timestamp: &DateTime<FixedOffset>) {
     // The format string is obtained from https://docs.rs/chrono/0.4.19/chrono/format/strftime/index.html#fn8.
     // The only change is that this only prints timestamps up to 3 decimal places (to match times).