clarify startup message in test mode (#687)

kemalcr · Jul 30, 2024 · 3243b8e · 3243b8e
1 parent 0afbd12
commit 3243b8e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/spec/run_spec.cr b/spec/run_spec.cr
@@ -26,15 +26,15 @@ describe "Run" do
   end
 
   it "runs without a block being specified" do
-    run(<<-CR).should eq "[test] Kemal is ready to lead at http://0.0.0.0:3000\ntrue\n"
+    run(<<-CR).should contain "[test] Kemal is running in test mode."
       Kemal.config.env = "test"
       Kemal.run
       puts Kemal.config.running
       CR
   end
 
   it "allows custom HTTP::Server bind" do
-    run(<<-CR).should eq "[test] Kemal is ready to lead at http://127.0.0.1:3000, http://0.0.0.0:3001\n"
+    run(<<-CR).should contain "[test] Kemal is running in test mode."
       Kemal.config.env = "test"
       Kemal.run do |config|
         server = config.server.not_nil!

diff --git a/src/kemal.cr b/src/kemal.cr
@@ -66,8 +66,12 @@ module Kemal
   end
 
   def self.display_startup_message(config, server)
-    addresses = server.addresses.join ", " { |address| "#{config.scheme}://#{address}" }
-    log "[#{config.env}] #{config.app_name} is ready to lead at #{addresses}"
+    if config.env != "test"
+      addresses = server.addresses.join ", " { |address| "#{config.scheme}://#{address}" }
+      log "[#{config.env}] #{config.app_name} is ready to lead at #{addresses}"
+    else
+      log "[#{config.env}] #{config.app_name} is running in test mode. Server not listening"
+    end
   end
 
   def self.stop