Remove legislation processes next filter

This commit is contained in:
decabeza
2019-02-06 16:53:30 +01:00
parent fd2c51cb4b
commit 060a4c684f
10 changed files with 5 additions and 58 deletions

View File

@@ -1,7 +1,7 @@
class Admin::Legislation::ProcessesController < Admin::Legislation::BaseController class Admin::Legislation::ProcessesController < Admin::Legislation::BaseController
include Translatable include Translatable
has_filters %w{open next past all}, only: :index has_filters %w[open past all], only: :index
load_and_authorize_resource :process, class: "Legislation::Process" load_and_authorize_resource :process, class: "Legislation::Process"

View File

@@ -1,5 +1,5 @@
class Legislation::ProcessesController < Legislation::BaseController class Legislation::ProcessesController < Legislation::BaseController
has_filters %w[open next past], only: :index has_filters %w[open past], only: :index
has_filters %w[random winners], only: :proposals has_filters %w[random winners], only: :proposals
load_and_authorize_resource load_and_authorize_resource

View File

@@ -50,7 +50,6 @@ class Legislation::Process < ActiveRecord::Base
validates :font_color, format: { allow_blank: true, with: CSS_HEX_COLOR } validates :font_color, format: { allow_blank: true, with: CSS_HEX_COLOR }
scope :open, -> { where("start_date <= ? and end_date >= ?", Date.current, Date.current) } scope :open, -> { where("start_date <= ? and end_date >= ?", Date.current, Date.current) }
scope :next, -> { where("start_date > ?", Date.current) }
scope :past, -> { where("end_date < ?", Date.current) } scope :past, -> { where("end_date < ?", Date.current) }
scope :published, -> { where(published: true) } scope :published, -> { where(published: true) }

View File

@@ -446,7 +446,6 @@ en:
title: Legislation processes title: Legislation processes
filters: filters:
open: Open open: Open
next: Next
past: Past past: Past
all: All all: All
new: new:

View File

@@ -62,10 +62,8 @@ en:
filter: Filter filter: Filter
filters: filters:
open: Open processes open: Open processes
next: Next
past: Past past: Past
no_open_processes: There aren't open processes no_open_processes: There aren't open processes
no_next_processes: There aren't planned processes
no_past_processes: There aren't past processes no_past_processes: There aren't past processes
section_header: section_header:
icon_alt: Legislation processes icon icon_alt: Legislation processes icon

View File

@@ -447,7 +447,6 @@ es:
title: Procesos de legislación colaborativa title: Procesos de legislación colaborativa
filters: filters:
open: Abiertos open: Abiertos
next: Próximamente
past: Pasados past: Pasados
all: Todos all: Todos
new: new:

View File

@@ -62,10 +62,8 @@ es:
filter: Filtro filter: Filtro
filters: filters:
open: Procesos activos open: Procesos activos
next: Próximamente
past: Terminados past: Terminados
no_open_processes: No hay procesos activos no_open_processes: No hay procesos activos
no_next_processes: No hay procesos planeados
no_past_processes: No hay procesos terminados no_past_processes: No hay procesos terminados
section_header: section_header:
icon_alt: Icono de Procesos legislativos icon_alt: Icono de Procesos legislativos

View File

@@ -34,17 +34,6 @@ FactoryBot.define do
result_publication_enabled true result_publication_enabled true
published true published true
trait :next do
start_date { Date.current + 2.days }
end_date { Date.current + 8.days }
debate_start_date { Date.current + 2.days }
debate_end_date { Date.current + 4.days }
draft_publication_date { Date.current + 5.days }
allegations_start_date { Date.current + 5.days }
allegations_end_date { Date.current + 7.days }
result_publication_date { Date.current + 8.days }
end
trait :past do trait :past do
start_date { Date.current - 12.days } start_date { Date.current - 12.days }
end_date { Date.current - 2.days } end_date { Date.current - 2.days }

View File

@@ -26,15 +26,9 @@ feature 'Legislation' do
context 'processes home page' do context 'processes home page' do
scenario 'No processes to be listed' do scenario "No processes to be listed" do
visit legislation_processes_path visit legislation_processes_path
expect(page).to have_text "There aren't open processes" expect(page).to have_text "There aren't open processes"
visit legislation_processes_path(filter: 'next')
expect(page).to have_text "There aren't planned processes"
visit legislation_processes_path(filter: 'past')
expect(page).to have_text "There aren't past processes"
end end
scenario 'Processes can be listed' do scenario 'Processes can be listed' do
@@ -90,24 +84,16 @@ feature 'Legislation' do
scenario 'Filtering processes' do scenario 'Filtering processes' do
create(:legislation_process, title: "Process open") create(:legislation_process, title: "Process open")
create(:legislation_process, :next, title: "Process next")
create(:legislation_process, :past, title: "Process past") create(:legislation_process, :past, title: "Process past")
create(:legislation_process, :in_draft_phase, title: "Process in draft phase") create(:legislation_process, :in_draft_phase, title: "Process in draft phase")
visit legislation_processes_path visit legislation_processes_path
expect(page).to have_content('Process open') expect(page).to have_content('Process open')
expect(page).not_to have_content('Process next')
expect(page).not_to have_content('Process past') expect(page).not_to have_content('Process past')
expect(page).not_to have_content('Process in draft phase') expect(page).not_to have_content('Process in draft phase')
visit legislation_processes_path(filter: 'next')
expect(page).not_to have_content('Process open')
expect(page).to have_content('Process next')
expect(page).not_to have_content('Process past')
visit legislation_processes_path(filter: 'past') visit legislation_processes_path(filter: 'past')
expect(page).not_to have_content('Process open') expect(page).not_to have_content('Process open')
expect(page).not_to have_content('Process next')
expect(page).to have_content('Process past') expect(page).to have_content('Process past')
end end
@@ -115,10 +101,8 @@ feature 'Legislation' do
before do before do
create(:legislation_process, title: "published") create(:legislation_process, title: "published")
create(:legislation_process, :not_published, title: "not published") create(:legislation_process, :not_published, title: "not published")
[:next, :past].each do |trait| create(:legislation_process, :past, title: "past published")
create(:legislation_process, trait, title: "#{trait} published") create(:legislation_process, :not_published, :past, title: "past not published")
create(:legislation_process, :not_published, trait, title: "#{trait} not published")
end
end end
it "aren't listed" do it "aren't listed" do
@@ -132,17 +116,6 @@ feature 'Legislation' do
expect(page).to have_content('published') expect(page).to have_content('published')
end end
it "aren't listed with next filter" do
visit legislation_processes_path(filter: 'next')
expect(page).not_to have_content('not published')
expect(page).to have_content('next published')
login_as(administrator)
visit legislation_processes_path(filter: 'next')
expect(page).not_to have_content('not published')
expect(page).to have_content('next published')
end
it "aren't listed with past filter" do it "aren't listed with past filter" do
visit legislation_processes_path(filter: 'past') visit legislation_processes_path(filter: 'past')
expect(page).not_to have_content('not published') expect(page).not_to have_content('not published')

View File

@@ -144,14 +144,6 @@ describe Legislation::Process do
expect(processes_not_in_draft).not_to include(process_with_draft_only_today) expect(processes_not_in_draft).not_to include(process_with_draft_only_today)
end end
it "filters next" do
next_processes = ::Legislation::Process.next
expect(next_processes).to include(process_2)
expect(next_processes).not_to include(process_1)
expect(next_processes).not_to include(process_3)
end
it "filters past" do it "filters past" do
past_processes = ::Legislation::Process.past past_processes = ::Legislation::Process.past