Refactor ICU4X code to allow configured code by version (#377)

Author: sffc

.github/workflows/run-rust.yml

@@ -18,7 +18,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions-rs/toolchain@v1
         with:
-          toolchain: 1.73
+          toolchain: "1.80"
           components: rustfmt, clippy
           default: true
       - uses: actions-rs/cargo@v1
@@ -31,4 +31,8 @@ jobs:
         with:
           command: fmt
           args: --manifest-path executors/rust/${{ matrix.icu4x-version }}/Cargo.toml -- --check
+      - name: "Run Rustfmt (executor code)"
+        run: rustfmt executors/rust/src/** --check
+      - name: "Run Rustfmt (common code)"
+        run: rustfmt executors/rust/common/** --check
 

executors/rust/1.3/build.rs

@@ -1,8 +1,4 @@
-// Getting the version of ICU4X
-
 #[path = "../common/print_icu4x_versions_1_3.rs"]
 mod print_icu4x_versions_1_3;
 
-fn main() {
-    print_icu4x_versions_1_3::print();
-}
+use print_icu4x_versions_1_3::main;

executors/rust/1.3/src/collator.rs

@@ -1,40 +1,4 @@
-// Read JSON test case from standard in.
-// Parse to get test and parameters
-// Run test
-// Return results
-
-use std::io;
-
-mod icu {
-    pub use ::icu::compactdecimal;
-    pub use ::icu::displaynames;
-    pub use ::icu::relativetime;
-}
-
-mod collator;
-mod datetimefmt;
-mod decimalfmt;
-mod displaynames;
-mod likelysubtags;
-mod listfmt;
-mod localenames;
-mod numberfmt;
-mod pluralrules;
-mod relativedatetime_fmt;
-
 #[path = "../../common/run_all_tests.rs"]
 mod run_all_tests;
 
-fn main() -> io::Result<()> {
-    let executor_fns = run_all_tests::ExecutorFns {
-        run_collation_test: collator::run_collation_test,
-        run_datetimeformat_test: datetimefmt::run_datetimeformat_test,
-        run_likelysubtags_test: likelysubtags::run_likelysubtags_test,
-        run_list_fmt_test: listfmt::run_list_fmt_test,
-        run_locale_name_test: localenames::run_locale_name_test,
-        run_numberformat_test: numberfmt::run_numberformat_test,
-        run_plural_rules_test: pluralrules::run_plural_rules_test,
-        run_relativedatetimeformat_test: relativedatetime_fmt::run_relativedatetimeformat_test,
-    };
-    run_all_tests::main(executor_fns)
-}
+use run_all_tests::main;