Stub out a components test, and use serde serialization for DateTime options

components/datetime/Cargo.toml

@@ -22,27 +22,37 @@ include = [
 [dependencies]
 icu_locid = { version = "0.1", path = "../locid" }
 icu_provider = { version = "0.1", path = "../provider" }
+serde = { version = "1.0", features = ["derive"], optional = true }
 
 [dev-dependencies]
 criterion = "0.3"
 icu_provider = { version = "0.1", path = "../provider" }
 icu_testdata = { version = "0.1", path = "../../resources/testdata" }
 icu_locid_macros = { version = "0.1", path = "../locid/macros" }
-serde = { version = "1.0", features = ["derive"] }
 serde_json = "1.0"
 
 [lib]
 bench = false  # This option is required for Benchmark CI
 
 [features]
-default = []
+default = ["serde"]
 bench = []
+serialize_none = []
 
 [[bench]]
 name = "datetime"
 harness = false
+required-features = ["serde"]
 
 [[bench]]
 name = "pattern"
 harness = false
 required-features = ["bench"]
+
+[[test]]
+name = "datetime"
+required-features = ["serde"]
+
+[[example]]
+name = "work_log"
+required-features = ["serde"]

components/datetime/src/options/components.rs

@@ -54,8 +54,14 @@
 //! time. Formatted result should be treated as opaque and displayed to the user as-is,
 //! and it is strongly recommended to never write tests that expect a particular formatted output.
 use super::preferences;
+#[cfg(all(not(feature = "serialize_none"), feature = "serde"))]
+use serde::{Deserialize, Serialize};
 
 #[derive(Debug)]
+#[cfg_attr(
+    all(not(feature = "serialize_none"), feature = "serde"),
+    derive(Clone, Serialize, Deserialize)
+)]
 pub struct Bag {
     pub era: Option<Text>,
     pub year: Option<Numeric>,
@@ -69,6 +75,10 @@ pub struct Bag {
 
     pub time_zone_name: Option<TimeZoneName>,
 
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(skip_serializing, skip_deserializing)
+    )]
     pub preferences: Option<preferences::Bag>,
 }
 
@@ -93,29 +103,93 @@ impl Default for Bag {
 }
 
 #[derive(Debug, Clone, Copy, PartialEq)]
+#[cfg_attr(
+    all(not(feature = "serialize_none"), feature = "serde"),
+    derive(Serialize, Deserialize)
+)]
 pub enum Numeric {
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "numeric")
+    )]
     Numeric,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "two-digit")
+    )]
     TwoDigit,
 }
 
 #[derive(Debug, Clone, Copy, PartialEq)]
+#[cfg_attr(
+    all(not(feature = "serialize_none"), feature = "serde"),
+    derive(Serialize, Deserialize)
+)]
 pub enum Text {
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "long")
+    )]
     Long,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "short")
+    )]
     Short,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "narrow")
+    )]
     Narrow,
 }
 
 #[derive(Debug, Clone, Copy, PartialEq)]
+#[cfg_attr(
+    all(not(feature = "serialize_none"), feature = "serde"),
+    derive(Serialize, Deserialize)
+)]
 pub enum Month {
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "numeric")
+    )]
     Numeric,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "two-digit")
+    )]
     TwoDigit,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "long")
+    )]
     Long,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "short")
+    )]
     Short,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "narrow")
+    )]
     Narrow,
 }
 
 #[derive(Debug, Clone, Copy, PartialEq)]
+#[cfg_attr(
+    all(not(feature = "serialize_none"), feature = "serde"),
+    derive(Serialize, Deserialize)
+)]
 pub enum TimeZoneName {
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "long")
+    )]
     Long,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "short")
+    )]
     Short,
 }

components/datetime/src/options/preferences.rs

@@ -32,6 +32,7 @@ use crate::fields;
 /// };
 /// ```
 #[derive(Debug)]
+#[cfg_attr(all(not(feature = "serialize_none"), feature = "serde"), derive(Clone))]
 pub struct Bag {
     pub hour_cycle: Option<HourCycle>,
 }

components/datetime/src/options/style.rs

@@ -41,6 +41,8 @@
 //! time. Formatted result should be treated as opaque and displayed to the user as-is,
 //! and it is strongly recommended to never write tests that expect a particular formatted output.
 use super::preferences;
+#[cfg(all(not(feature = "serialize_none"), feature = "serde"))]
+use serde::{Deserialize, Serialize};
 /// `style::Bag` is a structure to represent the set of styles in which the `DateTime` should
 /// be formatted to.
 ///
@@ -72,9 +74,17 @@ use super::preferences;
 /// [`UTS #35: Unicode LDML 4. Dates`]: https://unicode.org/reports/tr35/tr35-dates.html
 /// [`Element dateFormats`]: https://unicode.org/reports/tr35/tr35-dates.html#dateFormats
 #[derive(Debug)]
+#[cfg_attr(
+    all(not(feature = "serialize_none"), feature = "serde"),
+    derive(Clone, Serialize, Deserialize)
+)]
 pub struct Bag {
     pub date: Option<Date>,
     pub time: Option<Time>,
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(skip_serializing, skip_deserializing)
+    )]
     pub preferences: Option<preferences::Bag>,
 }
 
@@ -114,6 +124,10 @@ impl Default for Bag {
 /// [`Element dateFormats`]: https://unicode.org/reports/tr35/tr35-dates.html#dateFormats
 /// [`DateTimeFormat`]: super::super::DateTimeFormat
 #[derive(Debug, Clone, Copy, PartialEq)]
+#[cfg_attr(
+    all(not(feature = "serialize_none"), feature = "serde"),
+    derive(Serialize, Deserialize)
+)]
 pub enum Date {
     /// Full length, usually with weekday name.
     ///
@@ -126,6 +140,10 @@ pub enum Date {
     /// "вторник, 21 января 2020 г.";      // ru
     /// "2020年1月21日火曜日";               // ja
     /// ```
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "full")
+    )]
     Full,
     /// Long length, with wide month name.
     ///
@@ -138,6 +156,10 @@ pub enum Date {
     /// "10 сентября 2020 г.";    // ru
     /// "2020年9月10日";           // ja
     /// ```
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "long")
+    )]
     Long,
     /// Medium length.
     ///
@@ -150,6 +172,10 @@ pub enum Date {
     /// "20 февр. 2020 г.";    // ru
     /// "2020/02/20";          // ja
     /// ```
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "medium")
+    )]
     Medium,
     /// Short length, usually with numeric month.
     ///
@@ -162,6 +188,10 @@ pub enum Date {
     /// "30.01.2020";   // ru
     /// "2020/01/30";   // ja
     /// ```
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "short")
+    )]
     Short,
 }
 
@@ -191,6 +221,10 @@ pub enum Date {
 /// [`Element dateFormats`]: https://unicode.org/reports/tr35/tr35-dates.html#timeFormats
 /// [`DateTimeFormat`]: super::super::DateTimeFormat
 #[derive(Debug, Clone, Copy, PartialEq)]
+#[cfg_attr(
+    all(not(feature = "serialize_none"), feature = "serde"),
+    derive(Serialize, Deserialize)
+)]
 pub enum Time {
     /// Full length, with spelled out time zone name.
     ///
@@ -203,6 +237,10 @@ pub enum Time {
     /// "08:25:07 Тихоокеанское стандартное время";      // ru
     /// "8時25分07秒 アメリカ太平洋標準時";                  // ja
     /// ```
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "full")
+    )]
     Full,
     /// Full length, usually with short time zone code.
     ///
@@ -215,6 +253,10 @@ pub enum Time {
     /// "08:25:07 GMT-8";       // ru
     /// "8:25:07 GMT-8";        // ja
     /// ```
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "long")
+    )]
     Long,
     /// Full length, usually with seconds.
     ///
@@ -227,6 +269,10 @@ pub enum Time {
     /// "08:25:07";     // ru
     /// "8:25:07";      // ja
     /// ```
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "medium")
+    )]
     Medium,
     /// Full length, usually without seconds.
     ///
@@ -239,5 +285,9 @@ pub enum Time {
     /// "08:25";     // ru
     /// "8:25";      // ja
     /// ```
+    #[cfg_attr(
+        all(not(feature = "serialize_none"), feature = "serde"),
+        serde(rename = "short")
+    )]
     Short,
 }

components/datetime/tests/datetime.rs

@@ -1,17 +1,18 @@
 // This file is part of ICU4X. For terms of use, please see the file
 // called LICENSE at the top level of the ICU4X source tree
 // (online at: https://github.com/unicode-org/icu4x/blob/master/LICENSE ).
+#![cfg(all(not(feature = "serialize_none"), feature = "serde"))]
+
 mod fixtures;
 
 use icu_datetime::date::MockDateTime;
 use icu_datetime::DateTimeFormat;
 use std::fmt::Write;
 
-#[test]
-fn test_fixtures() {
+fn test_fixture(fixture_name: &str) {
     let provider = icu_testdata::get_provider();
 
-    for fx in fixtures::get_fixture("styles").unwrap().0 {
+    for fx in fixtures::get_fixture(fixture_name).unwrap().0 {
         let langid = fx.input.locale.parse().unwrap();
         let options = fixtures::get_options(&fx.input.options);
         let dtf = DateTimeFormat::try_new(langid, &provider, &options).unwrap();
@@ -33,3 +34,16 @@ fn test_fixtures() {
         assert_eq!(s, fx.output.value);
     }
 }
+
+#[test]
+fn test_style_fixtures() {
+    test_fixture("styles");
+}
+
+// Expected panic: 'not implemented', components/datetime/src/provider.rs:49:53
+// https://github.com/unicode-org/icu4x/issues/272
+#[test]
+#[should_panic]
+fn test_components_fixtures() {
+    test_fixture("components");
+}

components/datetime/tests/fixtures/mod.rs

@@ -1,9 +1,10 @@
 // This file is part of ICU4X. For terms of use, please see the file
 // called LICENSE at the top level of the ICU4X source tree
 // (online at: https://github.com/unicode-org/icu4x/blob/master/LICENSE ).
+#![cfg(all(not(feature = "serialize_none"), feature = "serde"))]
+
 pub mod structs;
 
-use icu_datetime::options::style;
 use icu_datetime::DateTimeFormatOptions;
 use std::fs::File;
 use std::io::BufReader;
@@ -16,20 +17,8 @@ pub fn get_fixture(name: &str) -> std::io::Result<structs::Fixture> {
 }
 
 pub fn get_options(input: &structs::TestOptions) -> DateTimeFormatOptions {
-    let style = style::Bag {
-        date: input.style.date.as_ref().map(|date| match date {
-            structs::TestStyleWidth::Full => style::Date::Full,
-            structs::TestStyleWidth::Long => style::Date::Long,
-            structs::TestStyleWidth::Medium => style::Date::Medium,
-            structs::TestStyleWidth::Short => style::Date::Short,
-        }),
-        time: input.style.time.as_ref().map(|time| match time {
-            structs::TestStyleWidth::Full => style::Time::Full,
-            structs::TestStyleWidth::Long => style::Time::Long,
-            structs::TestStyleWidth::Medium => style::Time::Medium,
-            structs::TestStyleWidth::Short => style::Time::Short,
-        }),
-        ..Default::default()
-    };
-    style.into()
+    match input {
+        structs::TestOptions::Style(bag) => (*bag).clone().into(),
+        structs::TestOptions::Components(bag) => (*bag).clone().into(),
+    }
 }