- <% if @account.organization? %>
- <%= f.fields_for :organization do |fo| %>
- <%= fo.text_field :name, autofocus: true, maxlength: Organization.name_max_length, placeholder: t("account.show.organization_name_label") %>
- <%= fo.text_field :responsible_name, autofocus: true, maxlength: Organization.responsible_name_max_length, placeholder: t("account.show.organization_responsible_name_placeholder") %>
- <% end %>
- <%= f.text_field :phone_number, placeholder: t("account.show.phone_number_label") %>
+
+ <% if @account.organization? %>
+ <%= f.fields_for :organization do |fo| %>
+ <%= fo.text_field :name, autofocus: true, maxlength: Organization.name_max_length, placeholder: t("account.show.organization_name_label") %>
+ <%= fo.text_field :responsible_name, autofocus: true, maxlength: Organization.responsible_name_max_length, placeholder: t("account.show.organization_responsible_name_placeholder") %>
+ <% end %>
+ <%= f.text_field :phone_number, placeholder: t("account.show.phone_number_label") %>
<% else %>
- <%= f.text_field :username, maxlength: User.username_max_length, placeholder: t("account.show.username_label") %>
- <% end %>
+ <%= f.text_field :username, maxlength: User.username_max_length, placeholder: t("account.show.username_label") %>
+ <% end %>
<%= t("account.show.notifications")%>
- <%= f.label :email_on_debate_comment do %>
- <%= f.check_box :email_on_debate_comment, label: false %>
- <%= t("account.show.email_on_debate_comment_label") %>
+ <%= f.label :email_on_comment do %>
+ <%= f.check_box :email_on_comment, label: false %>
+ <%= t("account.show.email_on_comment_label") %>
<% end %>
diff --git a/app/views/admin/_menu.html.erb b/app/views/admin/_menu.html.erb
index 89cf9694e..42d979209 100644
--- a/app/views/admin/_menu.html.erb
+++ b/app/views/admin/_menu.html.erb
@@ -11,6 +11,13 @@
<% end %>
+
>
+ <%= link_to admin_proposals_path do %>
+
+ <%= t("admin.menu.hidden_proposals") %>
+ <% end %>
+
+
>
<%= link_to admin_debates_path do %>
diff --git a/app/views/admin/proposals/index.html.erb b/app/views/admin/proposals/index.html.erb
new file mode 100644
index 000000000..2c6c8fd84
--- /dev/null
+++ b/app/views/admin/proposals/index.html.erb
@@ -0,0 +1,28 @@
+<%= t("admin.proposals.index.title") %>
+
+<%= render 'shared/filter_subnav', i18n_namespace: "admin.proposals.index" %>
+
+<%= page_entries_info @proposals %>
+
+
+ <% @proposals.each do |proposal| %>
+ -
+ <%= link_to proposal.title, proposal_path(proposal) %>
+
+ <%= link_to t("admin.actions.restore"),
+ restore_admin_proposal_path(proposal, request.query_parameters),
+ method: :put,
+ data: { confirm: t("admin.actions.confirm") },
+ class: "button radius tiny success right" %>
+
+ <% unless proposal.confirmed_hide? %>
+ <%= link_to t("admin.actions.confirm_hide"),
+ confirm_hide_admin_proposal_path(proposal, request.query_parameters),
+ method: :put,
+ class: "button radius tiny warning right" %>
+ <% end %>
+
+ <% end %>
+
+
+<%= paginate @proposals %>
diff --git a/app/views/comments/_actions.html.erb b/app/views/comments/_actions.html.erb
index 36ec42e01..80302a70a 100644
--- a/app/views/comments/_actions.html.erb
+++ b/app/views/comments/_actions.html.erb
@@ -11,7 +11,7 @@
<% if can? :hide, comment.user %>
•
- <%= link_to t("admin.actions.hide_author").capitalize, hide_moderation_user_path(comment.user_id, debate_id: @debate.id),
+ <%= link_to t("admin.actions.hide_author").capitalize, hide_moderation_user_path(comment.user_id),
method: :put, data: { confirm: t('admin.actions.confirm') } %>
<% end %>
diff --git a/app/views/comments/_comment.html.erb b/app/views/comments/_comment.html.erb
index 25d7585f3..5a87dde7d 100644
--- a/app/views/comments/_comment.html.erb
+++ b/app/views/comments/_comment.html.erb
@@ -5,7 +5,7 @@
<% if comment.hidden? || comment.user.hidden? %>
<% if select_children(@comments, comment).size > 0 %>
-
<%= t("debates.comment.deleted") %>
+
<%= t("comments.comment.deleted") %>
<%= t("debates.index.select_order") %>
+ <%= render 'shared/order_selector', i18n_namespace: "debates.index" %> +<%= t("debates.index.select_order_long") %>
+ <%= render 'shared/order_selector', i18n_namespace: "debates.index" %> +<%= t("layouts.header.open_city_title") %>
-<%= t("layouts.header.open_city_slogan") %>
- <%= link_to t("layouts.header.see_all_debates"), debates_path, class: "button radius see-more" %> +<%= t("layouts.header.open_city_slogan_html") %>
+ <%= link_to t("layouts.header.see_all"), highlights_path, class: "button radius see-more warning" %> <%= link_to t("layouts.header.more_information"), "/more_information", class: "more-info" %><%= t("moderation.proposals.index.title") %>
+ +<%= render 'shared/filter_subnav', i18n_namespace: "moderation.proposals.index" %> + +<%= page_entries_info @proposals %>
++ <%= t('shared.check') %>: + <%= link_to t('shared.check_all'), '#', data: {check_all: "proposal_ids[]"} %> + | + <%= link_to t('shared.check_none'), '#', data: {check_none: "proposal_ids[]"} %> +
+ ++ <%= l proposal.updated_at.to_date %> + • + <%= proposal.flags_count %> + • + <%= proposal.author.username %> +
+
¿Cómo funciona este Portal de Gobierno Abierto?
-+ + <%= link_to t("debates.show.back_link"), "/more_information", class: 'left back' %> +
-
I. Participación
- I.1. Espacio de debate
+ - I.2. Espacio de propuestas
@@ -39,6 +41,14 @@Tanto los hilos, como los comentarios podrán ser valorados por cualquiera, de tal manera que será la propia ciudadanía, y nadie en su nombre, la que decida cuáles son los temas más importantes en cada momento. Estos serán presentados en la portada del espacio, pudiendo por supuesto accederse a todos los demás temas en páginas posteriores, o usando otros criterios de ordenación (los temas con más comentarios, los más nuevos, los más controvertidos, etc.).
Cada uno de los trabajadores del Ayuntamiento tiene un usuario propio, que será resaltado como tal, permitiendo que participen en los debates al mismo nivel que todos los demás ciudadanos. Esto permitirá crear espacios de comunicación directos entre unos y otros, evitando los inconvenientes que implica la comunicación medidada, y respondiendo a un planteamiento claro por parte del nuevo gobierno de Madrid por el cual el Ayuntamiento trabaja para la ciudadanía, y ante ella debe responder.
+ +I.I. Espacio de propuestas
+ +En este espacio, cualquier persona puede proponer una iniciativa con la intención de recabar los suficientes apoyos como para que la idea pase a ser consultada a toda la ciudadanía con caracter vinculante.
+ +Las propuestas pueden ser apoyadas por ciudadanos empadronados en Madrid que hayan verificado su cuenta en la plataforma de participación, de tal manera que será la propia ciudadanía, y nadie en su nombre, la que decida cuáles son las propuestas que merecen la pena ser llevadas a cabo.
+ +Una vez que una propuesta alcance una cantidad de apoyos equivalente al 2% del censo de Madrid (unos 53000), automaticamente pasa a ser estudiada por un grupo de trabajo del ayuntamiento y pasará a la siguiente fase de consulta popular, en la que la ciudadanía de Madrid votará si se lleva a cabo o no. El plazo máximo para recabar los apoyos necesarios será de 12 meses.
Utilízalo en tu municipio
+ + <%= link_to t("debates.show.back_link"), "/more_information", class: 'left back' %> + +Utilízalo en tu municipio
Utilízalo en tu municipio libremente o ayúdanos a mejorarlo, es software libre.
Este Portal de Gobierno Abierto usa la aplicación Consul que es software libre, con licencia AGPLv3, esto significa en palabras sencillas, que cualquiera puede libremente usar el código, copiarlo, verlo en detalle, modificarlo, y redistribuirlo al mundo con las modificaciones que quiera (manteniendo el que otros puedan a su vez hacer lo mismo). Porque creemos que la cultura es mejor y más rica cuando se libera.
diff --git a/app/views/pages/more_information.html.erb b/app/views/pages/more_information.html.erb index c00e5f108..4b64b2137 100644 --- a/app/views/pages/more_information.html.erb +++ b/app/views/pages/more_information.html.erb @@ -21,7 +21,27 @@ Participación Ciudadana, Transparencia y Gobierno Abierto + +Si tienes problemas con la web contacta con el servicio técnico.
--
- I. Participación
-
- -
- II. Transparencia
-
-
+ + <%= link_to t("debates.show.back_link"), "/more_information", class: 'left back' %> +-- I.1. Propuestas ciudadanas
- - I.2. Presupuestos participativos
- - I.3. Legislacion colaborativa
- - I.4. Co-gobierno ciudadano
- - I.5. Participación sectorial
- - I.6. Inclusión, neutralidad y privacidad
- - I.7. Innovación social
-
--- II.1. Ordenanza de transparencia
- - II.2. Portal de transparencia
- - II.3. Datos abiertos
- - II.4. Agendas públicas
- - II.5. Transparencia del Lobby
-
-+-
+ I. Participación
+
+ -
+ II. Transparencia
+
+
+- I.1. Propuestas ciudadanas
+ - I.2. Presupuestos participativos
+ - I.3. Legislacion colaborativa
+ - I.4. Co-gobierno ciudadano
+ - I.5. Participación sectorial
+ - I.6. Inclusión, neutralidad y privacidad
+ - I.7. Innovación social
+
++- II.1. Ordenanza de transparencia
+ - II.2. Portal de transparencia
+ - II.3. Datos abiertos
+ - II.4. Agendas públicas
+ - II.5. Transparencia del Lobby
+
+Participación y Transparencia en Madrid - Próximas novedades
diff --git a/app/views/pages/participation_facts.html.erb b/app/views/pages/participation_facts.html.erb new file mode 100644 index 000000000..019076022 --- /dev/null +++ b/app/views/pages/participation_facts.html.erb @@ -0,0 +1,33 @@ +Hechos sobre participación ciudadana y democracia directa
++
La democracia directa produce gente más informada y con más cultura política.
+El poner en marcha mecanismos de participación reales hace que la gente se preocupe por las decisiones que tiene que tomar, y que por lo tanto se informe sobre ellas. Esto se observa sistemáticamente al comparar diferentes regiones de un mismo país con diferentes grados de democracia directa, o en procesos particulares como por ejemplo la votación del Tratado por una Constitución para Europa, comparando los países que lo votaron en referéndum y los que no.
+En ocasiones se argumenta que no se deben crear mecanismos de decisión ciudadana, porque un posible bajo nivel cultural o de conocimiento político haría que se tomaran malas decisiones. Esto se ha argumentado tradicionalmente en contra del voto femenino, del voto inmigrante, del voto de la gente sin renta, etc. pero lo que se ha demostrado es que permitir a toda esa gente decidir ha sido precisamente lo que ha permitido que escaparan de su situación de desigualdad cultural y de derechos, o al menos que la mejoraran considerablemente.
+M. Benz / A. Stutzer (2004), «Are voters better informed when they have a larger say in politics?», Public Choice 119, p. 31-59
+ +La democracia directa es más efectiva para tomar decisiones que la representativa. Por ejemplo genera menos deuda.
+Los proyectos disparatados y los gastos desmesurados suelen venir por parte de políticos electos en lugar de por procesos de participación ciudadana. Los estados de Estados Unidos con sistemas de iniciativas populares efectivos generan un 7% menos de deuda que los que no los tienen, y las regiones en Suiza con referéndum obligatorios para gastos públicos importantes gastan un 19% menos que los demás.
+R.K. von Weizsäcker (1992), «Staatsverschuldung und Demokratie», Kyklos 45, p. 51-67
+L.P. Feld / J.G. Matsusaka (2003), »Budget referendums and government spending: evidence from Swiss cantons», Journal of Public Economics 87, p. 2703-2724
+J. G. Matsusaka (2004), «For the Many or the Few. The Initiative, Public Policy, and American Democracy» University of Chicago Press
+ +La democracia directa protege mejor los derechos humanos y los de las minorías que la representativa
+Estudios de todos los referéndum suizos durante treinta años muestran que los referéndum cuando tratan sobre aumentar los derechos de las minorías se aprueban aproximadamente un 25% más que los que tratan de temas generales. En concreto, el 80% de los referéndum de este tipo a nivel federal fueron aprobados.
+Decisiones como eliminar el derecho a la vivienda de manera efectiva, el derecho de sanidad para las personas migrantes, o el iniciar guerras contra otros países, suelen darse a través de gobiernos representativos, y no en referéndum ciudadanos.
+B.S. Frey / M. Goette (1998), «Does the popular vote destroy civil rights?», American Journal of Political Science 42, p. 1343-1348
+ +En cualquier caso, la democracia directa no sustituye ni amenaza a la democracia representativa, la complementa. Y soluciona algunos problemas que no consigue la otra.
+El año que más iniciativas ciudadanas se votaron en los diferentes estados con democracia directa de Estados Unidos, estas decisiones sólo representaron el 0,6% de todas las decisiones que tomaron los políticos en esos estados.
+Las decisiones directas de la ciudadanía no sustituyen en ningún lugar del mundo el sistema político representativo, sólo lo amplían y mejoran. Tampoco pretenden ser la solución a todos los problemas, también se toman decisiones equivocadas a través de las decisiones ciudadanas, pero se reduce el número de las decisiones erróneas que se toman teniendo sólo un sistema representativo.
+M.D. Waters (2002), «Initiative and referendum in the United States: a primer», Washington: Citizen Lawmaker Press
+ +Si los ciudadanos son capaces de elegir entre políticos que toman decisiones buenas o malas para el país, deben ser capaces de elegir directamente entre buenas y malas decisiones.
++-
+ La nueva ola global de participación ciudadana
+
+ -
+ Participación ciudadana directa con experiencia
+
+
+Participación ciudadana directa en el mundo
+En el mundo existen sistemas de participación ciudadana muy similares al que vamos a implementar en Madrid, que llevan funcionando desde hace más de cien años y en países muy diferentes. La experiencia larga y variada de dichos sistemas demuestra que lo que ponemos en marcha en Madrid tendrá un impacto muy positivo en la sociedad, como ha tenido en esos otros países.
+ +Además de los ejemplos clásicos, como Suiza, asistimos a un desarrollo muy fuerte de los sistemas de participación ciudadana en todo el mundo, en particular en los últimos años, gracias especialmente a las nuevas posibilidades que nos brinda Internet. Islandia, Finlandia, Brasil, Estados Unidos, son algunos de los países que más están apostando por una participación directa de la ciudadanía en la toma de decisiones.
+ +La nueva ola global de participación ciudadana
+ +Las nuevas formas de participación se están dirigiendo principalmente a que sean la ciudadanía quien decida qué caminos debe tomar la política de su país, a través de mecanismos de iniciativas ciudadanas. Finlandia es uno de los países donde se están desarrollando nuevas herramientas similares al nuevo portal de gobierno abierto de Madrid. Su plataforma Open Ministry permite a la población presentar y apoyar propuestas, y ha conseguido por ejemplo que se apruebe gracias a él la ley de matrimonio igualitario. Islandia también ha tenido una gran repercusión desde 2011, cuando lanzaron su plataforma Better Reykjavik, que ha permitido que el 58% de la población participe en el proceso de propuestas, seleccionando cada mes las 15 ideas más votadas.
+ +Estonia es uno de los países que gracias a una apuesta clara por las nuevas tecnologías, ha podido situarse en cabeza de Europa en el nivel de uso por parte de la ciudadanía de Internet para la interacción con el gobierno. No sólo los ciudadanos y ciudadanas resuelven diariamente todos sus trámites a través de Internet, sino que han puesto en marcha plataformas como Rahvakogu, donde después de los escándalos políticos de 2012, 50.000 personas (de un total de 1.3 millones) participaron proponiendo medidas para mejorar la situación democrática del país.
+ +Otra de las principales experiencias que se están extendiendo rápidamente por todo el mundo son los presupuestos participativos. Estos consisten en mecanismos, generalmente acompañados de una plataforma en Internet, por los cuales el gobierno reserva parte de sus presupuestos de inversión (los que no están comprometidos ya en cuestiones como limpieza o servicios sociales), para que sea la ciudadanía quien decida en qué se gasta dicho dinero.
+ +Islandia es uno de los países de referencia al respecto, a través de su plataforma Betri Reykjavík-Betri Hverfi (Better District). Cada año se gastan 1.8 millones de euros en alrededor de 200 proyectos propuestos por la población para los distintos barrios de Reykjavík. Las ciudadanas y ciudadanos pueden participar y seguir el proyecto a través de la plataforma digital betrireykjavik, en Facebook o a través de centros en sus distritos o comités distritales.
+ +Otras experiencias de referencia incluyen París, donde los residentes decidirán hasta el 2020 cómo se gastan 426 millones de euros (lo que corresponde al 5% del presupuesto municipal de París); Nueva York, donde el año pasado dedicaron 32 millones de euros a presupuestos participativos; más de 100 ciudades brasileñas donde se han manejado presupuestos participativos que han oscilado entre el 5% y el 15%; entre muchas otras ciudades del mundo.
+ +Participación ciudadana directa con experiencia
+ +En diferentes países del mundo existen sistemas de participación ciudadana directa que vienen funcionando desde hace mucho tiempo sin pasar por las nuevas tecnologías. Aunque estos sistemas no tengan la agilidad de las nuevas plataformas, los procesos que se dan en ellos son básicamente los mismos, y nos aseguran la calidad y los resultados de los mecanismos que se van a poner en marcha. Comentamos a continuación tres de los casos más importantes y resaltamos algunas cifras para entender de lo que hablamos:
+ +Suiza
+Desde 1848 se han votado unos 600 referéndum a nivel federal (compilación en francés), y se celebran cada año en todo el país del orden de 200 referéndum a todos los niveles (municipal, cantonal y federal).
+Aproximadamente una de cada dos leyes aprobadas en el Parlamento directamente, y luego consultada a la ciudadanía, fue anulada. Por otro lado, incluso aunque las iniciativas populares no tengan éxito, el gobierno acaba concediendo parte de las demandas propuestas, y se genera una atención nacional sobre el asunto tratado. Esto hace que aproximadamente la mitad de la gente que lanzó las iniciativas que no tuvieron éxito consideren que mereció la pena el esfuerzo y se obtuvo algo que no hubiera sido posible sin la iniciativa.
+ +Estados Unidos
+En este país no se cuenta con procesos de iniciativas ciudadanas a nivel federal, pero 27 de los 51 estados cuentan con algún tipo sistema de democracia directa (cambiando mucho la regulación y el alcance de un estado a otro). A nivel local, aproximadamente la mitad de las ciudades cuentan con un sistema de iniciativas ciudadanas vinculantes.
+Entre 1904 y 2000 se convocaron casi 2.000 referéndum iniciados por la ciudadanía. En 1996 en los estados con mecanismos de iniciativas ciudadanas se convocaron 96 referéndum, frente a las más de 14.000 leyes y resoluciones aprobadas por los representantes de esos estados.
+ +Alemania
+No existe en Alemania ningún mecanismo de democracia directa a nivel nacional, pero sí a nivel regional y local. Estos sistemas se introdujeron en su mayoría en los años 90. En 1996 el número de iniciativas ciudadanas fue de 318, reduciéndose en años posteriores, una vez tratados tantos temas pendientes anteriores a la introducción de estos mecanismos, a una media de 100 por año.
+ ++-
+ Explicación detallada del proceso
+
+ -
+ Preguntas Frecuentes
+
+ -
+ Hojas de firmas para recoger apoyos
+
+
+¿Cómo funcionan las propuestas ciudadanas?
+El mecanismo de propuestas ciudadanas se resume en cuatro pasos muy sencillos:
++- ¡Propones! Creas una propuesta en esta web.
+ - ¡Apoyas! La gente hace click en el botón de apoyar tu propuesta (necesitas el apoyo del 2% de los empadronados mayores de 16 años para pasar a la siguiente fase).
+ - ¡Decides! Si has conseguido suficientes apoyos, dejamos 45 días para que la gente pueda debatir sobre la propuesta, y después durante una semana se invita a toda la gente de Madrid a decidir si están a favor o en contra de tu propuesta, en esta misma web.
+ - ¡Se hace! Si hay más gente a favor de tu propuesta que en contra, el gobierno del Ayuntamiento de Madrid asumirá como propia la propuesta y la llevará a cabo.
+
+ +Además no hace falta ni que tengas Internet, todos los pasos se pueden hacer en cualquiera de las 26 Oficinas de Atención al Ciudadano que hay por todo Madrid.
+ +Explicación detallada del proceso
+ ++- Creación de una propuesta. Cualquier persona (sin necesidad siquiera de estar empadronada en Madrid) puede crear una propuesta. Lo único que hay que hacer es pulsar el botón “Crear una propuesta” y rellenar los campos requeridos. La propuesta puede ser tan sencilla como una simple frase, pero te recomendamos detallarla todo lo posible, incluso añadiendo material adicional, para que sea más completa e interesante. Una vez creada aparecerá en esta web para que cualquiera pueda apoyarla.
+ - Apoyo de propuestas. Para apoyar una de las propuestas que aparece en la web, pulsamos el botón “apoyar esta propuesta” que aparece en cada una. Para este paso tendremos que estar empadronados en Madrid, así que al llevarlo a cabo por primera vez se nos pedirá que verifiquemos nuestra cuenta para estar seguros de este requerimiento. Se nos va a pedir que introduzcamos algunos datos para comprobar nuestra información de empadronamiento, y se nos enviará un código personal para que el proceso sea seguro. Las propuestas necesitan una cierta cantidad de apoyos para pasar a la siguiente fase; concretamente el 2% de los empadronados mayores de 16 años (que suponen 53.726 apoyos).
+ - Decisión sobre propuestas. Cuando una propuesta consigue los apoyos necesarios, se anuncia en la web. Desde ese momento se dejan 45 días para que todo el mundo pueda debatir e informarse sobre la propuesta. Todas las otras propuestas que hayan conseguido los apoyos necesarios en los primeros 30 días del tiempo de debate se agruparán junto a la primera para decidir sobre ellas al mismo tiempo. Pasados los 45 días se exponen estas propuestas en un espacio especial de votación de la web, donde durante una semana cualquier persona empadronada en Madrid y mayor de 16 años podrá decidir si está a favor o rechaza la propuesta. Para participar en este paso tendrás que tener tu cuenta de usuario verificada completamente, de tal forma que nos aseguremos que cada persona no tiene más de una cuenta y el proceso es seguro.
+ - Realización de las propuestas. En caso de que haya más gente a favor de una propuesta que rechazándola se aceptará como propuesta colectiva de la ciudadanía de Madrid, y el gobierno del Ayuntamiento de Madrid la asumirá como propia y la llevará a cabo. Para ello en un plazo máximo de un mes, se realizarán los informes técnicos correspondientes sobre su legalidad, viabilidad y coste económico, teniendo en cuenta a los sectores afectados y a la persona que haya lanzado la propuesta, para detallar la actuación correspondiente por parte del Ayuntamiento. Se publicarán en la web todos los informes realizados, y un seguimiento de las actuaciones que se lleven a cabo, para asegurar un correcto desarrollo de la propuesta.
+
+ +Todas las acciones relacionadas con el proceso de propuestas ciudadanas pueden realizarse a través del portal de gobierno abierto, o presencialmente en cualquiera de las 26 Oficinas de Atención al Ciudadano existentes en Madrid. Ver la lista completa de oficinas y su ubicación.
+ +El proceso de recogida de apoyos de una propuesta puede realizarse también a través de hojas de firmas, cuyo modelo puede ser descargado en este enlace. Los apoyos recogidos de esta manera se sumarán a los apoyos ya existentes en el portal de gobierno abierto. Las hojas pueden ser entregadas en cualquiera de los Registros del Ayuntamiento, presentes en cada una de las Juntas de Distrito. Ver la lista completa de Oficinas de Registro.
+ + +Preguntas Frecuentes
++- ¿Cuáles son los requisitos que se solicitan para poder apoyar o votar propuestas?
+
+ - ¿Cómo va a controlarse que cada persona vote una única vez por propuesta?
+
+ - ¿Se llevará a cabo cualquier propuesta que se acepte por mayoría en la web? ¿incluso aunque sean ilegales o atenten contra los derechos humanos? ¿y si no hay presupuesto para llevarla a cabo?
+
+ - ¿Qué diferencia existe entre los debates y las propuestas?
+
+ - ¿Vota toda la gente de Madrid en cada propuesta? ¿incluso aunque haga referencia a un único distrito?
+
+ - ¿Y si se presentan varias propuestas iguales? ¿Se plantea la posibilidad de unificarlas para evitar que se diversifiquen los apoyos y/o votos?
+
+ - ¿Durante cuanto tiempo se pueden recoger apoyos?
+
+ - ¿Durante cuanto tiempo se votará cada propuesta?
+
+ - ¿Pueden participar asociaciones, fundaciones y ONG´s? ¿Y empresas?
+
+ - ¿Cuánta gente tiene que votar una propuesta para que sea aprobada? ¿Qué quórum mínimo se necesita para que las votaciones sean vinculantes?
+
+ - ¿Existen mecanismos presenciales para participar? ¿Se ha planteado llegar a los ciudadanos y ciudadanas con dificultades de acceso a Internet o en situación de exclusión?
+
+ - ¿Cómo puede participar la gente que no esté empadronada en Madrid?
+
+ - ¿El voto es secreto? ¿el Ayuntamiento podrá tener acceso a los votos de los ciudadanos?
+
+ - ¿Al ser electrónico el voto no se aumenta el riesgo de fraude?
+ No, incluso lo reducimos. Hay una sensación subjetiva muy fuerte de diferencia de seguridad entre procesos en papel y procesos electrónicos que no se corresponde con la realidad. En las elecciones generales donde el voto es en papel, después de que las mesas hayan hecho el recuento, las papeletas de los votos se tiran a la basura, y la persona responsable se dirige en persona con el recuento en la mano al centro donde se comparten los datos. Y sin embargo pensemos por ejemplo en el sistema de tarjetas de crédito y cuentas bancarias, donde cada día confiamos la seguridad de nuestro dinero a un sistema electrónico. En el caso del Ayuntamiento, la votación está asegurada mediante diferentes entidades externas, y además cualquier votante puede tanto comprobar que su voto particular está en el recuento final, como realizar el recuento completo de todos los votos por su cuenta. Dos garantías de seguridad completamente imposibles en un votación tradicional.
+
+ - ¿Cómo se sabe que una propuesta se ha cumplido?
+
+
+ Estar empadronado en Madrid y ser mayor de 16 años. En caso de que se haga a través de Internet se requerirá que se haya verificado la cuenta de usuario de la web (encontramos el botón de verificación en el apartado “Mi cuenta” en la esquina superior derecha), proporcionando la información del padrón, y un medio de comunicación para obtener un código seguro de participación que introduciremos en la web para validar nuestra cuenta. +
+ Las votaciones presenciales en las Oficinas de Atención al Ciudadano están controladas porque la persona tiene que presentar su DNI. Para la votación a través de Internet se verifica que la cuenta de usuario corresponde a un único ciudadano/a. Para ello se le facilita un código personal seguro de verificación de la cuenta que se comunica a través de canales de comunicación privados, como son teléfonos móviles que constan en el Ayuntamiento que pertenecen a la persona adecuada, o por correo a través del buzón que figura en la dirección de empadronamiento. De esta forma nos aseguramos de que sólo esa persona ha recibido el código. En caso de que dichos medios de comunicación no estén disponibles se solicitará a la persona que acuda presencialmente a alguna de las Oficinas de Atención al Ciudadano, para obtener su código. +
+ Se establecen una serie de criterios objetivos por los que una propuesta no podrá ser aceptada como, por ejemplo, que tenga fines delictivos o que atente contra derechos fundamentales y libertades públicas. Además se excluyen del proceso las propuestas sobre otras modalidades de participación tales como presupuestos participativos, procesos revocatorios, audiencia pública o iniciativas normativas populares. +
Este mecanismo de participación no cambia los límites ya existentes para el Ayuntamiento, lo que hace es trasladar quién toma la decisión de lo que haya que hacer, incluyendo a toda la ciudadanía en el proceso de toma de decisiones. +
En el caso de las propuestas que quedan fuera de las competencias municipales el Ayuntamiento no podrá llevarlas a cabo, pero emprenderá actuaciones alternativas dentro de sus capacidades que intenten cumplir con la decisión de la propuesta. Lo mismo sucede con el presupuesto para llevar a cabo la propuesta; el ayuntamiento utilizará los recursos necesarios en cuanto sea posible, de la misma manera que se costean las decisiones adoptadas sin participación ciudadana. +
También puede ocurrir que la actuación requiera la aprobación del Pleno, por lo que el resultado dependerá del posicionamiento de todas las fuerzas políticas respecto a la actuación presentada.
+ Aunque ambos pueden ser apoyados, los primeros no activan ningún mecanismo de actuación concreto, mientras que las segundas pasan a una fase de decisión colectiva y en caso de que sean aprobadas, son asumidas por el Ayuntamiento.
+ Sí. El mecanismo actual se desarrolla a nivel de toda la ciudad. Lo que no entra en colisión con que a nivel distrital se puedan desarrollar mecanismos de participación ciudadana que afecten únicamente a cada distrito.
+ Propuestas similares pueden ser marcadas como tales en la plataforma, lo que se señala al visualizar cada una de ellas. Más allá de eso no se unifican, dejando que sea la gente la que decida si apoyar a una o a otra.
+ Las propuestas pueden recoger apoyos durante 12 meses. Si en ese tiempo no consiguen alcanzar el 2% de los apoyos necesarios dejan de poder recibir más apoyos.
+ Una vez que las propuestas reciben el apoyo del 2% pasan a la fase de decisión final donde la gente puede votar aceptando o rechazando la propuesta durante un plazo de una semana.
+ Las propuestas tienen que ser presentadas individualmente, sin problema de que la persona que la presenta represente a una organización de cualquier tipo. Al registrar una propuesta en la web existe la posibilidad de señalar este caso como registro de asociaciones/colectivos.
+ El quórum es el mínimo de participación necesaria para considerar una votación vinculante de manera legal. Ningún reglamento del Ayuntamiento puede hacer que este mecanismo sea vinculante juridicamente, porque eso está en contra de la legislación española. La vinculación con el mecanismo es política y se asume de manera personal por los concejales y la Alcaldesa. Por ello no se considerará ningún quórum.
+ Todas las acciones relacionadas con el proceso de propuestas ciudadanas pueden realizarse presencialmente en cualquiera de las 26 Oficinas de Atención al Ciudadano repartidas por todos los distritos de Madrid. Además, el proceso de recogida de apoyos de una propuesta puede realizarse también a través de hojas de firmas, cuyo modelo puede ser descargado en este enlace. +
Adicionalmente se ha creado en el Área de Gobierno de Participación Ciudadana, Transparencia y Gobierno Abierto el Servicio de Inclusión, Neutralidad y Privacidad que pondrá en marcha una mesa de inclusión con personal del Ayuntamiento y asociaciones que trabajan con colectivos en situación de exclusión, para diseñar mecanismos especiales para que puedan participar dichos colectivos.
+ Presentando propuestas, participando en los debates, y difundiendo lo que ocurra en la plataforma.
+ El voto es totalmente secreto. Se encripta de manera conjunta con diferentes autoridades externas al Ayuntamiento, de tal manera que para poder conocer la identidad de un votante todas las autoridades tendrían que realizar fraude conjuntamente.
+ Cuando se aprueba una propuesta, el Ayuntamiento publica en la página web cuáles son los pasos que se van a realizar para ponerla en práctica, incluyendo los informes que las correspondientes áreas redacten y las acciones que se emprenden. La información es completa y transparente para que puedas hacer un seguimiento de cómo evoluciona cada propuesta ciudadana.
Hojas de firmas para recoger apoyos
+ +El proceso de recogida de apoyos de una propuesta, además de en la web, puede realizarse a través de hojas de firmas, cuyo modelo puede ser descargado en este enlace.
+La hoja debe contener en las casillas superiores el código de la propuesta y su título, según figura en la página web específica de la propuesta, dentro del Portal de Gobierno Abierto.
+Los apoyos recogidos de esta manera se sumarán a los apoyos ya existentes en el portal de gobierno abierto. Las hojas pueden ser entregadas en cualquiera de los Registros del Ayuntamiento, presentes en cada una de las Juntas de Distrito. Ver la lista completa de Oficinas de Registro.
+ ++ <%= t("proposals.show.comments_title") %> + (<%= @proposal.comments_count %>) +
+ + <% if user_signed_in? %> + <%= render 'comments/form', {commentable: @proposal, parent_id: nil, toggeable: false} %> + <% else %> ++ +
+-
+ <%= link_to t('shared.flag'), flag_proposal_path(proposal), method: :put, remote: true, id: "flag-proposal-#{ proposal.id }" %>
+
+
+ <% end %> + + <% if show_unflag_action? proposal %> + + + ++-
+ <%= link_to t('shared.unflag'), unflag_proposal_path(proposal), method: :put, remote: true, id: "unflag-proposal-#{ proposal.id }" %>
+
+
+ <% end %> + diff --git a/app/views/proposals/_form.html.erb b/app/views/proposals/_form.html.erb new file mode 100644 index 000000000..1d98a05d6 --- /dev/null +++ b/app/views/proposals/_form.html.erb @@ -0,0 +1,84 @@ +<%= form_for(@proposal) do |f| %> + <%= render 'shared/errors', resource: @proposal %> + +<%= link_to proposal.title, proposal %>
++ + <%= link_to t("proposals.proposal.comments", count: proposal.comments_count), proposal_path(proposal, anchor: "comments") %> + • + <%= l proposal.created_at.to_date %> +
+<%= t("proposals.edit.editing") %>
+ + <%= render "form" %> ++ <%= page_entries_info @proposals %> + <%= t("proposals.index.search_results", count: @proposals.size, search_term: @search_terms) %> +
+ <% elsif @tag_filter %> ++ <%= page_entries_info @proposals %> + <%= t("proposals.index.filter_topic", count: @proposals.size, topic: @tag_filter) %> +
+ <% end %> ++ <%= t("proposals.index.select_order") %> +
+ <%= render 'shared/order_selector', i18n_namespace: "proposals.index" %> ++ <%= t("proposals.index.select_order_long") %> +
+ <%= render 'shared/order_selector', i18n_namespace: "proposals.index" %> +<%= t("proposals.new.start_new") %>
+<%= t("proposals.new.recommendations_title") %>
++- <%= t("proposals.new.recommendation_one") %>
+ - <%= t("proposals.new.recommendation_two") %>
+ - <%= t("proposals.new.recommendation_three") %>
+
+<%= @proposal.title %>
+ <% if @proposal.conflictive? %> +<%= @proposal.question %>
+ + <%= render 'shared/tags', proposal: @proposal %> + +<%= t("shared.search_form.search_title") %>
+<%= t("#{i18n_namespace}.title") %>
<%= t("shared.tags_cloud.tags") %>
<% tag_cloud @tag_cloud, %w[s m l] do |tag, css_class| %> - <%= link_to sanitize("#{tag.name} #{tag.taggings_count}"), debates_path(tag: tag.name), class: css_class %> + <%= link_to sanitize("#{tag.name} #{tag.taggings_count}"), taggable_path(taggable, tag.name), class: css_class %> <% end %>
<%= t("verification.letter.edit.title") %>
+ <%= t("verification.letter.new.explanation_html") %> + + <%= link_to t("verification.letter.new.offices"), t("verification.letter.new.offices_url"), + target: "_blank", class: "button radius inline-block" + %> + +<%= link_to featured_debate.title, featured_debate %>
-- - <%= link_to t("debates.show.comments", count: featured_debate.comments_count), debate_path(featured_debate, anchor: "comments") %> -
-+ <%= t('welcome.signed_in_home_title') %> +
+<%= t("welcome.last_debates") %>
+<%= t("welcome.debates.title") %>
+<%= t("welcome.debates.description") %>
+ +<%= t("welcome.proposal.title") %>
+<%= t("welcome.proposal.description") %>
+<%= t("welcome.decide.title") %>
+<%= t("welcome.decide.description") %>
+<%= t("welcome.do.title") %>
+<%= t("welcome.do.description") %>
#{Faker::Lorem.paragraphs.join('
')}" + description = "#{Faker::Lorem.paragraphs.join('
')}
" debate = Debate.create!(author: author, title: Faker::Lorem.sentence(3), created_at: rand((Time.now - 1.week) .. Time.now), @@ -67,6 +76,26 @@ tags = Faker::Lorem.words(25) puts " #{debate.title}" end +puts "Creating Proposals" + +tags = Faker::Lorem.words(25) + +(1..30).each do |i| + author = User.reorder("RANDOM()").first + description = "#{Faker::Lorem.paragraphs.join('
')}
" + proposal = Proposal.create!(author: author, + title: Faker::Lorem.sentence(3), + question: Faker::Lorem.sentence(3), + summary: Faker::Lorem.sentence(3), + responsible_name: Faker::Name.name, + external_url: Faker::Internet.url, + description: description, + created_at: rand((Time.now - 1.week) .. Time.now), + tag_list: tags.sample(3).join(','), + terms_of_service: "1") + puts " #{proposal.title}" +end + puts "Commenting Debates" @@ -80,9 +109,21 @@ puts "Commenting Debates" end -puts "Commenting Comments" +puts "Commenting Proposals" (1..100).each do |i| + author = User.reorder("RANDOM()").first + proposal = Proposal.reorder("RANDOM()").first + Comment.create!(user: author, + created_at: rand(proposal.created_at .. Time.now), + commentable: proposal, + body: Faker::Lorem.sentence) +end + + +puts "Commenting Comments" + +(1..200).each do |i| author = User.reorder("RANDOM()").first parent = Comment.reorder("RANDOM()").first Comment.create!(user: author, @@ -94,7 +135,7 @@ puts "Commenting Comments" end -puts "Voting Debates & Comments" +puts "Voting Debates, Proposals & Comments" (1..100).each do |i| voter = not_org_users.reorder("RANDOM()").first @@ -110,6 +151,12 @@ end comment.vote_by(voter: voter, vote: vote) end +(1..100).each do |i| + voter = User.level_two_or_three_verified.reorder("RANDOM()").first + proposal = Proposal.reorder("RANDOM()").first + proposal.vote_by(voter: voter, vote: true) +end + puts "Flagging Debates & Comments" @@ -125,23 +172,32 @@ end Flag.flag(flagger, comment) end +(1..40).each do |i| + proposal = Proposal.reorder("RANDOM()").first + flagger = User.where(["users.id <> ?", proposal.author_id]).reorder("RANDOM()").first + Flag.flag(flagger, proposal) +end -puts "Ignoring flags in Debates & comments" + +puts "Ignoring flags in Debates, comments & proposals" Debate.flagged.reorder("RANDOM()").limit(10).each(&:ignore_flag) Comment.flagged.reorder("RANDOM()").limit(30).each(&:ignore_flag) +Proposal.flagged.reorder("RANDOM()").limit(10).each(&:ignore_flag) -puts "Hiding debates & comments" +puts "Hiding debates, comments & proposals" Comment.with_hidden.flagged.reorder("RANDOM()").limit(30).each(&:hide) Debate.with_hidden.flagged.reorder("RANDOM()").limit(5).each(&:hide) +Proposal.with_hidden.flagged.reorder("RANDOM()").limit(10).each(&:hide) -puts "Confirming hiding in debates & comments" +puts "Confirming hiding in debates, comments & proposals" Comment.only_hidden.flagged.reorder("RANDOM()").limit(10).each(&:confirm_hide) Debate.only_hidden.flagged.reorder("RANDOM()").limit(5).each(&:confirm_hide) +Proposal.only_hidden.flagged.reorder("RANDOM()").limit(5).each(&:confirm_hide) diff --git a/db/migrate/20150911171301_create_proposal.rb b/db/migrate/20150911171301_create_proposal.rb new file mode 100644 index 000000000..37dd1bb85 --- /dev/null +++ b/db/migrate/20150911171301_create_proposal.rb @@ -0,0 +1,22 @@ +class CreateProposal < ActiveRecord::Migration + def change + create_table :proposals do |t| + t.string "title", limit: 80 + t.text "description" + t.string "question" + t.string "external_url" + t.integer "author_id" + t.datetime "hidden_at" + t.integer "flags_count", default: 0 + t.datetime "ignored_flag_at" + t.integer "cached_votes_up", default: 0 + t.integer "comments_count", default: 0 + t.datetime "confirmed_hide_at" + t.integer "hot_score", limit: 8, default: 0 + t.integer "confidence_score", default: 0 + + t.datetime "created_at", null: false + t.datetime "updated_at", null: false + end + end +end diff --git a/db/migrate/20150914113251_add_responsible_to_proposals.rb b/db/migrate/20150914113251_add_responsible_to_proposals.rb new file mode 100644 index 000000000..c565c29df --- /dev/null +++ b/db/migrate/20150914113251_add_responsible_to_proposals.rb @@ -0,0 +1,5 @@ +class AddResponsibleToProposals < ActiveRecord::Migration + def change + add_column :proposals, :responsible_name, :string, limit: 60 + end +end diff --git a/db/migrate/20150914114019_change_email_config_field_in_user.rb b/db/migrate/20150914114019_change_email_config_field_in_user.rb new file mode 100644 index 000000000..90c875d60 --- /dev/null +++ b/db/migrate/20150914114019_change_email_config_field_in_user.rb @@ -0,0 +1,5 @@ +class ChangeEmailConfigFieldInUser < ActiveRecord::Migration + def change + rename_column :users, :email_on_debate_comment, :email_on_comment + end +end diff --git a/db/migrate/20150914173834_add_summary_to_proposals.rb b/db/migrate/20150914173834_add_summary_to_proposals.rb new file mode 100644 index 000000000..7ebe2df9c --- /dev/null +++ b/db/migrate/20150914173834_add_summary_to_proposals.rb @@ -0,0 +1,5 @@ +class AddSummaryToProposals < ActiveRecord::Migration + def change + add_column :proposals, :summary, :text, limit: 280 + end +end diff --git a/db/migrate/20150914181921_add_more_indexes_for_ahoy.rb b/db/migrate/20150914181921_add_more_indexes_for_ahoy.rb new file mode 100644 index 000000000..7fa8bedc4 --- /dev/null +++ b/db/migrate/20150914181921_add_more_indexes_for_ahoy.rb @@ -0,0 +1,7 @@ +class AddMoreIndexesForAhoy < ActiveRecord::Migration + def change + add_index :ahoy_events, [:name, :time] + add_index :visits, [:started_at] + end +end + diff --git a/db/migrate/20150914182652_adds_indexes.rb b/db/migrate/20150914182652_adds_indexes.rb new file mode 100644 index 000000000..a22d0d16e --- /dev/null +++ b/db/migrate/20150914182652_adds_indexes.rb @@ -0,0 +1,19 @@ +class AddsIndexes < ActiveRecord::Migration + def change + add_index :debates, :author_id + add_index :debates, [:author_id, :hidden_at] + + add_index :proposals, :author_id + add_index :proposals, [:author_id, :hidden_at] + add_index :proposals, :cached_votes_up + add_index :proposals, :confidence_score + add_index :proposals, :hidden_at + add_index :proposals, :hot_score + + add_index :settings, :key + + add_index :verified_users, :document_number + add_index :verified_users, :phone + add_index :verified_users, :email + end +end diff --git a/db/migrate/20150914184018_add_indexes_for_searches.rb b/db/migrate/20150914184018_add_indexes_for_searches.rb new file mode 100644 index 000000000..64c46370c --- /dev/null +++ b/db/migrate/20150914184018_add_indexes_for_searches.rb @@ -0,0 +1,11 @@ +class AddIndexesForSearches < ActiveRecord::Migration + def change + add_index :debates, :title + add_index :debates, :description + + add_index :proposals, :title + add_index :proposals, :question + add_index :proposals, :summary + add_index :proposals, :description + end +end diff --git a/db/migrate/20150914191003_add_video_url_to_proposal.rb b/db/migrate/20150914191003_add_video_url_to_proposal.rb new file mode 100644 index 000000000..298038e97 --- /dev/null +++ b/db/migrate/20150914191003_add_video_url_to_proposal.rb @@ -0,0 +1,5 @@ +class AddVideoUrlToProposal < ActiveRecord::Migration + def change + add_column :proposals, :video_url, :string + end +end diff --git a/db/schema.rb b/db/schema.rb index dca1e8354..0bfa35d54 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -11,7 +11,7 @@ # # It's strongly recommended that you check this file into your version control system. -ActiveRecord::Schema.define(version: 20150912145218) do +ActiveRecord::Schema.define(version: 20150914191003) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" @@ -51,6 +51,7 @@ ActiveRecord::Schema.define(version: 20150912145218) do t.string "ip" end + add_index "ahoy_events", ["name", "time"], name: "index_ahoy_events_on_name_and_time", using: :btree add_index "ahoy_events", ["time"], name: "index_ahoy_events_on_time", using: :btree add_index "ahoy_events", ["user_id"], name: "index_ahoy_events_on_user_id", using: :btree add_index "ahoy_events", ["visit_id"], name: "index_ahoy_events_on_visit_id", using: :btree @@ -104,13 +105,17 @@ ActiveRecord::Schema.define(version: 20150912145218) do t.integer "confidence_score", default: 0 end + add_index "debates", ["author_id", "hidden_at"], name: "index_debates_on_author_id_and_hidden_at", using: :btree + add_index "debates", ["author_id"], name: "index_debates_on_author_id", using: :btree add_index "debates", ["cached_votes_down"], name: "index_debates_on_cached_votes_down", using: :btree add_index "debates", ["cached_votes_score"], name: "index_debates_on_cached_votes_score", using: :btree add_index "debates", ["cached_votes_total"], name: "index_debates_on_cached_votes_total", using: :btree add_index "debates", ["cached_votes_up"], name: "index_debates_on_cached_votes_up", using: :btree add_index "debates", ["confidence_score"], name: "index_debates_on_confidence_score", using: :btree + add_index "debates", ["description"], name: "index_debates_on_description", using: :btree add_index "debates", ["hidden_at"], name: "index_debates_on_hidden_at", using: :btree add_index "debates", ["hot_score"], name: "index_debates_on_hot_score", using: :btree + add_index "debates", ["title"], name: "index_debates_on_title", using: :btree create_table "delayed_jobs", force: :cascade do |t| t.integer "priority", default: 0, null: false @@ -188,11 +193,45 @@ ActiveRecord::Schema.define(version: 20150912145218) do add_index "organizations", ["user_id"], name: "index_organizations_on_user_id", using: :btree + create_table "proposals", force: :cascade do |t| + t.string "title", limit: 80 + t.text "description" + t.string "question" + t.string "external_url" + t.integer "author_id" + t.datetime "hidden_at" + t.integer "flags_count", default: 0 + t.datetime "ignored_flag_at" + t.integer "cached_votes_up", default: 0 + t.integer "comments_count", default: 0 + t.datetime "confirmed_hide_at" + t.integer "hot_score", limit: 8, default: 0 + t.integer "confidence_score", default: 0 + t.datetime "created_at", null: false + t.datetime "updated_at", null: false + t.string "responsible_name", limit: 60 + t.text "summary" + t.string "video_url" + end + + add_index "proposals", ["author_id", "hidden_at"], name: "index_proposals_on_author_id_and_hidden_at", using: :btree + add_index "proposals", ["author_id"], name: "index_proposals_on_author_id", using: :btree + add_index "proposals", ["cached_votes_up"], name: "index_proposals_on_cached_votes_up", using: :btree + add_index "proposals", ["confidence_score"], name: "index_proposals_on_confidence_score", using: :btree + add_index "proposals", ["description"], name: "index_proposals_on_description", using: :btree + add_index "proposals", ["hidden_at"], name: "index_proposals_on_hidden_at", using: :btree + add_index "proposals", ["hot_score"], name: "index_proposals_on_hot_score", using: :btree + add_index "proposals", ["question"], name: "index_proposals_on_question", using: :btree + add_index "proposals", ["summary"], name: "index_proposals_on_summary", using: :btree + add_index "proposals", ["title"], name: "index_proposals_on_title", using: :btree + create_table "settings", force: :cascade do |t| t.string "key" t.string "value" end + add_index "settings", ["key"], name: "index_settings_on_key", using: :btree + create_table "simple_captcha_data", force: :cascade do |t| t.string "key", limit: 40 t.string "value", limit: 6 @@ -240,7 +279,7 @@ ActiveRecord::Schema.define(version: 20150912145218) do t.datetime "confirmed_at" t.datetime "confirmation_sent_at" t.string "unconfirmed_email" - t.boolean "email_on_debate_comment", default: false + t.boolean "email_on_comment", default: false t.boolean "email_on_comment_reply", default: false t.string "phone_number", limit: 30 t.string "official_position" @@ -275,6 +314,10 @@ ActiveRecord::Schema.define(version: 20150912145218) do t.datetime "updated_at", null: false end + add_index "verified_users", ["document_number"], name: "index_verified_users_on_document_number", using: :btree + add_index "verified_users", ["email"], name: "index_verified_users_on_email", using: :btree + add_index "verified_users", ["phone"], name: "index_verified_users_on_phone", using: :btree + create_table "visits", id: :uuid, default: nil, force: :cascade do |t| t.uuid "visitor_id" t.string "ip" @@ -303,6 +346,7 @@ ActiveRecord::Schema.define(version: 20150912145218) do t.datetime "started_at" end + add_index "visits", ["started_at"], name: "index_visits_on_started_at", using: :btree add_index "visits", ["user_id"], name: "index_visits_on_user_id", using: :btree create_table "votes", force: :cascade do |t| diff --git a/db/seeds.rb b/db/seeds.rb index 7714981c7..27da98e1c 100644 --- a/db/seeds.rb +++ b/db/seeds.rb @@ -1,6 +1,8 @@ # Default admin user (change password after first deploy to a server!) -admin = User.create!(username: 'admin', email: 'admin@madrid.es', password: '12345678', password_confirmation: '12345678', confirmed_at: Time.now, terms_of_service: "1") -admin.create_administrator +if Administrator.count == 0 + admin = User.create!(username: 'admin', email: 'admin@madrid.es', password: '12345678', password_confirmation: '12345678', confirmed_at: Time.now, terms_of_service: "1") + admin.create_administrator +end # Names for the moderation console, as a hint for moderators # to know better how to assign users with official positions @@ -12,3 +14,12 @@ Setting.create(key: 'official_level_5_name', value: 'Alcaldesa') # Max percentage of allowed anonymous votes on a debate Setting.create(key: 'max_ratio_anon_votes_on_debates', value: '50') + +# Max votes where a proposal is still editable +Setting.create(key: 'max_votes_for_proposal_edit', value: '1000') + +# Prefix for the Proposal codes +Setting.create(key: 'proposal_code_prefix', value: 'MAD') + +# Number of votes needed for proposal success +Setting.create(key: 'votes_for_proposal_success', value: '53726') diff --git a/lib/score_calculator.rb b/lib/score_calculator.rb new file mode 100644 index 000000000..7d77e82d6 --- /dev/null +++ b/lib/score_calculator.rb @@ -0,0 +1,30 @@ +module ScoreCalculator + + EPOC = Time.new(2015, 6, 15) + COMMENT_WEIGHT = 1.0/20 # 1 positive vote / x comments + TIME_UNIT = 12.hours.to_f + + def self.hot_score(date, votes_total, votes_up, comments_count) + total = (votes_total + COMMENT_WEIGHT * comments_count).to_f + ups = (votes_up + COMMENT_WEIGHT * comments_count).to_f + downs = total - ups + score = ups - downs + offset = Math.log([score.abs, 1].max, 10) * (ups / [total, 1].max) + sign = score <=> 0 + seconds = ((date || Time.now) - EPOC).to_f + + (((offset * sign) + (seconds/TIME_UNIT)) * 10000000).round + end + + def self.confidence_score(votes_total, votes_up) + return 0 unless votes_total > 0 + + votes_total = votes_total.to_f + votes_up = votes_up.to_f + votes_down = votes_total - votes_up + score = votes_up - votes_down + + score * (votes_up / votes_total) * 100 + end + +end diff --git a/lib/verification.rb b/lib/verification.rb deleted file mode 100644 index 67b0f4422..000000000 --- a/lib/verification.rb +++ /dev/null @@ -1,36 +0,0 @@ -module Verification - - def verification_email_sent? - email_verification_token.present? - end - - def verification_sms_sent? - unconfirmed_phone.present? && sms_confirmation_code.present? - end - - def verification_letter_sent? - letter_requested_at.present? && letter_verification_code.present? - end - - def residence_verified? - residence_verified_at.present? - end - - def sms_verified? - confirmed_phone.present? - end - - def level_two_verified? - residence_verified? && sms_verified? - end - - def level_three_verified? - verified_at.present? - end - - def unverified? - !level_two_verified? && !level_three_verified? - end - - -end \ No newline at end of file diff --git a/public/docs/formulario_propuestas_ciudadanas.pdf b/public/docs/formulario_propuestas_ciudadanas.pdf new file mode 100644 index 000000000..f8296d9c5 Binary files /dev/null and b/public/docs/formulario_propuestas_ciudadanas.pdf differ diff --git a/spec/controllers/concerns/has_orders_spec.rb b/spec/controllers/concerns/has_orders_spec.rb new file mode 100644 index 000000000..082c4c068 --- /dev/null +++ b/spec/controllers/concerns/has_orders_spec.rb @@ -0,0 +1,37 @@ +require 'rails_helper' + +describe 'HasOrders' do + + class FakeController < ActionController::Base; end + + controller(FakeController) do + include HasOrders + has_orders ['created_at', 'votes_count', 'flags_count'], only: :index + + def index + render text: "#{@current_order} (#{@valid_orders.join(' ')})" + end + end + + it "has the valid orders set up" do + get :index + expect(response.body).to eq('created_at (created_at votes_count flags_count)') + end + + describe "the current order" do + it "defaults to the first one on the list" do + get :index + expect(response.body).to eq('created_at (created_at votes_count flags_count)') + end + + it "can be changed by the order param" do + get :index, order: 'votes_count' + expect(response.body).to eq('votes_count (created_at votes_count flags_count)') + end + + it "defaults to the first one on the list if given a bogus order" do + get :index, order: 'foobar' + expect(response.body).to eq('created_at (created_at votes_count flags_count)') + end + end +end diff --git a/spec/factories.rb b/spec/factories.rb index 8513d4d16..cda9c01a8 100644 --- a/spec/factories.rb +++ b/spec/factories.rb @@ -7,6 +7,17 @@ FactoryGirl.define do terms_of_service '1' confirmed_at { Time.now } + trait :level_two do + residence_verified_at Time.now + confirmed_phone "611111111" + document_number "12345678Z" + end + + trait :level_three do + verified_at Time.now + document_number "12345678Z" + end + trait :hidden do hidden_at Time.now end @@ -101,6 +112,50 @@ FactoryGirl.define do end end + factory :proposal do + sequence(:title) { |n| "Proposal #{n} title" } + summary 'In summary, what we want is...' + description 'Proposal description' + question 'Proposal question' + external_url 'http://external_documention.es' + responsible_name 'John Snow' + terms_of_service '1' + association :author, factory: :user + + trait :hidden do + hidden_at Time.now + end + + trait :with_ignored_flag do + ignored_flag_at Time.now + end + + trait :with_confirmed_hide do + confirmed_hide_at Time.now + end + + trait :flagged do + after :create do |debate| + Flag.flag(FactoryGirl.create(:user), debate) + end + end + + trait :with_hot_score do + before(:save) { |d| d.calculate_hot_score } + end + + trait :with_confidence_score do + before(:save) { |d| d.calculate_confidence_score } + end + + trait :conflictive do + after :create do |debate| + Flag.flag(FactoryGirl.create(:user), debate) + 4.times { create(:vote, votable: debate) } + end + end + end + factory :vote do association :votable, factory: :debate association :voter, factory: :user diff --git a/spec/features/account_spec.rb b/spec/features/account_spec.rb index 93cfe3dd0..e0249072c 100644 --- a/spec/features/account_spec.rb +++ b/spec/features/account_spec.rb @@ -33,7 +33,7 @@ feature 'Account' do visit account_path fill_in 'account_username', with: 'Larry Bird' - check 'account_email_on_debate_comment' + check 'account_email_on_comment' check 'account_email_on_comment_reply' click_button 'Save changes' @@ -42,7 +42,7 @@ feature 'Account' do visit account_path expect(page).to have_selector("input[value='Larry Bird']") - expect(page).to have_selector("input[id='account_email_on_debate_comment'][value='1']") + expect(page).to have_selector("input[id='account_email_on_comment'][value='1']") expect(page).to have_selector("input[id='account_email_on_comment_reply'][value='1']") end @@ -51,7 +51,7 @@ feature 'Account' do visit account_path fill_in 'account_organization_attributes_name', with: 'Google' - check 'account_email_on_debate_comment' + check 'account_email_on_comment' check 'account_email_on_comment_reply' click_button 'Save changes' @@ -60,7 +60,7 @@ feature 'Account' do visit account_path expect(page).to have_selector("input[value='Google']") - expect(page).to have_selector("input[id='account_email_on_debate_comment'][value='1']") + expect(page).to have_selector("input[id='account_email_on_comment'][value='1']") expect(page).to have_selector("input[id='account_email_on_comment_reply'][value='1']") end diff --git a/spec/features/admin/proposals_spec.rb b/spec/features/admin/proposals_spec.rb new file mode 100644 index 000000000..e31b84578 --- /dev/null +++ b/spec/features/admin/proposals_spec.rb @@ -0,0 +1,85 @@ +require 'rails_helper' + +feature 'Admin proposals' do + + background do + admin = create(:administrator) + login_as(admin.user) + end + + scenario 'Restore' do + proposal = create(:proposal, :hidden) + visit admin_proposals_path + + click_link 'Restore' + + expect(page).to_not have_content(proposal.title) + + expect(proposal.reload).to_not be_hidden + end + + scenario 'Confirm hide' do + proposal = create(:proposal, :hidden) + visit admin_proposals_path + + click_link 'Confirm' + + expect(page).to_not have_content(proposal.title) + click_link('Confirmed') + expect(page).to have_content(proposal.title) + + expect(proposal.reload).to be_confirmed_hide + end + + scenario "Current filter is properly highlighted" do + visit admin_proposals_path + expect(page).to_not have_link('Pending') + expect(page).to have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_proposals_path(filter: 'Pending') + expect(page).to_not have_link('Pending') + expect(page).to have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_proposals_path(filter: 'all') + expect(page).to have_link('Pending') + expect(page).to_not have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_proposals_path(filter: 'with_confirmed_hide') + expect(page).to have_link('All') + expect(page).to have_link('Pending') + expect(page).to_not have_link('Confirmed') + end + + scenario "Filtering proposals" do + create(:proposal, :hidden, title: "Unconfirmed proposal") + create(:proposal, :hidden, :with_confirmed_hide, title: "Confirmed proposal") + + visit admin_proposals_path(filter: 'pending') + expect(page).to have_content('Unconfirmed proposal') + expect(page).to_not have_content('Confirmed proposal') + + visit admin_proposals_path(filter: 'all') + expect(page).to have_content('Unconfirmed proposal') + expect(page).to have_content('Confirmed proposal') + + visit admin_proposals_path(filter: 'with_confirmed_hide') + expect(page).to_not have_content('Unconfirmed proposal') + expect(page).to have_content('Confirmed proposal') + end + + scenario "Action links remember the pagination setting and the filter" do + per_page = Kaminari.config.default_per_page + (per_page + 2).times { create(:proposal, :hidden, :with_confirmed_hide) } + + visit admin_proposals_path(filter: 'with_confirmed_hide', page: 2) + + click_on('Restore', match: :first, exact: true) + + expect(current_url).to include('filter=with_confirmed_hide') + expect(current_url).to include('page=2') + end + +end diff --git a/spec/features/admin_spec.rb b/spec/features/admin_spec.rb index a096b2b83..ec991e388 100644 --- a/spec/features/admin_spec.rb +++ b/spec/features/admin_spec.rb @@ -15,7 +15,8 @@ feature 'Admin' do login_as(user) visit admin_root_path - expect(current_path).to eq(root_path) + expect(current_path).not_to eq(admin_root_path) + expect(current_path).to eq(proposals_path) expect(page).to have_content "not authorized" end @@ -23,7 +24,8 @@ feature 'Admin' do login_as(moderator) visit admin_root_path - expect(current_path).to eq(root_path) + expect(current_path).not_to eq(admin_root_path) + expect(current_path).to eq(proposals_path) expect(page).to have_content "not authorized" end diff --git a/spec/features/ckeditor_spec.rb b/spec/features/ckeditor_spec.rb new file mode 100644 index 000000000..7f5edc6e8 --- /dev/null +++ b/spec/features/ckeditor_spec.rb @@ -0,0 +1,19 @@ +require 'rails_helper' + +feature 'CKEditor' do + + scenario 'is present before & after turbolinks update page', :js do + author = create(:user) + login_as(author) + + visit new_debate_path + + expect(page).to have_css "#cke_debate_description" + + click_link 'Debates' + click_link 'Start a debate' + + expect(page).to have_css "#cke_debate_description" + end + +end \ No newline at end of file diff --git a/spec/features/comments_spec.rb b/spec/features/comments/debates.rb similarity index 82% rename from spec/features/comments_spec.rb rename to spec/features/comments/debates.rb index 437286bf7..4b8b23ce8 100644 --- a/spec/features/comments_spec.rb +++ b/spec/features/comments/debates.rb @@ -1,7 +1,7 @@ require 'rails_helper' include ActionView::Helpers::DateHelper -feature 'Comments' do +feature 'Commenting debates' do let(:user) { create :user } let(:debate) { create :debate } @@ -305,4 +305,90 @@ feature 'Comments' do end end + feature 'Voting comments' do + background do + @manuela = create(:user, verified_at: Time.now) + @pablo = create(:user) + @debate = create(:debate) + @comment = create(:comment, commentable: @debate) + + login_as(@manuela) + end + + scenario 'Show' do + create(:vote, voter: @manuela, votable: @comment, vote_flag: true) + create(:vote, voter: @pablo, votable: @comment, vote_flag: false) + + visit debate_path(@debate) + + within("#comment_#{@comment.id}_votes") do + within(".in_favor") do + expect(page).to have_content "1" + end + + within(".against") do + expect(page).to have_content "1" + end + + expect(page).to have_content "2 votes" + end + end + + scenario 'Create', :js do + visit debate_path(@debate) + + within("#comment_#{@comment.id}_votes") do + find(".in_favor a").click + + within(".in_favor") do + expect(page).to have_content "1" + end + + within(".against") do + expect(page).to have_content "0" + end + + expect(page).to have_content "1 vote" + end + end + + scenario 'Update', :js do + visit debate_path(@debate) + + within("#comment_#{@comment.id}_votes") do + find('.in_favor a').click + find('.against a').click + + within('.in_favor') do + expect(page).to have_content "0" + end + + within('.against') do + expect(page).to have_content "1" + end + + expect(page).to have_content "1 vote" + end + end + + scenario 'Trying to vote multiple times', :js do + visit debate_path(@debate) + + within("#comment_#{@comment.id}_votes") do + find('.in_favor a').click + find('.in_favor a').click + + within('.in_favor') do + expect(page).to have_content "1" + end + + within('.against') do + expect(page).to have_content "0" + end + + expect(page).to have_content "1 vote" + end + end + end + end diff --git a/spec/features/comments/proposals.rb b/spec/features/comments/proposals.rb new file mode 100644 index 000000000..9c6b99a6a --- /dev/null +++ b/spec/features/comments/proposals.rb @@ -0,0 +1,395 @@ +require 'rails_helper' +include ActionView::Helpers::DateHelper + +feature 'Commenting proposals' do + let(:user) { create :user } + let(:proposal) { create :proposal } + + scenario 'Index' do + 3.times { create(:comment, commentable: proposal) } + + visit proposal_path(proposal) + + expect(page).to have_css('.comment', count: 3) + + comment = Comment.last + within first('.comment') do + expect(page).to have_content comment.user.name + expect(page).to have_content I18n.l(comment.created_at, format: :datetime) + expect(page).to have_content comment.body + end + end + + scenario 'Turns links into html links' do + create :comment, commentable: proposal, body: 'Built with http://rubyonrails.org/' + + visit proposal_path(proposal) + + within first('.comment') do + expect(page).to have_content 'Built with http://rubyonrails.org/' + expect(page).to have_link('http://rubyonrails.org/', href: 'http://rubyonrails.org/') + expect(find_link('http://rubyonrails.org/')[:rel]).to eq('nofollow') + expect(find_link('http://rubyonrails.org/')[:target]).to eq('_blank') + end + end + + scenario 'Sanitizes comment body for security' do + create :comment, commentable: proposal, body: " click me http://madrid.es" + + visit proposal_path(proposal) + + within first('.comment') do + expect(page).to have_content "click me http://madrid.es" + expect(page).to have_link('http://madrid.es', href: 'http://madrid.es') + expect(page).not_to have_link('click me') + end + end + + scenario 'Paginated comments' do + per_page = 10 + (per_page + 2).times { create(:comment, commentable: proposal)} + + visit proposal_path(proposal) + + expect(page).to have_css('.comment', count: per_page) + within("ul.pagination") do + expect(page).to have_content("1") + expect(page).to have_content("2") + expect(page).to_not have_content("3") + click_link "Next", exact: false + end + + expect(page).to have_css('.comment', count: 2) + end + + feature 'Not logged user' do + scenario 'can not see comments forms' do + create(:comment, commentable: proposal) + visit proposal_path(proposal) + + expect(page).to have_content 'You need to sign in or sign up to comment' + within('#comments') do + expect(page).to_not have_content 'Write a comment' + expect(page).to_not have_content 'Reply' + expect(page).to_not have_css('form') + end + end + end + + scenario 'Create', :js do + login_as(user) + visit proposal_path(proposal) + + fill_in "comment-body-proposal_#{proposal.id}", with: 'Have you thought about...?' + click_button 'Publish comment' + + within "#comments" do + expect(page).to have_content 'Have you thought about...?' + end + end + + scenario 'Errors on create', :js do + login_as(user) + visit proposal_path(proposal) + + click_button 'Publish comment' + + expect(page).to have_content "Can't be blank" + end + + scenario 'Reply', :js do + citizen = create(:user, username: 'Ana') + manuela = create(:user, username: 'Manuela') + comment = create(:comment, commentable: proposal, user: citizen) + + login_as(manuela) + visit proposal_path(proposal) + + click_link "Reply" + + within "#js-comment-form-comment_#{comment.id}" do + fill_in "comment-body-comment_#{comment.id}", with: 'It will be done next week.' + click_button 'Publish reply' + end + + within "#comment_#{comment.id}" do + expect(page).to have_content 'It will be done next week.' + end + + expect(page).to_not have_selector("#js-comment-form-comment_#{comment.id}", visible: true) + end + + scenario 'Errors on reply', :js do + comment = create(:comment, commentable: proposal, user: user) + + login_as(user) + visit proposal_path(proposal) + + click_link "Reply" + + within "#js-comment-form-comment_#{comment.id}" do + click_button 'Publish reply' + expect(page).to have_content "Can't be blank" + end + + end + + scenario "N replies", :js do + parent = create(:comment, commentable: proposal) + + 7.times do + create(:comment, commentable: proposal, parent: parent) + parent = parent.children.first + end + + visit proposal_path(proposal) + expect(page).to have_css(".comment.comment.comment.comment.comment.comment.comment.comment") + end + + scenario "Flagging as inappropriate", :js do + comment = create(:comment, commentable: proposal) + + login_as(user) + visit proposal_path(proposal) + + within "#comment_#{comment.id}" do + page.find("#flag-expand-comment-#{comment.id}").click + page.find("#flag-comment-#{comment.id}").click + + expect(page).to have_css("#unflag-expand-comment-#{comment.id}") + end + + expect(Flag.flagged?(user, comment)).to be + end + + scenario "Undoing flagging as inappropriate", :js do + comment = create(:comment, commentable: proposal) + Flag.flag(user, comment) + + login_as(user) + visit proposal_path(proposal) + + within "#comment_#{comment.id}" do + page.find("#unflag-expand-comment-#{comment.id}").click + page.find("#unflag-comment-#{comment.id}").click + + expect(page).to have_css("#flag-expand-comment-#{comment.id}") + end + + expect(Flag.flagged?(user, comment)).to_not be + end + + scenario "Flagging turbolinks sanity check", :js do + proposal = create(:proposal, title: "Should we change the world?") + comment = create(:comment, commentable: proposal) + + login_as(user) + visit proposals_path + click_link "Should we change the world?" + + within "#comment_#{comment.id}" do + page.find("#flag-expand-comment-#{comment.id}").click + expect(page).to have_selector("#flag-comment-#{comment.id}") + end + end + + feature "Moderators" do + scenario "can create comment as a moderator", :js do + moderator = create(:moderator) + + login_as(moderator.user) + visit proposal_path(proposal) + + fill_in "comment-body-proposal_#{proposal.id}", with: "I am moderating!" + check "comment-as-moderator-proposal_#{proposal.id}" + click_button "Publish comment" + + within "#comments" do + expect(page).to have_content "I am moderating!" + expect(page).to have_content "Moderator ##{moderator.id}" + expect(page).to have_css "p.is-moderator" + expect(page).to have_css "img.moderator-avatar" + end + end + + scenario "can create reply as a moderator", :js do + citizen = create(:user, username: "Ana") + manuela = create(:user, username: "Manuela") + moderator = create(:moderator, user: manuela) + comment = create(:comment, commentable: proposal, user: citizen) + + login_as(manuela) + visit proposal_path(proposal) + + click_link "Reply" + + within "#js-comment-form-comment_#{comment.id}" do + fill_in "comment-body-comment_#{comment.id}", with: "I am moderating!" + check "comment-as-moderator-comment_#{comment.id}" + click_button 'Publish reply' + end + + within "#comment_#{comment.id}" do + expect(page).to have_content "I am moderating!" + expect(page).to have_content "Moderator ##{moderator.id}" + expect(page).to have_css "p.is-moderator" + expect(page).to have_css "img.moderator-avatar" + end + + expect(page).to_not have_selector("#js-comment-form-comment_#{comment.id}", visible: true) + end + + scenario "can not comment as an administrator" do + moderator = create(:moderator) + + login_as(moderator.user) + visit proposal_path(proposal) + + expect(page).to_not have_content "Comment as administrator" + end + end + + feature "Administrators" do + scenario "can create comment as an administrator", :js do + admin = create(:administrator) + + login_as(admin.user) + visit proposal_path(proposal) + + fill_in "comment-body-proposal_#{proposal.id}", with: "I am your Admin!" + check "comment-as-administrator-proposal_#{proposal.id}" + click_button "Publish comment" + + within "#comments" do + expect(page).to have_content "I am your Admin!" + expect(page).to have_content "Administrator ##{admin.id}" + expect(page).to have_css "p.is-admin" + expect(page).to have_css "img.admin-avatar" + end + end + + scenario "can create reply as an administrator", :js do + citizen = create(:user, username: "Ana") + manuela = create(:user, username: "Manuela") + admin = create(:administrator, user: manuela) + comment = create(:comment, commentable: proposal, user: citizen) + + login_as(manuela) + visit proposal_path(proposal) + + click_link "Reply" + + within "#js-comment-form-comment_#{comment.id}" do + fill_in "comment-body-comment_#{comment.id}", with: "Top of the world!" + check "comment-as-administrator-comment_#{comment.id}" + click_button 'Publish reply' + end + + within "#comment_#{comment.id}" do + expect(page).to have_content "Top of the world!" + expect(page).to have_content "Administrator ##{admin.id}" + expect(page).to have_css "p.is-admin" + expect(page).to have_css "img.admin-avatar" + end + + expect(page).to_not have_selector("#js-comment-form-comment_#{comment.id}", visible: true) + end + + scenario "can not comment as a moderator" do + admin = create(:administrator) + + login_as(admin.user) + visit proposal_path(proposal) + + expect(page).to_not have_content "Comment as moderator" + end + end + + feature 'Voting comments' do + + background do + @manuela = create(:user, verified_at: Time.now) + @pablo = create(:user) + @proposal = create(:proposal) + @comment = create(:comment, commentable: @proposal) + + login_as(@manuela) + end + + scenario 'Show' do + create(:vote, voter: @manuela, votable: @comment, vote_flag: true) + create(:vote, voter: @pablo, votable: @comment, vote_flag: false) + + visit proposal_path(@proposal) + + within("#comment_#{@comment.id}_votes") do + within(".in_favor") do + expect(page).to have_content "1" + end + + within(".against") do + expect(page).to have_content "1" + end + + expect(page).to have_content "2 votes" + end + end + + scenario 'Create', :js do + visit proposal_path(@proposal) + + within("#comment_#{@comment.id}_votes") do + find(".in_favor a").click + + within(".in_favor") do + expect(page).to have_content "1" + end + + within(".against") do + expect(page).to have_content "0" + end + + expect(page).to have_content "1 vote" + end + end + + scenario 'Update', :js do + visit proposal_path(@proposal) + + within("#comment_#{@comment.id}_votes") do + find('.in_favor a').click + find('.against a').click + + within('.in_favor') do + expect(page).to have_content "0" + end + + within('.against') do + expect(page).to have_content "1" + end + + expect(page).to have_content "1 vote" + end + end + + scenario 'Trying to vote multiple times', :js do + visit proposal_path(@proposal) + + within("#comment_#{@comment.id}_votes") do + find('.in_favor a').click + find('.in_favor a').click + + within('.in_favor') do + expect(page).to have_content "1" + end + + within('.against') do + expect(page).to have_content "0" + end + + expect(page).to have_content "1 vote" + end + end + end + +end diff --git a/spec/features/debates_spec.rb b/spec/features/debates_spec.rb index 84d7413e0..934a602c9 100644 --- a/spec/features/debates_spec.rb +++ b/spec/features/debates_spec.rb @@ -55,34 +55,20 @@ feature 'Debates' do login_as(author) visit new_debate_path - fill_in 'debate_title', with: 'Acabar con los desahucios' - fill_in 'debate_description', with: 'Esto es un tema muy importante porque...' + fill_in 'debate_title', with: 'End evictions' + fill_in 'debate_description', with: 'This is very important because...' fill_in 'debate_captcha', with: correct_captcha_text check 'debate_terms_of_service' click_button 'Start a debate' expect(page).to have_content 'Debate was successfully created.' - expect(page).to have_content 'Acabar con los desahucios' - expect(page).to have_content 'Esto es un tema muy importante porque...' + expect(page).to have_content 'End evictions' + expect(page).to have_content 'This is very important because...' expect(page).to have_content author.name expect(page).to have_content I18n.l(Debate.last.created_at.to_date) end - scenario 'CKEditor is present before & after turbolinks update page', :js do - author = create(:user) - login_as(author) - - visit new_debate_path - - expect(page).to have_css "#cke_debate_description" - - click_link 'Debates' - click_link 'Start a debate' - - expect(page).to have_css "#cke_debate_description" - end - scenario 'Captcha is required for debate creation' do login_as(create(:user)) @@ -152,6 +138,48 @@ feature 'Debates' do expect(page.html).to_not include '<p>This is' end + scenario 'Autolinking is applied to description' do + author = create(:user) + login_as(author) + + visit new_debate_path + fill_in 'debate_title', with: 'Testing auto link' + fill_in 'debate_description', with: 'This is a link www.example.org
' + fill_in 'debate_captcha', with: correct_captcha_text + check 'debate_terms_of_service' + + click_button 'Start a debate' + + expect(page).to have_content 'Debate was successfully created.' + expect(page).to have_content 'Testing auto link' + expect(page).to have_link('www.example.org', href: 'http://www.example.org') + end + + scenario 'JS injection is prevented but autolinking is respected' do + author = create(:user) + login_as(author) + + visit new_debate_path + fill_in 'debate_title', with: 'Testing auto link' + fill_in 'debate_description', with: " click me http://example.org" + fill_in 'debate_captcha', with: correct_captcha_text + check 'debate_terms_of_service' + + click_button 'Start a debate' + + expect(page).to have_content 'Debate was successfully created.' + expect(page).to have_content 'Testing auto link' + expect(page).to have_link('http://example.org', href: 'http://example.org') + expect(page).not_to have_link('click me') + expect(page.html).to_not include "" + + click_link 'Edit' + + expect(current_path).to eq edit_debate_path(Debate.last) + expect(page).not_to have_link('click me') + expect(page.html).to_not include "" + end + context 'Tagging debates' do let(:author) { create(:user) } @@ -209,7 +237,8 @@ feature 'Debates' do login_as(create(:user)) visit edit_debate_path(debate) - expect(current_path).to eq(root_path) + expect(current_path).not_to eq(edit_debate_path(debate)) + expect(current_path).to eq(proposals_path) expect(page).to have_content 'not authorized' end @@ -221,7 +250,8 @@ feature 'Debates' do visit edit_debate_path(debate) - expect(current_path).to eq(root_path) + expect(current_path).not_to eq(edit_debate_path(debate)) + expect(current_path).to eq(proposals_path) expect(page).to have_content 'not authorized' end @@ -378,7 +408,9 @@ feature 'Debates' do visit debates_path select 'most active', from: 'order-selector' - within '#debates.js-order-hot-score' do + expect(page).to have_selector('.js-order-selector[data-order="hot_score"]') + + within '#debates' do expect('Best').to appear_before('Medium') expect('Medium').to appear_before('Worst') end @@ -395,7 +427,9 @@ feature 'Debates' do visit debates_path select 'most commented', from: 'order-selector' - within '#debates.js-order-most-commented' do + expect(page).to have_selector('.js-order-selector[data-order="most_commented"]') + + within '#debates' do expect('Best').to appear_before('Medium') expect('Medium').to appear_before('Worst') end @@ -412,7 +446,9 @@ feature 'Debates' do visit debates_path select 'newest', from: 'order-selector' - within '#debates.js-order-created-at' do + expect(page).to have_selector('.js-order-selector[data-order="created_at"]') + + within '#debates' do expect('Best').to appear_before('Medium') expect('Medium').to appear_before('Worst') end @@ -426,13 +462,15 @@ feature 'Debates' do visit debates_path select 'random', from: 'order-selector' - debates_first_time = find("#debates.js-order-random").text + expect(page).to have_selector('.js-order-selector[data-order="random"]') + debates_first_time = find("#debates").text select 'most commented', from: 'order-selector' - expect(page).to have_selector('#debates.js-order-most-commented') + expect(page).to have_selector('.js-order-selector[data-order="most_commented"]') select 'random', from: 'order-selector' - debates_second_time = find("#debates.js-order-random").text + expect(page).to have_selector('.js-order-selector[data-order="random"]') + debates_second_time = find("#debates").text expect(debates_first_time).to_not eq(debates_second_time) expect(current_url).to include('page=1') diff --git a/spec/features/emails_spec.rb b/spec/features/emails_spec.rb index b6a9d1db5..1ce4b529f 100644 --- a/spec/features/emails_spec.rb +++ b/spec/features/emails_spec.rb @@ -26,7 +26,7 @@ feature 'Emails' do context 'Debate comments' do scenario "Send email on debate comment", :js do - user = create(:user, email_on_debate_comment: true) + user = create(:user, email_on_comment: true) debate = create(:debate, author: user) comment_on(debate) @@ -37,7 +37,7 @@ feature 'Emails' do end scenario 'Do not send email about own debate comments', :js do - user = create(:user, email_on_debate_comment: true) + user = create(:user, email_on_comment: true) debate = create(:debate, author: user) comment_on(debate, user) @@ -45,7 +45,7 @@ feature 'Emails' do end scenario 'Do not send email about debate comment unless set in preferences', :js do - user = create(:user, email_on_debate_comment: false) + user = create(:user, email_on_comment: false) debate = create(:debate, author: user) comment_on(debate) diff --git a/spec/features/highlights_spec.rb b/spec/features/highlights_spec.rb new file mode 100644 index 000000000..33ade0063 --- /dev/null +++ b/spec/features/highlights_spec.rb @@ -0,0 +1,34 @@ +require 'rails_helper' + +feature "Highlights" do + + scenario 'Debates and proposals order by hot_score' do + create(:debate, title: 'best debate 100').update_column(:hot_score, 100) + create(:debate, title: 'worst debate 50').update_column(:hot_score, 50) + create(:debate, title: 'medium debate 70').update_column(:hot_score, 70) + + create(:proposal, title: 'best proposal 90').update_column(:hot_score, 90) + create(:proposal, title: 'worst proposal 60').update_column(:hot_score, 60) + create(:proposal, title: 'medium proposal 80').update_column(:hot_score, 80) + + login_as(create(:user)) + + visit highlights_path + + expect('best debate 100').to appear_before('best proposal 90') + expect('best proposal 90').to appear_before('medium proposal 80') + expect('medium proposal 80').to appear_before('medium debate 70') + expect('medium debate 70').to appear_before('worst proposal 60') + expect('worst proposal 60').to appear_before('worst debate 50') + end + + scenario 'create debate and create proposal links' do + login_as(create(:user)) + + visit highlights_path + + expect(page).to have_link("Start a proposal") + expect(page).to have_link("Start a debate") + end + +end \ No newline at end of file diff --git a/spec/features/home_spec.rb b/spec/features/home_spec.rb index ff87a294b..2c99a7083 100644 --- a/spec/features/home_spec.rb +++ b/spec/features/home_spec.rb @@ -2,30 +2,21 @@ require 'rails_helper' feature "Home" do - scenario 'featured debates' do - featured_debates = [create(:debate), create(:debate), create(:debate)] + feature "For not logged users" do + scenario 'Welcome message' do + visit root_path - visit root_path - - expect(page).to have_selector('#featured-debates .debate-featured', count: 3) - featured_debates.each do |debate| - within('#featured-debates') do - expect(page).to have_content debate.title - expect(page).to have_css("a[href='#{debate_path(debate)}']", text: debate.description) - end + expect(page).to have_content "we open this digital Puerta del Sol" end - end - scenario "Order by confidence score" do - create(:debate, confidence_score: 100, title: 'best') - create(:debate, confidence_score: -20, title: 'worst') - create(:debate, confidence_score: 50, title: 'medium') + feature "For signed in users" do + scenario 'Redirect to proposals' do + login_as(create(:user)) + visit root_path - visit root_path - - expect('best').to appear_before('medium') - expect('medium').to appear_before('worst') + expect(current_path).to eq proposals_path + end end end diff --git a/spec/features/moderation/proposals_spec.rb b/spec/features/moderation/proposals_spec.rb new file mode 100644 index 000000000..50572c49e --- /dev/null +++ b/spec/features/moderation/proposals_spec.rb @@ -0,0 +1,177 @@ +require 'rails_helper' + +feature 'Moderate proposals' do + + scenario 'Hide', :js do + citizen = create(:user) + moderator = create(:moderator) + + proposal = create(:proposal) + + login_as(moderator.user) + visit proposal_path(proposal) + + within("#proposal_#{proposal.id}") do + click_link 'Hide' + end + + expect(page).to have_css("#proposal_#{proposal.id}.faded") + + login_as(citizen) + visit proposals_path + + expect(page).to have_css('.proposal', count: 0) + end + + scenario 'Can not hide own proposal' do + moderator = create(:moderator) + proposal = create(:proposal, author: moderator.user) + + login_as(moderator.user) + visit proposal_path(proposal) + + within("#proposal_#{proposal.id}") do + expect(page).to_not have_link('Hide') + expect(page).to_not have_link('Block author') + end + end + + feature '/moderation/ screen' do + + background do + moderator = create(:moderator) + login_as(moderator.user) + end + + feature 'moderate in bulk' do + feature "When a proposal has been selected for moderation" do + background do + @proposal = create(:proposal) + visit moderation_proposals_path + + within("#proposal_#{@proposal.id}") do + check "proposal_#{@proposal.id}_check" + end + + expect(page).to_not have_css("proposal_#{@proposal.id}") + end + + scenario 'Hide the proposal' do + click_on "Hide proposals" + expect(page).to_not have_css("proposal_#{@proposal.id}") + expect(@proposal.reload).to be_hidden + expect(@proposal.author).to_not be_hidden + end + + scenario 'Block the author' do + click_on "Block authors" + expect(page).to_not have_css("proposal_#{@proposal.id}") + expect(@proposal.reload).to be_hidden + expect(@proposal.author).to be_hidden + end + + scenario 'Ignore the proposal' do + click_on "Ignore flags" + expect(page).to_not have_css("proposal_#{@proposal.id}") + expect(@proposal.reload).to be_ignored_flag + expect(@proposal.reload).to_not be_hidden + expect(@proposal.author).to_not be_hidden + end + end + + scenario "select all/none", :js do + create_list(:proposal, 20) + + visit moderation_proposals_path + + within('.js-check') { click_on 'All' } + + all('input[type=checkbox]').each do |checkbox| + expect(checkbox).to be_checked + end + + within('.js-check') { click_on 'None' } + + all('input[type=checkbox]').each do |checkbox| + expect(checkbox).to_not be_checked + end + end + + scenario "remembering page, filter and order" do + create_list(:proposal, 55) + + visit moderation_proposals_path(filter: 'all', page: '2', order: 'created_at') + + click_on "Ignore flags" + + expect(page).to have_selector('.js-order-selector[data-order="created_at"]') + + expect(current_url).to include('filter=all') + expect(current_url).to include('page=2') + expect(current_url).to include('order=created_at') + end + end + + scenario "Current filter is properly highlighted" do + visit moderation_proposals_path + expect(page).to_not have_link('Pending') + expect(page).to have_link('All') + expect(page).to have_link('Ignored') + + visit moderation_proposals_path(filter: 'all') + within('.sub-nav') do + expect(page).to_not have_link('All') + expect(page).to have_link('Pending') + expect(page).to have_link('Ignored') + end + + visit moderation_proposals_path(filter: 'pending_flag_review') + within('.sub-nav') do + expect(page).to have_link('All') + expect(page).to_not have_link('Pending') + expect(page).to have_link('Ignored') + end + + visit moderation_proposals_path(filter: 'with_ignored_flag') + within('.sub-nav') do + expect(page).to have_link('All') + expect(page).to have_link('Pending') + expect(page).to_not have_link('Ignored') + end + end + + scenario "Filtering proposals" do + create(:proposal, title: "Pending proposal") + create(:proposal, :hidden, title: "Hidden proposal") + create(:proposal, :with_ignored_flag, title: "Ignored proposal") + + visit moderation_proposals_path(filter: 'all') + expect(page).to have_content('Pending proposal') + expect(page).to_not have_content('Hidden proposal') + expect(page).to have_content('Ignored proposal') + + visit moderation_proposals_path(filter: 'pending_flag_review') + expect(page).to have_content('Pending proposal') + expect(page).to_not have_content('Hidden proposal') + expect(page).to_not have_content('Ignored proposal') + + visit moderation_proposals_path(filter: 'with_ignored_flag') + expect(page).to_not have_content('Pending proposal') + expect(page).to_not have_content('Hidden proposal') + expect(page).to have_content('Ignored proposal') + end + + scenario "sorting proposals" do + create(:proposal, title: "Flagged proposal", created_at: Time.now - 1.day, flags_count: 5) + create(:proposal, title: "Newer proposal", created_at: Time.now) + + visit moderation_proposals_path(order: 'created_at') + + expect("Newer proposal").to appear_before("Flagged proposal") + + visit moderation_proposals_path(order: 'flags') + + expect("Flagged proposal").to appear_before("Newer proposal") + end + end +end diff --git a/spec/features/moderation_spec.rb b/spec/features/moderation_spec.rb index 78df5e56f..9cb2ff19d 100644 --- a/spec/features/moderation_spec.rb +++ b/spec/features/moderation_spec.rb @@ -10,7 +10,8 @@ feature 'Admin' do expect(page).to_not have_link("Moderation") visit moderation_root_path - expect(current_path).to eq(root_path) + expect(current_path).not_to eq(moderation_root_path) + expect(current_path).to eq(proposals_path) expect(page).to have_content "not authorized" end diff --git a/spec/features/proposals_spec.rb b/spec/features/proposals_spec.rb new file mode 100644 index 000000000..a968cc9ea --- /dev/null +++ b/spec/features/proposals_spec.rb @@ -0,0 +1,599 @@ +require 'rails_helper' + +feature 'Proposals' do + + scenario 'Index' do + proposal = [create(:proposal), create(:proposal), create(:proposal)] + + visit proposals_path + + expect(page).to have_selector('#proposals .proposal', count: 3) + proposal.each do |proposal| + within('#proposals') do + expect(page).to have_content proposal.title + expect(page).to have_css("a[href='#{proposal_path(proposal)}']", text: proposal.summary) + end + end + end + + scenario 'Paginated Index' do + per_page = Kaminari.config.default_per_page + (per_page + 2).times { create(:proposal) } + + visit proposals_path + + expect(page).to have_selector('#proposals .proposal', count: per_page) + + within("ul.pagination") do + expect(page).to have_content("1") + expect(page).to have_content("2") + expect(page).to_not have_content("3") + click_link "Next", exact: false + end + + expect(page).to have_selector('#proposals .proposal', count: 2) + end + + scenario 'Show' do + proposal = create(:proposal) + + visit proposal_path(proposal) + + expect(page).to have_content proposal.title + expect(page).to have_content proposal.code + expect(page).to have_content "Proposal question" + expect(page).to have_content "Proposal description" + expect(page).to have_content "http://external_documention.es" + expect(page).to have_content proposal.author.name + expect(page).to have_content I18n.l(proposal.created_at.to_date) + expect(page).to have_selector(avatar(proposal.author.name)) + + within('.social-share-button') do + expect(page.all('a').count).to be(3) # Twitter, Facebook, Google+ + end + end + + scenario 'Create' do + author = create(:user) + login_as(author) + + visit new_proposal_path + fill_in 'proposal_title', with: 'Help refugees' + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: 'This is very important because...' + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_video_url', with: 'http://youtube.com' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + check 'proposal_terms_of_service' + + click_button 'Start a proposal' + + expect(page).to have_content 'Proposal was successfully created.' + expect(page).to have_content 'Help refugees' + expect(page).to have_content '¿Would you like to give assistance to war refugees?' + expect(page).to have_content 'In summary, what we want is...' + expect(page).to have_content 'This is very important because...' + expect(page).to have_content 'http://rescue.org/refugees' + expect(page).to have_content 'http://youtube.com' + expect(page).to have_content author.name + expect(page).to have_content I18n.l(Proposal.last.created_at.to_date) + end + + scenario 'Responsible name is stored for anonymous users' do + author = create(:user) + login_as(author) + + visit new_proposal_path + fill_in 'proposal_title', with: 'Help refugees' + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: 'This is very important because...' + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + check 'proposal_terms_of_service' + + click_button 'Start a proposal' + + expect(page).to have_content 'Proposal was successfully created.' + expect(Proposal.last.responsible_name).to eq('Isabel Garcia') + end + + scenario 'Responsible name field is not shown for verified users' do + author = create(:user, :level_two) + login_as(author) + + visit new_proposal_path + expect(page).to_not have_selector('#proposal_responsible_name') + + fill_in 'proposal_title', with: 'Help refugees' + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: 'This is very important because...' + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_captcha', with: correct_captcha_text + check 'proposal_terms_of_service' + + click_button 'Start a proposal' + + expect(page).to have_content 'Proposal was successfully created.' + end + + scenario 'Captcha is required for proposal creation' do + login_as(create(:user)) + + visit new_proposal_path + fill_in 'proposal_title', with: "Great title" + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: 'Very important issue...' + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: "wrongText!" + check 'proposal_terms_of_service' + + click_button "Start a proposal" + + expect(page).to_not have_content "Proposal was successfully created." + expect(page).to have_content "1 error" + + fill_in 'proposal_captcha', with: correct_captcha_text + click_button "Start a proposal" + + expect(page).to have_content "Proposal was successfully created." + end + + scenario 'Failed creation goes back to new showing featured tags' do + featured_tag = create(:tag, :featured) + tag = create(:tag) + login_as(create(:user)) + + visit new_proposal_path + fill_in 'proposal_title', with: "" + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: 'Very important issue...' + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + check 'proposal_terms_of_service' + + click_button "Start a proposal" + + expect(page).to_not have_content "Proposal was successfully created." + expect(page).to have_content "error" + within(".tags") do + expect(page).to have_content featured_tag.name + expect(page).to_not have_content tag.name + end + end + + scenario 'Errors on create' do + author = create(:user) + login_as(author) + + visit new_proposal_path + click_button 'Start a proposal' + expect(page).to have_content error_message + end + + scenario 'JS injection is prevented but safe html is respected' do + author = create(:user) + login_as(author) + + visit new_proposal_path + fill_in 'proposal_title', with: 'Testing an attack' + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: 'This is
' + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + check 'proposal_terms_of_service' + + click_button 'Start a proposal' + + expect(page).to have_content 'Proposal was successfully created.' + expect(page).to have_content 'Testing an attack' + expect(page.html).to include 'This is alert("an attack");
' + expect(page.html).to_not include '' + expect(page.html).to_not include '<p>This is' + end + + scenario 'Autolinking is applied to description' do + author = create(:user) + login_as(author) + + visit new_proposal_path + fill_in 'proposal_title', with: 'Testing auto link' + fill_in 'proposal_question', with: 'Should I stay or should I go?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: 'This is a link www.example.org
' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + check 'proposal_terms_of_service' + + click_button 'Start a proposal' + + expect(page).to have_content 'Proposal was successfully created.' + expect(page).to have_content 'Testing auto link' + expect(page).to have_link('www.example.org', href: 'http://www.example.org') + end + + scenario 'JS injection is prevented but autolinking is respected' do + author = create(:user) + login_as(author) + + visit new_proposal_path + fill_in 'proposal_title', with: 'Testing auto link' + fill_in 'proposal_question', with: 'Should I stay or should I go?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: " click me http://example.org" + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + check 'proposal_terms_of_service' + + click_button 'Start a proposal' + + expect(page).to have_content 'Proposal was successfully created.' + expect(page).to have_content 'Testing auto link' + expect(page).to have_link('http://example.org', href: 'http://example.org') + expect(page).not_to have_link('click me') + expect(page.html).to_not include "" + + click_link 'Edit' + + expect(current_path).to eq edit_proposal_path(Proposal.last) + expect(page).not_to have_link('click me') + expect(page.html).to_not include "" + end + + context 'Tagging proposals' do + let(:author) { create(:user) } + + background do + login_as(author) + end + + scenario 'using featured tags', :js do + ['Medio Ambiente', 'Ciencia'].each do |tag_name| + create(:tag, :featured, name: tag_name) + end + + visit new_proposal_path + + fill_in 'proposal_title', with: 'A test with enough characters' + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in_ckeditor 'proposal_description', with: 'A description with enough characters' + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + check 'proposal_terms_of_service' + + ['Medio Ambiente', 'Ciencia'].each do |tag_name| + find('.js-add-tag-link', text: tag_name).click + end + + click_button 'Start a proposal' + + expect(page).to have_content 'Proposal was successfully created.' + ['Medio Ambiente', 'Ciencia'].each do |tag_name| + expect(page).to have_content tag_name + end + end + + scenario 'using dangerous strings' do + visit new_proposal_path + + fill_in 'proposal_title', with: 'A test of dangerous strings' + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'In summary, what we want is...' + fill_in 'proposal_description', with: 'A description suitable for this test' + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + check 'proposal_terms_of_service' + + fill_in 'proposal_tag_list', with: 'user_id=1, &a=3, ' + + click_button 'Start a proposal' + + expect(page).to have_content 'Proposal was successfully created.' + expect(page).to have_content 'user_id1' + expect(page).to have_content 'a3' + expect(page).to have_content 'scriptalert("hey");script' + expect(page.html).to_not include 'user_id=1, &a=3, ' + end + end + + scenario 'Update should not be posible if logged user is not the author' do + proposal = create(:proposal) + expect(proposal).to be_editable + login_as(create(:user)) + + visit edit_proposal_path(proposal) + expect(current_path).not_to eq(edit_proposal_path(proposal)) + expect(current_path).to eq(proposals_path) + expect(page).to have_content 'not authorized' + end + + scenario 'Update should not be posible if proposal is not editable' do + proposal = create(:proposal) + Setting.find_by(key: "max_votes_for_proposal_edit").update(value: 10) + 11.times { create(:vote, votable: proposal) } + + expect(proposal).to_not be_editable + + login_as(proposal.author) + visit edit_proposal_path(proposal) + + expect(current_path).not_to eq(edit_proposal_path(proposal)) + expect(current_path).to eq(proposals_path) + expect(page).to have_content 'not authorized' + end + + scenario 'Update should be posible for the author of an editable proposal' do + proposal = create(:proposal) + login_as(proposal.author) + + visit edit_proposal_path(proposal) + expect(current_path).to eq(edit_proposal_path(proposal)) + + fill_in 'proposal_title', with: "End child poverty" + fill_in 'proposal_question', with: '¿Would you like to give assistance to war refugees?' + fill_in 'proposal_summary', with: 'Basically...' + fill_in 'proposal_description', with: "Let's do something to end child poverty" + fill_in 'proposal_external_url', with: 'http://rescue.org/refugees' + fill_in 'proposal_responsible_name', with: 'Isabel Garcia' + fill_in 'proposal_captcha', with: correct_captcha_text + + click_button "Save changes" + + expect(page).to have_content "Proposal was successfully updated." + expect(page).to have_content "Basically..." + expect(page).to have_content "End child poverty" + expect(page).to have_content "Let's do something to end child poverty" + end + + scenario 'Errors on update' do + proposal = create(:proposal) + login_as(proposal.author) + + visit edit_proposal_path(proposal) + fill_in 'proposal_title', with: "" + click_button "Save changes" + + expect(page).to have_content error_message + end + + scenario 'Captcha is required to update a proposal' do + proposal = create(:proposal) + login_as(proposal.author) + + visit edit_proposal_path(proposal) + expect(current_path).to eq(edit_proposal_path(proposal)) + + fill_in 'proposal_title', with: "New cool title" + fill_in 'proposal_captcha', with: "wrong!" + click_button "Save changes" + + expect(page).to_not have_content "Proposal was successfully updated." + expect(page).to have_content "error" + + fill_in 'proposal_captcha', with: correct_captcha_text + click_button "Save changes" + + expect(page).to have_content "Proposal was successfully updated." + end + + scenario 'Failed update goes back to edit showing featured tags' do + proposal = create(:proposal) + featured_tag = create(:tag, :featured) + tag = create(:tag) + login_as(proposal.author) + + visit edit_proposal_path(proposal) + expect(current_path).to eq(edit_proposal_path(proposal)) + + fill_in 'proposal_title', with: "" + fill_in 'proposal_captcha', with: correct_captcha_text + click_button "Save changes" + + expect(page).to_not have_content "Proposal was successfully updated." + expect(page).to have_content "error" + within(".tags") do + expect(page).to have_content featured_tag.name + expect(page).to_not have_content tag.name + end + end + + describe 'Limiting tags shown' do + scenario 'Index page shows up to 5 tags per proposal' do + tag_list = ["Hacienda", "Economía", "Medio Ambiente", "Corrupción", "Fiestas populares", "Prensa", "Huelgas"] + create :proposal, tag_list: tag_list + + visit proposals_path + + within('.proposal .tags') do + expect(page).to have_content '2+' + end + end + + scenario 'Index page shows 3 tags with no plus link' do + tag_list = ["Medio Ambiente", "Corrupción", "Fiestas populares"] + create :proposal, tag_list: tag_list + + visit proposals_path + + within('.proposal .tags') do + tag_list.each do |tag| + expect(page).to have_content tag + end + expect(page).not_to have_content '+' + end + end + end + + feature 'Proposal index order filters' do + + scenario 'Default order is confidence_score', :js do + create(:proposal, title: 'Best proposal').update_column(:confidence_score, 10) + create(:proposal, title: 'Worst proposal').update_column(:confidence_score, 2) + create(:proposal, title: 'Medium proposal').update_column(:confidence_score, 5) + + visit proposals_path + + expect('Best proposal').to appear_before('Medium proposal') + expect('Medium proposal').to appear_before('Worst proposal') + end + + scenario 'Proposals are ordered by hot_score', :js do + create(:proposal, title: 'Best proposal').update_column(:hot_score, 10) + create(:proposal, title: 'Worst proposal').update_column(:hot_score, 2) + create(:proposal, title: 'Medium proposal').update_column(:hot_score, 5) + + visit proposals_path + select 'most active', from: 'order-selector' + + expect(page).to have_selector('.js-order-selector[data-order="hot_score"]') + + within '#proposals' do + expect('Best proposal').to appear_before('Medium proposal') + expect('Medium proposal').to appear_before('Worst proposal') + end + + expect(current_url).to include('order=hot_score') + expect(current_url).to include('page=1') + end + + scenario 'Proposals are ordered by most commented', :js do + create(:proposal, title: 'Best proposal', comments_count: 10) + create(:proposal, title: 'Medium proposal', comments_count: 5) + create(:proposal, title: 'Worst proposal', comments_count: 2) + + visit proposals_path + select 'most commented', from: 'order-selector' + + expect(page).to have_selector('.js-order-selector[data-order="most_commented"]') + + within '#proposals' do + expect('Best proposal').to appear_before('Medium proposal') + expect('Medium proposal').to appear_before('Worst proposal') + end + + expect(current_url).to include('order=most_commented') + expect(current_url).to include('page=1') + end + + scenario 'Proposals are ordered by newest', :js do + create(:proposal, title: 'Best proposal', created_at: Time.now) + create(:proposal, title: 'Medium proposal', created_at: Time.now - 1.hour) + create(:proposal, title: 'Worst proposal', created_at: Time.now - 1.day) + + visit proposals_path + select 'newest', from: 'order-selector' + + expect(page).to have_selector('.js-order-selector[data-order="created_at"]') + + within '#proposals' do + expect('Best proposal').to appear_before('Medium proposal') + expect('Medium proposal').to appear_before('Worst proposal') + end + + expect(current_url).to include('order=created_at') + expect(current_url).to include('page=1') + end + + scenario 'Proposals are ordered randomly', :js do + create_list(:proposal, 12) + visit proposals_path + + select 'random', from: 'order-selector' + expect(page).to have_selector('.js-order-selector[data-order="random"]') + proposals_first_time = find("#proposals").text + + select 'most commented', from: 'order-selector' + expect(page).to have_selector('.js-order-selector[data-order="most_commented"]') + + select 'random', from: 'order-selector' + expect(page).to have_selector('.js-order-selector[data-order="random"]') + proposals_second_time = find("#proposals").text + + expect(proposals_first_time).to_not eq(proposals_second_time) + expect(current_url).to include('page=1') + end + end + + scenario 'proposal index search' do + proposal1 = create(:proposal, title: "Show me what you got") + proposal2 = create(:proposal, title: "Get Schwifty") + proposal3 = create(:proposal) + proposal4 = create(:proposal, description: "Schwifty in here") + proposal5 = create(:proposal, question: "Schwifty in here") + + visit proposals_path + fill_in "search", with: "Schwifty" + click_button "Search" + + expect(current_path).to eq(proposals_path) + + within("#proposals") do + expect(page).to have_css('.proposal', count: 3) + expect(page).to have_content(proposal2.title) + expect(page).to have_content(proposal4.title) + expect(page).to have_content(proposal5.title) + expect(page).to_not have_content(proposal1.title) + expect(page).to_not have_content(proposal3.title) + end + end + + scenario 'Conflictive' do + good_proposal = create(:proposal) + conflictive_proposal = create(:proposal, :conflictive) + + visit proposal_path(conflictive_proposal) + expect(page).to have_content "This proposal has been flag as innapropiate for some users." + + visit proposal_path(good_proposal) + expect(page).to_not have_content "This proposal has been flag as innapropiate for some users." + end + + scenario "Flagging", :js do + user = create(:user) + proposal = create(:proposal) + + login_as(user) + visit proposal_path(proposal) + + within "#proposal_#{proposal.id}" do + page.find("#flag-expand-proposal-#{proposal.id}").click + page.find("#flag-proposal-#{proposal.id}").click + + expect(page).to have_css("#unflag-expand-proposal-#{proposal.id}") + end + + expect(Flag.flagged?(user, proposal)).to be + end + + scenario "Unflagging", :js do + user = create(:user) + proposal = create(:proposal) + Flag.flag(user, proposal) + + login_as(user) + visit proposal_path(proposal) + + within "#proposal_#{proposal.id}" do + page.find("#unflag-expand-proposal-#{proposal.id}").click + page.find("#unflag-proposal-#{proposal.id}").click + + expect(page).to have_css("#flag-expand-proposal-#{proposal.id}") + end + + expect(Flag.flagged?(user, proposal)).to_not be + end +end diff --git a/spec/features/votes_spec.rb b/spec/features/votes_spec.rb index a3cf2e43a..81995397d 100644 --- a/spec/features/votes_spec.rb +++ b/spec/features/votes_spec.rb @@ -10,53 +10,6 @@ feature 'Votes' do end feature 'Debates' do - scenario "Home shows user votes on featured debates" do - debate1 = create(:debate) - debate2 = create(:debate) - debate3 = create(:debate) - create(:vote, voter: @manuela, votable: debate1, vote_flag: true) - create(:vote, voter: @manuela, votable: debate3, vote_flag: false) - - visit root_path - - within("#featured-debates") do - within("#debate_#{debate1.id}_votes") do - within(".in-favor") do - expect(page).to have_css("a.voted") - expect(page).to_not have_css("a.no-voted") - end - - within(".against") do - expect(page).to have_css("a.no-voted") - expect(page).to_not have_css("a.voted") - end - end - - within("#debate_#{debate2.id}_votes") do - within(".in-favor") do - expect(page).to_not have_css("a.voted") - expect(page).to_not have_css("a.no-voted") - end - - within(".against") do - expect(page).to_not have_css("a.no-voted") - expect(page).to_not have_css("a.voted") - end - end - - within("#debate_#{debate3.id}_votes") do - within(".in-favor") do - expect(page).to have_css("a.no-voted") - expect(page).to_not have_css("a.voted") - end - - within(".against") do - expect(page).to have_css("a.voted") - expect(page).to_not have_css("a.no-voted") - end - end - end - end scenario "Index shows user votes on debates" do debate1 = create(:debate) @@ -202,27 +155,6 @@ feature 'Votes' do expect(page).to have_content "1 vote" end - scenario 'Create in featured', :js do - visit root_path - - within("#featured-debates") do - find('.in-favor a').click - - within('.in-favor') do - expect(page).to have_content "100%" - expect(page).to have_css("a.voted") - end - - within('.against') do - expect(page).to have_content "0%" - expect(page).to have_css("a.no-voted") - end - - expect(page).to have_content "1 vote" - end - expect(current_path).to eq(root_path) - end - scenario 'Create in index', :js do visit debates_path @@ -245,87 +177,167 @@ feature 'Votes' do expect(current_path).to eq(debates_path) end end + + scenario 'Not logged user trying to vote', :js do + debate = create(:debate) + + visit "/" + click_link "Logout" + + visit debates_path + within("#debate_#{debate.id}") do + find("div.votes").hover + expect_message_you_need_to_sign_in + end + + visit debate_path(debate) + within("#debate_#{debate.id}") do + find("div.votes").hover + expect_message_you_need_to_sign_in + end + end + + scenario 'Anonymous user trying to vote', :js do + user = create(:user) + debate = create(:debate) + + Setting.find_by(key: "max_ratio_anon_votes_on_debates").update(value: 50) + debate.update(cached_anonymous_votes_total: 520, cached_votes_total: 1000) + + login_as(user) + + visit debates_path + within("#debate_#{debate.id}") do + find("div.votes").hover + expect_message_to_many_anonymous_votes + end + + visit debate_path(debate) + within("#debate_#{debate.id}") do + find("div.votes").hover + expect_message_to_many_anonymous_votes + end + end end - feature 'Comments' do - background do - @debate = create(:debate) - @comment = create(:comment, commentable: @debate) - end + feature 'Proposals' do - scenario 'Show' do - create(:vote, voter: @manuela, votable: @comment, vote_flag: true) - create(:vote, voter: @pablo, votable: @comment, vote_flag: false) + scenario "Index shows user votes on proposals" do + proposal1 = create(:proposal) + proposal2 = create(:proposal) + proposal3 = create(:proposal) + create(:vote, voter: @manuela, votable: proposal1, vote_flag: true) - visit debate_path(@debate) + visit proposals_path - within("#comment_#{@comment.id}_votes") do - within(".in_favor") do - expect(page).to have_content "1" + within("#proposals") do + within("#proposal_#{proposal1.id}_votes") do + expect(page).to have_content "You already supported this proposal!" end - within(".against") do - expect(page).to have_content "1" + within("#proposal_#{proposal2.id}_votes") do + expect(page).to_not have_content "You already supported this proposal!" end - expect(page).to have_content "2 votes" + within("#proposal_#{proposal3.id}_votes") do + expect(page).to_not have_content "You already supported this proposal!" + end end end - scenario 'Create', :js do - visit debate_path(@debate) + feature 'Single proposal' do + background do + @proposal = create(:proposal) + end - within("#comment_#{@comment.id}_votes") do - find(".in_favor a").click + scenario 'Show no votes' do + visit proposal_path(@proposal) + expect(page).to have_content "No supports" + end - within(".in_favor") do - expect(page).to have_content "1" + scenario 'Trying to vote multiple times', :js do + visit proposal_path(@proposal) + + within('.supports') do + find('.in-favor a').click + find('.in-favor a').click + + expect(page).to have_content "1 support" end + end - within(".against") do - expect(page).to have_content "0" + scenario 'Show' do + create(:vote, voter: @manuela, votable: @proposal, vote_flag: true) + create(:vote, voter: @pablo, votable: @proposal, vote_flag: true) + + visit proposal_path(@proposal) + + within('.supports') do + expect(page).to have_content "2 supports" end + end - expect(page).to have_content "1 vote" + scenario 'Create from proposal show', :js do + visit proposal_path(@proposal) + + within('.supports') do + find('.in-favor a').click + + expect(page).to have_content "1 support" + expect(page).to have_content "You already supported this proposal!" + end + end + + scenario 'Create in index', :js do + visit proposals_path + + within("#proposals") do + find('.in-favor a').click + + expect(page).to have_content "1 support" + expect(page).to have_content "You already supported this proposal!" + end + expect(URI.parse(current_url).path).to eq(proposals_path) end end + end - scenario 'Update', :js do - visit debate_path(@debate) + scenario 'Not logged user trying to vote', :js do + proposal = create(:proposal) - within("#comment_#{@comment.id}_votes") do - find('.in_favor a').click - find('.against a').click + visit "/" + click_link "Logout" + expect(page).to have_content "Signed out successfully." - within('.in_favor') do - expect(page).to have_content "0" - end - - within('.against') do - expect(page).to have_content "1" - end - - expect(page).to have_content "1 vote" - end + visit proposals_path + within("#proposal_#{proposal.id}") do + find("div.supports").hover + expect_message_you_need_to_sign_in end - scenario 'Trying to vote multiple times', :js do - visit debate_path(@debate) + visit proposal_path(proposal) + within("#proposal_#{proposal.id}") do + find("div.supports").hover + expect_message_you_need_to_sign_in + end + end - within("#comment_#{@comment.id}_votes") do - find('.in_favor a').click - find('.in_favor a').click + scenario "Anonymous user trying to vote", :js do + user = create(:user) + proposal = create(:proposal) - within('.in_favor') do - expect(page).to have_content "1" - end + login_as(user) + visit proposals_path - within('.against') do - expect(page).to have_content "0" - end + within("#proposal_#{proposal.id}") do + find("div.supports").hover + expect_message_only_verified_can_vote_proposals + end - expect(page).to have_content "1 vote" - end + visit proposal_path(proposal) + within("#proposal_#{proposal.id}") do + find("div.supports").hover + expect_message_only_verified_can_vote_proposals end end end diff --git a/spec/features/welcome_spec.rb b/spec/features/welcome_spec.rb index 2635ef0df..cef26e035 100644 --- a/spec/features/welcome_spec.rb +++ b/spec/features/welcome_spec.rb @@ -15,7 +15,7 @@ feature "Welcome screen" do login_through_form_as(user) - expect(current_path).to eq(root_path) + expect(current_path).to eq(proposals_path) end scenario 'is not shown to organizations' do @@ -23,7 +23,7 @@ feature "Welcome screen" do login_through_form_as(organization.user) - expect(current_path).to eq(root_path) + expect(current_path).to eq(proposals_path) end scenario 'it is not shown to level-2 users' do @@ -31,7 +31,7 @@ feature "Welcome screen" do login_through_form_as(user) - expect(current_path).to eq(root_path) + expect(current_path).to eq(proposals_path) end scenario 'it is not shown to level-3 users' do @@ -39,7 +39,7 @@ feature "Welcome screen" do login_through_form_as(user) - expect(current_path).to eq(root_path) + expect(current_path).to eq(proposals_path) end end diff --git a/spec/helpers/proposals_helper_spec.rb b/spec/helpers/proposals_helper_spec.rb new file mode 100644 index 000000000..983f43572 --- /dev/null +++ b/spec/helpers/proposals_helper_spec.rb @@ -0,0 +1,44 @@ +require 'rails_helper' + +describe ProposalsHelper do + + describe "#progress_bar_percentage" do + it "should be 0 if no votes" do + proposal = create(:proposal) + expect(progress_bar_percentage(proposal)).to eq 0 + 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) + 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) + expect(progress_bar_percentage(proposal)).to eq 100 + end + end + + describe "#supports_percentage" do + it "should be 0 if no votes" do + proposal = create(:proposal) + expect(supports_percentage(proposal)).to eq "0%" + end + + it "should be a between 0.1 from 1 to 0.1% of needed votes" do + proposal = create(:proposal, cached_votes_up: 1) + expect(supports_percentage(proposal)).to eq "0.1%" + 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) + 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) + expect(supports_percentage(proposal)).to eq "100%" + end + end + +end \ No newline at end of file diff --git a/spec/helpers/votes_helper_spec.rb b/spec/helpers/votes_helper_spec.rb new file mode 100644 index 000000000..96be3b3fe --- /dev/null +++ b/spec/helpers/votes_helper_spec.rb @@ -0,0 +1,21 @@ +require 'rails_helper' + +describe VotesHelper do + + describe "#voted_for?" do + it "should return true if voted for a proposal" do + proposal = create(:proposal) + votes = {proposal.id => true} + + expect(voted_for?(votes, proposal)).to eq(true) + end + + it "should return false if not voted for a proposals" do + proposal = create(:proposal) + votes = {proposal.id => nil} + + expect(voted_for?(votes, proposal)).to eq(nil) + end + end + +end \ No newline at end of file diff --git a/spec/models/ability_spec.rb b/spec/models/ability_spec.rb index f106ef14e..a2e6212b5 100644 --- a/spec/models/ability_spec.rb +++ b/spec/models/ability_spec.rb @@ -3,8 +3,17 @@ require 'cancan/matchers' describe Ability do subject(:ability) { Ability.new(user) } - let(:debate) { Debate.new } + let(:debate) { create(:debate) } let(:comment) { create(:comment) } + let(:proposal) { create(:proposal) } + + let(:own_debate) { create(:debate, author: user) } + let(:own_comment) { create(:comment, author: user) } + let(:own_proposal) { create(:proposal, author: user) } + + let(:hidden_debate) { create(:debate, :hidden) } + let(:hidden_comment) { create(:comment, :hidden) } + let(:hidden_proposal) { create(:proposal, :hidden) } describe "Non-logged in user" do let(:user) { nil } @@ -13,6 +22,15 @@ describe Ability do it { should be_able_to(:show, debate) } it { should_not be_able_to(:edit, Debate) } it { should_not be_able_to(:vote, Debate) } + it { should_not be_able_to(:flag, Debate) } + it { should_not be_able_to(:unflag, Debate) } + + it { should be_able_to(:index, Proposal) } + it { should be_able_to(:show, proposal) } + it { should_not be_able_to(:edit, Proposal) } + it { should_not be_able_to(:vote, Proposal) } + it { should_not be_able_to(:flag, Proposal) } + it { should_not be_able_to(:unflag, Proposal) } end describe "Citizen" do @@ -28,8 +46,14 @@ describe Ability do it { should be_able_to(:create, Comment) } it { should be_able_to(:vote, Comment) } + it { should be_able_to(:index, Proposal) } + it { should be_able_to(:show, proposal) } + it { should_not be_able_to(:vote, Proposal) } + it { should_not be_able_to(:comment_as_administrator, debate) } it { should_not be_able_to(:comment_as_moderator, debate) } + it { should_not be_able_to(:comment_as_administrator, proposal) } + it { should_not be_able_to(:comment_as_moderator, proposal) } describe 'flagging content' do it { should be_able_to(:flag, debate) } @@ -38,18 +62,18 @@ describe Ability do it { should be_able_to(:flag, comment) } it { should be_able_to(:unflag, comment) } - describe "own comments" do - let(:own_comment) { create(:comment, author: user) } + it { should be_able_to(:flag, proposal) } + it { should be_able_to(:unflag, proposal) } + describe "own content" do it { should_not be_able_to(:flag, own_comment) } it { should_not be_able_to(:unflag, own_comment) } - end - - describe "own debates" do - let(:own_debate) { create(:debate, author: user) } it { should_not be_able_to(:flag, own_debate) } it { should_not be_able_to(:unflag, own_debate) } + + it { should_not be_able_to(:flag, own_proposal) } + it { should_not be_able_to(:unflag, own_proposal) } end end @@ -60,15 +84,34 @@ describe Ability do end describe "editing debates" do - let(:own_debate) { create(:debate, author: user) } let(:own_debate_non_editable) { create(:debate, author: user) } - before { allow(own_debate_non_editable).to receive(:editable?).and_return(false) } it { should be_able_to(:edit, own_debate) } it { should_not be_able_to(:edit, debate) } # Not his it { should_not be_able_to(:edit, own_debate_non_editable) } end + + describe "editing proposals" do + let(:own_proposal_non_editable) { create(:proposal, author: user) } + before { allow(own_proposal_non_editable).to receive(:editable?).and_return(false) } + + it { should be_able_to(:edit, own_proposal) } + it { should_not be_able_to(:edit, proposal) } # Not his + it { should_not be_able_to(:edit, own_proposal_non_editable) } + end + + describe "when level 2 verified" do + before{ user.update(residence_verified_at: Time.now, confirmed_phone: "1") } + + it { should be_able_to(:vote, Proposal) } + end + + describe "when level 3 verified" do + before{ user.update(verified_at: Time.now) } + + it { should be_able_to(:vote, Proposal) } + end end describe "Organization" do @@ -82,6 +125,10 @@ describe Ability do it { should be_able_to(:show, debate) } it { should_not be_able_to(:vote, debate) } + it { should be_able_to(:index, Proposal) } + it { should be_able_to(:show, proposal) } + it { should_not be_able_to(:vote, Proposal) } + it { should be_able_to(:create, Comment) } it { should_not be_able_to(:vote, Comment) } end @@ -96,6 +143,9 @@ describe Ability do it { should be_able_to(:show, debate) } it { should be_able_to(:vote, debate) } + it { should be_able_to(:index, Proposal) } + it { should be_able_to(:show, proposal) } + it { should be_able_to(:read, Organization) } describe "organizations" do @@ -114,12 +164,9 @@ describe Ability do end describe "hiding, reviewing and restoring" do - let(:own_comment) { create(:comment, author: user) } - let(:own_debate) { create(:debate, author: user) } - let(:hidden_comment) { create(:comment, :hidden) } - let(:hidden_debate) { create(:debate, :hidden) } let(:ignored_comment) { create(:comment, :with_ignored_flag) } let(:ignored_debate) { create(:debate, :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) } @@ -131,6 +178,11 @@ describe Ability do it { should_not be_able_to(:hide, hidden_debate) } it { should_not be_able_to(:hide, own_debate) } + it { should be_able_to(:hide, proposal) } + it { should be_able_to(:hide_in_moderation_screen, proposal) } + it { should_not be_able_to(:hide, hidden_proposal) } + it { should_not be_able_to(:hide, own_proposal) } + it { should be_able_to(:ignore_flag, comment) } it { should_not be_able_to(:ignore_flag, hidden_comment) } it { should_not be_able_to(:ignore_flag, ignored_comment) } @@ -141,15 +193,29 @@ describe Ability do it { should_not be_able_to(:ignore_flag, ignored_debate) } it { should_not be_able_to(:ignore_flag, own_debate) } + it { should be_able_to(:ignore_flag, proposal) } + it { should_not be_able_to(:ignore_flag, hidden_proposal) } + it { should_not be_able_to(:ignore_flag, ignored_proposal) } + it { should_not be_able_to(:ignore_flag, own_proposal) } + + it { should be_able_to(:moderate, proposal) } + it { should_not be_able_to(:moderate, own_proposal) } + it { should_not be_able_to(:hide, user) } it { should be_able_to(:hide, other_user) } + it { should_not be_able_to(:block, user) } + it { should be_able_to(:block, other_user) } + it { should_not be_able_to(:restore, comment) } it { should_not be_able_to(:restore, debate) } + it { should_not be_able_to(:restore, proposal) } it { should_not be_able_to(:restore, other_user) } it { should be_able_to(:comment_as_moderator, debate) } + it { should be_able_to(:comment_as_moderator, proposal) } it { should_not be_able_to(:comment_as_administrator, debate) } + it { should_not be_able_to(:comment_as_administrator, proposal) } end end @@ -160,32 +226,37 @@ describe Ability do let(:other_user) { create(:user) } let(:hidden_user) { create(:user, :hidden) } - let(:hidden_debate) { create(:debate, :hidden) } - let(:hidden_comment) { create(:comment, :hidden) } - let(:own_debate) { create(:debate, author: user)} - let(:own_comment) { create(:comment, author: user)} - it { should be_able_to(:index, Debate) } it { should be_able_to(:show, debate) } it { should be_able_to(:vote, debate) } + it { should be_able_to(:index, Proposal) } + it { should be_able_to(:show, proposal) } + it { should_not be_able_to(:restore, comment) } it { should_not be_able_to(:restore, debate) } + it { should_not be_able_to(:restore, proposal) } it { should_not be_able_to(:restore, other_user) } it { should be_able_to(:restore, hidden_comment) } it { should be_able_to(:restore, hidden_debate) } + it { should be_able_to(:restore, hidden_proposal) } it { should be_able_to(:restore, hidden_user) } it { should_not be_able_to(:confirm_hide, comment) } it { should_not be_able_to(:confirm_hide, debate) } + it { should_not be_able_to(:confirm_hide, proposal) } it { should_not be_able_to(:confirm_hide, other_user) } it { should be_able_to(:confirm_hide, hidden_comment) } it { should be_able_to(:confirm_hide, hidden_debate) } + it { should be_able_to(:confirm_hide, hidden_proposal) } it { should be_able_to(:confirm_hide, hidden_user) } it { should be_able_to(:comment_as_administrator, debate) } it { should_not be_able_to(:comment_as_moderator, debate) } + + it { should be_able_to(:comment_as_administrator, proposal) } + it { should_not be_able_to(:comment_as_moderator, proposal) } end end diff --git a/spec/models/debate_spec.rb b/spec/models/debate_spec.rb index f9af3c149..907bcef5b 100644 --- a/spec/models/debate_spec.rb +++ b/spec/models/debate_spec.rb @@ -78,6 +78,42 @@ describe Debate do end end + describe "#votable_by?" do + let(:debate) { create(:debate) } + + before(:each) do + Setting.find_by(key: "max_ratio_anon_votes_on_debates").update(value: 50) + end + + it "should be true for level two verified users" do + user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111") + expect(debate.votable_by?(user)).to be true + end + + it "should be true for level three verified users" do + user = create(:user, verified_at: Time.now) + expect(debate.votable_by?(user)).to be true + end + + it "should be true for anonymous users if allowed anonymous votes" do + debate.update(cached_anonymous_votes_total: 420, cached_votes_total: 1000) + user = create(:user) + expect(debate.votable_by?(user)).to be true + end + + it "should be true for anonymous users if less than 100 votes" do + debate.update(cached_anonymous_votes_total: 90, cached_votes_total: 92) + user = create(:user) + expect(debate.votable_by?(user)).to be true + end + + it "should be false for anonymous users if too many anonymous votes" do + debate.update(cached_anonymous_votes_total: 520, cached_votes_total: 1000) + user = create(:user) + expect(debate.votable_by?(user)).to be false + end + end + describe "#register_vote" do let(:debate) { create(:debate) } @@ -138,42 +174,6 @@ describe Debate do end end - describe "#votable_by?" do - let(:debate) { create(:debate) } - - before(:each) do - Setting.find_by(key: "max_ratio_anon_votes_on_debates").update(value: 50) - end - - it "should be true for level two verified users" do - user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111") - expect(debate.votable_by?(user)).to be true - end - - it "should be true for level three verified users" do - user = create(:user, verified_at: Time.now) - expect(debate.votable_by?(user)).to be true - end - - it "should be true for anonymous users if allowed anonymous votes" do - debate.update(cached_anonymous_votes_total: 420, cached_votes_total: 1000) - user = create(:user) - expect(debate.votable_by?(user)).to be true - end - - it "should be true for anonymous users if less than 100 votes" do - debate.update(cached_anonymous_votes_total: 90, cached_votes_total: 92) - user = create(:user) - expect(debate.votable_by?(user)).to be true - end - - it "should be false for anonymous users if too many anonymous votes" do - debate.update(cached_anonymous_votes_total: 520, cached_votes_total: 1000) - user = create(:user) - expect(debate.votable_by?(user)).to be false - end - end - describe '#anonymous_votes_ratio' do it "returns the percentage of anonymous votes of the total votes" do debate = create(:debate, cached_anonymous_votes_total: 25, cached_votes_total: 100) @@ -244,16 +244,16 @@ describe Debate do debate = create(:debate, :with_confidence_score, cached_votes_up: 100, cached_votes_score: 100, cached_votes_total: 100) expect(debate.confidence_score).to eq(10000) - debate = create(:debate, :with_confidence_score, cached_votes_up: 0, cached_votes_score: -100, cached_votes_total: 100) + debate = create(:debate, :with_confidence_score, cached_votes_up: 0, cached_votes_total: 100) expect(debate.confidence_score).to eq(0) - debate = create(:debate, :with_confidence_score, cached_votes_up: 50, cached_votes_score: 50, cached_votes_total: 100) - expect(debate.confidence_score).to eq(2500) + debate = create(:debate, :with_confidence_score, cached_votes_up: 75, cached_votes_total: 100) + expect(debate.confidence_score).to eq(3750) - debate = create(:debate, :with_confidence_score, cached_votes_up: 500, cached_votes_score: 500, cached_votes_total: 1000) - expect(debate.confidence_score).to eq(25000) + debate = create(:debate, :with_confidence_score, cached_votes_up: 750, cached_votes_total: 1000) + expect(debate.confidence_score).to eq(37500) - debate = create(:debate, :with_confidence_score, cached_votes_up: 10, cached_votes_score: -80, cached_votes_total: 100) + debate = create(:debate, :with_confidence_score, cached_votes_up: 10, cached_votes_total: 100) expect(debate.confidence_score).to eq(-800) end diff --git a/spec/models/lock_spec.rb b/spec/models/lock_spec.rb index 0c072e9c8..d3c238484 100644 --- a/spec/models/lock_spec.rb +++ b/spec/models/lock_spec.rb @@ -10,7 +10,7 @@ describe Lock do expect(lock.locked?).to be true end - it "return false if locked_until is before curren time" do + it "return false if locked_until is before current time" do lock.locked_until = 1.day.ago expect(lock.locked?).to be false end diff --git a/spec/models/proposal_spec.rb b/spec/models/proposal_spec.rb new file mode 100644 index 000000000..f713f3ce5 --- /dev/null +++ b/spec/models/proposal_spec.rb @@ -0,0 +1,226 @@ +require 'rails_helper' + +describe Proposal do + let(:proposal) { build(:proposal) } + + it "should be valid" do + expect(proposal).to be_valid + end + + it "should not be valid without an author" do + proposal.author = nil + expect(proposal).to_not be_valid + end + + it "should not be valid without a title" do + proposal.title = nil + expect(proposal).to_not be_valid + end + + it "should not be valid without a question" do + proposal.question = nil + expect(proposal).to_not be_valid + end + + it "should not be valid without a summary" do + proposal.summary = nil + expect(proposal).to_not be_valid + end + + describe "#description" do + it "should be sanitized" do + proposal.description = "" + proposal.valid? + expect(proposal.description).to eq("alert('danger');") + end + end + + describe "#responsible" do + it "should be mandatory" do + proposal.responsible_name = nil + expect(proposal).to_not be_valid + end + + it "should be the document_number if level two user" do + author = create(:user, :level_two, document_number: "12345678Z") + proposal.author = author + proposal.responsible_name = nil + + expect(proposal).to be_valid + proposal.responsible_name = "12345678Z" + end + + it "should be the document_number if level two user" do + author = create(:user, :level_three, document_number: "12345678Z") + proposal.author = author + proposal.responsible_name = nil + + expect(proposal).to be_valid + proposal.responsible_name = "12345678Z" + end + end + + it "should sanitize the tag list" do + proposal.tag_list = "user_id=1" + proposal.valid? + expect(proposal.tag_list).to eq(['user_id1']) + end + + it "should not be valid without accepting terms of service" do + proposal.terms_of_service = nil + expect(proposal).to_not be_valid + end + + it "should have a code" do + Setting.find_by(key: "proposal_code_prefix").update(value: "TEST") + proposal = create(:proposal) + expect(proposal.code).to eq "TEST-#{proposal.created_at.strftime('%Y-%M')}-#{proposal.id}" + end + + describe "#editable?" do + let(:proposal) { create(:proposal) } + before(:each) {Setting.find_by(key: "max_votes_for_proposal_edit").update(value: 100)} + + it "should be true if proposal has no votes yet" do + expect(proposal.total_votes).to eq(0) + expect(proposal.editable?).to be true + end + + it "should be true if proposal has less than limit votes" do + create_list(:vote, 91, votable: proposal) + expect(proposal.total_votes).to eq(91) + expect(proposal.editable?).to be true + end + + it "should be false if proposal has more than limit votes" do + create_list(:vote, 102, votable: proposal) + expect(proposal.total_votes).to eq(102) + expect(proposal.editable?).to be false + end + end + + describe "#votable_by?" do + let(:proposal) { create(:proposal) } + + it "should be true for level two verified users" do + user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111") + expect(proposal.votable_by?(user)).to be true + end + + it "should be true for level three verified users" do + user = create(:user, verified_at: Time.now) + expect(proposal.votable_by?(user)).to be true + end + + it "should be false for anonymous users" do + user = create(:user) + expect(proposal.votable_by?(user)).to be false + end + end + + describe "#register_vote" do + let(:proposal) { create(:proposal) } + + describe "from level two verified users" do + it "should register vote" do + user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111") + expect {proposal.register_vote(user, 'yes')}.to change{proposal.reload.votes_for.size}.by(1) + end + end + + describe "from level three verified users" do + it "should register vote" do + user = create(:user, verified_at: Time.now) + expect {proposal.register_vote(user, 'yes')}.to change{proposal.reload.votes_for.size}.by(1) + end + end + + describe "from anonymous users" do + it "should not register vote" do + user = create(:user) + expect {proposal.register_vote(user, 'yes')}.to change{proposal.reload.votes_for.size}.by(0) + end + end + end + + describe '#hot_score' do + let(:now) { Time.now } + + it "increases for newer proposals" do + old = create(:proposal, :with_hot_score, created_at: now - 1.day) + new = create(:proposal, :with_hot_score, created_at: now) + expect(new.hot_score).to be > old.hot_score + end + + it "increases for proposals with more comments" do + more_comments = create(:proposal, :with_hot_score, created_at: now, comments_count: 25) + less_comments = create(:proposal, :with_hot_score, created_at: now, comments_count: 1) + expect(more_comments.hot_score).to be > less_comments.hot_score + end + + it "increases for proposals with more positive votes" do + more_likes = create(:proposal, :with_hot_score, created_at: now, cached_votes_up: 5) + less_likes = create(:proposal, :with_hot_score, created_at: now, cached_votes_up: 1) + expect(more_likes.hot_score).to be > less_likes.hot_score + end + + it "increases for proposals with more confidence" do + more_confidence = create(:proposal, :with_hot_score, created_at: now, cached_votes_up: 700) + less_confidence = create(:proposal, :with_hot_score, created_at: now, cached_votes_up: 9) + expect(more_confidence.hot_score).to be > less_confidence.hot_score + end + + it "decays in older proposals, even if they have more votes" do + older_more_voted = create(:proposal, :with_hot_score, created_at: now - 2.days, cached_votes_up: 900) + new_less_voted = create(:proposal, :with_hot_score, created_at: now, cached_votes_up: 9) + expect(new_less_voted.hot_score).to be > older_more_voted.hot_score + end + + describe 'actions which affect it' do + let(:proposal) { create(:proposal, :with_hot_score) } + + it "increases with votes" do + previous = proposal.hot_score + 5.times { proposal.register_vote(create(:user, verified_at: Time.now), true) } + expect(previous).to be < proposal.reload.hot_score + end + + it "increases with comments" do + previous = proposal.hot_score + 25.times{ Comment.create(user: create(:user), commentable: proposal, body: 'foobarbaz') } + expect(previous).to be < proposal.reload.hot_score + end + end + end + + describe "#confidence_score" do + + it "takes into account votes" do + proposal = create(:proposal, :with_confidence_score, cached_votes_up: 100) + expect(proposal.confidence_score).to eq(10000) + + proposal = create(:proposal, :with_confidence_score, cached_votes_up: 0) + expect(proposal.confidence_score).to eq(0) + + proposal = create(:proposal, :with_confidence_score, cached_votes_up: 75) + expect(proposal.confidence_score).to eq(7500) + + proposal = create(:proposal, :with_confidence_score, cached_votes_up: 750) + expect(proposal.confidence_score).to eq(75000) + + proposal = create(:proposal, :with_confidence_score, cached_votes_up: 10) + expect(proposal.confidence_score).to eq(1000) + end + + describe 'actions which affect it' do + let(:proposal) { create(:proposal, :with_confidence_score) } + + it "increases with like" do + previous = proposal.confidence_score + 5.times { proposal.register_vote(create(:user, verified_at: Time.now), true) } + expect(previous).to be < proposal.confidence_score + end + end + + end +end diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index 756810395..f7ef59121 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -66,9 +66,9 @@ describe User do end describe 'preferences' do - describe 'email_on_debate_comment' do + describe 'email_on_comment' do it 'should be false by default' do - expect(subject.email_on_debate_comment).to eq(false) + expect(subject.email_on_comment).to eq(false) end end diff --git a/spec/support/common_actions.rb b/spec/support/common_actions.rb index 2de06ce92..4e2dcdb18 100644 --- a/spec/support/common_actions.rb +++ b/spec/support/common_actions.rb @@ -134,4 +134,20 @@ module CommonActions expect(page).to have_content 'Correct code' end + + def expect_message_you_need_to_sign_in + expect(page).to have_content 'You need to sign in or sign up before continuing' + expect(page).to have_selector('.in-favor a', visible: false) + end + + def expect_message_to_many_anonymous_votes + expect(page).to have_content 'Too many anonymous votes, verify your account to vote.' + expect(page).to have_selector('.in-favor a', visible: false) + end + + def expect_message_only_verified_can_vote_proposals + expect(page).to have_content 'Proposals can only be voted by verified users, verify your account.' + expect(page).to have_selector('.in-favor a', visible: false) + end + end