From 8a668e62541a521aaa117c085510b70664d03e4d Mon Sep 17 00:00:00 2001 From: f Date: Fri, 6 Oct 2023 11:39:58 -0300 Subject: [PATCH] fix: testear arrays MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit las arrays se usan para campos estilo categorías, etiquetas, autorxs --- app/models/metadata_array.rb | 11 ++-- test/models/metadata_array_test.rb | 100 +++++++++++++++++++++++++++++ 2 files changed, 107 insertions(+), 4 deletions(-) create mode 100644 test/models/metadata_array_test.rb diff --git a/app/models/metadata_array.rb b/app/models/metadata_array.rb index 68f4effb..e090feaa 100644 --- a/app/models/metadata_array.rb +++ b/app/models/metadata_array.rb @@ -25,12 +25,15 @@ class MetadataArray < MetadataTemplate [super].flatten(1).compact end - alias indexable_values value - private - # TODO: Sanitizar otros valores - # XXX: Por qué eliminamos el punto del final? + # Eliminar valores vacíos + # + # XXX: Por qué eliminábamos el punto del final? + # + # @todo Sanitizar otros valores + # @param :values [Array] + # @return [Array] def sanitize(values) values.map do |v| case v diff --git a/test/models/metadata_array_test.rb b/test/models/metadata_array_test.rb new file mode 100644 index 00000000..046aaeca --- /dev/null +++ b/test/models/metadata_array_test.rb @@ -0,0 +1,100 @@ +# frozen_string_literal: true + +require 'test_helper' + +class MetadataArrayTest < ActiveSupport::TestCase + setup do + @site = build(:site) + @document = build(:document, site: @site.jekyll) + @name = SecureRandom.hex + @layout = build(:layout, site: @site) + @metadata = MetadataArray.new(site: @site, document: @document, name: @name, type: 'array', layout: @layout) + 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 'no puede estar vacía si es obligatoria' do + @layout = build(:layout, site: @site, metadata: { @name => { 'default' => { 'required' => true } } }) + @metadata = MetadataArray.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: 'string', 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 = ["#{content}"] + + assert @metadata.save + assert_not_equal value, @metadata.value + assert_equal [content], @metadata.value + end +end