refactor: use indoc for unit tests

Author: maltesander

rust/operator-binary/src/controller/build/properties/hbase_site.rs

@@ -137,6 +137,8 @@ pub fn build(
 
 #[cfg(test)]
 mod tests {
+    use indoc::indoc;
+
     use super::*;
     use crate::test_utils::{merged_config, validated_cluster};
 
@@ -153,11 +155,15 @@ mod tests {
             KeyValueConfigOverrides::default(),
         );
         assert!(
-            xml.contains("<name>hbase.cluster.distributed</name>\n    <value>true</value>"),
+            xml.contains(indoc! {"
+                <name>hbase.cluster.distributed</name>
+                    <value>true</value>"}),
             "{xml}"
         );
         assert!(
-            xml.contains("<name>hbase.master.ipc.address</name>\n    <value>0.0.0.0</value>"),
+            xml.contains(indoc! {"
+                <name>hbase.master.ipc.address</name>
+                    <value>0.0.0.0</value>"}),
             "{xml}"
         );
     }
@@ -175,13 +181,15 @@ mod tests {
             KeyValueConfigOverrides::default(),
         );
         assert!(
-            xml.contains("<name>hbase.regionserver.ipc.address</name>\n    <value>0.0.0.0</value>"),
+            xml.contains(indoc! {"
+                <name>hbase.regionserver.ipc.address</name>
+                    <value>0.0.0.0</value>"}),
             "{xml}"
         );
         assert!(
-            xml.contains(
-                "<name>hbase.unsafe.regionserver.hostname</name>\n    <value>${env:HBASE_SERVICE_HOST}</value>"
-            ),
+            xml.contains(indoc! {"
+                <name>hbase.unsafe.regionserver.hostname</name>
+                    <value>${env:HBASE_SERVICE_HOST}</value>"}),
             "{xml}"
         );
     }
@@ -199,9 +207,9 @@ mod tests {
             KeyValueConfigOverrides::default(),
         );
         assert!(
-            xml.contains(
-                "<name>hbase.rest.endpoint</name>\n    <value>${env:HBASE_SERVICE_HOST}:${env:HBASE_SERVICE_PORT}</value>"
-            ),
+            xml.contains(indoc! {"
+                <name>hbase.rest.endpoint</name>
+                    <value>${env:HBASE_SERVICE_HOST}:${env:HBASE_SERVICE_PORT}</value>"}),
             "{xml}"
         );
     }
@@ -219,7 +227,9 @@ mod tests {
             [("hbase.cluster.distributed", "false")].into(),
         );
         assert!(
-            xml.contains("<name>hbase.cluster.distributed</name>\n    <value>false</value>"),
+            xml.contains(indoc! {"
+                <name>hbase.cluster.distributed</name>
+                    <value>false</value>"}),
             "{xml}"
         );
     }

rust/operator-binary/src/controller/build/properties/ssl_client.rs

@@ -16,6 +16,8 @@ pub fn build(
 
 #[cfg(test)]
 mod tests {
+    use indoc::indoc;
+
     use super::*;
 
     #[test]
@@ -36,7 +38,9 @@ mod tests {
         )
         .expect("settings present, so ssl-client.xml is rendered");
         assert!(
-            xml.contains("<name>ssl.client.truststore.type</name>\n    <value>pkcs12</value>"),
+            xml.contains(indoc! {"
+                <name>ssl.client.truststore.type</name>
+                    <value>pkcs12</value>"}),
             "{xml}"
         );
     }
@@ -49,7 +53,9 @@ mod tests {
         )
         .expect("override present, so ssl-client.xml is rendered");
         assert!(
-            xml.contains("<name>ssl.client.keystore.type</name>\n    <value>jks</value>"),
+            xml.contains(indoc! {"
+                <name>ssl.client.keystore.type</name>
+                    <value>jks</value>"}),
             "{xml}"
         );
     }

rust/operator-binary/src/controller/build/properties/ssl_server.rs

@@ -16,6 +16,8 @@ pub fn build(
 
 #[cfg(test)]
 mod tests {
+    use indoc::indoc;
+
     use super::*;
 
     #[test]
@@ -33,7 +35,9 @@ mod tests {
         )
         .expect("settings present, so ssl-server.xml is rendered");
         assert!(
-            xml.contains("<name>ssl.server.keystore.type</name>\n    <value>pkcs12</value>"),
+            xml.contains(indoc! {"
+                <name>ssl.server.keystore.type</name>
+                    <value>pkcs12</value>"}),
             "{xml}"
         );
     }
@@ -46,7 +50,9 @@ mod tests {
         )
         .expect("override present, so ssl-server.xml is rendered");
         assert!(
-            xml.contains("<name>ssl.server.keystore.type</name>\n    <value>jks</value>"),
+            xml.contains(indoc! {"
+                <name>ssl.server.keystore.type</name>
+                    <value>jks</value>"}),
             "{xml}"
         );
     }