Merge pull request #37 from hartwork/fix-symbol-visibility

Fix symbol visibility

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "rust-for-it"
 description = "Wait for one or more services to be available before executing a command."
-version = "1.3.0"
+version = "2.0.0"
 edition = "2021"
 authors = ["Sebastian Pipping <sebastian@pipping.org>"]
 license = "MIT"

src/command_line_parser.rs

@@ -47,7 +47,7 @@ fn test_parse_service_syntax_for_invalid() {
     assert_eq!(parse_service_syntax(":123"), expected_error);
 }
 
-pub fn command() -> Command {
+pub(crate) fn command() -> Command {
     command!()
         .arg(
             Arg::new("quiet")

src/exec.rs

@@ -73,7 +73,7 @@ fn test_process_popen_result() {
     );
 }
 
-pub fn run_command(command: &str, args: Vec<&str>) -> i32 {
+pub(crate) fn run_command(command: &str, args: Vec<&str>) -> i32 {
     let popen_result = Exec::cmd(command).args(args.as_slice()).join();
     process_popen_result(popen_result, command)
 }

src/main.rs

@@ -120,7 +120,7 @@ fn test_middle_main() {
     );
     assert_eq!(
         capture_main(["rust-for-it", "--version"]),
-        (0, String::from("rust-for-it 1.3.0\n"), String::new())
+        (0, String::from("rust-for-it 2.0.0\n"), String::new())
     );
 
     // Does bad usage produce exit code 2?

src/network.rs

@@ -9,7 +9,7 @@ use std::net::{Shutdown, SocketAddr, TcpStream, ToSocketAddrs};
 use std::result::Result;
 use std::thread::sleep;
 use std::time::{Duration, Instant};
-pub type TimeoutSeconds = u64;
+pub(crate) type TimeoutSeconds = u64;
 
 fn resolve_address(host_and_port: &str, timeout: Duration) -> Result<SocketAddr, std::io::Error> {
     let timer = Instant::now();
@@ -116,7 +116,7 @@ fn test_wait_for_tcp_socket_for_bad() {
     assert!(wait_result.is_err());
 }
 
-pub fn wait_for_service(
+pub(crate) fn wait_for_service(
     host_and_port: &str,
     timeout_seconds: TimeoutSeconds,
 ) -> Result<(), std::io::Error> {