diff --git a/Gemfile b/Gemfile index b92fc4cbe..dbefab4c0 100644 --- a/Gemfile +++ b/Gemfile @@ -62,6 +62,7 @@ gem 'fog' gem 'fog-openstack' gem 'pg' +gem 'scenic' gem 'rgeo-geojson' gem 'leaflet-rails' diff --git a/Gemfile.lock b/Gemfile.lock index 706c620e8..a42023ff3 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -482,6 +482,9 @@ GEM sprockets (>= 2.8, < 4.0) sprockets-rails (>= 2.0, < 4.0) tilt (>= 1.1, < 3) + scenic (1.3.0) + activerecord (>= 4.0.0) + railties (>= 4.0.0) sdoc (0.4.1) json (~> 1.7, >= 1.7.7) rdoc (~> 4.0) @@ -635,6 +638,7 @@ DEPENDENCIES rubocop-checkstyle_formatter rubocop-rspec sass-rails (~> 5.0) + scenic sdoc (~> 0.4.0) selenium-webdriver sentry-raven @@ -654,4 +658,4 @@ DEPENDENCIES will_paginate-bootstrap BUNDLED WITH - 1.12.5 + 1.13.2 diff --git a/app/assets/javascripts/dossiers_list_link.js b/app/assets/javascripts/dossiers_list_link.js new file mode 100644 index 000000000..1764dd571 --- /dev/null +++ b/app/assets/javascripts/dossiers_list_link.js @@ -0,0 +1,9 @@ +$(document).on('page:load', link_init); +$(document).ready(link_init); + + +function link_init() { + $('#dossiers_list tr').on('click', function () { + $(location).attr('href', $(this).data('dossier_url')) + }); +} \ No newline at end of file diff --git a/app/assets/stylesheets/dossiers.scss b/app/assets/stylesheets/dossiers.scss index 98ed7d31f..76ce739db 100644 --- a/app/assets/stylesheets/dossiers.scss +++ b/app/assets/stylesheets/dossiers.scss @@ -27,4 +27,9 @@ h5 span { #validate_button { float: right; } +} + +#dossiers_list tr:hover{ + background-color: #eeeeee; + cursor: pointer; } \ No newline at end of file diff --git a/app/controllers/backoffice/dossiers_controller.rb b/app/controllers/backoffice/dossiers_controller.rb index 9c4895798..4de0b31a2 100644 --- a/app/controllers/backoffice/dossiers_controller.rb +++ b/app/controllers/backoffice/dossiers_controller.rb @@ -27,7 +27,12 @@ class Backoffice::DossiersController < Backoffice::DossiersListController def search @search_terms = params[:q] - @dossier = Dossier.search(current_gestionnaire, @search_terms) + + @dossier = Search.new( + gestionnaire: current_gestionnaire, + query: @search_terms, + page: params[:page] + ).results smartlisting_dossier @dossier, 'search' diff --git a/app/facades/dossiers_list_facades.rb b/app/facades/dossiers_list_facades.rb index 50d2848d7..4f96caa4d 100644 --- a/app/facades/dossiers_list_facades.rb +++ b/app/facades/dossiers_list_facades.rb @@ -58,6 +58,10 @@ class DossiersListFacades (@liste == 'a_traiter' ? 'active' : '') end + def fige_class + (@liste == 'fige' ? 'active' : '') + end + def en_attente_class (@liste == 'en_attente' ? 'active' : '') end @@ -103,13 +107,17 @@ class DossiersListFacades end def a_traiter_total - service.waiting_for_gestionnaire.count + service.ouvert.count end def en_construction_total service.en_construction.count end + def fige_total + service.fige.count + end + def en_attente_total service.waiting_for_user.count end @@ -158,6 +166,10 @@ class DossiersListFacades base_url 'a_traiter' end + def fige_url + base_url 'fige' + end + def en_attente_url base_url 'en_attente' end diff --git a/app/models/dossier.rb b/app/models/dossier.rb index e7b9599d0..c6ef99b8c 100644 --- a/app/models/dossier.rb +++ b/app/models/dossier.rb @@ -45,6 +45,7 @@ class Dossier < ActiveRecord::Base BROUILLON = %w(draft) NOUVEAUX = %w(initiated) + OUVERT = %w(updated replied) WAITING_FOR_GESTIONNAIRE = %w(updated) WAITING_FOR_USER = %w(replied validated) EN_CONSTRUCTION = %w(initiated updated replied) @@ -194,6 +195,10 @@ class Dossier < ActiveRecord::Base EN_CONSTRUCTION.include?(state) end + def ouvert? + OUVERT.include?(state) + end + def deposes? DEPOSES.include?(state) end @@ -202,6 +207,10 @@ class Dossier < ActiveRecord::Base VALIDES.include?(state) end + def fige? + VALIDES.include?(state) + end + def a_instruire? A_INSTRUIRE.include?(state) end @@ -238,10 +247,18 @@ class Dossier < ActiveRecord::Base where(state: EN_CONSTRUCTION, archived: false).order("updated_at #{order}") end + def self.ouvert order = 'ASC' + where(state: OUVERT, archived: false).order("updated_at #{order}") + end + def self.valides order = 'ASC' where(state: VALIDES, archived: false).order("updated_at #{order}") end + def self.fige order = 'ASC' + where(state: VALIDES, archived: false).order("updated_at #{order}") + end + def self.deposes order = 'ASC' where(state: DEPOSES, archived: false).order("updated_at #{order}") end @@ -258,37 +275,6 @@ class Dossier < ActiveRecord::Base where(state: TERMINE, archived: false).order("updated_at #{order}") end - def self.search current_gestionnaire, terms - return [] if terms.blank? - - dossiers = Dossier.arel_table - users = User.arel_table - etablissements = Etablissement.arel_table - entreprises = Entreprise.arel_table - - composed_scope = self.joins('LEFT OUTER JOIN users ON users.id = dossiers.user_id') - .joins('LEFT OUTER JOIN entreprises ON entreprises.dossier_id = dossiers.id') - .joins('LEFT OUTER JOIN etablissements ON etablissements.dossier_id = dossiers.id') - - terms.split.each do |word| - query_string = "%#{word}%" - query_string_start_with = "#{word}%" - - composed_scope = composed_scope.where( - users[:email].matches(query_string).or\ - etablissements[:siret].matches(query_string_start_with).or\ - entreprises[:raison_sociale].matches(query_string).or\ - dossiers[:id].eq(word_is_an_integer word)) - end - - composed_scope = composed_scope.where( - dossiers[:id].eq_any(current_gestionnaire.dossiers.ids).and\ - dossiers[:state].does_not_match('draft').and\ - dossiers[:archived].eq(false)) - - composed_scope - end - def cerfa_available? procedure.cerfa_flag? && cerfa.size != 0 end diff --git a/app/models/search.rb b/app/models/search.rb new file mode 100644 index 000000000..6b4557677 --- /dev/null +++ b/app/models/search.rb @@ -0,0 +1,82 @@ +# See: +# - https://robots.thoughtbot.com/implementing-multi-table-full-text-search-with-postgres +# - http://calebthompson.io/talks/search.html +class Search < ActiveRecord::Base + # :nodoc: + # + # Englobs a search result (actually a collection of Search objects) so it acts + # like a collection of regular Dossier objects, which can be decorated, + # paginated, ... + class Results + include Enumerable + + def initialize(results) + @results = results + end + + def each + @results.each do |search| + yield search.dossier + end + end + + def method_missing(name, *args, &block) + @results.__send__(name, *args, &block) + end + + def decorate! + @results.each do |search| + search.dossier = search.dossier.decorate + end + end + end + + attr_accessor :gestionnaire + attr_accessor :query + attr_accessor :page + + belongs_to :dossier + + def results + unless @query.present? + return Search.none + end + + search_term = Search.connection.quote(to_tsquery) + + dossier_ids = @gestionnaire.dossiers + .select(:id) + .where(archived: false) + .where.not(state: "draft") + + q = Search + .select("DISTINCT(searches.dossier_id)") + .select("COALESCE(ts_rank(to_tsvector('french', searches.term::text), to_tsquery('french', #{search_term})), 0) AS rank") + .joins(:dossier) + .where(dossier_id: dossier_ids) + .where("to_tsvector('french', searches.term::text) @@ to_tsquery('french', #{search_term})") + .order("rank DESC") + .preload(:dossier) + + if @page.present? + q = q.paginate(page: @page) + end + + Results.new(q) + end + + #def self.refresh + # # TODO: could be executed concurrently + # # See https://github.com/thoughtbot/scenic#what-about-materialized-views + # Scenic.database.refresh_materialized_view(table_name, concurrently: false) + #end + + private + + def to_tsquery + @query.gsub(/['?\\:&|!]/, "") # drop disallowed characters + .split(/\s+/) # split words + .map { |x| "#{x}:*" } # enable prefix matching + .join(" & ") + end +end diff --git a/app/services/dossiers_list_gestionnaire_service.rb b/app/services/dossiers_list_gestionnaire_service.rb index bc7923d08..7cb09366f 100644 --- a/app/services/dossiers_list_gestionnaire_service.rb +++ b/app/services/dossiers_list_gestionnaire_service.rb @@ -7,8 +7,8 @@ class DossiersListGestionnaireService def dossiers_to_display {'nouveaux' => nouveaux, - 'a_traiter' => waiting_for_gestionnaire, - 'en_attente' => waiting_for_user, + 'a_traiter' => ouvert, + 'fige' => fige, 'deposes' => deposes, 'a_instruire' => a_instruire, 'termine' => termine, @@ -16,7 +16,7 @@ class DossiersListGestionnaireService end def self.dossiers_liste_libelle - ['nouveaux', 'a_traiter', 'en_attente', 'deposes', 'a_instruire', 'termine', 'all_state'] + ['nouveaux', 'a_traiter', 'fige' ,'deposes', 'a_instruire', 'termine', 'all_state'] end def all_state @@ -27,12 +27,12 @@ class DossiersListGestionnaireService @nouveaux ||= filter_dossiers.nouveaux end - def waiting_for_gestionnaire - @waiting_for_gestionnaire ||= filter_dossiers.waiting_for_gestionnaire + def ouvert + @ouvert ||= filter_dossiers.ouvert end - def waiting_for_user - @waiting_for_user ||= filter_dossiers.waiting_for_user + def fige + @fige ||= filter_dossiers.fige end def deposes diff --git a/app/views/backoffice/dossiers/_list.html.haml b/app/views/backoffice/dossiers/_list.html.haml index 3b6793ef5..7b2f7dbd5 100644 --- a/app/views/backoffice/dossiers/_list.html.haml +++ b/app/views/backoffice/dossiers/_list.html.haml @@ -1,4 +1,4 @@ -%table.table +%table#dossiers_list.table %thead - @dossiers_list_facade.preference_list_dossiers_filter.each do |preference| %th{class: "col-md-#{preference.bootstrap_lg} col-lg-#{preference.bootstrap_lg}"} @@ -15,7 +15,7 @@ - unless smart_listing.empty? - @dossiers.each do |dossier| - %tr + %tr{id: "tr_dossier_#{dossier.id}", 'data-dossier_url' => backoffice_dossier_url(id: dossier.id)} - @dossiers_list_facade.preference_list_dossiers_filter.each_with_index do |preference, index| %td - if preference.table.nil? || preference.table.empty? @@ -30,10 +30,7 @@ - rescue NoMethodError - value = '' - - if index == 0 - = link_to value, backoffice_dossier_path(id: dossier.id) - - else - = value + = value %td.center - if current_gestionnaire.follow?(dossier.id) @@ -48,4 +45,4 @@ - if smart_listing.empty? %h4.center - Aucun dossier \ No newline at end of file + Aucun dossier diff --git a/app/views/backoffice/dossiers/_onglets.html.haml b/app/views/backoffice/dossiers/_onglets.html.haml index fa32c51b0..b35281247 100644 --- a/app/views/backoffice/dossiers/_onglets.html.haml +++ b/app/views/backoffice/dossiers/_onglets.html.haml @@ -19,18 +19,18 @@ =@dossiers_list_facade.nouveaux_total %li{ class: (@dossiers_list_facade.a_traiter_class) } - %a{:href => "#{url_for @dossiers_list_facade.a_traiter_url}", 'data-toggle' => :tooltip, title: 'Les dossiers qui requièrent une action de votre part.'} + %a{:href => "#{url_for @dossiers_list_facade.a_traiter_url}", 'data-toggle' => :tooltip, title: 'Les dossiers qui ne sont pas encore déclarés complets.'} %h5.text-danger - = "Action requise" + ="Ouverts" .badge.progress-bar-danger =@dossiers_list_facade.a_traiter_total - %li{ class: (@dossiers_list_facade.en_attente_class) } - %a{:href => "#{url_for @dossiers_list_facade.en_attente_url}", 'data-toggle' => :tooltip, title: 'Les dossiers en attentes d\'une action de la part de l\'usager.'} + %li{ class: (@dossiers_list_facade.fige_class) } + %a{:href => "#{url_for @dossiers_list_facade.fige_url}", 'data-toggle' => :tooltip, title: 'Les dossiers qui sont déclarés complets et donc figés.'} %h5.text-default - ="Attente usager " + ="Figés" .badge.progress-bar-default - =@dossiers_list_facade.en_attente_total + =@dossiers_list_facade.fige_total %li{ class: (@dossiers_list_facade.deposes_class) } %a{:href => "#{url_for @dossiers_list_facade.deposes_url}", 'data-toggle' => :tooltip, title: 'Les dossiers qui ont été validés et déposés par les usager qui attendent une réponse de bonne réception avant examen.'} diff --git a/app/views/backoffice/dossiers/_state_description.html.haml b/app/views/backoffice/dossiers/_state_description.html.haml index 13d1cf58f..b3d1946d1 100644 --- a/app/views/backoffice/dossiers/_state_description.html.haml +++ b/app/views/backoffice/dossiers/_state_description.html.haml @@ -11,12 +11,18 @@ n'ont jamais été ouvert par votre service. Il attende une première lecture et intervention de votre part. -elsif dossiers_list_facade.liste == 'a_traiter' - Tous les dossiers présents dans cette liste sont ceux qui + Tous les dossiers présents dans cette liste sont ceux qui sont %b - attendent une action de votre part. - Cela peut être par exemple une demande client ou une relecture pour validation de complétude. - -elsif dossiers_list_facade.liste == 'en_attente' - Tous les dossiers présents dans cette liste sont ceux qui requière une action de la part de l'usager. À priori, vous n'avez donc pas d'intervention particulière à réaliser. + en cours de construction avec l'usager. + Ils ne sont pas figés et ne sont donc pas complets. + -elsif dossiers_list_facade.liste == 'fige' + Tous les dossiers présents dans cette liste ont été déclarés + %b + complets + et ne sont + %b + plus modifiables par l'usager. + Ils attendent donc leurs dépots officiels qui doit être effectué par l'usager. -elsif dossiers_list_facade.liste == 'deposes' Tous les dossiers présents dans cette liste ont été %b diff --git a/app/views/backoffice/dossiers/index.js.erb b/app/views/backoffice/dossiers/index.js.erb index 0f7d43419..876f677fd 100644 --- a/app/views/backoffice/dossiers/index.js.erb +++ b/app/views/backoffice/dossiers/index.js.erb @@ -1,3 +1,4 @@ <%= smart_listing_update :dossiers %> -filters_init(); \ No newline at end of file +filters_init(); +link_init(); \ No newline at end of file diff --git a/app/views/layouts/application.html.haml b/app/views/layouts/application.html.haml index d1bda0bb0..baf25cfeb 100644 --- a/app/views/layouts/application.html.haml +++ b/app/views/layouts/application.html.haml @@ -2,7 +2,8 @@ %html %head %meta{:content => "text/html; charset=UTF-8", "http-equiv" => "Content-Type"}/ - %title TPS - Téléprocédures simplifiées + %title + =t('dynamics.page_title') %meta{'http-equiv' => "X-UA-Compatible", :content => "IE=edge"} = stylesheet_link_tag 'application', media: 'all', 'data-turbolinks-track' => true diff --git a/app/views/users/dossiers/_list.html.haml b/app/views/users/dossiers/_list.html.haml index 34139cce4..398b31876 100644 --- a/app/views/users/dossiers/_list.html.haml +++ b/app/views/users/dossiers/_list.html.haml @@ -1,5 +1,5 @@ - unless smart_listing.empty? - %table.table + %table#dossiers_list.table %thead %th.col-md-1.col-lg-1= smart_listing.sortable 'Numéro', 'id' %th.col-md-5.col-lg-5= smart_listing.sortable 'Procédure', 'procedure.libelle' @@ -13,14 +13,17 @@ -dossier = invite.dossier.decorate - else - dossier = dossier.decorate - %tr + + - dossier_url = users_dossiers_invite_path(id: invite.id) unless invite.nil? + - if invite.nil? + - dossier_url = users_dossier_recapitulatif_path(dossier) unless dossier.brouillon? + - dossier_url = users_dossier_description_path(dossier) if dossier.brouillon? + + %tr{id: "tr_dossier_#{dossier.id}", 'data-dossier_url' => dossier_url} %td.center = dossier.id %td - = link_to(dossier.procedure.libelle, users_dossiers_invite_path(id: invite.id)) unless invite.nil? - - if invite.nil? - = link_to(dossier.procedure.libelle, users_dossier_recapitulatif_path(dossier)) unless dossier.brouillon? - = link_to(dossier.procedure.libelle, users_dossier_description_path(dossier)) if dossier.brouillon? + = dossier.procedure.libelle %td{id: "dossier_#{dossier.id}_state"}= dossier.display_state %td= dossier.last_update diff --git a/app/views/users/dossiers/index.js.erb b/app/views/users/dossiers/index.js.erb index 1517a60d3..5a2505268 100644 --- a/app/views/users/dossiers/index.js.erb +++ b/app/views/users/dossiers/index.js.erb @@ -1 +1,2 @@ <%= smart_listing_update :dossiers %> +link_init(); \ No newline at end of file diff --git a/config/deploy.rb b/config/deploy.rb index cf3617318..cfa3be8aa 100644 --- a/config/deploy.rb +++ b/config/deploy.rb @@ -98,7 +98,8 @@ set :shared_paths, [ 'config/france_connect.yml', 'config/initializers/mailjet.rb', 'config/initializers/storage_url.rb', - 'app/views/root/landing.html.haml' + 'app/views/root/landing.html.haml', + 'app/views/cgu/index.html.haml' ] @@ -139,6 +140,9 @@ task :setup => :environment do queue! %[mkdir -p "#{deploy_to}/shared/app"] queue! %[chmod g+rx,u+rwx "#{deploy_to}/shared/app"] + queue! %[mkdir -p "#{deploy_to}/shared/views/cgu"] + queue! %[chmod g+rx,u+rwx "#{deploy_to}/shared/views/cgu"] + queue! %[mkdir -p "#{deploy_to}/shared/config/locales/dynamics"] queue! %[chmod g+rx,u+rwx "#{deploy_to}/shared/config/locales/dynamics"] diff --git a/config/locales/dynamics/fr.yml b/config/locales/dynamics/fr.yml index 0d8e42293..c16ec0ab7 100644 --- a/config/locales/dynamics/fr.yml +++ b/config/locales/dynamics/fr.yml @@ -1,5 +1,6 @@ fr: dynamics: + page_title: TPS - Téléprocédures simplifiées contact_email: contact@tps.apientreprise.fr users: connexion_title: Connexion diff --git a/config/locales/dynamics/fr_opensimplif.yml b/config/locales/dynamics/fr_opensimplif.yml index ed8b4c7e0..09b11be21 100644 --- a/config/locales/dynamics/fr_opensimplif.yml +++ b/config/locales/dynamics/fr_opensimplif.yml @@ -1,5 +1,6 @@ fr_opensimplif: dynamics: + page_title: OpenSimplif contact_email: simplification.sgmap@modernisation.gouv.fr users: connexion_title: "Connectez-vous en tant qu'utilisateur" diff --git a/db/migrate/20161025150900_create_searches.rb b/db/migrate/20161025150900_create_searches.rb new file mode 100644 index 000000000..911948c7d --- /dev/null +++ b/db/migrate/20161025150900_create_searches.rb @@ -0,0 +1,27 @@ +class CreateSearches < ActiveRecord::Migration + def up + add_index :champs, :dossier_id + add_index :champs, :type_de_champ_id + add_index :drop_down_lists, :type_de_champ_id + add_index :etablissements, :dossier_id + add_index :entreprises, :dossier_id + add_index :france_connect_informations, :user_id + add_index :individuals, :dossier_id + add_index :pieces_justificatives, :dossier_id + add_index :rna_informations, :entreprise_id + create_view :searches #, materialized: true + end + + def down + remove_index :champs, :dossier_id + remove_index :champs, :type_de_champ_id + remove_index :drop_down_lists, :type_de_champ_id + remove_index :etablissements, :dossier_id + remove_index :entreprises, :dossier_id + remove_index :france_connect_informations, :user_id + remove_index :individuals, :dossier_id + remove_index :pieces_justificatives, :dossier_id + remove_index :rna_informations, :entreprise_id + drop_view :searches #, materialized: true + end +end diff --git a/db/migrate/20161102154835_update_searches_to_version_2.rb b/db/migrate/20161102154835_update_searches_to_version_2.rb new file mode 100644 index 000000000..55223b4cf --- /dev/null +++ b/db/migrate/20161102154835_update_searches_to_version_2.rb @@ -0,0 +1,9 @@ +class UpdateSearchesToVersion2 < ActiveRecord::Migration + def up + replace_view :searches, version: 2 + end + + def down + replace_view :searches, version: 1 + end +end diff --git a/db/schema.rb b/db/schema.rb index 17762a2b4..7ef1e735e 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: 20161011125345) do +ActiveRecord::Schema.define(version: 20161102154835) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" @@ -102,6 +102,9 @@ ActiveRecord::Schema.define(version: 20161011125345) do t.string "type" end + add_index "champs", ["dossier_id"], name: "index_champs_on_dossier_id", using: :btree + add_index "champs", ["type_de_champ_id"], name: "index_champs_on_type_de_champ_id", using: :btree + create_table "commentaires", force: :cascade do |t| t.string "email" t.datetime "created_at", null: false @@ -135,6 +138,8 @@ ActiveRecord::Schema.define(version: 20161011125345) do t.integer "type_de_champ_id" end + add_index "drop_down_lists", ["type_de_champ_id"], name: "index_drop_down_lists_on_type_de_champ_id", using: :btree + create_table "entreprises", force: :cascade do |t| t.string "siren" t.integer "capital_social" @@ -151,6 +156,8 @@ ActiveRecord::Schema.define(version: 20161011125345) do t.integer "dossier_id" end + add_index "entreprises", ["dossier_id"], name: "index_entreprises_on_dossier_id", using: :btree + create_table "etablissements", force: :cascade do |t| t.string "siret" t.boolean "siege_social" @@ -168,6 +175,8 @@ ActiveRecord::Schema.define(version: 20161011125345) do t.integer "entreprise_id" end + add_index "etablissements", ["dossier_id"], name: "index_etablissements_on_dossier_id", using: :btree + create_table "exercices", force: :cascade do |t| t.string "ca" t.datetime "dateFinExercice" @@ -194,6 +203,8 @@ ActiveRecord::Schema.define(version: 20161011125345) do t.string "email_france_connect" end + add_index "france_connect_informations", ["user_id"], name: "index_france_connect_informations_on_user_id", using: :btree + create_table "gestionnaires", force: :cascade do |t| t.string "email", default: "", null: false t.string "encrypted_password", default: "", null: false @@ -221,6 +232,8 @@ ActiveRecord::Schema.define(version: 20161011125345) do t.string "gender" end + add_index "individuals", ["dossier_id"], name: "index_individuals_on_dossier_id", using: :btree + create_table "invites", force: :cascade do |t| t.string "email" t.string "email_sender" @@ -255,6 +268,7 @@ ActiveRecord::Schema.define(version: 20161011125345) do t.string "content_secure_token" end + add_index "pieces_justificatives", ["dossier_id"], name: "index_pieces_justificatives_on_dossier_id", using: :btree add_index "pieces_justificatives", ["type_de_piece_justificative_id"], name: "index_pieces_justificatives_on_type_de_piece_justificative_id", using: :btree create_table "preference_list_dossiers", force: :cascade do |t| @@ -324,6 +338,8 @@ ActiveRecord::Schema.define(version: 20161011125345) do t.integer "entreprise_id" end + add_index "rna_informations", ["entreprise_id"], name: "index_rna_informations_on_entreprise_id", using: :btree + create_table "types_de_champ", force: :cascade do |t| t.string "libelle" t.string "type_champ" @@ -369,4 +385,21 @@ ActiveRecord::Schema.define(version: 20161011125345) do add_foreign_key "dossiers", "users" add_foreign_key "procedure_paths", "administrateurs" add_foreign_key "procedure_paths", "procedures" + + create_view :searches, sql_definition: <<-SQL + SELECT dossiers.id AS dossier_id, + (((((((((((((((((((((((((((((((((((((((((((((((((((((((COALESCE(users.email, ''::character varying))::text || ' '::text) || (COALESCE(france_connect_informations.given_name, ''::character varying))::text) || ' '::text) || (COALESCE(france_connect_informations.family_name, ''::character varying))::text) || ' '::text) || (COALESCE(cerfas.content, ''::character varying))::text) || ' '::text) || (COALESCE(champs.value, ''::character varying))::text) || ' '::text) || (COALESCE(drop_down_lists.value, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.siren, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.numero_tva_intracommunautaire, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.forme_juridique, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.forme_juridique_code, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.nom_commercial, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.raison_sociale, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.siret_siege_social, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.nom, ''::character varying))::text) || ' '::text) || (COALESCE(entreprises.prenom, ''::character varying))::text) || ' '::text) || (COALESCE(rna_informations.association_id, ''::character varying))::text) || ' '::text) || (COALESCE(rna_informations.titre, ''::character varying))::text) || ' '::text) || COALESCE(rna_informations.objet, ''::text)) || ' '::text) || (COALESCE(etablissements.siret, ''::character varying))::text) || ' '::text) || (COALESCE(etablissements.naf, ''::character varying))::text) || ' '::text) || (COALESCE(etablissements.libelle_naf, ''::character varying))::text) || ' '::text) || (COALESCE(etablissements.adresse, ''::character varying))::text) || ' '::text) || (COALESCE(etablissements.code_postal, ''::character varying))::text) || ' '::text) || (COALESCE(etablissements.localite, ''::character varying))::text) || ' '::text) || (COALESCE(etablissements.code_insee_localite, ''::character varying))::text) || ' '::text) || (COALESCE(individuals.nom, ''::character varying))::text) || ' '::text) || (COALESCE(individuals.prenom, ''::character varying))::text) || ' '::text) || (COALESCE(pieces_justificatives.content, ''::character varying))::text) AS term + FROM ((((((((((dossiers + JOIN users ON ((users.id = dossiers.user_id))) + LEFT JOIN france_connect_informations ON ((france_connect_informations.user_id = dossiers.user_id))) + LEFT JOIN cerfas ON ((cerfas.dossier_id = dossiers.id))) + LEFT JOIN champs ON ((champs.dossier_id = dossiers.id))) + LEFT JOIN drop_down_lists ON ((drop_down_lists.type_de_champ_id = champs.type_de_champ_id))) + LEFT JOIN entreprises ON ((entreprises.dossier_id = dossiers.id))) + LEFT JOIN rna_informations ON ((rna_informations.entreprise_id = entreprises.id))) + LEFT JOIN etablissements ON ((etablissements.dossier_id = dossiers.id))) + LEFT JOIN individuals ON ((individuals.dossier_id = dossiers.id))) + LEFT JOIN pieces_justificatives ON ((pieces_justificatives.dossier_id = dossiers.id))); + SQL + end diff --git a/db/views/searches_v01.sql b/db/views/searches_v01.sql new file mode 100644 index 000000000..e5f2cfd71 --- /dev/null +++ b/db/views/searches_v01.sql @@ -0,0 +1,59 @@ +-- this version allows to search for a single term within many tables, +-- but behaves badly with multiple terms scattered in multiple tables. + +SELECT dossiers.id AS dossier_id, + dossiers.id::text || ' ' || + COALESCE(users.email, '') AS term + FROM dossiers + INNER JOIN users ON users.id = dossiers.user_id + +UNION SELECT cerfas.dossier_id, + COALESCE(cerfas.content, '') AS term + FROM cerfas + +UNION SELECT champs.dossier_id, + COALESCE(champs.value, '') || ' ' || + COALESCE(drop_down_lists.value, '') AS term + FROM champs + INNER JOIN drop_down_lists ON drop_down_lists.type_de_champ_id = champs.type_de_champ_id + +UNION SELECT entreprises.dossier_id, + COALESCE(entreprises.siren, '') || ' ' || + COALESCE(entreprises.numero_tva_intracommunautaire, '') || ' ' || + COALESCE(entreprises.forme_juridique, '') || ' ' || + COALESCE(entreprises.forme_juridique_code, '') || ' ' || + COALESCE(entreprises.nom_commercial, '') || ' ' || + COALESCE(entreprises.raison_sociale, '') || ' ' || + COALESCE(entreprises.siret_siege_social, '') || ' ' || + COALESCE(entreprises.nom, '') || ' ' || + COALESCE(entreprises.prenom, '') || ' ' || + COALESCE(rna_informations.association_id, '') || ' ' || + COALESCE(rna_informations.titre, '') || ' ' || + COALESCE(rna_informations.objet, '') AS term + FROM entreprises + LEFT JOIN rna_informations ON rna_informations.entreprise_id = entreprises.id + +UNION SELECT etablissements.dossier_id, + COALESCE(etablissements.siret, '') || ' ' || + COALESCE(etablissements.naf, '') || ' ' || + COALESCE(etablissements.libelle_naf, '') || ' ' || + COALESCE(etablissements.adresse, '') || ' ' || + COALESCE(etablissements.code_postal, '') || ' ' || + COALESCE(etablissements.localite, '') || ' ' || + COALESCE(etablissements.code_insee_localite, '') AS term + FROM etablissements + +UNION SELECT individuals.dossier_id, + COALESCE(individuals.nom, '') || ' ' || + COALESCE(individuals.prenom, '') AS term + FROM individuals + +UNION SELECT pieces_justificatives.dossier_id, + COALESCE(pieces_justificatives.content, '') AS term + FROM pieces_justificatives + +UNION SELECT dossiers.id, + COALESCE(france_connect_informations.given_name, '') || ' ' || + COALESCE(france_connect_informations.family_name, '') AS term + FROM france_connect_informations + INNER JOIN dossiers ON dossiers.user_id = france_connect_informations.user_id diff --git a/db/views/searches_v02.sql b/db/views/searches_v02.sql new file mode 100644 index 000000000..8a168f465 --- /dev/null +++ b/db/views/searches_v02.sql @@ -0,0 +1,43 @@ +-- this version merges all possible search terms together, complicating the +-- view, but enables searching for multiple terms from multiple tables at once. + +SELECT dossiers.id AS dossier_id, + COALESCE(users.email, '') || ' ' || + COALESCE(france_connect_informations.given_name, '') || ' ' || + COALESCE(france_connect_informations.family_name, '') || ' ' || + COALESCE(cerfas.content, '') || ' ' || + COALESCE(champs.value, '') || ' ' || + COALESCE(drop_down_lists.value, '') || ' ' || + COALESCE(entreprises.siren, '') || ' ' || + COALESCE(entreprises.numero_tva_intracommunautaire, '') || ' ' || + COALESCE(entreprises.forme_juridique, '') || ' ' || + COALESCE(entreprises.forme_juridique_code, '') || ' ' || + COALESCE(entreprises.nom_commercial, '') || ' ' || + COALESCE(entreprises.raison_sociale, '') || ' ' || + COALESCE(entreprises.siret_siege_social, '') || ' ' || + COALESCE(entreprises.nom, '') || ' ' || + COALESCE(entreprises.prenom, '') || ' ' || + COALESCE(rna_informations.association_id, '') || ' ' || + COALESCE(rna_informations.titre, '') || ' ' || + COALESCE(rna_informations.objet, '') || ' ' || + COALESCE(etablissements.siret, '') || ' ' || + COALESCE(etablissements.naf, '') || ' ' || + COALESCE(etablissements.libelle_naf, '') || ' ' || + COALESCE(etablissements.adresse, '') || ' ' || + COALESCE(etablissements.code_postal, '') || ' ' || + COALESCE(etablissements.localite, '') || ' ' || + COALESCE(etablissements.code_insee_localite, '') || ' ' || + COALESCE(individuals.nom, '') || ' ' || + COALESCE(individuals.prenom, '') || ' ' || + COALESCE(pieces_justificatives.content, '') AS term +FROM dossiers +INNER JOIN users ON users.id = dossiers.user_id +LEFT JOIN france_connect_informations ON france_connect_informations.user_id = dossiers.user_id +LEFT JOIN cerfas ON cerfas.dossier_id = dossiers.id +LEFT JOIN champs ON champs.dossier_id = dossiers.id +LEFT JOIN drop_down_lists ON drop_down_lists.type_de_champ_id = champs.type_de_champ_id +LEFT JOIN entreprises ON entreprises.dossier_id = dossiers.id +LEFT JOIN rna_informations ON rna_informations.entreprise_id = entreprises.id +LEFT JOIN etablissements ON etablissements.dossier_id = dossiers.id +LEFT JOIN individuals ON individuals.dossier_id = dossiers.id +LEFT JOIN pieces_justificatives ON pieces_justificatives.dossier_id = dossiers.id diff --git a/lib/tasks/opensimplif_import.rake b/lib/tasks/opensimplif_import.rake new file mode 100644 index 000000000..7131153ed --- /dev/null +++ b/lib/tasks/opensimplif_import.rake @@ -0,0 +1,118 @@ +require 'csv' +require 'json' + +namespace :opensimplif_import do + task :import_all => :environment do + puts 'start opensimplif' + + Rake::Task['opensimplif_import:import_proposition'].invoke + Rake::Task['opensimplif_import:import_piste'].invoke + Rake::Task['opensimplif_import:import_mesure'].invoke + + puts 'end import opensimplif' + end + + task :import_proposition do + file_path = "lib/tasks/161102_OS_Inputs_test_propositions.csv" + procedure_id = 35 + + matching = [ + {id: 44, key: 'Intitulé de la proposition'}, + {id: 43, key: 'Champ concerné'}, + {id: 45, key: 'Champ ministériel chef de file'}, + {id: 59, key: 'Date de la proposition'}, + {id: 60, key: 'Moment de vie'}, + {id: 61, key: 'Source'}, + {id: 48, key: 'Description de la proposition'} + ] + + puts 'start propositions' + import file_path, procedure_id, matching + puts 'done propositions' + end + + task :import_piste do + file_path = "lib/tasks/161102_OS_Inputs_test_pistes.csv" + procedure_id = 36 + + matching = [ + {id: 81, key: 'Intitulé de la piste *'}, + {id: 82, key: 'Usager concerné *'}, + {id: 83, key: 'Champ ministériel chef de file *'}, + {id: 84, key: 'Champ ministériel contributeur'}, + {id: 85, key: 'Date de saisine'}, + {id: 66, key: 'Moment de vie'}, + {id: 80, key: 'Source de la piste'}, + {id: 70, key: 'Description de la piste '}, + {id: 68, key: 'Objectifs / bénéfices attendus'}, + {id: 65, key: 'Description détaillée des démarches impactées par la piste'}, + {id: 69, key: 'Levier de mise en oeuvre'}, + {id: 67, key: 'Précision sur le levier de meo'}, + {id: 64, key: 'Calendrier de mise en oeuvre'} + ] + + puts 'start piste' + import file_path, procedure_id, matching + puts 'done pistes' + end + + task :import_mesure do + file_path = "lib/tasks/161102_OS_Inputs_test_mesures.csv" + procedure_id = 37 + + matching = [ + {id: 107, key: 'Intitulé projet / mesure'}, + {id: 104, key: 'Champ concerné'}, + {id: 105, key: 'Champ ministériel chef de file'}, + {id: 112, key: 'Direction chef de file'}, + {id: 106, key: 'Champ ministériel contributeur'}, + {id: 113, key: 'Direction contributrice'}, + {id: 92, key: 'Moment de vie'}, + {id: 109, key: 'Date d\'annonce'}, + {id: 114, key: 'N° de la mesure'}, + {id: 115, key: 'Responsable ministère'}, + {id: 116, key: 'Responsable SGMAP'}, + {id: 89, key: 'Actions réalisées'}, + {id: 95, key: 'Etapes nécessaires à l\'atteinte de la cible et alertes'}, + {id: 102, key: 'Alertes'}, + {id: 101, key: 'Échéance initiale'}, + {id: 96, key: 'Échéance prévisionnelle / réelle'}, + {id: 94, key: 'Appréciation avancement'}, + {id: 91, key: 'Etat d\'avancement LOLF'}, + {id: 111, key: '§ de com'} + ] + + puts 'start mesures' + import file_path, procedure_id, matching + puts 'done mesures' + end + + def self.import file_path, procedure_id, matching + user = User.find_or_create_by(email: 'import@opensimplif.modernisation.fr') + + unless user.valid? + user.password = 'TPSpassword2016' + user.save + end + + file ||= CSV.open(file_path, :col_sep => ";", :headers => true).map { |x| x.to_h }.to_json + file = JSON.parse(file) + + procedure = Procedure.find(procedure_id) + + user.dossiers.where(procedure_id: procedure.id).destroy_all + + file.each do |proposition| + dossier = Dossier.create procedure: procedure, user: user, state: :initiated + + dossier.champs.each do |champ| + matching.each do |match| + if match[:id] == champ.type_de_champ.id + champ.update_column :value, proposition[match[:key]] + break + end + end + end + end + end +end diff --git a/spec/controllers/backoffice/dossiers_controller_spec.rb b/spec/controllers/backoffice/dossiers_controller_spec.rb index 61c5c0428..542b7fc12 100644 --- a/spec/controllers/backoffice/dossiers_controller_spec.rb +++ b/spec/controllers/backoffice/dossiers_controller_spec.rb @@ -63,14 +63,14 @@ describe Backoffice::DossiersController, type: :controller do end end - describe 'GET #en_attente' do + describe 'GET #fige' do context 'when gestionnaire is connected' do before do sign_in gestionnaire end it 'returns http success' do - get :index, liste: :en_attente + get :index, liste: :fige expect(response).to have_http_status(200) end end diff --git a/spec/features/backoffice/navigate_to_dossier_spec.rb b/spec/features/backoffice/navigate_to_dossier_spec.rb index 972c70f58..1c19cb3f9 100644 --- a/spec/features/backoffice/navigate_to_dossier_spec.rb +++ b/spec/features/backoffice/navigate_to_dossier_spec.rb @@ -20,9 +20,9 @@ feature 'on backoffice page' do page.click_on 'Se connecter' end - context 'when he click on first dossier' do + context 'when he click on first dossier', js: true do before do - page.click_on dossier.id + page.find("#tr_dossier_#{dossier.id}").click end scenario 'it redirect to dossier page' do @@ -30,7 +30,7 @@ feature 'on backoffice page' do end end - context 'when gestionnaire have enterprise and individual dossier in his inbox' do + context 'when gestionnaire have enterprise and individual dossier in his inbox', js: true do let!(:procedure_individual) { create :procedure, libelle: 'procedure individual', administrateur: administrateur, for_individual: true } let!(:dossier_individual) { create :dossier, procedure: procedure_individual, state: 'updated' } @@ -38,7 +38,7 @@ feature 'on backoffice page' do create :assign_to, gestionnaire: gestionnaire, procedure: procedure_individual visit backoffice_path - page.click_on dossier_individual.id + page.find("#tr_dossier_#{dossier_individual.id}").click end scenario 'it redirect to dossier page' do diff --git a/spec/features/backoffice/onglets_link_spec.rb b/spec/features/backoffice/onglets_link_spec.rb index 3232d02c4..087bb3d95 100644 --- a/spec/features/backoffice/onglets_link_spec.rb +++ b/spec/features/backoffice/onglets_link_spec.rb @@ -37,7 +37,7 @@ feature 'on click on tabs button' do context 'when he click on tabs a traite' do before do visit backoffice_dossiers_url(liste: :a_traiter) - page.click_on 'Action requise 1' + page.click_on 'Ouverts 2' end scenario 'it redirect to backoffice dossier termine' do @@ -47,8 +47,8 @@ feature 'on click on tabs button' do context 'when he click on tabs en attente' do before do - visit backoffice_dossiers_url(liste: :en_attente) - page.click_on 'Attente usager 2' + visit backoffice_dossiers_url(liste: :figes) + page.click_on 'Figés 1' end scenario 'it redirect to backoffice dossier en attente' do diff --git a/spec/features/users/list_dossiers_spec.rb b/spec/features/users/list_dossiers_spec.rb index 85fdc963f..b3a767551 100644 --- a/spec/features/users/list_dossiers_spec.rb +++ b/spec/features/users/list_dossiers_spec.rb @@ -32,9 +32,9 @@ feature 'user access to the list of his dossier' do expect(page).to have_css("#dossier_#{dossier1.id}_state") end - context 'when user clicks on a projet in list' do + context 'when user clicks on a projet in list', js: true do before do - page.click_on dossier1.procedure.libelle + page.find("#tr_dossier_#{dossier1.id}").click end scenario 'user is redirected to dossier page' do expect(page).to have_css('#recap_dossier') diff --git a/spec/models/dossier_spec.rb b/spec/models/dossier_spec.rb index 5fb127019..b4ee3c005 100644 --- a/spec/models/dossier_spec.rb +++ b/spec/models/dossier_spec.rb @@ -556,87 +556,6 @@ describe Dossier do end end - describe '.search' do - subject { liste_dossiers } - - let(:liste_dossiers) { described_class.search(gestionnaire_1, terms) } - # let(:dossier) { described_class.search(gestionnaire_1, terms)[1] } - - let(:administrateur_1) { create(:administrateur) } - let(:administrateur_2) { create(:administrateur) } - - let(:gestionnaire_1) { create(:gestionnaire, administrateurs: [administrateur_1]) } - let(:gestionnaire_2) { create(:gestionnaire, administrateurs: [administrateur_2]) } - - before do - create :assign_to, gestionnaire: gestionnaire_1, procedure: procedure_1 - create :assign_to, gestionnaire: gestionnaire_2, procedure: procedure_2 - end - - let(:procedure_1) { create(:procedure, administrateur: administrateur_1) } - let(:procedure_2) { create(:procedure, administrateur: administrateur_2) } - - let!(:dossier_0) { create(:dossier, state: 'draft', procedure: procedure_1, user: create(:user, email: 'brouillon@clap.fr')) } - let!(:dossier_1) { create(:dossier, state: 'initiated', procedure: procedure_1, user: create(:user, email: 'contact@test.com')) } - let!(:dossier_2) { create(:dossier, state: 'initiated', procedure: procedure_1, user: create(:user, email: 'plop@gmail.com')) } - let!(:dossier_3) { create(:dossier, state: 'initiated', procedure: procedure_2, user: create(:user, email: 'peace@clap.fr')) } - let!(:dossier_archived) { create(:dossier, state: 'initiated', procedure: procedure_1, archived: true, user: create(:user, email: 'brouillonArchived@clap.fr')) } - - let!(:etablissement_1) { create(:etablissement, entreprise: create(:entreprise, raison_sociale: 'OCTO Academy', dossier: dossier_1), dossier: dossier_1, siret: '41636169600051') } - let!(:etablissement_2) { create(:etablissement, entreprise: create(:entreprise, raison_sociale: 'Plop octo', dossier: dossier_2), dossier: dossier_2, siret: '41816602300012') } - let!(:etablissement_3) { create(:etablissement, entreprise: create(:entreprise, raison_sociale: 'OCTO Technology', dossier: dossier_3), dossier: dossier_3, siret: '41816609600051') } - - describe 'search is empty' do - let(:terms) { '' } - - it { expect(subject.size).to eq(0) } - end - - describe 'search draft file' do - let(:terms) { 'brouillon' } - - it { expect(subject.size).to eq(0) } - it { expect(subject.class).to eq Dossier::ActiveRecord_Relation } - end - - describe 'search on contact email' do - let(:terms) { 'clap' } - - it { expect(subject.size).to eq(0) } - end - - describe 'search on ID dossier' do - let(:terms) { "#{dossier_2.id}" } - - it { expect(subject.size).to eq(1) } - end - - describe 'search on SIRET' do - context 'when is part of SIRET' do - let(:terms) { '4181' } - - it { expect(subject.size).to eq(1) } - end - - context 'when is a complet SIRET' do - let(:terms) { '41816602300012' } - - it { expect(subject.size).to eq(1) } - end - end - - describe 'search on raison social' do - let(:terms) { 'OCTO' } - - it { expect(subject.size).to eq(2) } - end - - describe 'search on multiple fields' do - let(:terms) { 'octo test' } - - it { expect(subject.size).to eq(1) } - end - end end describe '#cerfa_available?' do diff --git a/spec/models/search_spec.rb b/spec/models/search_spec.rb new file mode 100644 index 000000000..4e7d20ec2 --- /dev/null +++ b/spec/models/search_spec.rb @@ -0,0 +1,79 @@ +require 'rails_helper' + +describe Search do + describe '.results' do + subject { liste_dossiers } + + let(:liste_dossiers) do + described_class.new(gestionnaire: gestionnaire_1, query: terms).results + end + + let(:administrateur_1) { create(:administrateur) } + let(:administrateur_2) { create(:administrateur) } + + let(:gestionnaire_1) { create(:gestionnaire, administrateurs: [administrateur_1]) } + let(:gestionnaire_2) { create(:gestionnaire, administrateurs: [administrateur_2]) } + + before do + create :assign_to, gestionnaire: gestionnaire_1, procedure: procedure_1 + create :assign_to, gestionnaire: gestionnaire_2, procedure: procedure_2 + end + + let(:procedure_1) { create(:procedure, administrateur: administrateur_1) } + let(:procedure_2) { create(:procedure, administrateur: administrateur_2) } + + let!(:dossier_0) { create(:dossier, state: 'draft', procedure: procedure_1, user: create(:user, email: 'brouillon@clap.fr')) } + let!(:dossier_1) { create(:dossier, state: 'initiated', procedure: procedure_1, user: create(:user, email: 'contact@test.com')) } + let!(:dossier_2) { create(:dossier, state: 'initiated', procedure: procedure_1, user: create(:user, email: 'plop@gmail.com')) } + let!(:dossier_3) { create(:dossier, state: 'initiated', procedure: procedure_2, user: create(:user, email: 'peace@clap.fr')) } + let!(:dossier_archived) { create(:dossier, state: 'initiated', procedure: procedure_1, archived: true, user: create(:user, email: 'brouillonArchived@clap.fr')) } + + let!(:etablissement_1) { create(:etablissement, entreprise: create(:entreprise, raison_sociale: 'OCTO Academy', dossier: dossier_1), dossier: dossier_1, siret: '41636169600051') } + let!(:etablissement_2) { create(:etablissement, entreprise: create(:entreprise, raison_sociale: 'Plop octo', dossier: dossier_2), dossier: dossier_2, siret: '41816602300012') } + let!(:etablissement_3) { create(:etablissement, entreprise: create(:entreprise, raison_sociale: 'OCTO Technology', dossier: dossier_3), dossier: dossier_3, siret: '41816609600051') } + + describe 'search is empty' do + let(:terms) { '' } + + it { expect(subject.size).to eq(0) } + end + + describe 'search draft file' do + let(:terms) { 'brouillon' } + + it { expect(subject.size).to eq(0) } + end + + describe 'search on contact email' do + let(:terms) { 'clap' } + + it { expect(subject.size).to eq(0) } + end + + describe 'search on SIRET' do + context 'when is part of SIRET' do + let(:terms) { '4181' } + + it { expect(subject.size).to eq(1) } + end + + context 'when is a complet SIRET' do + let(:terms) { '41816602300012' } + + it { expect(subject.size).to eq(1) } + end + end + + describe 'search on raison social' do + let(:terms) { 'OCTO' } + + it { expect(subject.size).to eq(2) } + end + + describe 'search on multiple fields' do + let(:terms) { 'octo plop' } + + it { expect(subject.size).to eq(1) } + end + end +end diff --git a/spec/views/backoffice/dossiers/index_html.haml_spec.rb b/spec/views/backoffice/dossiers/index_html.haml_spec.rb index ddca9e976..879b76cf2 100644 --- a/spec/views/backoffice/dossiers/index_html.haml_spec.rb +++ b/spec/views/backoffice/dossiers/index_html.haml_spec.rb @@ -95,29 +95,27 @@ describe 'backoffice/dossiers/index.html.haml', type: :view do end describe 'on tab a_traiter' do - let(:total_dossiers) { 1 } + let(:total_dossiers) { 2 } let(:active_class) { '.active .text-danger' } - let(:dossiers_to_display) { gestionnaire.dossiers.waiting_for_gestionnaire } + let(:dossiers_to_display) { gestionnaire.dossiers.ouvert } let(:liste) { 'a_traiter' } it_behaves_like 'check_tab_content' do let(:decorate_dossier_at_check) { decorate_dossier_updated } end + + it_behaves_like 'check_tab_content' do + let(:decorate_dossier_at_check) { decorate_dossier_replied } + end end - describe 'on tab en_attente' do - let(:total_dossiers) { 2 } + describe 'on tab figes' do + let(:total_dossiers) { 1 } let(:active_class) { '.active .text-default' } - let(:dossiers_to_display) { gestionnaire.dossiers.waiting_for_user } - let(:liste) { 'en_attente' } + let(:dossiers_to_display) { gestionnaire.dossiers.fige } + let(:liste) { 'fige' } describe 'for state replied' do - it_behaves_like 'check_tab_content' do - let(:decorate_dossier_at_check) { decorate_dossier_replied } - end - end - - describe 'for state validated' do it_behaves_like 'check_tab_content' do let(:decorate_dossier_at_check) { decorate_dossier_validated } end