From f0f1d3239dfb18994e414352f8293026428e93b5 Mon Sep 17 00:00:00 2001
From: finga <finga@onders.org>
Date: Tue, 9 Nov 2021 13:58:57 +0100
Subject: [PATCH] Use `BTreeMap` instead of `HashMap`

As ordering of `BTreeMap` is easier for testing and the difference
does not really matter.
---
 src/main.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/main.rs b/src/main.rs
index a681d5d..2bfbf6f 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -28,7 +28,7 @@ use sha2::Sha256;
 use thiserror::Error;
 
 use std::{
-    collections::HashMap,
+    collections::BTreeMap,
     fs::File,
     io::BufReader,
     net::{IpAddr, Ipv4Addr, SocketAddr},
@@ -122,7 +122,7 @@ impl IpFilter {
 #[derive(Debug, Deserialize, Serialize)]
 #[serde(deny_unknown_fields)]
 struct Config {
-    hooks: HashMap<String, Hook>,
+    hooks: BTreeMap<String, Hook>,
 }
 
 #[derive(Debug, Deserialize, Serialize)]
@@ -239,7 +239,7 @@ impl Hook {
 
 #[derive(Debug)]
 struct Hooks {
-    inner: HashMap<String, String>,
+    inner: BTreeMap<String, String>,
 }
 
 impl Hooks {
@@ -254,7 +254,7 @@ impl Hooks {
 
         let config = request.guard::<&State<Config>>().await.unwrap(); // should never fail
         let mut valid = false;
-        let mut result = HashMap::new();
+        let mut result = BTreeMap::new();
         let client_ip = &request
             .client_ip()
             .unwrap_or(IpAddr::V4(Ipv4Addr::UNSPECIFIED));
@@ -636,7 +636,7 @@ mod tests {
 
     #[rocket::async_test]
     async fn secret() {
-        let mut hooks = HashMap::new();
+        let mut hooks = BTreeMap::new();
 
         hooks.insert(
             "test_hook".to_string(),
@@ -804,7 +804,7 @@ mod tests {
 
     #[rocket::async_test]
     async fn parse_command_request() {
-        let mut hooks = HashMap::new();
+        let mut hooks = BTreeMap::new();
 
         hooks.insert(
             "test_hook".to_string(),
@@ -878,7 +878,7 @@ mod tests {
 
     #[rocket::async_test]
     async fn parse_invalid_command_request() {
-        let mut hooks = HashMap::new();
+        let mut hooks = BTreeMap::new();
 
         hooks.insert(
             "test_hook".to_string(),