klastera 1.5.5.2 → 1.5.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7f593dccae852483eb2b18e8254ee94c1e44804bcb59780cd26a8987ef383443
-  data.tar.gz: f28976b48568045dfe38b3feb2915de5e87b079279b0a31e481f48741fdbd0a0
+  metadata.gz: c4b18c6803bcea9323fe454de8293d219fca55d5fade1913dc31e01dadf16efd
+  data.tar.gz: 184638381c45f93db22d2bec40c8d2536c0722dd8ec54d7f3f508502ebe73514
 SHA512:
-  metadata.gz: a2b1d549df310a5bf569316f5e9de36f66c9be7b5d052108d85aa1345f6d05527a536494ca39ea37e2f19f663e8973dd5d62a107ea101168f63d5a2ace6566b4
-  data.tar.gz: 1353ffb7147a62d4e35cf28a0ccafa2e976460e96db0dc566cc48ed1db0e16e2ea315d65fcd8b12c2642cea76f53e3a2b2a8c10d48000f08efb93f72f4f9c749
+  metadata.gz: 27bd431138c5c6f17f296e2e49a3aaff2846011aca1feb53e500f1eba5e60c0b98f1b69dfc0f9fad9b8c886069882744152fc3b89a4a14951dd21f87a2f3d2c6
+  data.tar.gz: d64254e8c67a7b54d49179f58721547ead68bd363dd4cd6174addd2a13957a21b795da9118289df8f9a228807388d4da8ab6e3639a00447dd3f04cb57e379f08

data/app/models/klastera/concerns/clusterizable.rb

@@ -54,14 +54,19 @@ module Klastera::Concerns::Clusterizable
       )
     end
 
+    # No cuenta las filas marcadas para destruir: quitar una zona con el basurero y agregar otra en
+    # el mismo guardado deja una sola zona al final, y no debe rechazarse.
     def has_one_cluster_entity
-      if cluster_entities.length > 1
+      if cluster_entities.reject(&:marked_for_destruction?).length > 1
         return errors.add(:cluster_entities, I18n.t('klastera.messages.has_one_cluster_entity'))
       end
     end
 
+    # No cuenta las filas marcadas para destruir: quitar una zona con el basurero y volver a agregar
+    # la misma en el mismo guardado deja una sola fila al final, y no es un duplicado.
     def uniqueness_of_cluster_entity_record
-      if cluster_entities.map(&:cluster_id).uniq.size != cluster_entities.size
+      vigentes = cluster_entities.reject(&:marked_for_destruction?)
+      if vigentes.map(&:cluster_id).uniq.size != vigentes.size
         return errors.add(:cluster_entities, I18n.t('klastera.messages.duplicated_cluster_entity') )
       end
     end

data/lib/klastera/version.rb

@@ -1,3 +1,3 @@
 module Klastera
-  VERSION = "1.5.5.2"
+  VERSION = "1.5.5.4"
 end

data/test/models/klastera/clusterizable_nested_attributes_test.rb

@@ -39,5 +39,43 @@ module Klastera
       assert entity.cluster_entities.any?(&:marked_for_destruction?)
       assert entity.cluster_entities.any? { |c| c.new_record? && c.cluster_id == otro.id }
     end
+
+    # Entidad que permite una sola zona (por defecto): quitar una y agregar otra en el mismo guardado
+    # deja una sola, y la validacion no debe contar la fila marcada para destruir.
+    test "acepta quitar una zona y agregar otra en el mismo guardado" do
+      entity, ce, = build_entity_with_cluster
+      otro = ::Cluster.create!(organization: entity.organization, name: "otro-#{SecureRandom.hex(2)}")
+      entity.assign_attributes(cluster_entities_attributes: {
+        "0" => { "id" => ce.id, "_destroy" => "1" },
+        "1" => { "cluster_id" => otro.id.to_s, "_destroy" => "false" }
+      })
+      entity.valid?
+      assert_not entity.errors[:cluster_entities].any? { |m| m == I18n.t('klastera.messages.has_one_cluster_entity') }
+    end
+
+    test "sigue rechazando dos zonas vigentes en una entidad de una sola zona" do
+      entity, _ce, = build_entity_with_cluster
+      otro = ::Cluster.create!(organization: entity.organization, name: "otro2-#{SecureRandom.hex(2)}")
+      entity.assign_attributes(cluster_entities_attributes: { "1" => { "cluster_id" => otro.id.to_s, "_destroy" => "false" } })
+      entity.valid?
+      assert entity.errors[:cluster_entities].any?
+    end
+
+    test "acepta quitar una zona y volver a agregar la misma en el mismo guardado" do
+      entity, ce, cluster = build_entity_with_cluster
+      entity.assign_attributes(cluster_entities_attributes: {
+        "0" => { "id" => ce.id, "_destroy" => "1" },
+        "1" => { "cluster_id" => cluster.id.to_s, "_destroy" => "false" }
+      })
+      entity.valid?
+      assert_not entity.errors[:cluster_entities].any? { |m| m == I18n.t('klastera.messages.duplicated_cluster_entity') }
+    end
+
+    test "sigue rechazando la misma zona dos veces vigente" do
+      entity, _ce, cluster = build_entity_with_cluster
+      entity.assign_attributes(cluster_entities_attributes: { "1" => { "cluster_id" => cluster.id.to_s, "_destroy" => "false" } })
+      entity.valid?
+      assert entity.errors[:cluster_entities].any?
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: klastera
 version: !ruby/object:Gem::Version
-  version: 1.5.5.2
+  version: 1.5.5.4
 platform: ruby
 authors:
 - Gino Barahona