use indexmap instead of hasmap in reports

Author: zeegomo

Cargo.lock

@@ -69,6 +69,7 @@ remove_dir_all = "0.5.2"
 ctrlc = "3.1.3"
 prometheus = "0.7.0"
 cargo_metadata = "0.9.1"
+indexmap = "1.4.0"
 
 [dev-dependencies]
 assert_cmd = "0.10.1"

Cargo.toml

@@ -4,8 +4,8 @@ use crate::results::{
     FailureReason,
     TestResult::{self, BuildFail},
 };
+use indexmap::IndexMap;
 use std::collections::BTreeSet;
-use std::collections::HashMap;
 
 pub enum ToolchainSelect {
     Start,
@@ -22,20 +22,20 @@ pub enum ReportConfig {
 pub enum ReportCrates {
     Plain(Vec<CrateResult>),
     Complete {
-        tree: HashMap<Crate, Vec<CrateResult>>,
-        results: HashMap<TestResult, Vec<CrateResult>>,
+        tree: IndexMap<Crate, Vec<CrateResult>>,
+        results: IndexMap<TestResult, Vec<CrateResult>>,
     },
 }
 
 #[cfg_attr(test, derive(Debug, PartialEq))]
 pub struct TestResults {
-    pub categories: HashMap<Comparison, ReportCrates>,
-    pub info: HashMap<Comparison, u32>,
+    pub categories: IndexMap<Comparison, ReportCrates>,
+    pub info: IndexMap<Comparison, u32>,
 }
 
 fn analyze_detailed(toolchain: usize, crates: Vec<CrateResult>) -> ReportCrates {
-    let mut tree = HashMap::new();
-    let mut results = HashMap::new();
+    let mut tree = IndexMap::new();
+    let mut results = IndexMap::new();
 
     let mut root = Vec::new();
     for krate in crates {
@@ -75,7 +75,7 @@ fn analyze_detailed(toolchain: usize, crates: Vec<CrateResult>) -> ReportCrates
 }
 
 pub fn analyze_report(test: RawTestResults) -> TestResults {
-    let mut comparison = HashMap::new();
+    let mut comparison = IndexMap::new();
     for krate in test.crates {
         comparison
             .entry(krate.res)
@@ -86,9 +86,9 @@ pub fn analyze_report(test: RawTestResults) -> TestResults {
     let info = comparison
         .iter()
         .map(|(&key, vec)| (key, vec.len() as u32))
-        .collect::<HashMap<_, _>>();
+        .collect::<IndexMap<_, _>>();
 
-    let mut categories = HashMap::new();
+    let mut categories = IndexMap::new();
     for (cat, crates) in comparison {
         if let ReportConfig::Complete(toolchain) = cat.report_config() {
             // variants in an enum are numbered following an
@@ -191,18 +191,18 @@ mod tests {
                     panic!("invalid crate type")
                 }
             })
-            .collect::<HashMap<_, _>>();
+            .collect::<IndexMap<_, _>>();
         let analyzed = analyze_report(raw);
 
-        let mut info = HashMap::new();
+        let mut info = IndexMap::new();
         info.insert(Comparison::Regressed, 5);
         info.insert(Comparison::Fixed, 2);
         info.insert(Comparison::SameTestPass, 1);
 
         macro_rules! create_results {
             ($src:expr, $($key:expr => ($($krate:expr),*)),*) => {
                 {
-                    let mut map = HashMap::new();
+                    let mut map = IndexMap::new();
                     $(
                         let mut crates = Vec::new();
                         $(
@@ -248,7 +248,7 @@ mod tests {
 
         let test_pass = ReportCrates::Plain(vec![crates.remove("test-pass").unwrap()]);
 
-        let mut categories = HashMap::new();
+        let mut categories = IndexMap::new();
         categories.insert(Comparison::Regressed, regressed);
         categories.insert(Comparison::Fixed, fixed);
         categories.insert(Comparison::SameTestPass, test_pass);

src/report/analyzer.rs

@@ -5,7 +5,7 @@ use crate::prelude::*;
 use crate::report::{compare, ReportWriter};
 use crate::results::{EncodedLog, EncodingType, ReadResults};
 use flate2::{write::GzEncoder, Compression};
-use std::collections::HashMap;
+use indexmap::IndexMap;
 use tar::{Builder as TarBuilder, Header as TarHeader};
 
 #[derive(Serialize)]
@@ -23,7 +23,7 @@ pub fn write_logs_archives<DB: ReadResults, W: ReportWriter>(
 ) -> Fallible<Vec<Archive>> {
     let mut archives = Vec::new();
     let mut all = TarBuilder::new(GzEncoder::new(Vec::new(), Compression::default()));
-    let mut by_comparison = HashMap::new();
+    let mut by_comparison = IndexMap::new();
 
     for krate in crates {
         if config.should_skip(krate) {
@@ -86,7 +86,7 @@ pub fn write_logs_archives<DB: ReadResults, W: ReportWriter>(
         path: "logs-archives/all.tar.gz".to_string(),
     });
 
-    for (comparison, archive) in by_comparison.drain() {
+    for (comparison, archive) in by_comparison.drain(..) {
         let data = archive.into_inner()?.finish()?;
         dest.write_bytes(
             &format!("logs-archives/{}.tar.gz", comparison),

src/report/archives.rs

@@ -6,7 +6,7 @@ use crate::report::{
     ResultColor, ResultName, TestResults,
 };
 use crate::results::{EncodingType, FailureReason, TestResult};
-use std::collections::HashMap;
+use indexmap::IndexMap;
 
 #[derive(Serialize)]
 struct NavbarItem {
@@ -27,11 +27,11 @@ enum ReportCratesHTML {
     Plain(Vec<CrateResultHTML>),
     Tree {
         count: u32,
-        tree: HashMap<String, Vec<CrateResultHTML>>,
+        tree: IndexMap<String, Vec<CrateResultHTML>>,
     },
     RootResults {
         count: u32,
-        results: HashMap<String, Vec<CrateResultHTML>>,
+        results: IndexMap<String, Vec<CrateResultHTML>>,
     },
 }
 
@@ -62,10 +62,10 @@ struct ResultsContext<'a> {
     ex: &'a Experiment,
     nav: Vec<NavbarItem>,
     categories: Vec<(Comparison, ReportCratesHTML)>,
-    info: HashMap<Comparison, u32>,
+    info: IndexMap<Comparison, u32>,
     full: bool,
     crates_count: usize,
-    comparison_colors: HashMap<Comparison, Color>,
+    comparison_colors: IndexMap<Comparison, Color>,
     result_colors: Vec<Color>,
     result_names: Vec<String>,
 }
@@ -103,8 +103,8 @@ fn write_report<W: ReportWriter>(
     dest: &W,
     output_templates: bool,
 ) -> Fallible<()> {
-    let mut comparison_colors = HashMap::new();
-    let mut test_results_to_int = HashMap::new();
+    let mut comparison_colors = IndexMap::new();
+    let mut test_results_to_int = IndexMap::new();
     let mut result_colors = Vec::new();
     let mut result_names = Vec::new();
 
@@ -165,7 +165,7 @@ fn write_report<W: ReportWriter>(
                                     .collect::<Vec<_>>(),
                             )
                         })
-                        .collect::<HashMap<_, _>>();
+                        .collect::<IndexMap<_, _>>();
                     let results = results
                         .into_iter()
                         .map(|(res, krates)| {
@@ -182,7 +182,7 @@ fn write_report<W: ReportWriter>(
                                     .collect::<Vec<_>>(),
                             )
                         })
-                        .collect::<HashMap<_, _>>();
+                        .collect::<IndexMap<_, _>>();
 
                     vec![
                         (