diff --git a/app/models/site/deployment.rb b/app/models/site/deployment.rb index f88d2a0c..b3b88987 100644 --- a/app/models/site/deployment.rb +++ b/app/models/site/deployment.rb @@ -17,6 +17,7 @@ class Site validates :name, hostname: true validates_presence_of :canonical_deploy validate :deploy_local_presence + validate :name_changed_is_unique_hostname, if: :name_changed? has_one :canonical_deploy, class_name: 'Deploy' has_many :deploys, dependent: :destroy @@ -68,6 +69,19 @@ class Site private + # Validar que al cambiar el nombre no estemos utilizando un + # hostname reservado por otro sitio. + # + # Al cambiar el nombre del DeployLocal se va a validar que el + # hostname nuevo sea único. + def name_changed_is_unique_hostname + deploy_local.hostname = nil + + return if deploy_local.valid? + + errors.add :name, I18n.t('activerecord.errors.models.site.attributes.name.duplicated_hostname') + end + # Si cambia el nombre queremos actualizarlo en el DeployLocal y # recargar el deploy canónico para tomar el nombre que # corresponda. diff --git a/config/locales/en.yml b/config/locales/en.yml index b59a5f21..27ffd43c 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -147,6 +147,7 @@ en: attributes: name: no_subdomains: 'Name cannot contain dots' + duplicated_hostname: 'There already is a site with this address' deploys: deploy_local_presence: 'We need to be build the site!' design_id: diff --git a/config/locales/es.yml b/config/locales/es.yml index 68ee3669..f98fd7db 100644 --- a/config/locales/es.yml +++ b/config/locales/es.yml @@ -147,6 +147,7 @@ es: attributes: name: no_subdomains: 'El nombre no puede contener puntos' + duplicated_hostname: 'Ya existe un sitio con ese nombre' deploys: deploy_local_presence: '¡Necesitamos poder generar el sitio!' design_id: diff --git a/test/models/site/deployment_test.rb b/test/models/site/deployment_test.rb new file mode 100644 index 00000000..8ec61e8a --- /dev/null +++ b/test/models/site/deployment_test.rb @@ -0,0 +1,25 @@ +# frozen_string_literal: true + +require 'test_helper' + +class Site::DeploymentTest < ActiveSupport::TestCase + def site + @site ||= create :site + end + + teardown do + @site&.destroy + end + + test 'al publicar el sitio se crea el directorio' do + assert site.deploy_local.deploy + assert site.deploy_local.exist? + end + + test 'al cambiar el nombre no puede pisar un dominio que ya existe' do + site_pre = create :site + dup_name = "test-#{SecureRandom.hex}" + assert site_pre.deploys.create(type: 'DeployAlternativeDomain', hostname: "#{dup_name}.#{Site.domain}") + assert_not site.update(name: dup_name) + end +end