From 82933194cf3c983d024a9392ee77341bbf64966d Mon Sep 17 00:00:00 2001
From: Sergey Kipnis <sergey.kipnis@checkmk.com>
Date: Wed, 18 Oct 2023 16:25:35 +0200
Subject: [PATCH] move test tools in separate mod

Change-Id: I4b4d679954c4d564761e271d60a79171fa9e009e
---
 packages/check-sql/tests/test_ms_sql.rs | 76 +------------------------
 packages/check-sql/tests/tools.rs       | 75 ++++++++++++++++++++++++
 2 files changed, 77 insertions(+), 74 deletions(-)
 create mode 100644 packages/check-sql/tests/tools.rs

diff --git a/packages/check-sql/tests/test_ms_sql.rs b/packages/check-sql/tests/test_ms_sql.rs
index 99b7b3a7766..abbbfb37829 100644
--- a/packages/check-sql/tests/test_ms_sql.rs
+++ b/packages/check-sql/tests/test_ms_sql.rs
@@ -2,81 +2,9 @@
 // This file is part of Checkmk (https://checkmk.com). It is subject to the terms and
 // conditions defined in the file COPYING, which is part of this source code package.
 
-use check_sql::{config::CheckConfig, ms_sql::api};
-
-mod tools {
-    use assert_cmd::Command;
-    use std::io::Write;
-    use tempfile::NamedTempFile;
-
-    pub fn run_bin() -> Command {
-        Command::cargo_bin("check-sql").unwrap()
-    }
+mod tools;
 
-    pub const SQL_DB_ENDPOINT: &str = "CI_TEST_SQL_DB_ENDPOINT";
-    const SQL_DB_ENDPOINT_SPLITTER: char = ':';
-    pub struct SqlDbEndpoint {
-        pub host: String,
-        pub user: String,
-        pub pwd: String,
-    }
-
-    pub fn get_remote_sql_from_env_var() -> Option<SqlDbEndpoint> {
-        if let Ok(content) = std::env::var(SQL_DB_ENDPOINT) {
-            let x: Vec<&str> = content.split(SQL_DB_ENDPOINT_SPLITTER).collect();
-            if x.len() == 3 {
-                return Some(SqlDbEndpoint {
-                    host: x[0].to_owned(),
-                    user: x[1].to_owned(),
-                    pwd: x[2].to_owned(),
-                });
-            } else {
-                println!(
-                    "Error: environment variable {} is invalid, must have format 'host:user:password' expected",
-                    SQL_DB_ENDPOINT
-                );
-            }
-        } else {
-            println!("Error: environment variable {} is absent", SQL_DB_ENDPOINT);
-        }
-        None
-    }
-
-    pub fn create_remote_config(end_point: &SqlDbEndpoint) -> NamedTempFile {
-        let mut l = NamedTempFile::new().unwrap();
-        let config = format!(
-            r#"
----
-mssql:
-  standard:
-    authentication:
-       username: {}
-       password: {}
-       type: "sql_server"
-    connection:
-       hostname: {}
-"#,
-            end_point.user, end_point.pwd, end_point.host
-        );
-        l.write_all(config.as_bytes()).unwrap();
-        l
-    }
-    pub fn create_local_config() -> NamedTempFile {
-        let mut l = NamedTempFile::new().unwrap();
-        let config = r#"
----
-mssql:
-  standard:
-    authentication:
-       username: "nobody"
-       type: "integrated"
-    connection:
-       hostname: "localhost"
-"#;
-        l.write_all(config.as_bytes()).unwrap();
-        l
-    }
-}
+use check_sql::{config::CheckConfig, ms_sql::api};
 #[cfg(windows)]
 #[tokio::test(flavor = "multi_thread")]
 async fn test_local_connection() {
diff --git a/packages/check-sql/tests/tools.rs b/packages/check-sql/tests/tools.rs
new file mode 100644
index 00000000000..0d22910216e
--- /dev/null
+++ b/packages/check-sql/tests/tools.rs
@@ -0,0 +1,75 @@
+// Copyright (C) 2023 Checkmk GmbH - License: GNU General Public License v2
+// This file is part of Checkmk (https://checkmk.com). It is subject to the terms and
+// conditions defined in the file COPYING, which is part of this source code package.
+
+use assert_cmd::Command;
+use std::io::Write;
+use tempfile::NamedTempFile;
+
+pub fn run_bin() -> Command {
+    Command::cargo_bin("check-sql").unwrap()
+}
+
+pub const SQL_DB_ENDPOINT: &str = "CI_TEST_SQL_DB_ENDPOINT";
+const SQL_DB_ENDPOINT_SPLITTER: char = ':';
+pub struct SqlDbEndpoint {
+    pub host: String,
+    pub user: String,
+    pub pwd: String,
+}
+
+pub fn get_remote_sql_from_env_var() -> Option<SqlDbEndpoint> {
+    if let Ok(content) = std::env::var(SQL_DB_ENDPOINT) {
+        let x: Vec<&str> = content.split(SQL_DB_ENDPOINT_SPLITTER).collect();
+        if x.len() == 3 {
+            return Some(SqlDbEndpoint {
+                host: x[0].to_owned(),
+                user: x[1].to_owned(),
+                pwd: x[2].to_owned(),
+            });
+        } else {
+            println!(
+                    "Error: environment variable {} is invalid, must have format 'host:user:password' expected",
+                    SQL_DB_ENDPOINT
+                );
+        }
+    } else {
+        println!("Error: environment variable {} is absent", SQL_DB_ENDPOINT);
+    }
+    None
+}
+
+pub fn create_remote_config(end_point: &SqlDbEndpoint) -> NamedTempFile {
+    let mut l = NamedTempFile::new().unwrap();
+    let config = format!(
+        r#"
+---
+mssql:
+  standard:
+    authentication:
+       username: {}
+       password: {}
+       type: "sql_server"
+    connection:
+       hostname: {}
+"#,
+        end_point.user, end_point.pwd, end_point.host
+    );
+    l.write_all(config.as_bytes()).unwrap();
+    l
+}
+pub fn create_local_config() -> NamedTempFile {
+    let mut l = NamedTempFile::new().unwrap();
+    let config = r#"
+---
+mssql:
+  standard:
+    authentication:
+       username: "nobody"
+       type: "integrated"
+    connection:
+       hostname: "localhost"
+"#;
+    l.write_all(config.as_bytes()).unwrap();
+    l
+}