klastera 1.5.5.2 → 1.5.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7f593dccae852483eb2b18e8254ee94c1e44804bcb59780cd26a8987ef383443
-  data.tar.gz: f28976b48568045dfe38b3feb2915de5e87b079279b0a31e481f48741fdbd0a0
+  metadata.gz: 1b7a266fca197bb8333a1bfa72e2a36b2bdec447429a1e50e95b5b89dc31fb5f
+  data.tar.gz: fd8ec6949e6fea5015fe33090ecf09a90e9815361d8e861fb42ac856e83ff111
 SHA512:
-  metadata.gz: a2b1d549df310a5bf569316f5e9de36f66c9be7b5d052108d85aa1345f6d05527a536494ca39ea37e2f19f663e8973dd5d62a107ea101168f63d5a2ace6566b4
-  data.tar.gz: 1353ffb7147a62d4e35cf28a0ccafa2e976460e96db0dc566cc48ed1db0e16e2ea315d65fcd8b12c2642cea76f53e3a2b2a8c10d48000f08efb93f72f4f9c749
+  metadata.gz: 29ed3030a1296a3fb95073255642e9bef489fea5d136291d8d6f8e71f5b2b6f8ab040034e56c5dc66e70d09451d1880290fedfbbc904ac105ec7dc58ae4810f8
+  data.tar.gz: 7bc4e57bdac9abc5cf68be34bc64c48dbce81f6db405485e4dca1e6a8079e1d97e28630e159fb5049158a5d0f280fd000849b89e0cf6dfdc87f2a56bd4ae4e6b

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

@@ -54,8 +54,10 @@ 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

data/lib/klastera/version.rb

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

data/test/models/klastera/clusterizable_nested_attributes_test.rb

@@ -39,5 +39,26 @@ 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
   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.3
 platform: ruby
 authors:
 - Gino Barahona