From 49810beb1aed9c58aef1bae82255b10aae5cf825 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Fri, 6 Oct 2023 12:22:15 -0300
Subject: [PATCH] fix: arrays predefinidas

---
 app/models/metadata_predefined_array.rb       |  10 +-
 test/models/metadata_predefined_array_test.rb | 135 ++++++++++++++++++
 2 files changed, 143 insertions(+), 2 deletions(-)
 create mode 100644 test/models/metadata_predefined_array_test.rb

diff --git a/app/models/metadata_predefined_array.rb b/app/models/metadata_predefined_array.rb
index b8e5050e..6ac30636 100644
--- a/app/models/metadata_predefined_array.rb
+++ b/app/models/metadata_predefined_array.rb
@@ -2,9 +2,15 @@
 
 # Una lista de valores predefinidos
 class MetadataPredefinedArray < MetadataArray
+  # Genera una lista de valores para usar en options_for_select
+  #
+  # @todo Unificar la definición de valores por defecto con respecto a
+  # otros campos (como MetadataPredefinedValue), por ahora mantenemos
+  # por retrocompatibilidad.
+  # @return [Hash]
   def values
-    @values ||= layout.dig(:metadata, name, 'values')&.map do |k, v|
+    @values ||= layout.dig(:metadata, name, 'values')&.to_h do |k, v|
       [v[I18n.locale.to_s], k]
-    end&.to_h
+    end
   end
 end
diff --git a/test/models/metadata_predefined_array_test.rb b/test/models/metadata_predefined_array_test.rb
new file mode 100644
index 00000000..8dd99e0b
--- /dev/null
+++ b/test/models/metadata_predefined_array_test.rb
@@ -0,0 +1,135 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class MetadataPredefinedArrayTest < ActiveSupport::TestCase
+  setup do
+    @site = build(:site)
+    @document = build(:document, site: @site.jekyll)
+    @name = SecureRandom.hex
+    @locale = I18n.available_locales.sample.to_s
+    @layout = build(:layout, site: @site, metadata: {
+                      @name => {
+                        'values' => random_values
+                      }
+                    })
+    @metadata = MetadataPredefinedArray.new(site: @site, document: @document, name: @name, type: 'predefined_array',
+                                            layout: @layout)
+  end
+
+  def random_values
+    random_value.to_h do |k|
+      [
+        k,
+        I18n.available_locales.map(&:to_s).to_h do |l|
+          [l, SecureRandom.hex]
+        end
+      ]
+    end
+  end
+
+  def random_value
+    (1..100).to_a.sample.times.map do
+      SecureRandom.hex
+    end
+  end
+
+  test 'se guarda en el encabezado' do
+    assert @metadata.front_matter?
+  end
+
+  test 'por defecto es un array vacía' do
+    assert_equal [], @metadata.default_value
+  end
+
+  test 'los valores posibles vienen desde el layout' do
+    assert_equal @layout.metadata.dig(@name, 'values').keys, @metadata.values.values
+  end
+
+  test 'no puede estar vacía si es obligatoria' do
+    @layout = build(:layout, site: @site, metadata: { @name => { 'default' => { 'required' => true } } })
+    @metadata = MetadataPredefinedArray.new(site: @site, document: @document, name: @name, type: 'string', layout: @layout,
+                                            required: true)
+
+    assert @metadata.required
+    assert @metadata.empty?
+    assert_not @metadata.valid?
+  end
+
+  test 'puede traer el valor desde el documento' do
+    @document.data[@name] = random_value
+
+    assert_equal @document[@name], @metadata.document_value
+    assert @metadata.save
+    assert_equal @document[@name], @metadata.value
+  end
+
+  test 'al cambiar el valor podemos obtenerlo' do
+    @metadata.value = value_was = random_value
+    @metadata.value = random_value
+
+    assert_equal value_was, @metadata.value_was
+    assert_not_equal value_was, @metadata.value
+  end
+
+  test 'pueden tener un valor por defecto desde el layout' do
+    default = random_value
+
+    @layout = build(:layout, site: @site, metadata: { @name => { 'default' => { I18n.locale.to_s => default } } })
+    @metadata = MetadataArray.new(site: @site, document: @document, name: @name, type: 'array', layout: @layout)
+
+    assert_equal default, @metadata.default_value
+  end
+
+  test 'se pueden indexar' do
+    assert @metadata.indexable?
+  end
+
+  test 'son públicos por defecto' do
+    assert_not @metadata.private?
+  end
+
+  test 'no se pueden hacer privados' do
+    @layout = build(:layout, site: @site, metadata: { @name => { 'private' => true } })
+    @metadata = MetadataArray.new(site: @site, document: @document, name: @name, type: 'predefined_array',
+                                  layout: @layout)
+
+    assert_not @metadata.private?
+  end
+
+  test 'se les puede asignar un valor' do
+    @metadata.value = value = random_value
+
+    assert_equal value, @metadata.value
+  end
+
+  test 'el valor asignado no puede tener espacios adelante ni atrás' do
+    @metadata.value = value = ["  	#{SecureRandom.hex}\n\n"]
+
+    assert @metadata.save
+    assert_equal value.map(&:strip), @metadata.value
+  end
+
+  test 'no pueden tener html' do
+    content = SecureRandom.hex
+
+    @metadata.value = value = ["<html>#{content}</html>"]
+
+    assert @metadata.save
+    assert_not_equal value, @metadata.value
+    assert_equal [content], @metadata.value
+  end
+
+  test 'los valores vacíos son eliminados' do
+    @metadata.value = [
+      nil,
+      '',
+      '      '
+    ]
+
+    @metadata.save
+
+    assert @metadata.value.empty?
+    assert @metadata.empty?
+  end
+end