diff --git a/app/views/users/registrations/new.html.erb b/app/views/users/registrations/new.html.erb
index 3ae3156f7..8e702a5f1 100644
--- a/app/views/users/registrations/new.html.erb
+++ b/app/views/users/registrations/new.html.erb
@@ -18,8 +18,11 @@
<%= f.hidden_field :locale, value: I18n.locale %>
<%= f.label :username %>
-
<%= t("devise_views.users.registrations.new.username_note") %>
- <%= f.text_field :username, autofocus: true, maxlength: User.username_max_length, placeholder: t("devise_views.users.registrations.new.username_label"), label: false %>
+
<%= t("devise_views.users.registrations.new.username_note") %>
+ <%= f.text_field :username, autofocus: true, maxlength: User.username_max_length,
+ placeholder: t("devise_views.users.registrations.new.username_label"),
+ label: false,
+ aria: {describedby: "username-help-text"} %>
<%= f.invisible_captcha :family_name %>
diff --git a/app/views/verification/residence/new.html.erb b/app/views/verification/residence/new.html.erb
index 703e4b98f..2159d1a01 100644
--- a/app/views/verification/residence/new.html.erb
+++ b/app/views/verification/residence/new.html.erb
@@ -76,9 +76,9 @@
<%= f.label t("verification.residence.new.postal_code") %>
-
<%= t("verification.residence.new.postal_code_note") %>
+
<%= t("verification.residence.new.postal_code_note") %>
- <%= f.text_field :postal_code, label: false %>
+ <%= f.text_field :postal_code, label: false, aria: {describedby: "postal-code-help-text"} %>
diff --git a/app/views/verification/sms/new.html.erb b/app/views/verification/sms/new.html.erb
index 06f679e36..820ed01fa 100644
--- a/app/views/verification/sms/new.html.erb
+++ b/app/views/verification/sms/new.html.erb
@@ -31,8 +31,10 @@
<%= f.label :phone, t("verification.sms.new.phone"), class: "inline-block" %>
<%= t("verification.sms.new.phone_format_html") %>
-
<%= t("verification.sms.new.phone_note") %>
- <%= f.text_field :phone, label: false, placeholder: t("verification.sms.new.phone_placeholder") %>
+
<%= t("verification.sms.new.phone_note") %>
+ <%= f.text_field :phone, label: false,
+ placeholder: t("verification.sms.new.phone_placeholder"),
+ aria: {describedby: "phone-help-text"} %>
<%= f.submit t("verification.sms.new.submit_button"), class: "button success" %>
diff --git a/bin/setup b/bin/setup
index acdb2c138..d3bb99760 100755
--- a/bin/setup
+++ b/bin/setup
@@ -2,7 +2,7 @@
require 'pathname'
# path to your application root.
-APP_ROOT = Pathname.new File.expand_path('../../', __FILE__)
+APP_ROOT = Pathname.new File.expand_path('../../', __FILE__)
Dir.chdir APP_ROOT do
# This script is a starting point to setup your application.
diff --git a/config/locales/activerecord.en.yml b/config/locales/activerecord.en.yml
index 0af387a50..d45c569d4 100644
--- a/config/locales/activerecord.en.yml
+++ b/config/locales/activerecord.en.yml
@@ -10,6 +10,9 @@ en:
budget/investment:
one: "Investment"
other: "Investments"
+ budget/investment/milestone:
+ one: "milestone"
+ other: "milestones"
comment:
one: "Comment"
other: "Comments"
@@ -98,6 +101,9 @@ en:
title: "Title"
location: "Location"
organization_name: "If you are proposing in the name of a collective/organization, write its name"
+ budget/investment/milestone:
+ title: "Title"
+ description: "Description"
comment:
body: "Comment"
user: "User"
diff --git a/config/locales/activerecord.es.yml b/config/locales/activerecord.es.yml
index b2b5fc20e..678053f84 100644
--- a/config/locales/activerecord.es.yml
+++ b/config/locales/activerecord.es.yml
@@ -10,6 +10,9 @@ es:
budget/investment:
one: "Proyecto de inversión"
other: "Proyectos de inversión"
+ budget/investment/milestone:
+ one: "hito"
+ other: "hitos"
comment:
one: "Comentario"
other: "Comentarios"
@@ -96,6 +99,9 @@ es:
comment:
body: "Comentario"
user: "Usuario"
+ budget/investment/milestone:
+ title: "Título"
+ description: "Descripción"
debate:
author: "Autor"
description: "Opinión"
diff --git a/config/locales/admin.en.yml b/config/locales/admin.en.yml
index 7638ce535..9cc37b359 100755
--- a/config/locales/admin.en.yml
+++ b/config/locales/admin.en.yml
@@ -97,10 +97,15 @@ en:
heading: Heading name
add_heading: Add heading
amount: Amount
+ population: Population
save_heading: Save heading
no_heading: This group has no assigned heading.
table_heading: Heading
table_amount: Amount
+ table_population: Population
+ winners:
+ calculate: Calculate Winner Investments
+ calculated: Winners being calculated, it may take a minute.
budget_investments:
index:
heading_filter_all: All headings
@@ -151,6 +156,8 @@ en:
edit_dossier: Edit dossier
tags: Tags
undefined: Undefined
+ milestone: Milestone
+ new_milestone: Create new milestone
edit:
classification: Clasification
assigned_valuators: Valuators
@@ -160,6 +167,24 @@ en:
tags_placeholder: "Write the tags you want separated by commas (,)"
undefined: Undefined
search_unfeasible: Search unfeasible
+ milestones:
+ index:
+ table_id: "ID"
+ table_title: "Title"
+ table_description: "Description"
+ table_actions: "Actions"
+ delete: "Delete milestone"
+ no_milestones: "Don't have defined milestones"
+ new:
+ creating: Create milestone
+ edit:
+ title: Edit milestone
+ create:
+ notice: New milestone created successfully!
+ update:
+ notice: Milestone updated successfully
+ delete:
+ notice: Milestone successfully deleted
comments:
index:
filter: Filter
diff --git a/config/locales/admin.es.yml b/config/locales/admin.es.yml
index 92d5fb534..cb852328e 100644
--- a/config/locales/admin.es.yml
+++ b/config/locales/admin.es.yml
@@ -97,10 +97,15 @@ es:
heading: Nombre de la partida
add_heading: Añadir partida
amount: Cantidad
+ population: Población
save_heading: Guardar partida
no_heading: Este grupo no tiene ninguna partida asignada.
table_heading: Partida
table_amount: Cantidad
+ table_population: Población
+ winners:
+ calculate: Calcular propuestas ganadoras
+ calculated: Calculando ganadoras, puede tardar un minuto.
budget_investments:
index:
heading_filter_all: Todas las partidas
@@ -151,6 +156,8 @@ es:
edit_dossier: Editar informe
tags: Etiquetas
undefined: Sin definir
+ milestone: Seguimiento
+ new_milestone: Crear nuevo hito
edit:
classification: Clasificación
assigned_valuators: Evaluadores
@@ -160,6 +167,24 @@ es:
tags_placeholder: "Escribe las etiquetas que desees separadas por comas (,)"
undefined: Sin definir
search_unfeasible: Buscar inviables
+ milestones:
+ index:
+ table_id: "ID"
+ table_title: "Título"
+ table_description: "Descripción"
+ table_actions: "Acciones"
+ delete: "Eliminar hito"
+ no_milestones: "No hay hitos definidos"
+ new:
+ creating: Crear hito
+ edit:
+ title: Editar hito
+ create:
+ notice: Nuevo hito creado con éxito!
+ update:
+ notice: Hito actualizado
+ delete:
+ notice: Hito borrado correctamente
comments:
index:
filter: Filtro
diff --git a/config/locales/admin.fr.yml b/config/locales/admin.fr.yml
index d020a5690..2c3b84786 100644
--- a/config/locales/admin.fr.yml
+++ b/config/locales/admin.fr.yml
@@ -97,10 +97,15 @@ fr:
heading: Nom de la rubrique
add_heading: Ajouter une rubrique
amount: Montant
+ population: Population
save_heading: Sauvegarder la rubrique
no_heading: Ce groupe n'a pas de rubrique assignée.
table_heading: Rubrique
table_amount: Montant
+ table_population: Population
+ winners:
+ calculate: Calculate Winner Investments
+ calculated: Winners being calculated, it may take a minute.
budget_investments:
index:
heading_filter_all: Toutes les rubriques
diff --git a/config/locales/admin.nl.yml b/config/locales/admin.nl.yml
index f31313cf4..4e35e5ace 100755
--- a/config/locales/admin.nl.yml
+++ b/config/locales/admin.nl.yml
@@ -98,10 +98,15 @@ nl:
heading: Heading name
add_heading: Add heading
amount: Amount
+ population: Population
save_heading: Save heading
no_heading: This group has no assigned heading.
table_heading: Heading
table_amount: Amount
+ table_population: Population
+ winners:
+ calculate: Calculate Winner Investments
+ calculated: Winners being calculated, it may take a minute.
budget_investments:
index:
heading_filter_all: All headings
diff --git a/config/locales/budgets.en.yml b/config/locales/budgets.en.yml
index 905ccce7e..75a56c919 100644
--- a/config/locales/budgets.en.yml
+++ b/config/locales/budgets.en.yml
@@ -93,6 +93,10 @@ en:
supports: Supports
votes: Votes
price: Price
+ comments_tab: Comments
+ milestones_tab: Milestones
+ no_milestones: Don't have defined milestones
+ milestone_publish_date: "Published %{publish_date}"
wrong_price_format: Only integer numbers
investment:
add: Vote
@@ -131,4 +135,4 @@ en:
price: Cost
amount_available: Available budget
accepted: "Accepted spending proposal: "
- discarded: "Discarded spending proposal: "
\ No newline at end of file
+ discarded: "Discarded spending proposal: "
diff --git a/config/locales/budgets.es.yml b/config/locales/budgets.es.yml
index 38f8601e8..4c35d8902 100644
--- a/config/locales/budgets.es.yml
+++ b/config/locales/budgets.es.yml
@@ -93,6 +93,10 @@ es:
supports: Apoyos
votes: Votos
price: Coste
+ comments_tab: Comentarios
+ milestones_tab: Seguimiento
+ no_milestones: No hay hitos definidos
+ milestone_publish_date: "Publicado el %{publish_date}"
wrong_price_format: Solo puede incluir caracteres numéricos
investment:
add: Votar
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 7da965fe4..33b457e32 100755
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -298,7 +298,7 @@ en:
geozone: Scope of operation
proposal_external_url: Link to additional documentation
proposal_question: Proposal question
- proposal_question_example_html: Must be summarised in one question with a Yes or No answer.
E.g. 'Do you agree with the pedestrianisation of Calle Mayor?'
+ proposal_question_example_html: "Must be summarised in one question with a Yes or No answer.
E.g. 'Do you agree with the pedestrianisation of Calle Mayor?'"
proposal_responsible_name: Full name of the person submitting the proposal
proposal_responsible_name_note: "(individually or as representative of a collective; will not be displayed publically)"
proposal_summary: Proposal summary
@@ -357,12 +357,13 @@ en:
notice:
retired: Proposal retired
proposal:
- created: "You've created a new proposal!"
+ created: "You've created a proposal!"
share:
- guide: "Now you can share it through Twitter, Facebook, Google+, Telegram or Whatsapp (if you're using a mobile device) so people can start supporting.
Before it gets shared you'll be able to change the text as you like."
+ guide: "Now you can share it so people can start supporting."
+ edit: "Before it gets shared you'll be able to change the text as you like."
view_proposal: Not now, go to my proposal
- improve_info: "You can also %{improve_info_link} about improving your campaign"
- improve_info_link: see more information
+ improve_info: "Improve your campaign and get more supports"
+ improve_info_link: "See more information"
already_supported: You have already supported this proposal. Share it!
comments:
one: 1 comment
diff --git a/config/locales/es.yml b/config/locales/es.yml
index 07c653669..d38184647 100755
--- a/config/locales/es.yml
+++ b/config/locales/es.yml
@@ -298,7 +298,7 @@ es:
geozone: "Ámbito de actuación"
proposal_external_url: Enlace a documentación adicional
proposal_question: Pregunta de la propuesta
- proposal_question_example_html: Debe ser resumida en una pregunta cuya respuesta sea Sí o No.
Ej. '¿Está usted de acuerdo en peatonalizar la calle Mayor?'
+ proposal_question_example_html: "Debe ser resumida en una pregunta cuya respuesta sea Sí o No.
Ej. '¿Está usted de acuerdo en peatonalizar la calle Mayor?'"
proposal_responsible_name: Nombre y apellidos de la persona que hace esta propuesta
proposal_responsible_name_note: "(individualmente o como representante de un colectivo; no se mostrará públicamente)"
proposal_summary: Resumen de la propuesta
@@ -357,12 +357,13 @@ es:
notice:
retired: Propuesta retirada
proposal:
- created: "¡Has creado una nueva propuesta!"
+ created: "¡Has creado una propuesta!"
share:
- guide: "Ahora puedes compartirla en Twitter, Facebook, Google+, Telegram o Whatsapp (si utilizas un dispositivo móvil) para que la gente empieze a apoyarla.
Antes de que se publique el texto en las redes sociales podrás modificarlo a tu gusto"
- view_proposal: "Ahora no, ver mi propuesta"
- improve_info: "También puedes %{improve_info_link} de como mejorar tu campaña"
- improve_info_link: ver más información
+ guide: "Compártela para que la gente empieze a apoyarla."
+ edit: "Antes de que se publique podrás modificar el texto a tu gusto."
+ view_proposal: "Ahora no, ir a mi propuesta"
+ improve_info: "Mejora tu campaña y consigue más apoyos"
+ improve_info_link: "Ver más información"
already_supported: "¡Ya has apoyado esta propuesta, compártela!"
comments:
one: 1 Comentario
diff --git a/config/locales/fr.yml b/config/locales/fr.yml
index b978f70e2..e4d0f9a51 100644
--- a/config/locales/fr.yml
+++ b/config/locales/fr.yml
@@ -1249,8 +1249,8 @@ fr:
proposal_external_url: Lien vers de la documentation complémentaire
proposal_question: Question proposée
proposal_question_example_html: 'Doit être résumé par une question appelant
- à une réponse par oui ou par non.
Exemple : ''Êtes-vous d''accord avec
- la piétonnisation de Calle Mayor?'''
+ à une réponse par oui ou par non.
Exemple : ''Êtes-vous d''accord avec
+ la piétonnisation de Calle Mayor?'
proposal_responsible_name: Nom complet de la personne qui soumet la proposition
proposal_responsible_name_note: "(individuellement ou comme représentant d'un
collectif; ne sera pas public)"
diff --git a/config/locales/nl.yml b/config/locales/nl.yml
index 48e1bcdc9..62221c280 100755
--- a/config/locales/nl.yml
+++ b/config/locales/nl.yml
@@ -14,7 +14,7 @@ nl:
phone_number_label: Telefoonnummer
public_activity_label: Mijn activiteiten zijn publiek
save_changes_submit: Sla veranderingen op
- subscription_to_website_newsletter_label: Stuur me email met relevante informatie over de website
+ subscription_to_website_newsletter_label: Stuur me email met relevante informatie over de website
email_on_direct_message_label: Stuur me email naar aanleiding van directe boodschappen
email_digest_label: Stuur me een samenvatting van meldingen naar aanleiding van voorstellen
official_position_badge_label: Show official position badge
@@ -179,7 +179,7 @@ nl:
conditions: Gebruiksvoorwaarden
consul: Consul
consul_url: https://github.com/consul/consul
- contact_us: Voor tech support klik
+ contact_us: Voor tech support klik
copyright: Consul, %{year}
description: Deze site gebruikt %{consul}; een %{open_source} toepassing.
faq: hier
@@ -217,7 +217,7 @@ nl:
other: U heeft %{count} nieuwe meldingen
no_notifications: "U heeft geen nieuwe meldingen"
open: open
- open_city_slogan_html: Er zijn steden die rechtstreeks door hun bewoners worden bestuurd, welke
discussiëren over de onderwerpen waarover ze zich zorgen maken, welke
voorstellen doen om hun leven te verbeteren en
zelf beslissen welke daarvan zullen worden uitgevoerd.
+ open_city_slogan_html: Er zijn steden die rechtstreeks door hun bewoners worden bestuurd, welke
discussiëren over de onderwerpen waarover ze zich zorgen maken, welke
voorstellen doen om hun leven te verbeteren en
zelf beslissen welke daarvan zullen worden uitgevoerd.
open_city_title: De gemeente waar u van houdt is de gemeente waarin u meedenkt.
open_gov: Open bestuur
proposals: Voorstellen
@@ -297,10 +297,10 @@ nl:
done: Is gebeurt
other: Anders
form:
- geozone: Regio
+ geozone: Regio
proposal_external_url: Link naar meet informatie
proposal_question: Vraag
- proposal_question_example_html: Moet samengevat worden in een vraag met een ja/nee antwoord.
E.g. 'Ben je het eens met autovrije Geldersekade?'
+ proposal_question_example_html: Moet samengevat worden in een vraag met een ja/nee antwoord.
E.g. 'Ben je het eens met autovrije Geldersekade?'
proposal_responsible_name: Volledige naam van persoon die het voorstel doet
proposal_responsible_name_note: "(individueel of als vertegenwoordiger van een collectief; niet publiek zichtbaar)"
proposal_summary: Samenvatting voorstel
@@ -506,7 +506,7 @@ nl:
suggest:
debate:
found:
-
+
one: "Er is een debat met de term '%{query}'; u kunt daar deelnemen in plaats van een nieuwe discussie te openen."
other: "Er zijn discussies met de term '%{query}'; u kunt daar deelnemen in plaats van een nieuwe discussie te openen."
message: "Er worden %{limit} uit %{count} discussies met de term '%{query}' getoond"
@@ -636,7 +636,7 @@ nl:
other: "%{count} Begrotingsvoorstellen"
no_activity: Deze deelnemer heeft geen publieke activiteit
no_private_messages: "Deze deelnemer ontvangt geen privé berichten."
- private_activity: Deze deelnemer heeft besloten activiteiten niet te delen
+ private_activity: Deze deelnemer heeft besloten activiteiten niet te delen
send_private_message: "Stuur privé bericht"
proposals:
send_notification: "Stuur melding"
diff --git a/config/locales/pt-BR.yml b/config/locales/pt-BR.yml
index 90e4f35d8..69ca72c02 100644
--- a/config/locales/pt-BR.yml
+++ b/config/locales/pt-BR.yml
@@ -1267,8 +1267,8 @@ pt-BR:
proposal_external_url: Link para documentação adicional
proposal_question: Questão proposta
proposal_question_example_html: 'Precisa ser sumarizado em uma questão com resposta
- Sim ou Não.
Ex.: ''Você concorda com a mudança da Calle Mayor como via
- exclusiva de pedestres?'''
+ Sim ou Não.
Ex.: ''Você concorda com a mudança da Calle Mayor como via
+ exclusiva de pedestres?'
proposal_responsible_name: Nome completo da pessoa que está submetendo a proposta
proposal_responsible_name_note: "(indivudualmente ou como representante de um
coletivo; não será mostrado publicamente)"
diff --git a/config/routes.rb b/config/routes.rb
index 5c5284a91..ef476cf28 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -193,16 +193,22 @@ Rails.application.routes.draw do
end
resources :budgets do
+ member do
+ put :calculate_winners
+ end
+
resources :budget_groups do
resources :budget_headings do
end
end
resources :budget_investments, only: [:index, :show, :edit, :update] do
+ resources :budget_investment_milestones
member { patch :toggle_selection }
end
end
+
resources :signature_sheets, only: [:index, :new, :create, :show]
resources :banners, only: [:index, :new, :create, :edit, :update, :destroy] do
diff --git a/db/dev_seeds.rb b/db/dev_seeds.rb
index 8f34fddd5..35257931a 100644
--- a/db/dev_seeds.rb
+++ b/db/dev_seeds.rb
@@ -390,7 +390,8 @@ Budget::PHASES.each_with_index do |phase, i|
geozones = Geozone.reorder("RANDOM()").limit([2, 5, 6, 7].sample)
geozones.each do |geozone|
group.headings << group.headings.create!(name: geozone.name,
- price: rand(1..100) * 100000)
+ price: rand(1..100) * 100000,
+ population: rand(1..50) * 10000)
end
end
end
diff --git a/db/migrate/20170620132731_create_budget_investment_milestones.rb b/db/migrate/20170620132731_create_budget_investment_milestones.rb
new file mode 100644
index 000000000..199d8e70e
--- /dev/null
+++ b/db/migrate/20170620132731_create_budget_investment_milestones.rb
@@ -0,0 +1,11 @@
+class CreateBudgetInvestmentMilestones < ActiveRecord::Migration
+ def change
+ create_table :budget_investment_milestones do |t|
+ t.integer :investment_id
+ t.string "title", limit: 80
+ t.text "description"
+
+ t.timestamps null: false
+ end
+ end
+end
diff --git a/db/migrate/20170621180611_add_population_to_budget_headings.rb b/db/migrate/20170621180611_add_population_to_budget_headings.rb
new file mode 100644
index 000000000..fe723bc98
--- /dev/null
+++ b/db/migrate/20170621180611_add_population_to_budget_headings.rb
@@ -0,0 +1,5 @@
+class AddPopulationToBudgetHeadings < ActiveRecord::Migration
+ def change
+ add_column :budget_headings, :population, :integer, default: nil
+ end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 98e563748..3c21882e9 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -110,10 +110,19 @@ ActiveRecord::Schema.define(version: 20170623141655) do
t.integer "group_id"
t.string "name", limit: 50
t.integer "price", limit: 8
+ t.integer "population"
end
add_index "budget_headings", ["group_id"], name: "index_budget_headings_on_group_id", using: :btree
+ create_table "budget_investment_milestones", force: :cascade do |t|
+ t.integer "investment_id"
+ t.string "title", limit: 80
+ t.text "description"
+ t.datetime "created_at", null: false
+ t.datetime "updated_at", null: false
+ end
+
create_table "budget_investments", force: :cascade do |t|
t.integer "author_id"
t.integer "administrator_id"
@@ -413,8 +422,8 @@ ActiveRecord::Schema.define(version: 20170623141655) do
t.date "allegations_end_date"
t.date "result_publication_date"
t.datetime "hidden_at"
- t.datetime "created_at", null: false
- t.datetime "updated_at", null: false
+ t.datetime "created_at", null: false
+ t.datetime "updated_at", null: false
t.text "summary"
t.boolean "debate_phase_enabled", default: false
t.boolean "allegations_phase_enabled", default: false
@@ -923,7 +932,7 @@ ActiveRecord::Schema.define(version: 20170623141655) do
t.boolean "email_digest", default: true
t.boolean "email_on_direct_message", default: true
t.boolean "official_position_badge", default: false
- t.datetime "password_changed_at", default: '2016-12-21 17:55:08', null: false
+ t.datetime "password_changed_at", default: '2017-06-22 11:21:30', null: false
t.boolean "created_from_signature", default: false
t.integer "failed_email_digests_count", default: 0
t.text "former_users_data_log", default: ""
diff --git a/lib/acts_as_paranoid_aliases.rb b/lib/acts_as_paranoid_aliases.rb
index cf0c0bac8..17587fca8 100644
--- a/lib/acts_as_paranoid_aliases.rb
+++ b/lib/acts_as_paranoid_aliases.rb
@@ -25,7 +25,7 @@ module ActsAsParanoidAliases
update_attribute(:confirmed_hide_at, Time.current)
end
- def restore(opts={})
+ def restore(opts = {})
return false unless hidden?
super(opts)
update_attribute(:confirmed_hide_at, nil)
diff --git a/lib/graph_ql/api_types_creator.rb b/lib/graph_ql/api_types_creator.rb
index f8f78355e..9f55be3cc 100644
--- a/lib/graph_ql/api_types_creator.rb
+++ b/lib/graph_ql/api_types_creator.rb
@@ -42,7 +42,7 @@ module GraphQL
field(field_name, SCALAR_TYPES[field_type], model.human_attribute_name(field_name))
when :singular_association
field(field_name, -> { created_types[field_type] }) do
- resolve -> (object, arguments, context) do
+ resolve ->(object, arguments, context) do
association_target = object.send(field_name)
association_target.present? ? field_type.public_for_api.find_by(id: association_target.id) : nil
end
@@ -50,7 +50,7 @@ module GraphQL
when :multiple_association
field_type = field_type.first
connection(field_name, -> { created_types[field_type].connection_type }, max_page_size: 50, complexity: 1000) do
- resolve -> (object, arguments, context) { object.send(field_name).public_for_api }
+ resolve ->(object, arguments, context) { object.send(field_name).public_for_api }
end
end
end
diff --git a/lib/graph_ql/query_type_creator.rb b/lib/graph_ql/query_type_creator.rb
index 71beb8033..8f4a497ef 100644
--- a/lib/graph_ql/query_type_creator.rb
+++ b/lib/graph_ql/query_type_creator.rb
@@ -14,13 +14,13 @@ module GraphQL
type created_type
description model.graphql_field_description
argument :id, !types.ID
- resolve -> (object, arguments, context) { model.public_for_api.find_by(id: arguments['id'])}
+ resolve ->(object, arguments, context) { model.public_for_api.find_by(id: arguments['id'])}
end
end
connection(model.graphql_pluralized_field_name, created_type.connection_type, max_page_size: 50, complexity: 1000) do
description model.graphql_pluralized_field_description
- resolve -> (object, arguments, context) { model.public_for_api }
+ resolve ->(object, arguments, context) { model.public_for_api }
end
end
diff --git a/lib/manager_authenticator.rb b/lib/manager_authenticator.rb
index bd61d97a5..8f9a1f55a 100644
--- a/lib/manager_authenticator.rb
+++ b/lib/manager_authenticator.rb
@@ -1,6 +1,6 @@
class ManagerAuthenticator
- def initialize(data={})
+ def initialize(data = {})
@manager = {login: data[:login], user_key: data[:clave_usuario], date: data[:fecha_conexion]}.with_indifferent_access
end
@@ -25,7 +25,7 @@ class ManagerAuthenticator
parsed_response = parser.parse((response[:get_applications_user_list_response][:get_applications_user_list_return]))
aplication_value = parsed_response["APLICACIONES"]["APLICACION"]
# aplication_value from UWEB can be an array of hashes or a hash
- aplication_value.include?( {"CLAVE_APLICACION" => application_key}) || aplication_value["CLAVE_APLICACION"] == application_key
+ aplication_value.include?("CLAVE_APLICACION" => application_key) || aplication_value["CLAVE_APLICACION"] == application_key
rescue
false
end
diff --git a/lib/migrate_spending_proposals_to_investments.rb b/lib/migrate_spending_proposals_to_investments.rb
index 9ef2f096d..6df75d3f7 100644
--- a/lib/migrate_spending_proposals_to_investments.rb
+++ b/lib/migrate_spending_proposals_to_investments.rb
@@ -50,7 +50,7 @@ class MigrateSpendingProposalsToInvestments
votes = ActsAsVotable::Vote.where(votable_type: 'SpendingProposal', votable_id: sp.id)
- votes.each {|v| investment.vote_by({voter: v.voter, vote: 'yes'}) }
+ votes.each {|v| investment.vote_by(voter: v.voter, vote: 'yes') }
# Spending proposals are not commentable in Consul so we can not test this
#
diff --git a/lib/score_calculator.rb b/lib/score_calculator.rb
index 32086392a..4f4e21e85 100644
--- a/lib/score_calculator.rb
+++ b/lib/score_calculator.rb
@@ -1,7 +1,7 @@
module ScoreCalculator
- EPOC = Time.new(2015, 6, 15)
- COMMENT_WEIGHT = 1.0/5 # 1 positive vote / x comments
+ EPOC = Time.new(2015, 6, 15).in_time_zone
+ COMMENT_WEIGHT = 1.0 / 5 # 1 positive vote / x comments
TIME_UNIT = 24.hours.to_f
def self.hot_score(date, votes_total, votes_up, comments_count)
@@ -13,7 +13,7 @@ module ScoreCalculator
sign = score <=> 0
seconds = ((date || Time.current) - EPOC).to_f
- (((offset * sign) + (seconds/TIME_UNIT)) * 10000000).round
+ (((offset * sign) + (seconds / TIME_UNIT)) * 10000000).round
end
def self.confidence_score(votes_total, votes_up)
diff --git a/lib/sms_api.rb b/lib/sms_api.rb
index de0e2c312..252d0749e 100644
--- a/lib/sms_api.rb
+++ b/lib/sms_api.rb
@@ -38,7 +38,20 @@ class SMSApi
end
def stubbed_response
- {:respuesta_sms=>{:identificador_mensaje=>"1234567", :fecha_respuesta=>"Thu, 20 Aug 2015 16:28:05 +0200", :respuesta_pasarela=>{:codigo_pasarela=>"0000", :descripcion_pasarela=>"Operación ejecutada correctamente."}, :respuesta_servicio_externo=>{:codigo_respuesta=>"1000", :texto_respuesta=>"Success"}}}
+ {
+ respuesta_sms: {
+ identificador_mensaje: "1234567",
+ fecha_respuesta: "Thu, 20 Aug 2015 16:28:05 +0200",
+ respuesta_pasarela: {
+ codigo_pasarela: "0000",
+ descripcion_pasarela: "Operación ejecutada correctamente."
+ },
+ respuesta_servicio_externo: {
+ codigo_respuesta: "1000",
+ texto_respuesta: "Success"
+ }
+ }
+ }
end
end
diff --git a/spec/controllers/admin/api/stats_controller_spec.rb b/spec/controllers/admin/api/stats_controller_spec.rb
index 7ba63ffb3..8dd948086 100644
--- a/spec/controllers/admin/api/stats_controller_spec.rb
+++ b/spec/controllers/admin/api/stats_controller_spec.rb
@@ -17,9 +17,9 @@ describe Admin::Api::StatsController do
context 'events present' do
before :each do
- time_1 = DateTime.parse("2015-01-01")
- time_2 = DateTime.parse("2015-01-02")
- time_3 = DateTime.parse("2015-01-03")
+ time_1 = DateTime.parse("2015-01-01").in_time_zone
+ time_2 = DateTime.parse("2015-01-02").in_time_zone
+ time_3 = DateTime.parse("2015-01-03").in_time_zone
create :ahoy_event, name: 'foo', time: time_1
create :ahoy_event, name: 'foo', time: time_1
@@ -36,7 +36,7 @@ describe Admin::Api::StatsController do
expect(response).to be_ok
data = JSON.parse(response.body)
- expect(data).to eq "x"=>["2015-01-01", "2015-01-02"], "Foo"=>[2, 1]
+ expect(data).to eq "x" => ["2015-01-01", "2015-01-02"], "Foo" => [2, 1]
end
it 'should return combined comma separated events formated for working with c3.js' do
@@ -46,14 +46,14 @@ describe Admin::Api::StatsController do
expect(response).to be_ok
data = JSON.parse(response.body)
- expect(data).to eq "x"=>["2015-01-01", "2015-01-02", "2015-01-03"], "Foo"=>[2, 1, 0], "Bar"=>[1, 0, 2]
+ expect(data).to eq "x" => ["2015-01-01", "2015-01-02", "2015-01-03"], "Foo" => [2, 1, 0], "Bar" => [1, 0, 2]
end
end
context 'visits present' do
it 'should return visits formated for working with c3.js' do
- time_1 = DateTime.parse("2015-01-01")
- time_2 = DateTime.parse("2015-01-02")
+ time_1 = DateTime.parse("2015-01-01").in_time_zone
+ time_2 = DateTime.parse("2015-01-02").in_time_zone
create :visit, started_at: time_1
create :visit, started_at: time_1
@@ -65,14 +65,14 @@ describe Admin::Api::StatsController do
expect(response).to be_ok
data = JSON.parse(response.body)
- expect(data).to eq "x"=>["2015-01-01", "2015-01-02"], "Visits"=>[2, 1]
+ expect(data).to eq "x" => ["2015-01-01", "2015-01-02"], "Visits" => [2, 1]
end
end
context 'visits and events present' do
it 'should return combined events and visits formated for working with c3.js' do
- time_1 = DateTime.parse("2015-01-01")
- time_2 = DateTime.parse("2015-01-02")
+ time_1 = DateTime.parse("2015-01-01").in_time_zone
+ time_2 = DateTime.parse("2015-01-02").in_time_zone
create :ahoy_event, name: 'foo', time: time_1
create :ahoy_event, name: 'foo', time: time_2
@@ -88,14 +88,14 @@ describe Admin::Api::StatsController do
expect(response).to be_ok
data = JSON.parse(response.body)
- expect(data).to eq "x"=>["2015-01-01", "2015-01-02"], "Foo"=>[1, 2], "Visits"=>[2, 1]
+ expect(data).to eq "x" => ["2015-01-01", "2015-01-02"], "Foo" => [1, 2], "Visits" => [2, 1]
end
end
context 'budget investments present' do
it 'should return budget investments formated for working with c3.js' do
- time_1 = DateTime.parse("2017-04-01")
- time_2 = DateTime.parse("2017-04-02")
+ time_1 = DateTime.parse("2017-04-01").in_time_zone
+ time_2 = DateTime.parse("2017-04-02").in_time_zone
budget_investment1 = create(:budget_investment, budget: @budget, created_at: time_1)
budget_investment2 = create(:budget_investment, budget: @budget, created_at: time_2)
@@ -107,7 +107,7 @@ describe Admin::Api::StatsController do
expect(response).to be_ok
data = JSON.parse(response.body)
- expect(data).to eq "x"=>["2017-04-01", "2017-04-02"], "Budget Investments"=>[1, 2]
+ expect(data).to eq "x" => ["2017-04-01", "2017-04-02"], "Budget Investments" => [1, 2]
end
end
end
diff --git a/spec/controllers/comments_controller_spec.rb b/spec/controllers/comments_controller_spec.rb
index abd2577c2..d06e1900f 100644
--- a/spec/controllers/comments_controller_spec.rb
+++ b/spec/controllers/comments_controller_spec.rb
@@ -4,7 +4,7 @@ describe CommentsController do
describe 'POST create' do
before(:each) do
- @process = create(:legislation_process, debate_start_date: Date.current - 3.day, debate_end_date: Date.current + 2.days)
+ @process = create(:legislation_process, debate_start_date: Date.current - 3.days, debate_end_date: Date.current + 2.days)
@question = create(:legislation_question, process: @process, title: "Question 1")
@user = create(:user, :level_two)
@unverified_user = create(:user)
diff --git a/spec/controllers/concerns/has_orders_spec.rb b/spec/controllers/concerns/has_orders_spec.rb
index 37d62fa84..fa2c49b07 100644
--- a/spec/controllers/concerns/has_orders_spec.rb
+++ b/spec/controllers/concerns/has_orders_spec.rb
@@ -7,7 +7,7 @@ describe 'HasOrders' do
controller(FakeController) do
include HasOrders
has_orders ['created_at', 'votes_count', 'flags_count', 'relevance'], only: :index
- has_orders -> (c) { ['votes_count', 'flags_count'] }, only: :new
+ has_orders ->(c) { ['votes_count', 'flags_count'] }, only: :new
def index
render text: "#{@current_order} (#{@valid_orders.join(' ')})"
diff --git a/spec/controllers/graphql_controller_spec.rb b/spec/controllers/graphql_controller_spec.rb
index 1b8c64793..0f15da038 100644
--- a/spec/controllers/graphql_controller_spec.rb
+++ b/spec/controllers/graphql_controller_spec.rb
@@ -71,19 +71,19 @@ describe GraphqlController, type: :request do
let(:query_string) { "{ proposal(id: #{proposal.id}) { title } }" }
specify "when absent" do
- get '/graphql', { query: query_string }
+ get '/graphql', query: query_string
expect(response).to have_http_status(:ok)
end
specify "when specified as the 'null' string" do
- get '/graphql', { query: query_string, variables: 'null' }
+ get '/graphql', query: query_string, variables: 'null'
expect(response).to have_http_status(:ok)
end
specify "when specified as an empty string" do
- get '/graphql', { query: query_string, variables: '' }
+ get '/graphql', query: query_string, variables: ''
expect(response).to have_http_status(:ok)
end
diff --git a/spec/controllers/legislation/annotations_controller_spec.rb b/spec/controllers/legislation/annotations_controller_spec.rb
index 5b1fd3d39..834ec046f 100644
--- a/spec/controllers/legislation/annotations_controller_spec.rb
+++ b/spec/controllers/legislation/annotations_controller_spec.rb
@@ -4,7 +4,7 @@ describe Legislation::AnnotationsController do
describe 'POST create' do
before(:each) do
- @process = create(:legislation_process, allegations_start_date: Date.current - 3.day, allegations_end_date: Date.current + 2.days)
+ @process = create(:legislation_process, allegations_start_date: Date.current - 3.days, allegations_end_date: Date.current + 2.days)
@draft_version = create(:legislation_draft_version, :published, process: @process, title: "Version 1")
@final_version = create(:legislation_draft_version, :published, :final_version, process: @process, title: "Final version")
@user = create(:user, :level_two)
@@ -16,8 +16,8 @@ describe Legislation::AnnotationsController do
post :create, process_id: @process.id,
draft_version_id: @draft_version.id,
legislation_annotation: {
- "quote"=>"ipsum",
- "ranges"=>[{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}],
+ "quote" => "ipsum",
+ "ranges" => [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}],
"text": "una anotacion"
}
expect(Ahoy::Event.where(name: :legislation_annotation_created).count).to eq 1
@@ -30,8 +30,8 @@ describe Legislation::AnnotationsController do
post :create, process_id: @process.id,
draft_version_id: @final_version.id,
legislation_annotation: {
- "quote"=>"ipsum",
- "ranges"=>[{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}],
+ "quote" => "ipsum",
+ "ranges" => [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}],
"text": "una anotacion"
}
@@ -45,8 +45,8 @@ describe Legislation::AnnotationsController do
xhr :post, :create, process_id: @process.id,
draft_version_id: @draft_version.id,
legislation_annotation: {
- "quote"=>"ipsum",
- "ranges"=>[{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}],
+ "quote" => "ipsum",
+ "ranges" => [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}],
"text": "una anotacion"
}
end.to change { @draft_version.annotations.count }.by(1)
@@ -60,8 +60,8 @@ describe Legislation::AnnotationsController do
xhr :post, :create, process_id: @process.id,
draft_version_id: @draft_version.id,
legislation_annotation: {
- "quote"=>"ipsum",
- "ranges"=>[{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}],
+ "quote" => "ipsum",
+ "ranges" => [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}],
"text": "una anotacion"
}
end.to_not change { @draft_version.annotations.count }
@@ -74,8 +74,8 @@ describe Legislation::AnnotationsController do
xhr :post, :create, process_id: @process.id,
draft_version_id: @draft_version.id,
legislation_annotation: {
- "quote"=>"ipsum",
- "ranges"=>[{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}].to_json,
+ "quote" => "ipsum",
+ "ranges" => [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}].to_json,
"text": "una anotacion"
}
end.to change { @draft_version.annotations.count }.by(1)
@@ -83,7 +83,7 @@ describe Legislation::AnnotationsController do
it 'should create a new comment on an existing annotation when range is the same' do
annotation = create(:legislation_annotation, draft_version: @draft_version, text: "my annotation",
- ranges: [{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}],
+ ranges: [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}],
range_start: "/p[1]", range_start_offset: 6, range_end: "/p[1]", range_end_offset: 11)
sign_in @user
@@ -91,8 +91,8 @@ describe Legislation::AnnotationsController do
xhr :post, :create, process_id: @process.id,
draft_version_id: @draft_version.id,
legislation_annotation: {
- "quote"=>"ipsum",
- "ranges"=>[{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}],
+ "quote" => "ipsum",
+ "ranges" => [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}],
"text": "una anotacion"
}
end.to_not change { @draft_version.annotations.count }
diff --git a/spec/controllers/legislation/answers_controller_spec.rb b/spec/controllers/legislation/answers_controller_spec.rb
index 77d037478..569ea2879 100644
--- a/spec/controllers/legislation/answers_controller_spec.rb
+++ b/spec/controllers/legislation/answers_controller_spec.rb
@@ -4,7 +4,7 @@ describe Legislation::AnswersController do
describe 'POST create' do
before(:each) do
- @process = create(:legislation_process, debate_start_date: Date.current - 3.day, debate_end_date: Date.current + 2.days)
+ @process = create(:legislation_process, debate_start_date: Date.current - 3.days, debate_end_date: Date.current + 2.days)
@question = create(:legislation_question, process: @process, title: "Question 1")
@question_option = create(:legislation_question_option, question: @question, value: "Yes")
@user = create(:user, :level_two)
diff --git a/spec/controllers/management/sessions_controller_spec.rb b/spec/controllers/management/sessions_controller_spec.rb
index d2d6b5f8b..0e2a18544 100644
--- a/spec/controllers/management/sessions_controller_spec.rb
+++ b/spec/controllers/management/sessions_controller_spec.rb
@@ -5,15 +5,15 @@ describe Management::SessionsController do
describe 'Sign in' do
it "should deny access if wrong manager credentials" do
allow_any_instance_of(ManagerAuthenticator).to receive(:auth).and_return(false)
- expect { get :create, login: "nonexistent" , clave_usuario: "wrong"}.to raise_error CanCan::AccessDenied
+ expect { get :create, login: "nonexistent", clave_usuario: "wrong"}.to raise_error CanCan::AccessDenied
expect(session[:manager]).to be_nil
end
it "should redirect to management root path if authorized manager with right credentials" do
- manager = {login: "JJB033", user_key: "31415926" , date: "20151031135905"}
+ manager = {login: "JJB033", user_key: "31415926", date: "20151031135905"}
allow_any_instance_of(ManagerAuthenticator).to receive(:auth).and_return(manager)
- get :create, login: "JJB033" , clave_usuario: "31415926", fecha_conexion: "20151031135905"
+ get :create, login: "JJB033", clave_usuario: "31415926", fecha_conexion: "20151031135905"
expect(response).to be_redirect
expect(session[:manager][:login]).to eq "JJB033"
end
@@ -44,7 +44,7 @@ describe Management::SessionsController do
describe 'Sign out' do
it "should destroy the session data and redirect" do
session[:manager] = {user_key: "31415926", date: "20151031135905", login: "JJB033"}
- session[:document_type] = "1"
+ session[:document_type] = "1"
session[:document_number] = "12345678Z"
delete :destroy
diff --git a/spec/controllers/management/users_controller_spec.rb b/spec/controllers/management/users_controller_spec.rb
index ed73aa053..221d302b6 100644
--- a/spec/controllers/management/users_controller_spec.rb
+++ b/spec/controllers/management/users_controller_spec.rb
@@ -4,13 +4,13 @@ describe Management::UsersController do
describe 'logout' do
it "should remove user data from the session" do
- session[:manager] = {user_key: "31415926" , date: "20151031135905", login: "JJB033"}
- session[:document_type] = "1"
+ session[:manager] = {user_key: "31415926", date: "20151031135905", login: "JJB033"}
+ session[:document_type] = "1"
session[:document_number] = "12345678Z"
get :logout
- expect(session[:manager]).to eq({user_key: "31415926" , date: "20151031135905", login: "JJB033"})
+ expect(session[:manager]).to eq(user_key: "31415926", date: "20151031135905", login: "JJB033")
expect(session[:document_type]).to be_nil
expect(session[:document_number]).to be_nil
expect(response).to be_redirect
diff --git a/spec/factories.rb b/spec/factories.rb
index 94d1a86ff..8ada920b8 100644
--- a/spec/factories.rb
+++ b/spec/factories.rb
@@ -99,7 +99,7 @@ FactoryGirl.define do
factory :verified_user do
document_number
- document_type 'dni'
+ document_type 'dni'
end
factory :debate do
@@ -254,10 +254,11 @@ FactoryGirl.define do
association :group, factory: :budget_group
sequence(:name) { |n| "Heading #{n}" }
price 1000000
+ population 1234
end
factory :budget_investment, class: 'Budget::Investment' do
- sequence(:title) { |n| "Budget Investment #{n} title" }
+ sequence(:title) { |n| "Budget Investment #{n} title" }
association :heading, factory: :budget_heading
association :author, factory: :user
description 'Spend money on this'
@@ -322,6 +323,12 @@ FactoryGirl.define do
reason "unfeasible"
end
+ factory :budget_investment_milestone, class: 'Budget::Investment::Milestone' do
+ association :investment, factory: :budget_investment
+ sequence(:title) { |n| "Budget investment milestone #{n} title" }
+ description 'Milestone description'
+ end
+
factory :vote do
association :votable, factory: :debate
association :voter, factory: :user
@@ -372,7 +379,7 @@ FactoryGirl.define do
factory :annotation do
quote "ipsum"
text "Loremp ipsum dolor"
- ranges [{"start"=>"/div[1]", "startOffset"=>5, "end"=>"/div[1]", "endOffset"=>10}]
+ ranges [{"start" => "/div[1]", "startOffset" => 5, "end" => "/div[1]", "endOffset" => 10}]
legacy_legislation
user
end
@@ -541,7 +548,7 @@ FactoryGirl.define do
sequence(:value) { |n| "Setting #{n} Value" }
end
- factory :ahoy_event, :class => Ahoy::Event do
+ factory :ahoy_event, class: Ahoy::Event do
id { SecureRandom.uuid }
time DateTime.current
sequence(:name) {|n| "Event #{n} type"}
@@ -574,7 +581,7 @@ FactoryGirl.define do
factory :banner do
sequence(:title) { |n| "Banner title #{n}" }
- sequence(:description) { |n| "This is the text of Banner #{n}" }
+ sequence(:description) { |n| "This is the text of Banner #{n}" }
style {["banner-style-one", "banner-style-two", "banner-style-three"].sample}
image {["banner.banner-img-one", "banner.banner-img-two", "banner.banner-img-three"].sample}
target_url {["/proposals", "/debates" ].sample}
@@ -628,7 +635,7 @@ FactoryGirl.define do
end_date Date.current + 8.days
debate_start_date Date.current + 2.days
debate_end_date Date.current + 4.days
- draft_publication_date Date.current + 5.day
+ draft_publication_date Date.current + 5.days
allegations_start_date Date.current + 5.days
allegations_end_date Date.current + 7.days
result_publication_date Date.current + 8.days
@@ -639,7 +646,7 @@ FactoryGirl.define do
end_date Date.current - 2.days
debate_start_date Date.current - 12.days
debate_end_date Date.current - 9.days
- draft_publication_date Date.current - 8.day
+ draft_publication_date Date.current - 8.days
allegations_start_date Date.current - 8.days
allegations_end_date Date.current - 4.days
result_publication_date Date.current - 2.days
@@ -649,7 +656,7 @@ FactoryGirl.define do
start_date Date.current - 5.days
end_date Date.current + 5.days
debate_start_date Date.current - 5.days
- debate_end_date Date.current + 1.days
+ debate_end_date Date.current + 1.day
draft_publication_date Date.current + 1.day
allegations_start_date Date.current + 2.days
allegations_end_date Date.current + 3.days
@@ -691,7 +698,7 @@ LOREM_IPSUM
author factory: :user
quote "ipsum"
text "a comment"
- ranges [{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}]
+ ranges [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}]
range_start "/p[1]"
range_start_offset 6
range_end "/p[1]"
diff --git a/spec/features/admin/banners_spec.rb b/spec/features/admin/banners_spec.rb
index 70d9f7621..28a105bed 100644
--- a/spec/features/admin/banners_spec.rb
+++ b/spec/features/admin/banners_spec.rb
@@ -29,7 +29,7 @@ feature 'Admin banners magement' do
target_url: "http://www.url.com",
style: "banner-style.banner-three",
image: "banner-img.banner-three",
- post_started_at: (Time.current - 1.days),
+ post_started_at: (Time.current - 1.day),
post_ended_at: (Time.current + 10.days))
@banner4 = create(:banner, title: "Banner number four",
diff --git a/spec/features/admin/budget_investment_milestones_spec.rb b/spec/features/admin/budget_investment_milestones_spec.rb
new file mode 100644
index 000000000..9bcb78479
--- /dev/null
+++ b/spec/features/admin/budget_investment_milestones_spec.rb
@@ -0,0 +1,92 @@
+require 'rails_helper'
+
+feature 'Admin budget investment milestones' do
+
+ background do
+ admin = create(:administrator)
+ login_as(admin.user)
+
+ @investment = create(:budget_investment)
+ end
+
+ context "Index" do
+ scenario 'Displaying milestones' do
+ milestone = create(:budget_investment_milestone, investment: @investment)
+
+ visit admin_budget_budget_investment_path(@investment.budget, @investment)
+
+ expect(page).to have_content("Milestone")
+ expect(page).to have_content(milestone.title)
+ expect(page).to have_content(milestone.id)
+ end
+
+ scenario 'Displaying no_milestones text' do
+ visit admin_budget_budget_investment_path(@investment.budget, @investment)
+
+ expect(page).to have_content("Milestone")
+ expect(page).to have_content("Don't have defined milestones")
+ end
+ end
+
+ context "New" do
+ scenario "Add milestone" do
+ visit admin_budget_budget_investment_path(@investment.budget, @investment)
+
+ click_link 'Create new milestone'
+
+ fill_in 'budget_investment_milestone_title', with: 'New title milestone'
+ fill_in 'budget_investment_milestone_description', with: 'New description milestone'
+
+ click_button 'Create milestone'
+
+ expect(page).to have_content 'New title milestone'
+ expect(page).to have_content 'New description milestone'
+ end
+
+ scenario "Show validation errors on milestone form" do
+ visit admin_budget_budget_investment_path(@investment.budget, @investment)
+
+ click_link 'Create new milestone'
+
+ fill_in 'budget_investment_milestone_description', with: 'New description milestone'
+
+ click_button 'Create milestone'
+
+ within "#new_budget_investment_milestone" do
+ expect(page).to have_content "can't be blank"
+ expect(page).to have_content 'New description milestone'
+ end
+ end
+ end
+
+ context "Edit" do
+ scenario "Change title and description" do
+ milestone = create(:budget_investment_milestone, investment: @investment)
+
+ visit admin_budget_budget_investment_path(@investment.budget, @investment)
+
+ click_link milestone.title
+
+ fill_in 'budget_investment_milestone_title', with: 'Changed title'
+ fill_in 'budget_investment_milestone_description', with: 'Changed description'
+
+ click_button 'Update milestone'
+
+ expect(page).to have_content 'Changed title'
+ expect(page).to have_content 'Changed description'
+ end
+ end
+
+ context "Delete" do
+ scenario "Remove milestone" do
+ milestone = create(:budget_investment_milestone, investment: @investment, title: "Title will it remove")
+
+ visit admin_budget_budget_investment_path(@investment.budget, @investment)
+
+ click_link "Delete milestone"
+
+ expect(page).to_not have_content 'Title will it remove'
+ end
+ end
+
+end
diff --git a/spec/features/admin/budget_investments_spec.rb b/spec/features/admin/budget_investments_spec.rb
index 52c008c18..727cc3d6c 100644
--- a/spec/features/admin/budget_investments_spec.rb
+++ b/spec/features/admin/budget_investments_spec.rb
@@ -27,7 +27,7 @@ feature 'Admin budget investments' do
context "Index" do
- scenario 'Displaying investmentss' do
+ scenario 'Displaying investments' do
budget_investment = create(:budget_investment, budget: @budget, cached_votes_up: 77)
visit admin_budget_budget_investments_path(budget_id: @budget.id)
expect(page).to have_content(budget_investment.title)
@@ -47,7 +47,7 @@ feature 'Admin budget investments' do
budget_investment1.valuators << valuator1
budget_investment2.valuator_ids = [valuator1.id, valuator2.id]
- budget_investment3.update({administrator_id: admin.id})
+ budget_investment3.update(administrator_id: admin.id)
visit admin_budget_budget_investments_path(budget_id: @budget.id)
diff --git a/spec/features/admin/budgets_spec.rb b/spec/features/admin/budgets_spec.rb
index 760817c26..d4ed2d2ac 100644
--- a/spec/features/admin/budgets_spec.rb
+++ b/spec/features/admin/budgets_spec.rb
@@ -109,6 +109,33 @@ feature 'Admin budgets' do
end
+ context "Calculate Budget's Winner Investments" do
+
+ scenario 'For a Budget in reviewing balloting' do
+ budget = create(:budget, phase: 'reviewing_ballots')
+ group = create(:budget_group, budget: budget)
+ heading = create(:budget_heading, group: group, price: 4)
+ unselected_investment = create(:budget_investment, :unselected, heading: heading, price: 1, ballot_lines_count: 3)
+ winner_investment = create(:budget_investment, :winner, heading: heading, price: 3, ballot_lines_count: 2)
+ selected_investment = create(:budget_investment, :selected, heading: heading, price: 2, ballot_lines_count: 1)
+
+ visit edit_admin_budget_path(budget)
+ click_link 'Calculate Winner Investments'
+ expect(page).to have_content 'Winners being calculated, it may take a minute.'
+ expect(page).to have_content winner_investment.title
+ expect(page).not_to have_content unselected_investment.title
+ expect(page).not_to have_content selected_investment.title
+ end
+
+ scenario 'For a finished Budget' do
+ budget = create(:budget, phase: 'finished')
+
+ visit edit_admin_budget_path(budget)
+ expect(page).not_to have_content 'Calculate Winner Investments'
+ end
+
+ end
+
context 'Manage groups and headings' do
scenario 'Create group', :js do
@@ -156,6 +183,7 @@ feature 'Admin budgets' do
fill_in 'budget_heading_name', with: 'District 9 reconstruction'
fill_in 'budget_heading_price', with: '6785'
+ fill_in 'budget_heading_population', with: '100500'
click_button 'Save heading'
end
@@ -167,6 +195,7 @@ feature 'Admin budgets' do
expect(page).to have_content 'District 9 reconstruction'
expect(page).to have_content '6785'
+ expect(page).to have_content '100500'
end
end
diff --git a/spec/features/admin/feature_flags_spec.rb b/spec/features/admin/feature_flags_spec.rb
index 956d80e6c..99c460477 100644
--- a/spec/features/admin/feature_flags_spec.rb
+++ b/spec/features/admin/feature_flags_spec.rb
@@ -13,7 +13,6 @@ feature 'Admin feature flags' do
Setting['feature.spending_proposal_features.voting_allowed'] = nil
end
-
scenario 'Enabled features are listed on menu' do
visit admin_root_path
diff --git a/spec/features/admin/poll/polls_spec.rb b/spec/features/admin/poll/polls_spec.rb
index d2041a583..4508e5bba 100644
--- a/spec/features/admin/poll/polls_spec.rb
+++ b/spec/features/admin/poll/polls_spec.rb
@@ -356,7 +356,7 @@ feature 'Admin polls' do
question_2.valid_answers.each_with_index do |answer, i|
within("#question_#{question_2.id}_#{i}_result") do
expect(page).to have_content(answer)
- expect(page).to have_content([0,15][i])
+ expect(page).to have_content([0, 15][i])
end
end
diff --git a/spec/features/admin/spending_proposals_spec.rb b/spec/features/admin/spending_proposals_spec.rb
index 239454902..c27e102ee 100644
--- a/spec/features/admin/spending_proposals_spec.rb
+++ b/spec/features/admin/spending_proposals_spec.rb
@@ -43,7 +43,7 @@ feature 'Admin spending proposals' do
spending_proposal1.valuators << valuator1
spending_proposal2.valuator_ids = [valuator1.id, valuator2.id]
- spending_proposal3.update({administrator_id: admin.id})
+ spending_proposal3.update(administrator_id: admin.id)
visit admin_spending_proposals_path
@@ -634,7 +634,6 @@ feature 'Admin spending proposals' do
proposal5 = create(:spending_proposal, geozone: new_york, valuation_finished: true)
proposal6 = create(:spending_proposal, geozone: new_york, valuation_finished: false)
-
create(:vote, votable: proposal1)
create(:vote, votable: proposal2)
create(:vote, votable: proposal3)
diff --git a/spec/features/admin/verifications_spec.rb b/spec/features/admin/verifications_spec.rb
index 50ff457a1..ce455fe87 100644
--- a/spec/features/admin/verifications_spec.rb
+++ b/spec/features/admin/verifications_spec.rb
@@ -36,7 +36,6 @@ feature 'Incomplete verifications' do
expect(page).to_not have_content("Isabel_anonymous")
end
-
scenario "Residence unverified" do
incompletely_verified_user = create(:user, :incomplete_verification)
diff --git a/spec/features/budgets/investments_spec.rb b/spec/features/budgets/investments_spec.rb
index e131973da..dfb64a755 100644
--- a/spec/features/budgets/investments_spec.rb
+++ b/spec/features/budgets/investments_spec.rb
@@ -5,7 +5,7 @@ feature 'Budget Investments' do
let(:author) { create(:user, :level_two, username: 'Isabel') }
let(:budget) { create(:budget, name: "Big Budget") }
let(:other_budget) { create(:budget, name: "What a Budget!") }
- let(:group) { create(:budget_group, name: "Health", budget: budget) }
+ let(:group) { create(:budget_group, name: "Health", budget: budget) }
let!(:heading) { create(:budget_heading, name: "More hospitals", group: group) }
scenario 'Index' do
@@ -389,6 +389,37 @@ feature 'Budget Investments' do
expect(page).to have_content(investment.unfeasibility_explanation)
end
+ scenario "Show milestones", :js do
+ user = create(:user)
+ investment = create(:budget_investment)
+ milestone = create(:budget_investment_milestone, investment: investment, title: "New text to show")
+
+ login_as(user)
+ visit budget_investment_path(budget_id: investment.budget.id, id: investment.id)
+
+ find("#tab-milestones-label").trigger('click')
+
+ within("#tab-milestones") do
+ expect(page).to have_content(milestone.title)
+ expect(page).to have_content(milestone.description)
+ expect(page).to have_content("Published #{milestone.created_at.strftime("%d/%m/%Y")}")
+ end
+ end
+
+ scenario "Show no_milestones text", :js do
+ user = create(:user)
+ investment = create(:budget_investment)
+
+ login_as(user)
+ visit budget_investment_path(budget_id: investment.budget.id, id: investment.id)
+
+ find("#tab-milestones-label").trigger('click')
+
+ within("#tab-milestones") do
+ expect(page).to have_content("Don't have defined milestones")
+ end
+ end
+
context "Destroy" do
scenario "Admin cannot destroy budget investments" do
@@ -610,11 +641,11 @@ feature 'Budget Investments' do
carabanchel_heading = create(:budget_heading, group: group, name: "Carabanchel")
new_york_heading = create(:budget_heading, group: group, name: "New York")
- sp1 = create(:budget_investment, :selected, price: 1, heading: global_heading)
- sp2 = create(:budget_investment, :selected, price: 10, heading: global_heading)
- sp3 = create(:budget_investment, :selected, price: 100, heading: global_heading)
- sp4 = create(:budget_investment, :selected, price: 1000, heading: carabanchel_heading)
- sp5 = create(:budget_investment, :selected, price: 10000, heading: carabanchel_heading)
+ sp1 = create(:budget_investment, :selected, price: 1, heading: global_heading)
+ sp2 = create(:budget_investment, :selected, price: 10, heading: global_heading)
+ sp3 = create(:budget_investment, :selected, price: 100, heading: global_heading)
+ sp4 = create(:budget_investment, :selected, price: 1000, heading: carabanchel_heading)
+ sp5 = create(:budget_investment, :selected, price: 10000, heading: carabanchel_heading)
sp6 = create(:budget_investment, :selected, price: 100000, heading: new_york_heading)
login_as(user)
diff --git a/spec/features/comments/budget_investments_spec.rb b/spec/features/comments/budget_investments_spec.rb
index b36f51b8f..7e96b1f80 100644
--- a/spec/features/comments/budget_investments_spec.rb
+++ b/spec/features/comments/budget_investments_spec.rb
@@ -2,7 +2,7 @@ require 'rails_helper'
include ActionView::Helpers::DateHelper
feature 'Commenting Budget::Investments' do
- let(:user) { create :user }
+ let(:user) { create :user }
let(:investment) { create :budget_investment }
scenario 'Index' do
@@ -166,9 +166,12 @@ feature 'Commenting Budget::Investments' do
fill_in "comment-body-budget_investment_#{investment.id}", with: 'Have you thought about...?'
click_button 'Publish comment'
+ within "#tab-comments-label" do
+ expect(page).to have_content 'Comments (1)'
+ end
+
within "#comments" do
expect(page).to have_content 'Have you thought about...?'
- expect(page).to have_content 'Comments (1)'
end
end
@@ -392,7 +395,7 @@ feature 'Commenting Budget::Investments' do
end
scenario "can not comment as a moderator" do
- admin = create(:administrator)
+ admin = create(:administrator)
login_as(admin.user)
visit budget_investment_path(investment.budget, investment)
@@ -489,4 +492,4 @@ feature 'Commenting Budget::Investments' do
end
end
-end
\ No newline at end of file
+end
diff --git a/spec/features/comments/debates_spec.rb b/spec/features/comments/debates_spec.rb
index 577ba6fd0..2a4d46ec3 100644
--- a/spec/features/comments/debates_spec.rb
+++ b/spec/features/comments/debates_spec.rb
@@ -407,7 +407,7 @@ feature 'Commenting debates' do
end
scenario "can not comment as a moderator" do
- admin = create(:administrator)
+ admin = create(:administrator)
login_as(admin.user)
visit debate_path(debate)
diff --git a/spec/features/comments/legislation_annotations_spec.rb b/spec/features/comments/legislation_annotations_spec.rb
index b7e57039d..7727d8204 100644
--- a/spec/features/comments/legislation_annotations_spec.rb
+++ b/spec/features/comments/legislation_annotations_spec.rb
@@ -2,7 +2,7 @@ require 'rails_helper'
include ActionView::Helpers::DateHelper
feature 'Commenting legislation questions' do
- let(:user) { create :user }
+ let(:user) { create :user }
let(:legislation_annotation) { create :legislation_annotation, author: user }
scenario 'Index' do
@@ -409,7 +409,7 @@ feature 'Commenting legislation questions' do
end
scenario "can not comment as a moderator" do
- admin = create(:administrator)
+ admin = create(:administrator)
login_as(admin.user)
visit legislation_process_draft_version_annotation_path(legislation_annotation.draft_version.process, legislation_annotation.draft_version, legislation_annotation)
@@ -510,8 +510,8 @@ feature 'Commenting legislation questions' do
feature "Merged comment threads", :js do
let!(:draft_version) { create(:legislation_draft_version, :published) }
- let!(:annotation1) { create(:legislation_annotation, draft_version: draft_version, text: "my annotation", ranges: [{"start"=>"/p[1]", "startOffset"=>1, "end"=>"/p[1]", "endOffset"=>5}]) }
- let!(:annotation2) { create(:legislation_annotation, draft_version: draft_version, text: "my other annotation", ranges: [{"start"=>"/p[1]", "startOffset"=>1, "end"=>"/p[1]", "endOffset"=>10}]) }
+ let!(:annotation1) { create(:legislation_annotation, draft_version: draft_version, text: "my annotation", ranges: [{"start" => "/p[1]", "startOffset" => 1, "end" => "/p[1]", "endOffset" => 5}]) }
+ let!(:annotation2) { create(:legislation_annotation, draft_version: draft_version, text: "my other annotation", ranges: [{"start" => "/p[1]", "startOffset" => 1, "end" => "/p[1]", "endOffset" => 10}]) }
background do
login_as user
diff --git a/spec/features/comments/legislation_questions_spec.rb b/spec/features/comments/legislation_questions_spec.rb
index 35521e89b..064a72ae0 100644
--- a/spec/features/comments/legislation_questions_spec.rb
+++ b/spec/features/comments/legislation_questions_spec.rb
@@ -2,7 +2,7 @@ require 'rails_helper'
include ActionView::Helpers::DateHelper
feature 'Commenting legislation questions' do
- let(:user) { create :user, :level_two }
+ let(:user) { create :user, :level_two }
let(:process) { create :legislation_process, :in_debate_phase }
let(:legislation_question) { create :legislation_question, process: process }
@@ -192,7 +192,7 @@ feature 'Commenting legislation questions' do
end
scenario "Can't create comments if debate phase is not open", :js do
- process.update_attributes(debate_start_date: Date.current - 2.days, debate_end_date: Date.current - 1.days)
+ process.update_attributes(debate_start_date: Date.current - 2.days, debate_end_date: Date.current - 1.day)
login_as(user)
visit legislation_process_question_path(legislation_question.process, legislation_question)
@@ -424,7 +424,7 @@ feature 'Commenting legislation questions' do
end
scenario "can not comment as a moderator" do
- admin = create(:administrator)
+ admin = create(:administrator)
login_as(admin.user)
visit legislation_process_question_path(legislation_question.process, legislation_question)
diff --git a/spec/features/comments/proposals_spec.rb b/spec/features/comments/proposals_spec.rb
index 9f3d22966..d10a11de7 100644
--- a/spec/features/comments/proposals_spec.rb
+++ b/spec/features/comments/proposals_spec.rb
@@ -2,7 +2,7 @@ require 'rails_helper'
include ActionView::Helpers::DateHelper
feature 'Commenting proposals' do
- let(:user) { create :user }
+ let(:user) { create :user }
let(:proposal) { create :proposal }
scenario 'Index' do
@@ -395,7 +395,7 @@ feature 'Commenting proposals' do
end
scenario "can not comment as a moderator" do
- admin = create(:administrator)
+ admin = create(:administrator)
login_as(admin.user)
visit proposal_path(proposal)
diff --git a/spec/features/debates_spec.rb b/spec/features/debates_spec.rb
index 6914a55f7..b1357bb1c 100644
--- a/spec/features/debates_spec.rb
+++ b/spec/features/debates_spec.rb
@@ -211,8 +211,6 @@ feature 'Debates' do
expect(page.html).to_not include ""
end
-
-
scenario 'Update should not be posible if logged user is not the author' do
debate = create(:debate)
expect(debate).to be_editable
@@ -629,7 +627,7 @@ feature 'Debates' do
click_link "Advanced search"
select "Customized", from: "js-advanced-search-date-min"
fill_in "advanced_search_date_min", with: 7.days.ago
- fill_in "advanced_search_date_max", with: 1.days.ago
+ fill_in "advanced_search_date_max", with: 1.day.ago
click_button "Filter"
within("#debates") do
@@ -676,7 +674,7 @@ feature 'Debates' do
click_link "Advanced search"
fill_in "Write the text", with: "Schwifty"
select Setting['official_level_1_name'], from: "advanced_search_official_level"
- select "Last 24 hours", from: "js-advanced-search-date-min"
+ select "Last 24 hours", from: "js-advanced-search-date-min"
click_button "Filter"
@@ -709,7 +707,7 @@ feature 'Debates' do
select "Customized", from: "js-advanced-search-date-min"
fill_in "advanced_search_date_min", with: 7.days.ago
- fill_in "advanced_search_date_max", with: 1.days.ago
+ fill_in "advanced_search_date_max", with: 1.day.ago
click_button "Filter"
within "#js-advanced-search" do
@@ -868,10 +866,10 @@ feature 'Debates' do
login_as(author)
debate1 = create(:debate, title: "First debate has 1 vote", cached_votes_up: 1)
- debate2 = create(:debate, title: "Second debate has 2 votes", cached_votes_up: 2)
- debate3 = create(:debate, title: "Third debate has 3 votes", cached_votes_up: 3)
- debate4 = create(:debate, title: "This one has 4 votes", description: "This is the fourth debate", cached_votes_up: 4)
- debate5 = create(:debate, title: "Fifth debate has 5 votes", cached_votes_up: 5)
+ debate2 = create(:debate, title: "Second debate has 2 votes", cached_votes_up: 2)
+ debate3 = create(:debate, title: "Third debate has 3 votes", cached_votes_up: 3)
+ debate4 = create(:debate, title: "This one has 4 votes", description: "This is the fourth debate", cached_votes_up: 4)
+ debate5 = create(:debate, title: "Fifth debate has 5 votes", cached_votes_up: 5)
debate6 = create(:debate, title: "Sixth debate has 6 votes", description: 'This is the sixth debate', cached_votes_up: 6)
debate7 = create(:debate, title: "This has seven votes, and is not suggest", description: 'This is the seven', cached_votes_up: 7)
@@ -947,7 +945,6 @@ feature 'Debates' do
end
end
-
scenario 'Index do not show featured debates if none is marked as featured' do
admin = create(:administrator)
login_as(admin.user)
diff --git a/spec/features/legacy_legislation_spec.rb b/spec/features/legacy_legislation_spec.rb
index e2f1ba915..5df3b8f76 100644
--- a/spec/features/legacy_legislation_spec.rb
+++ b/spec/features/legacy_legislation_spec.rb
@@ -77,8 +77,8 @@ feature 'Legacy Legislation' do
scenario 'Search' do
legacy_legislation = create(:legacy_legislation)
- annotation1 = create(:annotation, legacy_legislation: legacy_legislation, text: "my annotation", ranges: [{"start"=>"/div[1]", "startOffset"=>5, "end"=>"/div[1]", "endOffset"=>10}])
- annotation2 = create(:annotation, legacy_legislation: legacy_legislation, text: "my other annotation", ranges: [{"start"=>"/div[1]", "startOffset"=>12, "end"=>"/div[1]", "endOffset"=>19}])
+ annotation1 = create(:annotation, legacy_legislation: legacy_legislation, text: "my annotation", ranges: [{"start" => "/div[1]", "startOffset" => 5, "end" => "/div[1]", "endOffset" => 10}])
+ annotation2 = create(:annotation, legacy_legislation: legacy_legislation, text: "my other annotation", ranges: [{"start" => "/div[1]", "startOffset" => 12, "end" => "/div[1]", "endOffset" => 19}])
visit legacy_legislation_path(legacy_legislation)
diff --git a/spec/features/legislation/draft_versions_spec.rb b/spec/features/legislation/draft_versions_spec.rb
index 7288c0cf0..f0a798b85 100644
--- a/spec/features/legislation/draft_versions_spec.rb
+++ b/spec/features/legislation/draft_versions_spec.rb
@@ -154,11 +154,11 @@ feature 'Legislation Draft Versions' do
page.find(:css, ".legislation-annotatable").double_click
page.find(:css, ".annotator-adder button").click
- page.click_button "Create Annotation"
+ page.click_button "Publish Comment"
expect(page).to have_content "Comment can't be blank"
fill_in 'legislation_annotation_text', with: 'this is my annotation'
- page.click_button "Create Annotation"
+ page.click_button "Publish Comment"
expect(page).to have_css ".annotator-hl"
first(:css, ".annotator-hl").click
@@ -173,8 +173,8 @@ feature 'Legislation Draft Versions' do
scenario 'View annotations and comments' do
draft_version = create(:legislation_draft_version, :published)
- annotation1 = create(:legislation_annotation, draft_version: draft_version, text: "my annotation", ranges: [{"start"=>"/p[1]", "startOffset"=>5, "end"=>"/p[1]", "endOffset"=>10}])
- annotation2 = create(:legislation_annotation, draft_version: draft_version, text: "my other annotation", ranges: [{"start"=>"/p[1]", "startOffset"=>12, "end"=>"/p[1]", "endOffset"=>19}])
+ annotation1 = create(:legislation_annotation, draft_version: draft_version, text: "my annotation", ranges: [{"start" => "/p[1]", "startOffset" => 5, "end" => "/p[1]", "endOffset" => 10}])
+ annotation2 = create(:legislation_annotation, draft_version: draft_version, text: "my other annotation", ranges: [{"start" => "/p[1]", "startOffset" => 12, "end" => "/p[1]", "endOffset" => 19}])
comment = create(:comment, commentable: annotation1)
visit legislation_process_draft_version_path(draft_version.process, draft_version)
@@ -190,7 +190,7 @@ feature 'Legislation Draft Versions' do
scenario "Publish new comment for an annotation from comments box" do
draft_version = create(:legislation_draft_version, :published)
- annotation = create(:legislation_annotation, draft_version: draft_version, text: "my annotation", ranges: [{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}])
+ annotation = create(:legislation_annotation, draft_version: draft_version, text: "my annotation", ranges: [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}])
visit legislation_process_draft_version_path(draft_version.process, draft_version)
@@ -212,8 +212,8 @@ feature 'Legislation Draft Versions' do
scenario 'View annotations and comments in an included range' do
draft_version = create(:legislation_draft_version, :published)
- annotation1 = create(:legislation_annotation, draft_version: draft_version, text: "my annotation", ranges: [{"start"=>"/p[1]", "startOffset"=>1, "end"=>"/p[1]", "endOffset"=>5}])
- annotation2 = create(:legislation_annotation, draft_version: draft_version, text: "my other annotation", ranges: [{"start"=>"/p[1]", "startOffset"=>1, "end"=>"/p[1]", "endOffset"=>10}])
+ annotation1 = create(:legislation_annotation, draft_version: draft_version, text: "my annotation", ranges: [{"start" => "/p[1]", "startOffset" => 1, "end" => "/p[1]", "endOffset" => 5}])
+ annotation2 = create(:legislation_annotation, draft_version: draft_version, text: "my other annotation", ranges: [{"start" => "/p[1]", "startOffset" => 1, "end" => "/p[1]", "endOffset" => 10}])
visit legislation_process_draft_version_path(draft_version.process, draft_version)
@@ -232,8 +232,8 @@ feature 'Legislation Draft Versions' do
context "Annotations page" do
background do
@draft_version = create(:legislation_draft_version, :published)
- @annotation_1 = create(:legislation_annotation, draft_version: @draft_version, text: "my annotation", quote: "ipsum", ranges: [{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}])
- @annotation_2 = create(:legislation_annotation, draft_version: @draft_version, text: "my other annotation", quote: "audiam", ranges: [{"start"=>"/p[3]", "startOffset"=>6, "end"=>"/p[3]", "endOffset"=>11}])
+ @annotation_1 = create(:legislation_annotation, draft_version: @draft_version, text: "my annotation", quote: "ipsum", ranges: [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}])
+ @annotation_2 = create(:legislation_annotation, draft_version: @draft_version, text: "my other annotation", quote: "audiam", ranges: [{"start" => "/p[3]", "startOffset" => 6, "end" => "/p[3]", "endOffset" => 11}])
end
scenario "See all annotations for a draft version" do
@@ -247,9 +247,9 @@ feature 'Legislation Draft Versions' do
background do
@process = create(:legislation_process)
@draft_version_1 = create(:legislation_draft_version, :published, process: @process, title: "Version 1", body: "Text with quote for version 1")
- @annotation_1 = create(:legislation_annotation, draft_version: @draft_version_1, text: "annotation for version 1", quote: "quote for version 1", ranges: [{"start"=>"/p[1]", "startOffset"=>11, "end"=>"/p[1]", "endOffset"=>30}])
+ @annotation_1 = create(:legislation_annotation, draft_version: @draft_version_1, text: "annotation for version 1", quote: "quote for version 1", ranges: [{"start" => "/p[1]", "startOffset" => 11, "end" => "/p[1]", "endOffset" => 30}])
@draft_version_2 = create(:legislation_draft_version, :published, process: @process, title: "Version 2", body: "Text with quote for version 2")
- @annotation_1 = create(:legislation_annotation, draft_version: @draft_version_2, text: "annotation for version 2", quote: "quote for version 2", ranges: [{"start"=>"/p[1]", "startOffset"=>11, "end"=>"/p[1]", "endOffset"=>30}])
+ @annotation_1 = create(:legislation_annotation, draft_version: @draft_version_2, text: "annotation for version 2", quote: "quote for version 2", ranges: [{"start" => "/p[1]", "startOffset" => 11, "end" => "/p[1]", "endOffset" => 30}])
end
scenario "without js" do
@@ -278,8 +278,8 @@ feature 'Legislation Draft Versions' do
context "Annotation comments page" do
background do
@draft_version = create(:legislation_draft_version, :published)
- @annotation_1 = create(:legislation_annotation, draft_version: @draft_version, text: "my annotation", quote: "ipsum", ranges: [{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[1]", "endOffset"=>11}])
- @annotation_2 = create(:legislation_annotation, draft_version: @draft_version, text: "my other annotation", quote: "audiam", ranges: [{"start"=>"/p[3]", "startOffset"=>6, "end"=>"/p[3]", "endOffset"=>11}])
+ @annotation_1 = create(:legislation_annotation, draft_version: @draft_version, text: "my annotation", quote: "ipsum", ranges: [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[1]", "endOffset" => 11}])
+ @annotation_2 = create(:legislation_annotation, draft_version: @draft_version, text: "my other annotation", quote: "audiam", ranges: [{"start" => "/p[3]", "startOffset" => 6, "end" => "/p[3]", "endOffset" => 11}])
end
scenario "See one annotation with replies for a draft version" do
diff --git a/spec/features/legislation/questions_spec.rb b/spec/features/legislation/questions_spec.rb
index 77503b5a9..17d55c16c 100644
--- a/spec/features/legislation/questions_spec.rb
+++ b/spec/features/legislation/questions_spec.rb
@@ -3,7 +3,7 @@ require 'rails_helper'
feature 'Legislation' do
context 'process debate page' do
before(:each) do
- @process = create(:legislation_process, debate_start_date: Date.current - 3.day, debate_end_date: Date.current + 2.days)
+ @process = create(:legislation_process, debate_start_date: Date.current - 3.days, debate_end_date: Date.current + 2.days)
create(:legislation_question, process: @process, title: "Question 1")
create(:legislation_question, process: @process, title: "Question 2")
create(:legislation_question, process: @process, title: "Question 3")
diff --git a/spec/features/management/users_spec.rb b/spec/features/management/users_spec.rb
index e7c43f4a9..cdb0ec879 100644
--- a/spec/features/management/users_spec.rb
+++ b/spec/features/management/users_spec.rb
@@ -29,7 +29,7 @@ feature 'Users' do
expect(user).to be_level_three_verified
expect(user).to be_residence_verified
expect(user).to_not be_confirmed
- expect(user.date_of_birth).to have_content (Date.new(1980,12,31))
+ expect(user.date_of_birth).to have_content (Date.new(1980, 12, 31))
sent_token = /.*confirmation_token=(.*)".*/.match(ActionMailer::Base.deliveries.last.body.to_s)[1]
visit user_confirmation_path(confirmation_token: sent_token)
@@ -69,7 +69,7 @@ feature 'Users' do
expect(user).to be_level_three_verified
expect(user).to be_residence_verified
expect(user).to be_confirmed
- expect(user.date_of_birth).to have_content (Date.new(1980,12,31))
+ expect(user.date_of_birth).to have_content (Date.new(1980, 12, 31))
end
scenario 'Delete a level 2 user account from document verification page', :js do
diff --git a/spec/features/organizations_spec.rb b/spec/features/organizations_spec.rb
index 0a7a2bcc7..f968d6cac 100644
--- a/spec/features/organizations_spec.rb
+++ b/spec/features/organizations_spec.rb
@@ -8,7 +8,7 @@ feature 'Organizations' do
visit new_organization_registration_path
- fill_in 'user_organization_attributes_name', with: 'Greenpeace'
+ fill_in 'user_organization_attributes_name', with: 'Greenpeace'
fill_in 'user_organization_attributes_responsible_name', with: 'Dorothy Stowe'
fill_in 'user_email', with: 'green@peace.com'
fill_in 'user_password', with: 'greenpeace'
@@ -45,7 +45,7 @@ feature 'Organizations' do
scenario 'Create organization too fast' do
allow(InvisibleCaptcha).to receive(:timestamp_threshold).and_return(Float::INFINITY)
visit new_organization_registration_path
- fill_in 'user_organization_attributes_name', with: 'robot'
+ fill_in 'user_organization_attributes_name', with: 'robot'
fill_in 'user_organization_attributes_responsible_name', with: 'Robots are more responsible than humans'
fill_in 'user_email', with: 'robot@robot.com'
fill_in 'user_password', with: 'destroyallhumans'
diff --git a/spec/features/proposals_spec.rb b/spec/features/proposals_spec.rb
index 6fb298303..8faa899f9 100644
--- a/spec/features/proposals_spec.rb
+++ b/spec/features/proposals_spec.rb
@@ -89,7 +89,7 @@ feature 'Proposals' do
end
end
- context "Embedded video" do
+ context "Embedded video" do
scenario "Show YouTube video" do
proposal = create(:proposal, video_url: "http://www.youtube.com/watch?v=a7UFm6ErMPU")
@@ -99,7 +99,7 @@ feature 'Proposals' do
end
scenario "Show Vimeo video" do
- proposal = create(:proposal, video_url: "https://vimeo.com/7232823" )
+ proposal = create(:proposal, video_url: "https://vimeo.com/7232823")
visit proposal_path(proposal)
expect(page).to have_selector("div[id='js-embedded-video']")
expect(page.html).to include 'https://player.vimeo.com/video/7232823'
@@ -175,7 +175,7 @@ feature 'Proposals' do
click_button 'Create proposal'
expect(page).to have_content 'Proposal created successfully.'
- expect(page).to have_content 'You can also see more information about improving your campaign'
+ expect(page).to have_content 'Improve your campaign and get more supports'
click_link 'Not now, go to my proposal'
@@ -1025,7 +1025,7 @@ feature 'Proposals' do
click_link "Advanced search"
select "Customized", from: "js-advanced-search-date-min"
fill_in "advanced_search_date_min", with: 7.days.ago
- fill_in "advanced_search_date_max", with: 1.days.ago
+ fill_in "advanced_search_date_max", with: 1.day.ago
click_button "Filter"
expect(page).to have_content("There are 2 citizen proposals")
@@ -1072,7 +1072,7 @@ feature 'Proposals' do
click_link "Advanced search"
fill_in "Write the text", with: "Schwifty"
select Setting['official_level_1_name'], from: "advanced_search_official_level"
- select "Last 24 hours", from: "js-advanced-search-date-min"
+ select "Last 24 hours", from: "js-advanced-search-date-min"
click_button "Filter"
@@ -1108,7 +1108,7 @@ feature 'Proposals' do
select "Customized", from: "js-advanced-search-date-min"
fill_in "advanced_search_date_min", with: 7.days.ago.to_date
- fill_in "advanced_search_date_max", with: 1.days.ago.to_date
+ fill_in "advanced_search_date_max", with: 1.day.ago.to_date
click_button "Filter"
expect(page).to have_content("citizen proposals cannot be found")
diff --git a/spec/features/registration_form_spec.rb b/spec/features/registration_form_spec.rb
index 81b9d9bec..2d3d16a25 100644
--- a/spec/features/registration_form_spec.rb
+++ b/spec/features/registration_form_spec.rb
@@ -78,5 +78,4 @@ feature 'Registration form' do
expect(current_path).to eq(new_user_registration_path)
end
-
end
diff --git a/spec/features/tags/budget_investments_spec.rb b/spec/features/tags/budget_investments_spec.rb
index ada2338b4..fe1eed0e1 100644
--- a/spec/features/tags/budget_investments_spec.rb
+++ b/spec/features/tags/budget_investments_spec.rb
@@ -87,7 +87,7 @@ feature 'Tags' do
select 'Health: More hospitals', from: 'budget_investment_heading_id'
fill_in 'budget_investment_title', with: 'Build a skyscraper'
fill_in_ckeditor 'budget_investment_description', with: 'If I had a gym near my place I could go do Zumba'
- check 'budget_investment_terms_of_service'
+ check 'budget_investment_terms_of_service'
find('.js-add-tag-link', text: 'Education').click
click_button 'Create Investment'
diff --git a/spec/features/users_auth_spec.rb b/spec/features/users_auth_spec.rb
index 617b1b1d4..566c1c233 100644
--- a/spec/features/users_auth_spec.rb
+++ b/spec/features/users_auth_spec.rb
@@ -200,7 +200,6 @@ feature 'Users' do
expect(current_path).to eq(finish_signup_path)
click_link 'Cancel login'
-
visit '/'
expect_to_not_be_signed_in
end
diff --git a/spec/features/valuation/budget_investments_spec.rb b/spec/features/valuation/budget_investments_spec.rb
index b74213d7e..3f91d8925 100644
--- a/spec/features/valuation/budget_investments_spec.rb
+++ b/spec/features/valuation/budget_investments_spec.rb
@@ -76,7 +76,6 @@ feature 'Valuation budget investments' do
expect(page).to have_link("Realocate visitors")
expect(page).to have_link("Destroy the city")
-
expect(page).to have_content "All headings (2)"
expect(page).to have_content "District 9 (1)"
expect(page).to have_content "Down to the river (1)"
@@ -289,10 +288,10 @@ feature 'Valuation budget investments' do
end
scenario 'Feasibility selection makes proper fields visible', :js do
- feasible_fields = ['Price (€)','Cost during the first year (€)','Price explanation','Time scope']
+ feasible_fields = ['Price (€)', 'Cost during the first year (€)', 'Price explanation', 'Time scope']
unfeasible_fields = ['Feasibility explanation']
- any_feasibility_fields = ['Valuation finished','Internal comments']
- undecided_fields = feasible_fields + unfeasible_fields + any_feasibility_fields
+ any_feasibility_fields = ['Valuation finished', 'Internal comments']
+ undecided_fields = feasible_fields + unfeasible_fields + any_feasibility_fields
visit edit_valuation_budget_budget_investment_path(@budget, @investment)
diff --git a/spec/features/valuation/spending_proposals_spec.rb b/spec/features/valuation/spending_proposals_spec.rb
index e19c4fb41..2b2e06908 100644
--- a/spec/features/valuation/spending_proposals_spec.rb
+++ b/spec/features/valuation/spending_proposals_spec.rb
@@ -311,9 +311,9 @@ feature 'Valuation spending proposals' do
end
scenario 'Feasibility selection makes proper fields visible', :js do
- feasible_true_fields = ['Price (€)','Cost during the first year (€)','Price explanation','Time scope']
+ feasible_true_fields = ['Price (€)', 'Cost during the first year (€)', 'Price explanation', 'Time scope']
feasible_false_fields = ['Feasibility explanation']
- feasible_any_fields = ['Valuation finished','Internal comments']
+ feasible_any_fields = ['Valuation finished', 'Internal comments']
feasible_nil_fields = feasible_true_fields + feasible_false_fields + feasible_any_fields
visit edit_valuation_spending_proposal_path(@spending_proposal)
@@ -403,7 +403,7 @@ feature 'Valuation spending proposals' do
end
scenario "Summary table" do
- scarlett = create(:valuator)
+ scarlett = create(:valuator)
john = create(:valuator)
finished_and_feasible1 = create(:spending_proposal, valuation_finished: true, feasible: true, price: '3000000')
@@ -452,7 +452,7 @@ feature 'Valuation spending proposals' do
scenario "Order by investment project count" do
isabel = create(:valuator)
john = create(:valuator)
- scarlett = create(:valuator)
+ scarlett = create(:valuator)
3.times { create(:spending_proposal, valuators: [scarlett])}
1.times { create(:spending_proposal, valuators: [john])}
diff --git a/spec/features/verification/verified_user_spec.rb b/spec/features/verification/verified_user_spec.rb
index 451ea03e5..3b020fcb0 100644
--- a/spec/features/verification/verified_user_spec.rb
+++ b/spec/features/verification/verified_user_spec.rb
@@ -69,7 +69,6 @@ feature 'Verified users' do
expect(current_path).to eq new_sms_path
end
-
scenario "Select a verified email" do
user = create(:user,
residence_verified_at: Time.current,
diff --git a/spec/helpers/proposals_helper_spec.rb b/spec/helpers/proposals_helper_spec.rb
index cee1ec648..d71178d9d 100644
--- a/spec/helpers/proposals_helper_spec.rb
+++ b/spec/helpers/proposals_helper_spec.rb
@@ -9,12 +9,12 @@ describe ProposalsHelper do
end
it "should be a between 1 and 100 if there are votes but less than needed" do
- proposal = create(:proposal, cached_votes_up: Proposal.votes_needed_for_success/2)
+ proposal = create(:proposal, cached_votes_up: Proposal.votes_needed_for_success / 2)
expect(progress_bar_percentage(proposal)).to eq 50
end
it "should be 100 if there are more votes than needed" do
- proposal = create(:proposal, cached_votes_up: Proposal.votes_needed_for_success*2)
+ proposal = create(:proposal, cached_votes_up: Proposal.votes_needed_for_success * 2)
expect(progress_bar_percentage(proposal)).to eq 100
end
end
@@ -31,12 +31,12 @@ describe ProposalsHelper do
end
it "should be a between 1 and 100 if there are votes but less than needed" do
- proposal = create(:proposal, cached_votes_up: Proposal.votes_needed_for_success/2)
+ proposal = create(:proposal, cached_votes_up: Proposal.votes_needed_for_success / 2)
expect(supports_percentage(proposal)).to eq "50%"
end
it "should be 100 if there are more votes than needed" do
- proposal = create(:proposal, cached_votes_up: Proposal.votes_needed_for_success*2)
+ proposal = create(:proposal, cached_votes_up: Proposal.votes_needed_for_success * 2)
expect(supports_percentage(proposal)).to eq "100%"
end
diff --git a/spec/lib/age_spec.rb b/spec/lib/age_spec.rb
index 6e1012fd4..02637966e 100644
--- a/spec/lib/age_spec.rb
+++ b/spec/lib/age_spec.rb
@@ -17,7 +17,7 @@ describe Age do
d = Date.new(1980, 12, 31)
expect(Age.in_years(d, Date.new(2000, 12, 30))).to eq(19)
expect(Age.in_years(d, Date.new(2000, 12, 31))).to eq(20)
- expect(Age.in_years(d, Date.new(2001, 1, 1))).to eq(20)
+ expect(Age.in_years(d, Date.new(2001, 1, 1))).to eq(20)
d = Date.new(1980, 1, 1)
expect(Age.in_years(d, Date.new(2000, 12, 31))).to eq(20)
@@ -31,28 +31,28 @@ describe Age do
expect(Age.in_years(d, Date.new(2000, 2, 27))).to eq(19)
expect(Age.in_years(d, Date.new(2000, 2, 28))).to eq(19)
expect(Age.in_years(d, Date.new(2000, 2, 29))).to eq(20)
- expect(Age.in_years(d, Date.new(2000, 3, 1))).to eq(20)
+ expect(Age.in_years(d, Date.new(2000, 3, 1))).to eq(20)
expect(Age.in_years(d, Date.new(2001, 2, 27))).to eq(20)
expect(Age.in_years(d, Date.new(2001, 2, 28))).to eq(20)
- expect(Age.in_years(d, Date.new(2001, 3, 1))).to eq(21)
+ expect(Age.in_years(d, Date.new(2001, 3, 1))).to eq(21)
d = Date.new(1980, 2, 28)
expect(Age.in_years(d, Date.new(2000, 2, 27))).to eq(19)
expect(Age.in_years(d, Date.new(2000, 2, 28))).to eq(20)
expect(Age.in_years(d, Date.new(2000, 2, 29))).to eq(20)
- expect(Age.in_years(d, Date.new(2000, 3, 1))).to eq(20)
+ expect(Age.in_years(d, Date.new(2000, 3, 1))).to eq(20)
expect(Age.in_years(d, Date.new(2001, 2, 27))).to eq(20)
expect(Age.in_years(d, Date.new(2001, 2, 28))).to eq(21)
- expect(Age.in_years(d, Date.new(2001, 3, 1))).to eq(21)
+ expect(Age.in_years(d, Date.new(2001, 3, 1))).to eq(21)
d = Date.new(1980, 3, 1)
expect(Age.in_years(d, Date.new(2000, 2, 27))).to eq(19)
expect(Age.in_years(d, Date.new(2000, 2, 28))).to eq(19)
expect(Age.in_years(d, Date.new(2000, 2, 29))).to eq(19)
- expect(Age.in_years(d, Date.new(2000, 3, 1))).to eq(20)
+ expect(Age.in_years(d, Date.new(2000, 3, 1))).to eq(20)
expect(Age.in_years(d, Date.new(2001, 2, 27))).to eq(20)
expect(Age.in_years(d, Date.new(2001, 2, 28))).to eq(20)
- expect(Age.in_years(d, Date.new(2001, 3, 1))).to eq(21)
+ expect(Age.in_years(d, Date.new(2001, 3, 1))).to eq(21)
end
end
end
diff --git a/spec/lib/census_api_spec.rb b/spec/lib/census_api_spec.rb
index fb14f39c5..274101f22 100644
--- a/spec/lib/census_api_spec.rb
+++ b/spec/lib/census_api_spec.rb
@@ -39,7 +39,7 @@ describe CensusApi do
response = api.call(1, "123456")
expect(response).to be_valid
- expect(response.date_of_birth).to eq(Date.new(1980,1,1))
+ expect(response.date_of_birth).to eq(Date.new(1980, 1, 1))
end
it "returns the last failed response" do
diff --git a/spec/lib/graph_ql/query_type_creator_spec.rb b/spec/lib/graph_ql/query_type_creator_spec.rb
index 6d3fe7a2a..be223c109 100644
--- a/spec/lib/graph_ql/query_type_creator_spec.rb
+++ b/spec/lib/graph_ql/query_type_creator_spec.rb
@@ -7,7 +7,7 @@ describe GraphQL::QueryTypeCreator do
Proposal => { fields: { id: :integer, title: :string } }
}
end
- let(:api_types) { GraphQL::ApiTypesCreator.create(api_type_definitions) }
+ let(:api_types) { GraphQL::ApiTypesCreator.create(api_type_definitions) }
describe "::create" do
let(:query_type) { GraphQL::QueryTypeCreator.create(api_types) }
diff --git a/spec/lib/graphql_spec.rb b/spec/lib/graphql_spec.rb
index b1e7073ea..d6ea071e2 100644
--- a/spec/lib/graphql_spec.rb
+++ b/spec/lib/graphql_spec.rb
@@ -321,7 +321,7 @@ describe 'ConsulSchema' do
it 'does not include hidden comments' do
visible_comment = create(:comment)
- hidden_comment = create(:comment, hidden_at: Time.now)
+ hidden_comment = create(:comment, hidden_at: Time.current)
response = execute('{ comments { edges { node { body } } } }')
received_comments = extract_fields(response, 'comments', 'body')
@@ -331,7 +331,7 @@ describe 'ConsulSchema' do
it 'does not include comments from hidden proposals' do
visible_proposal = create(:proposal)
- hidden_proposal = create(:proposal, hidden_at: Time.now)
+ hidden_proposal = create(:proposal, hidden_at: Time.current)
visible_proposal_comment = create(:comment, commentable: visible_proposal)
hidden_proposal_comment = create(:comment, commentable: hidden_proposal)
@@ -344,7 +344,7 @@ describe 'ConsulSchema' do
it 'does not include comments from hidden debates' do
visible_debate = create(:debate)
- hidden_debate = create(:debate, hidden_at: Time.now)
+ hidden_debate = create(:debate, hidden_at: Time.current)
visible_debate_comment = create(:comment, commentable: visible_debate)
hidden_debate_comment = create(:comment, commentable: hidden_debate)
@@ -567,7 +567,7 @@ describe 'ConsulSchema' do
it 'does not include votes of hidden proposals' do
visible_proposal = create(:proposal)
- hidden_proposal = create(:proposal, hidden_at: Time.now)
+ hidden_proposal = create(:proposal, hidden_at: Time.current)
visible_proposal_vote = create(:vote, votable: visible_proposal)
hidden_proposal_vote = create(:vote, votable: hidden_proposal)
@@ -580,7 +580,7 @@ describe 'ConsulSchema' do
it 'does not include votes of hidden comments' do
visible_comment = create(:comment)
- hidden_comment = create(:comment, hidden_at: Time.now)
+ hidden_comment = create(:comment, hidden_at: Time.current)
visible_comment_vote = create(:vote, votable: visible_comment)
hidden_comment_vote = create(:vote, votable: hidden_comment)
diff --git a/spec/lib/manager_authenticator_spec.rb b/spec/lib/manager_authenticator_spec.rb
index 256303fdd..c53a26f5e 100644
--- a/spec/lib/manager_authenticator_spec.rb
+++ b/spec/lib/manager_authenticator_spec.rb
@@ -3,24 +3,24 @@ require 'rails_helper'
describe ManagerAuthenticator do
describe 'initialization params' do
it 'should cause auth to return false if blank login' do
- authenticator = ManagerAuthenticator.new({login: "", clave_usuario: "31415926", fecha_conexion: "20151031135905"})
+ authenticator = ManagerAuthenticator.new(login: "", clave_usuario: "31415926", fecha_conexion: "20151031135905")
expect(authenticator.auth).to be false
end
it 'should cause auth to return false if blank user_key' do
- authenticator = ManagerAuthenticator.new({login: "JJB033", clave_usuario: "", fecha_conexion: "20151031135905"})
+ authenticator = ManagerAuthenticator.new(login: "JJB033", clave_usuario: "", fecha_conexion: "20151031135905")
expect(authenticator.auth).to be false
end
it 'should cause auth to return false if blank date' do
- authenticator = ManagerAuthenticator.new({login: "JJB033", clave_usuario: "31415926", fecha_conexion: ""})
+ authenticator = ManagerAuthenticator.new(login: "JJB033", clave_usuario: "31415926", fecha_conexion: "")
expect(authenticator.auth).to be false
end
end
describe '#auth' do
before(:all) do
- @authenticator = ManagerAuthenticator.new({login: "JJB033", clave_usuario: "31415926", fecha_conexion: "20151031135905"})
+ @authenticator = ManagerAuthenticator.new(login: "JJB033", clave_usuario: "31415926", fecha_conexion: "20151031135905")
end
it 'should return false if not manager_exists' do
@@ -47,7 +47,7 @@ describe ManagerAuthenticator do
describe 'SOAP' do
before(:all) do
- @authenticator = ManagerAuthenticator.new({login: "JJB033", clave_usuario: "31415926", fecha_conexion: "20151031135905"})
+ @authenticator = ManagerAuthenticator.new(login: "JJB033", clave_usuario: "31415926", fecha_conexion: "20151031135905")
end
it 'should call the verification user method' do
diff --git a/spec/models/abilities/common_spec.rb b/spec/models/abilities/common_spec.rb
index 7c02f50fd..da398e610 100644
--- a/spec/models/abilities/common_spec.rb
+++ b/spec/models/abilities/common_spec.rb
@@ -35,14 +35,14 @@ describe "Abilities::Common" do
let(:incoming_poll) { create(:poll, :incoming) }
let(:incoming_poll_from_own_geozone) { create(:poll, :incoming, geozone_restricted: true, geozones: [geozone]) }
let(:incoming_poll_from_other_geozone) { create(:poll, :incoming, geozone_restricted: true, geozones: [create(:geozone)]) }
- let(:expired_poll) { create(:poll, :expired) }
+ let(:expired_poll) { create(:poll, :expired) }
let(:expired_poll_from_own_geozone) { create(:poll, :expired, geozone_restricted: true, geozones: [geozone]) }
let(:expired_poll_from_other_geozone) { create(:poll, :expired, geozone_restricted: true, geozones: [create(:geozone)]) }
let(:poll) { create(:poll, geozone_restricted: false) }
let(:poll_from_own_geozone) { create(:poll, geozone_restricted: true, geozones: [geozone]) }
let(:poll_from_other_geozone) { create(:poll, geozone_restricted: true, geozones: [create(:geozone)]) }
- let(:poll_question_from_own_geozone) { create(:poll_question, poll: poll_from_own_geozone) }
+ let(:poll_question_from_own_geozone) { create(:poll_question, poll: poll_from_own_geozone) }
let(:poll_question_from_other_geozone) { create(:poll_question, poll: poll_from_other_geozone) }
let(:poll_question_from_all_geozones) { create(:poll_question, poll: poll) }
@@ -197,7 +197,7 @@ describe "Abilities::Common" do
it { should_not be_able_to(:vote, investment_in_accepting_budget) }
it { should be_able_to(:vote, investment_in_selecting_budget) }
it { should_not be_able_to(:vote, investment_in_balloting_budget) }
-
+
it { should_not be_able_to(:destroy, investment_in_accepting_budget) }
it { should_not be_able_to(:destroy, investment_in_reviewing_budget) }
it { should_not be_able_to(:destroy, investment_in_selecting_budget) }
@@ -222,7 +222,7 @@ describe "Abilities::Common" do
it { should be_able_to(:vote, Proposal) }
it { should be_able_to(:vote_featured, Proposal) }
- it { should be_able_to(:create, SpendingProposal) }
+ it { should be_able_to(:create, SpendingProposal) }
it { should_not be_able_to(:destroy, create(:spending_proposal)) }
it { should_not be_able_to(:destroy, own_spending_proposal) }
diff --git a/spec/models/abilities/moderator_spec.rb b/spec/models/abilities/moderator_spec.rb
index 332f69450..f3135c8d9 100644
--- a/spec/models/abilities/moderator_spec.rb
+++ b/spec/models/abilities/moderator_spec.rb
@@ -35,20 +35,20 @@ describe "Abilities::Moderator" do
let(:rejected_organization) { create(:organization, :rejected) }
let(:verified_organization) { create(:organization, :verified) }
- it { should be_able_to( :verify, pending_organization) }
- it { should be_able_to( :reject, pending_organization) }
+ it { should be_able_to(:verify, pending_organization) }
+ it { should be_able_to(:reject, pending_organization) }
it { should_not be_able_to(:verify, verified_organization) }
- it { should be_able_to( :reject, verified_organization) }
+ it { should be_able_to(:reject, verified_organization) }
it { should_not be_able_to(:reject, rejected_organization) }
- it { should be_able_to( :verify, rejected_organization) }
+ it { should be_able_to(:verify, rejected_organization) }
end
describe "hiding, reviewing and restoring" do
let(:ignored_comment) { create(:comment, :with_ignored_flag) }
let(:ignored_debate) { create(:debate, :with_ignored_flag) }
- let(:ignored_proposal) { create(:proposal,:with_ignored_flag) }
+ let(:ignored_proposal) { create(:proposal, :with_ignored_flag) }
it { should be_able_to(:hide, comment) }
it { should be_able_to(:hide_in_moderation_screen, comment) }
diff --git a/spec/models/ahoy/data_source_spec.rb b/spec/models/ahoy/data_source_spec.rb
index deb1fd056..863820710 100644
--- a/spec/models/ahoy/data_source_spec.rb
+++ b/spec/models/ahoy/data_source_spec.rb
@@ -3,9 +3,9 @@ require 'rails_helper'
describe Ahoy::DataSource do
describe '#build' do
before :each do
- time_1 = DateTime.parse("2015-01-01")
- time_2 = DateTime.parse("2015-01-02")
- time_3 = DateTime.parse("2015-01-03")
+ time_1 = DateTime.parse("2015-01-01").in_time_zone
+ time_2 = DateTime.parse("2015-01-02").in_time_zone
+ time_3 = DateTime.parse("2015-01-03").in_time_zone
create :ahoy_event, name: 'foo', time: time_1
create :ahoy_event, name: 'foo', time: time_1
@@ -23,14 +23,14 @@ describe Ahoy::DataSource do
it 'should work with single data sources' do
ds = Ahoy::DataSource.new
ds.add 'foo', Ahoy::Event.where(name: 'foo').group_by_day(:time).count
- expect(ds.build).to eq :x=>["2015-01-01", "2015-01-02"], "foo"=>[2, 1]
+ expect(ds.build).to eq :x => ["2015-01-01", "2015-01-02"], "foo" => [2, 1]
end
it 'should combine data sources' do
ds = Ahoy::DataSource.new
ds.add 'foo', Ahoy::Event.where(name: 'foo').group_by_day(:time).count
ds.add 'bar', Ahoy::Event.where(name: 'bar').group_by_day(:time).count
- expect(ds.build).to eq :x=>["2015-01-01", "2015-01-02", "2015-01-03"], "foo"=>[2, 1, 0], "bar"=>[1, 0, 2]
+ expect(ds.build).to eq :x => ["2015-01-01", "2015-01-02", "2015-01-03"], "foo" => [2, 1, 0], "bar" => [1, 0, 2]
end
end
end
diff --git a/spec/models/budget/investment/milestone_spec.rb b/spec/models/budget/investment/milestone_spec.rb
new file mode 100644
index 000000000..32f040eaf
--- /dev/null
+++ b/spec/models/budget/investment/milestone_spec.rb
@@ -0,0 +1,23 @@
+require 'rails_helper'
+
+describe "Budget::Investment::Milestone" do
+
+ describe "Validations" do
+ let(:milestone) { build(:budget_investment_milestone) }
+
+ it "Should be valid" do
+ expect(milestone).to be_valid
+ end
+
+ it "Should not be valid without a title" do
+ milestone.title = nil
+ expect(milestone).to_not be_valid
+ end
+
+ it "Should not be valid without an investment" do
+ milestone.investment_id = nil
+ expect(milestone).to_not be_valid
+ end
+ end
+
+end
diff --git a/spec/models/budget/investment_spec.rb b/spec/models/budget/investment_spec.rb
index 2290abba9..827ca9d21 100644
--- a/spec/models/budget/investment_spec.rb
+++ b/spec/models/budget/investment_spec.rb
@@ -249,7 +249,7 @@ describe Budget::Investment do
by_valuator = Budget::Investment.by_valuator(valuator1.id)
expect(by_valuator.size).to eq(2)
- expect(by_valuator.sort).to eq([investment1,investment3].sort)
+ expect(by_valuator.sort).to eq([investment1, investment3].sort)
end
end
@@ -486,7 +486,6 @@ describe Budget::Investment do
end
-
describe 'Permissions' do
let(:budget) { create(:budget) }
let(:group) { create(:budget_group, budget: budget) }
@@ -591,11 +590,10 @@ describe Budget::Investment do
most_voted2 = create(:budget_investment, cached_votes_up: 10)
least_voted2 = create(:budget_investment, cached_votes_up: 1)
-
- expect(Budget::Investment.sort_by_confidence_score.first).to eq most_voted2
- expect(Budget::Investment.sort_by_confidence_score.second).to eq most_voted
- expect(Budget::Investment.sort_by_confidence_score.third).to eq least_voted2
- expect(Budget::Investment.sort_by_confidence_score.fourth).to eq least_voted
+ expect(Budget::Investment.sort_by_confidence_score.first).to eq most_voted2
+ expect(Budget::Investment.sort_by_confidence_score.second).to eq most_voted
+ expect(Budget::Investment.sort_by_confidence_score.third).to eq least_voted2
+ expect(Budget::Investment.sort_by_confidence_score.fourth).to eq least_voted
end
end
end
diff --git a/spec/models/debate_spec.rb b/spec/models/debate_spec.rb
index 612775ee4..8d24f9945 100644
--- a/spec/models/debate_spec.rb
+++ b/spec/models/debate_spec.rb
@@ -545,7 +545,7 @@ describe Debate do
title_some_votes = create(:debate, title: 'stop corruption', cached_votes_up: 5)
title_least_voted = create(:debate, title: 'stop corruption', cached_votes_up: 2)
title_most_voted = create(:debate, title: 'stop corruption', cached_votes_up: 10)
- description_most_voted = create(:debate, description: 'stop corruption', cached_votes_up: 10)
+ description_most_voted = create(:debate, description: 'stop corruption', cached_votes_up: 10)
results = Debate.search('stop corruption')
diff --git a/spec/models/legislation/annotation_spec.rb b/spec/models/legislation/annotation_spec.rb
index 4cf5c6f59..0daca383a 100644
--- a/spec/models/legislation/annotation_spec.rb
+++ b/spec/models/legislation/annotation_spec.rb
@@ -17,7 +17,7 @@ RSpec.describe Legislation::Annotation, type: :model do
Expetenda tincidunt in sed, ex partem placerat sea, porro commodo ex eam. His putant aeterno interesset at. Usu ea mundi tincidunt, omnium virtute aliquando ius ex. Ea aperiri sententiae duo. Usu nullam dolorum quaestio ei, sit vidit facilisis ea. Per ne impedit iracundia neglegentur. Consetetur neglegentur eum ut, vis animal legimus inimicus id.
His audiam",
- ranges: [{"start"=>"/p[1]", "startOffset"=>6, "end"=>"/p[3]", "endOffset"=>11}]
+ ranges: [{"start" => "/p[1]", "startOffset" => 6, "end" => "/p[3]", "endOffset" => 11}]
)
expect(annotation.context).to eq("Lorem
ipsum dolor sit amet, consectetuer adipiscing elit, sed diam nonummy nibh euismod tincidunt ut laoreet dolore magna aliquam erat volutpat. Ut wisi enim ad minim veniam, quis nostrud exerci tation ullamcorper suscipit lobortis nisl ut aliquip ex ea commodo consequat. Duis autem vel eum iriure dolor in hendrerit in vulputate velit esse molestie consequat, vel illum dolore eu feugiat nulla facilisis at vero eros et accumsan et iusto odio dignissim qui blandit praesent luptatum zzril delenit augue duis dolore te feugait nulla facilisi.\n\nExpetenda tincidunt in sed, ex partem placerat sea, porro commodo ex eam. His putant aeterno interesset at. Usu ea mundi tincidunt, omnium virtute aliquando ius ex. Ea aperiri sententiae duo. Usu nullam dolorum quaestio ei, sit vidit facilisis ea. Per ne impedit iracundia neglegentur. Consetetur neglegentur eum ut, vis animal legimus inimicus id.\n\nHis audiamdeserunt in, eum ubique voluptatibus te. In reque dicta usu. Ne rebum dissentiet eam, vim omnis deseruisse id. Ullum deleniti vituperata at quo, insolens complectitur te eos, ea pri dico munere propriae. Vel ferri facilis ut, qui paulo ridens praesent ad. Possim alterum qui cu. Accusamus consulatu ius te, cu decore soleat appareat usu.")
@@ -27,7 +27,7 @@ His audiam",
annotation = create(:legislation_annotation,
draft_version: draft_version,
quote: "Duis autem vel eum iriure dolor in hendrerit in vulputate velit esse molestie consequat, vel illum dolore eu feugiat nulla facilisis at vero eros et accumsan et iusto odio dignissim qui blandit praesent luptatum zzril delenit augue duis dolore te feugait nulla facilisi.\r\n\r\nExpetenda tincidunt in sed, ex partem placerat sea, porro commodo ex eam. His putant aeterno interesset at. Usu ea mundi tincidunt, omnium virtute aliquando ius ex. Ea aperiri sententiae duo",
- ranges: [{"start"=>"/p[1]", "startOffset"=>273, "end"=>"/p[2]", "endOffset"=>190}]
+ ranges: [{"start" => "/p[1]", "startOffset" => 273, "end" => "/p[2]", "endOffset" => 190}]
)
expect(annotation.context).to eq("Lorem ipsum dolor sit amet, consectetuer adipiscing elit, sed diam nonummy nibh euismod tincidunt ut laoreet dolore magna aliquam erat volutpat. Ut wisi enim ad minim veniam, quis nostrud exerci tation ullamcorper suscipit lobortis nisl ut aliquip ex ea commodo consequat.
Duis autem vel eum iriure dolor in hendrerit in vulputate velit esse molestie consequat, vel illum dolore eu feugiat nulla facilisis at vero eros et accumsan et iusto odio dignissim qui blandit praesent luptatum zzril delenit augue duis dolore te feugait nulla facilisi.\r\n\r\nExpetenda tincidunt in sed, ex partem placerat sea, porro commodo ex eam. His putant aeterno interesset at. Usu ea mundi tincidunt, omnium virtute aliquando ius ex. Ea aperiri sententiae duo. Usu nullam dolorum quaestio ei, sit vidit facilisis ea. Per ne impedit iracundia neglegentur. Consetetur neglegentur eum ut, vis animal legimus inimicus id.")
@@ -39,7 +39,7 @@ His audiam",
annotation = create(:legislation_annotation,
draft_version: draft_version,
quote: "By contrast, our General Public Licenses are intended to guarantee your freedom to share and change all versions of a program--to make sure it remains free software for all its users.\r\n\r\nWhen we speak of free software, we are referring to freedom, not price. Our General Public Licenses are designed to make sure that you have the freedom to distribute copies of free software (and charge for them if you wish)",
- ranges: [{"start"=>"/p[2]", "startOffset"=>127, "end"=>"/p[3]", "endOffset"=>223}]
+ ranges: [{"start" => "/p[2]", "startOffset" => 127, "end" => "/p[3]", "endOffset" => 223}]
)
expect(annotation.context).to eq("The licenses for most software and other practical works are designed to take away your freedom to share and change the works.
By contrast, our General Public Licenses are intended to guarantee your freedom to share and change all versions of a program--to make sure it remains free software for all its users.\r\n\r\nWhen we speak of free software, we are referring to freedom, not price. Our General Public Licenses are designed to make sure that you have the freedom to distribute copies of free software (and charge for them if you wish), that you receive source code or can get it if you want it, that you can change the software or use pieces of it in new free programs, and that you know you can do these things.")
diff --git a/spec/models/legislation/process_spec.rb b/spec/models/legislation/process_spec.rb
index f14e81c8e..33460832b 100644
--- a/spec/models/legislation/process_spec.rb
+++ b/spec/models/legislation/process_spec.rb
@@ -71,7 +71,7 @@ RSpec.describe Legislation::Process, type: :model do
describe "filter scopes" do
before(:each) do
@process_1 = create(:legislation_process, start_date: Date.current - 2.days, end_date: Date.current + 1.day)
- @process_2 = create(:legislation_process, start_date: Date.current + 1.days, end_date: Date.current + 3.days)
+ @process_2 = create(:legislation_process, start_date: Date.current + 1.day, end_date: Date.current + 3.days)
@process_3 = create(:legislation_process, start_date: Date.current - 4.days, end_date: Date.current - 3.days)
end
diff --git a/spec/models/letter_spec.rb b/spec/models/letter_spec.rb
index fdec7f390..eca9f815e 100644
--- a/spec/models/letter_spec.rb
+++ b/spec/models/letter_spec.rb
@@ -2,7 +2,7 @@ require 'rails_helper'
describe 'Verification::Letter' do
- let(:user) { create(:user) }
+ let(:user) { create(:user) }
describe "validations" do
diff --git a/spec/models/officing/residence_spec.rb b/spec/models/officing/residence_spec.rb
index dee63b85a..c39711e30 100644
--- a/spec/models/officing/residence_spec.rb
+++ b/spec/models/officing/residence_spec.rb
@@ -44,12 +44,12 @@ describe Officing::Residence do
describe "new" do
it "should upcase document number" do
- residence = Officing::Residence.new({document_number: "x1234567z"})
+ residence = Officing::Residence.new(document_number: "x1234567z")
expect(residence.document_number).to eq("X1234567Z")
end
it "should remove all characters except numbers and letters" do
- residence = Officing::Residence.new({document_number: " 12.345.678 - B"})
+ residence = Officing::Residence.new(document_number: " 12.345.678 - B")
expect(residence.document_number).to eq("12345678B")
end
end
@@ -108,13 +108,13 @@ describe Officing::Residence do
residence.save
expect(FailedCensusCall.count).to eq(1)
- expect(FailedCensusCall.first).to have_attributes({
+ expect(FailedCensusCall.first).to have_attributes(
user_id: residence.user.id,
poll_officer_id: residence.officer.id,
document_number: "12345678Z",
document_type: "1",
year_of_birth: Time.current.year
- })
+ )
end
end
diff --git a/spec/models/poll/officer_spec.rb b/spec/models/poll/officer_spec.rb
index 83bdbecc2..b096aebc3 100644
--- a/spec/models/poll/officer_spec.rb
+++ b/spec/models/poll/officer_spec.rb
@@ -32,7 +32,7 @@ describe :officer do
poll_2 = create(:poll)
booth_assignment_1 = create(:poll_booth_assignment, poll: poll_1)
- booth_assignment_2 = create(:poll_booth_assignment, poll: poll_2)
+ booth_assignment_2 = create(:poll_booth_assignment, poll: poll_2)
create(:poll_officer_assignment, booth_assignment: booth_assignment_1, officer: officer, date: poll_1.starts_at)
create(:poll_officer_assignment, booth_assignment: booth_assignment_2, officer: officer, final: true)
@@ -48,7 +48,7 @@ describe :officer do
poll_1 = create(:poll, ends_at: 1.day.ago)
poll_2 = create(:poll, ends_at: 10.days.from_now)
- poll_3 = create(:poll, ends_at: 10.day.ago)
+ poll_3 = create(:poll, ends_at: 10.days.ago)
[poll_1, poll_2, poll_3].each do |p|
create(:poll_officer_assignment, officer: officer, booth_assignment: create(:poll_booth_assignment, poll: p))
@@ -92,7 +92,7 @@ describe :officer do
poll_2 = create(:poll)
booth_assignment_1 = create(:poll_booth_assignment, poll: poll_1)
- booth_assignment_2 = create(:poll_booth_assignment, poll: poll_2)
+ booth_assignment_2 = create(:poll_booth_assignment, poll: poll_2)
create(:poll_officer_assignment, booth_assignment: booth_assignment_1, officer: officer, date: poll_1.starts_at)
create(:poll_officer_assignment, booth_assignment: booth_assignment_2, officer: officer, final: true)
@@ -108,7 +108,7 @@ describe :officer do
poll_1 = create(:poll, ends_at: 1.day.ago)
poll_2 = create(:poll, ends_at: 10.days.from_now)
- poll_3 = create(:poll, ends_at: 10.day.ago)
+ poll_3 = create(:poll, ends_at: 10.days.ago)
[poll_1, poll_2, poll_3].each do |p|
create(:poll_officer_assignment, officer: officer, booth_assignment: create(:poll_booth_assignment, poll: p), final: true)
diff --git a/spec/models/poll/poll_spec.rb b/spec/models/poll/poll_spec.rb
index 80a57e916..61f9fce52 100644
--- a/spec/models/poll/poll_spec.rb
+++ b/spec/models/poll/poll_spec.rb
@@ -93,7 +93,7 @@ describe :poll do
let(:non_user) { nil }
let(:level1) { create(:user) }
let(:level2) { create(:user, :level_two) }
- let(:level2_from_geozone) { create(:user, :level_two, geozone: geozone) }
+ let(:level2_from_geozone) { create(:user, :level_two, geozone: geozone) }
let(:all_users) { [non_user, level1, level2, level2_from_geozone] }
describe 'instance method' do
diff --git a/spec/models/poll/voter_spec.rb b/spec/models/poll/voter_spec.rb
index 752ac5bbc..c1c248550 100644
--- a/spec/models/poll/voter_spec.rb
+++ b/spec/models/poll/voter_spec.rb
@@ -33,7 +33,7 @@ describe :voter do
user = create(:user, :level_two)
voter1 = create(:poll_voter, user: user, poll: poll)
- voter2 = build(:poll_voter, user: user, poll: poll)
+ voter2 = build(:poll_voter, user: user, poll: poll)
expect(voter2).to_not be_valid
expect(voter2.errors.messages[:document_number]).to eq(["User has already voted"])
@@ -43,7 +43,7 @@ describe :voter do
user = create(:user, :level_two)
voter1 = create(:poll_voter, user: user, poll: poll, booth_assignment: booth_assignment)
- voter2 = build(:poll_voter, user: user, poll: poll, booth_assignment: booth_assignment)
+ voter2 = build(:poll_voter, user: user, poll: poll, booth_assignment: booth_assignment)
expect(voter2).to_not be_valid
expect(voter2.errors.messages[:document_number]).to eq(["User has already voted"])
@@ -56,7 +56,7 @@ describe :voter do
user = create(:user, :level_two)
voter1 = create(:poll_voter, user: user, poll: poll, booth_assignment: booth_assignment1)
- voter2 = build(:poll_voter, user: user, poll: poll, booth_assignment: booth_assignment2)
+ voter2 = build(:poll_voter, user: user, poll: poll, booth_assignment: booth_assignment2)
expect(voter2).to_not be_valid
expect(voter2.errors.messages[:document_number]).to eq(["User has already voted"])
@@ -69,7 +69,7 @@ describe :voter do
user = create(:user, :level_two)
voter1 = create(:poll_voter, user: user, booth_assignment: booth_assignment1)
- voter2 = build(:poll_voter, user: user, booth_assignment: booth_assignment2)
+ voter2 = build(:poll_voter, user: user, booth_assignment: booth_assignment2)
expect(voter2).to be_valid
end
diff --git a/spec/models/residence_spec.rb b/spec/models/residence_spec.rb
index 6841f636b..072782508 100644
--- a/spec/models/residence_spec.rb
+++ b/spec/models/residence_spec.rb
@@ -13,19 +13,19 @@ describe Verification::Residence do
describe "dates" do
it "should be valid with a valid date of birth" do
- residence = Verification::Residence.new({"date_of_birth(3i)"=>"1", "date_of_birth(2i)"=>"1", "date_of_birth(1i)"=>"1980"})
+ residence = Verification::Residence.new("date_of_birth(3i)" => "1", "date_of_birth(2i)" => "1", "date_of_birth(1i)" => "1980")
expect(residence.errors[:date_of_birth].size).to eq(0)
end
it "should not be valid without a date of birth" do
- residence = Verification::Residence.new({"date_of_birth(3i)"=>"", "date_of_birth(2i)"=>"", "date_of_birth(1i)"=>""})
+ residence = Verification::Residence.new("date_of_birth(3i)" => "", "date_of_birth(2i)" => "", "date_of_birth(1i)" => "")
expect(residence).to_not be_valid
expect(residence.errors[:date_of_birth]).to include("can't be blank")
end
end
it "should validate user has allowed age" do
- residence = Verification::Residence.new({"date_of_birth(3i)"=>"1", "date_of_birth(2i)"=>"1", "date_of_birth(1i)"=>"#{5.year.ago.year}"})
+ residence = Verification::Residence.new("date_of_birth(3i)" => "1", "date_of_birth(2i)" => "1", "date_of_birth(1i)" => "#{5.years.ago.year}")
expect(residence).to_not be_valid
expect(residence.errors[:date_of_birth]).to include("You don't have the required age to participate")
end
@@ -50,12 +50,12 @@ describe Verification::Residence do
describe "new" do
it "should upcase document number" do
- residence = Verification::Residence.new({document_number: "x1234567z"})
+ residence = Verification::Residence.new(document_number: "x1234567z")
expect(residence.document_number).to eq("X1234567Z")
end
it "should remove all characters except numbers and letters" do
- residence = Verification::Residence.new({document_number: " 12.345.678 - B"})
+ residence = Verification::Residence.new(document_number: " 12.345.678 - B")
expect(residence.document_number).to eq("12345678B")
end
end
@@ -99,13 +99,13 @@ describe Verification::Residence do
residence.save
expect(FailedCensusCall.count).to eq(1)
- expect(FailedCensusCall.first).to have_attributes({
+ expect(FailedCensusCall.first).to have_attributes(
user_id: residence.user.id,
document_number: "12345678Z",
document_type: "1",
date_of_birth: Date.new(1980, 12, 31),
postal_code: "28001"
- })
+ )
end
end
diff --git a/spec/models/tag_cloud_spec.rb b/spec/models/tag_cloud_spec.rb
index c99e6a919..7229b1b40 100644
--- a/spec/models/tag_cloud_spec.rb
+++ b/spec/models/tag_cloud_spec.rb
@@ -24,7 +24,7 @@ describe TagCloud do
it "returns budget investment tags" do
create(:budget_investment, tag_list: 'participation')
- create(:debate, tag_list: 'world hunger')
+ create(:debate, tag_list: 'world hunger')
tag_cloud = TagCloud.new(Budget::Investment)
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
index 2bf09f378..2e92e5c96 100644
--- a/spec/models/user_spec.rb
+++ b/spec/models/user_spec.rb
@@ -220,7 +220,7 @@ describe User do
subject.username = nil
expect(subject).to be_valid
- subject.organization.name= nil
+ subject.organization.name = nil
expect(subject).to_not be_valid
end
end
@@ -434,13 +434,13 @@ describe User do
describe "document_number" do
it "should upcase document number" do
- user = User.new({document_number: "x1234567z"})
+ user = User.new(document_number: "x1234567z")
user.valid?
expect(user.document_number).to eq("X1234567Z")
end
it "should remove all characters except numbers and letters" do
- user = User.new({document_number: " 12.345.678 - B"})
+ user = User.new(document_number: " 12.345.678 - B")
user.valid?
expect(user.document_number).to eq("12345678B")
end
diff --git a/spec/models/verification/management/email_spec.rb b/spec/models/verification/management/email_spec.rb
index 924f63501..652c094ba 100644
--- a/spec/models/verification/management/email_spec.rb
+++ b/spec/models/verification/management/email_spec.rb
@@ -39,7 +39,7 @@ describe Verification::Management::Email do
allow(validation).to receive(:user).and_return user
expect(mail).to receive(:deliver_later)
- expect(Devise.token_generator).to receive(:generate).with(User, :email_verification_token).and_return(["1","2"])
+ expect(Devise.token_generator).to receive(:generate).with(User, :email_verification_token).and_return(["1", "2"])
expect(Mailer).to receive(:email_verification).with(user, user.email, "2", "1", "1234").and_return(mail)
validation.save
diff --git a/spec/support/common_actions.rb b/spec/support/common_actions.rb
index 327943931..209ddf800 100644
--- a/spec/support/common_actions.rb
+++ b/spec/support/common_actions.rb
@@ -1,6 +1,6 @@
module CommonActions
- def sign_up(email='manuela@consul.dev', password='judgementday')
+ def sign_up(email = 'manuela@consul.dev', password = 'judgementday')
visit '/'
click_link 'Register'
@@ -109,7 +109,7 @@ module CommonActions
SCRIPT
end
- def error_message(resource_model=nil)
+ def error_message(resource_model = nil)
resource_model ||= "(.*)"
/\d errors? prevented this #{resource_model} from being saved:/
end
diff --git a/spec/support/verifiable.rb b/spec/support/verifiable.rb
index 53c5931cc..9112c67cf 100644
--- a/spec/support/verifiable.rb
+++ b/spec/support/verifiable.rb
@@ -67,7 +67,6 @@ shared_examples_for "verifiable" do
user3 = create(:user, verified_at: nil, confirmed_phone: nil)
user4 = create(:user, verified_at: Time.current, residence_verified_at: Time.current, unconfirmed_phone: "123456789", confirmed_phone: "123456789")
-
expect(model.incomplete_verification).to include(user1)
expect(model.incomplete_verification).to include(user2)
expect(model.incomplete_verification).to_not include(user3)
- <%= comment_tree_title_text(commentable) %> - (<%= commentable.comments_count %>) -
+ <% if display_comments_count %> ++ <%= comment_tree_title_text(commentable) %> + (<%= commentable.comments_count %>) +
+ <% end %> <%= render 'shared/wide_order_selector', i18n_namespace: "comments" %> diff --git a/app/views/debates/_form.html.erb b/app/views/debates/_form.html.erb index c88045f26..96a5337ca 100644 --- a/app/views/debates/_form.html.erb +++ b/app/views/debates/_form.html.erb @@ -17,11 +17,12 @@<%= t("debates.form.tags_instructions") %>
+<%= t("debates.form.tags_instructions") %>
<%= f.text_field :tag_list, value: @debate.tag_list.to_s, label: false, - placeholder: t("debates.form.tags_placeholder") %> + placeholder: t("debates.form.tags_placeholder"), + aria: {describedby: "tag-list-help-text"} %>