spacebar/kamal
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

No need for run_id

Browse Source
This commit is contained in:
Donal McBreen
2024-09-12 14:31:21 +01:00
parent 109339189a
commit cb73c730f9
3 changed files with 0 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/kamal/configuration.rb
View File

@@ -189,10 +189,6 @@ class Kamal::Configuration
raw_config.readiness_delay || 7 raw_config.readiness_delay || 7
end end
def run_id
@run_id ||= SecureRandom.hex(16)
end
def run_directory def run_directory
".kamal" ".kamal"

1
test/commands/app_test.rb
View File

@@ -3,7 +3,6 @@ require "test_helper"
class CommandsAppTest < ActiveSupport::TestCase class CommandsAppTest < ActiveSupport::TestCase
setup do setup do
setup_test_secrets("secrets" => "RAILS_MASTER_KEY=456") setup_test_secrets("secrets" => "RAILS_MASTER_KEY=456")
Kamal::Configuration.any_instance.stubs(:run_id).returns("12345678901234567890123456789012")
@config = { service: "app", image: "dhh/app", registry: { "username" => "dhh", "password" => "secret" }, servers: [ "1.1.1.1" ], env: { "secret" => [ "RAILS_MASTER_KEY" ] }, builder: { "arch" => "amd64" } } @config = { service: "app", image: "dhh/app", registry: { "username" => "dhh", "password" => "secret" }, servers: [ "1.1.1.1" ], env: { "secret" => [ "RAILS_MASTER_KEY" ] }, builder: { "arch" => "amd64" } }
end end

5
test/configuration_test.rb
View File

@@ -296,11 +296,6 @@ class ConfigurationTest < ActiveSupport::TestCase
assert_equal "$(pwd)/.kamal", config.run_directory_as_docker_volume assert_equal "$(pwd)/.kamal", config.run_directory_as_docker_volume
end end
test "run id" do
SecureRandom.expects(:hex).with(16).returns("09876543211234567890098765432112")
assert_equal "09876543211234567890098765432112", @config.run_id
end
test "asset path" do test "asset path" do
assert_nil @config.asset_path assert_nil @config.asset_path
assert_equal "foo", Kamal::Configuration.new(@deploy.merge!(asset_path: "foo")).asset_path assert_equal "foo", Kamal::Configuration.new(@deploy.merge!(asset_path: "foo")).asset_path