From f20f561e6ce900e06f76401c9314fc7b3df4acfe Mon Sep 17 00:00:00 2001 From: Xavier J Date: Thu, 24 Sep 2015 11:37:18 +0200 Subject: [PATCH] Fix liens page description et recapitulatif --- .../users/description_controller.rb | 1 + app/views/dossiers/_infos_dossier.html.haml | 2 +- .../_modifications_terminees.html.haml | 2 +- app/views/users/recapitulatif/show.html.haml | 4 -- .../recapitulatif/show.html_BACKUP_4662.haml | 29 --------- .../recapitulatif/show.html_BASE_4662.haml | 24 -------- .../recapitulatif/show.html_LOCAL_4662.haml | 25 -------- .../recapitulatif/show.html_REMOTE_4662.haml | 23 -------- .../description_page/show_page_spec.rb | 2 +- .../_commentaires_flux_spec.rb | 38 ------------ .../recapitulatif_page/show_page_spec.rb | 59 ------------------- .../recapitulatif/show.html.haml_spec.rb | 57 ++++++++++++++++-- 12 files changed, 57 insertions(+), 209 deletions(-) delete mode 100644 app/views/users/recapitulatif/show.html_BACKUP_4662.haml delete mode 100644 app/views/users/recapitulatif/show.html_BASE_4662.haml delete mode 100644 app/views/users/recapitulatif/show.html_LOCAL_4662.haml delete mode 100644 app/views/users/recapitulatif/show.html_REMOTE_4662.haml delete mode 100644 spec/features/recapitulatif_page/_commentaires_flux_spec.rb delete mode 100644 spec/features/recapitulatif_page/show_page_spec.rb diff --git a/app/controllers/users/description_controller.rb b/app/controllers/users/description_controller.rb index d952cab2d..f2be1d8d6 100644 --- a/app/controllers/users/description_controller.rb +++ b/app/controllers/users/description_controller.rb @@ -46,6 +46,7 @@ class Users::DescriptionController < ApplicationController commentaire.save end + flash.notice = 'Félicitation, votre demande a bien été enregistrée.' redirect_to url_for(controller: :recapitulatif, action: :show, dossier_id: @dossier.id) end diff --git a/app/views/dossiers/_infos_dossier.html.haml b/app/views/dossiers/_infos_dossier.html.haml index 634639366..972f06a09 100644 --- a/app/views/dossiers/_infos_dossier.html.haml +++ b/app/views/dossiers/_infos_dossier.html.haml @@ -38,5 +38,5 @@ -#= pie_chart({"Montant à charge #{(100 - @dossier.montant_aide_demande.to_f/@dossier.montant_projet.to_f*100).round(2)}%" => (@dossier.montant_projet.to_f - @dossier.montant_aide_demande.to_f), "Montant souhaité #{(@dossier.montant_aide_demande.to_f/@dossier.montant_projet.to_f*100).round(2)}%" => @dossier.montant_aide_demande}) %div.row{style: 'text-align:right'} - %a#maj_infos.btn.btn-info{href: "/dossiers/#{@dossier.id}/description?back_url=recapitulatif"} + %a#maj_infos.btn.btn-info{href: "/users/dossiers/#{@dossier.id}/description?back_url=recapitulatif"} = 'Editer mon dossier' \ No newline at end of file diff --git a/app/views/layouts/_modifications_terminees.html.haml b/app/views/layouts/_modifications_terminees.html.haml index 34705d839..c4bfc57fb 100644 --- a/app/views/layouts/_modifications_terminees.html.haml +++ b/app/views/layouts/_modifications_terminees.html.haml @@ -1,3 +1,3 @@ %div{style: 'float:right'} - %a.btn{href: "/dossiers/#{@dossier.id}/recapitulatif"} Retour + %a.btn{href: "/users/dossiers/#{@dossier.id}/recapitulatif"} Retour = submit_tag 'Modification terminée', class: %w(btn btn btn-info), id: 'modification_terminee', data: { disable_with: 'Modification terminée', submit: true}, onclick: 'submit_check_draw(event)' diff --git a/app/views/users/recapitulatif/show.html.haml b/app/views/users/recapitulatif/show.html.haml index f5e5f09e2..1c415d7f3 100644 --- a/app/views/users/recapitulatif/show.html.haml +++ b/app/views/users/recapitulatif/show.html.haml @@ -14,10 +14,6 @@ %button#action_button.btn.btn-success = 'Soumettre mon dossier' -%div{style: 'text-align:center'} - -if (request.referer != nil) && (request.referer.include?'/description') && !(request.referer.include?'back_url=recapitulatif') - %h3.text-success Félicitation, votre demande a bien été enregistrée. - %br = render partial: '/dossiers/infos_dossier' diff --git a/app/views/users/recapitulatif/show.html_BACKUP_4662.haml b/app/views/users/recapitulatif/show.html_BACKUP_4662.haml deleted file mode 100644 index 8e2f661b2..000000000 --- a/app/views/users/recapitulatif/show.html_BACKUP_4662.haml +++ /dev/null @@ -1,29 +0,0 @@ -<<<<<<< HEAD:app/views/recapitulatif/show.html.haml -%div.row - %div.col-md-2.col-lg-2 - %h2 - ='Récapitulatif' -======= -%h2#recap_dossier Récapitulatif ->>>>>>> 6eeac88c95dce079195f596e05a7e652fd9720c9:app/views/users/recapitulatif/show.html.haml - - %div.col-md-8.col-lg-8 - - %div.col-md-2.col-lg-2 - %h2#dossier_id{:class => 'text-info', :style => 'text-align:right; margin-bottom:15px'} - = "Dossier n°#{@dossier.id}" - -#%h3{:class => 'text-success', :style => 'text-align:right'} - -# = @dossier.state_fr - - unless gestionnaire_signed_in? - %button#action_button.btn.btn-success - = 'Soumettre mon dossier' - -%div{style: 'text-align:center'} - -if (request.referer != nil) && (request.referer.include?'/description') && !(request.referer.include?'back_url=recapitulatif') - %h3.text-success Félicitation, votre demande a bien été enregistrée. - -%br - -= render partial: '/dossiers/infos_dossier' -%br -= render partial: 'commentaires_flux' \ No newline at end of file diff --git a/app/views/users/recapitulatif/show.html_BASE_4662.haml b/app/views/users/recapitulatif/show.html_BASE_4662.haml deleted file mode 100644 index 571490579..000000000 --- a/app/views/users/recapitulatif/show.html_BASE_4662.haml +++ /dev/null @@ -1,24 +0,0 @@ -%h2 - ='Récapitulatif' - -%div{style: 'text-align:center'} - -if request.referer != nil - -if (request.referer.include?'/description') && !(request.referer.include?'back_url=recapitulatif') - %h4.text-success Félicitation, votre demande a bien été enregistrée. - - %h3{style: 'text-align:center; line-height:1.5em'} - ='Votre dossier est le ' - %br - %span{id: 'dossier_id', style: 'font-weight:bold;', class: 'text-success'} - ="n°#{@dossier.id}" - -else - %h2#dossier_id{:class => 'text-info', :style => 'text-align:right'} - = "Dossier n°#{@dossier.id}" - -else - %h2#dossier_id{:class => 'text-info', :style => 'text-align:right'} - = "Dossier n°#{@dossier.id}" - -= render partial: '/dossiers/infos_dossier' -%br - -= render partial: 'commentaires_flux' \ No newline at end of file diff --git a/app/views/users/recapitulatif/show.html_LOCAL_4662.haml b/app/views/users/recapitulatif/show.html_LOCAL_4662.haml deleted file mode 100644 index 400b8e6b0..000000000 --- a/app/views/users/recapitulatif/show.html_LOCAL_4662.haml +++ /dev/null @@ -1,25 +0,0 @@ -%div.row - %div.col-md-2.col-lg-2 - %h2 - ='Récapitulatif' - - %div.col-md-8.col-lg-8 - - %div.col-md-2.col-lg-2 - %h2#dossier_id{:class => 'text-info', :style => 'text-align:right; margin-bottom:15px'} - = "Dossier n°#{@dossier.id}" - -#%h3{:class => 'text-success', :style => 'text-align:right'} - -# = @dossier.state_fr - - unless gestionnaire_signed_in? - %button#action_button.btn.btn-success - = 'Soumettre mon dossier' - -%div{style: 'text-align:center'} - -if (request.referer != nil) && (request.referer.include?'/description') && !(request.referer.include?'back_url=recapitulatif') - %h3.text-success Félicitation, votre demande a bien été enregistrée. - -%br - -= render partial: '/dossiers/infos_dossier' -%br -= render partial: 'commentaires_flux' \ No newline at end of file diff --git a/app/views/users/recapitulatif/show.html_REMOTE_4662.haml b/app/views/users/recapitulatif/show.html_REMOTE_4662.haml deleted file mode 100644 index 96d1e714f..000000000 --- a/app/views/users/recapitulatif/show.html_REMOTE_4662.haml +++ /dev/null @@ -1,23 +0,0 @@ -%h2#recap_dossier Récapitulatif - -%div{style: 'text-align:center'} - -if request.referer != nil - -if (request.referer.include?'/description') && !(request.referer.include?'back_url=recapitulatif') - %h4.text-success Félicitation, votre demande a bien été enregistrée. - - %h3{style: 'text-align:center; line-height:1.5em'} - ='Votre dossier est le ' - %br - %span{id: 'dossier_id', style: 'font-weight:bold;', class: 'text-success'} - ="n°#{@dossier.id}" - -else - %h2#dossier_id{:class => 'text-info', :style => 'text-align:right'} - = "Dossier n°#{@dossier.id}" - -else - %h2#dossier_id{:class => 'text-info', :style => 'text-align:right'} - = "Dossier n°#{@dossier.id}" - -= render partial: '/dossiers/infos_dossier' -%br - -= render partial: 'commentaires_flux' \ No newline at end of file diff --git a/spec/features/description_page/show_page_spec.rb b/spec/features/description_page/show_page_spec.rb index d70447fb7..cd8f99e96 100644 --- a/spec/features/description_page/show_page_spec.rb +++ b/spec/features/description_page/show_page_spec.rb @@ -79,7 +79,7 @@ feature 'Description#Show Page' do end scenario 'le lien de retour au récapitulatif est présent' do - expect(page).to have_selector("a[href='/dossiers/#{dossier_id}/recapitulatif']") + expect(page).to have_selector("a[href='/users/dossiers/#{dossier_id}/recapitulatif']") end end diff --git a/spec/features/recapitulatif_page/_commentaires_flux_spec.rb b/spec/features/recapitulatif_page/_commentaires_flux_spec.rb deleted file mode 100644 index 4da764172..000000000 --- a/spec/features/recapitulatif_page/_commentaires_flux_spec.rb +++ /dev/null @@ -1,38 +0,0 @@ -require 'spec_helper' - -feature '_Commentaires_Flux Recapitulatif#Show Page' do - let(:dossier) { create(:dossier, :with_user, :with_procedure) } - let(:dossier_id) { dossier.id } - let(:email_commentaire) { 'mon_mail_de_commentaire@test.com' } - let!(:commentaire) { create(:commentaire, dossier: dossier, email: email_commentaire, body: 'ma super description') } - let(:body) { 'Commentaire de test' } - - before do - login_as(dossier.user, scope: :user) - visit "/users/dossiers/#{dossier_id}/recapitulatif" - end - - context 'Affichage du flux de commentaire' do - scenario 'l\'email du contact est présent' do - expect(page).to have_css('#email_contact') - end - - scenario 'la date du commentaire est présent' do - expect(page).to have_selector('span[id=created_at]') - end - - scenario 'le corps du commentaire est présent' do - expect(page).to have_css('.description#body') - end - end - - context 'Affichage du formulaire de commentaire' do - scenario 'Le formulaire envoie vers /dossiers/:dossier_id/commentaire en #POST' do - expect(page).to have_selector("form[action='/users/dossiers/#{dossier_id}/commentaire'][method=post]") - end - - scenario 'Champs de texte' do - expect(page).to have_selector('textarea[id=texte_commentaire][name=texte_commentaire]') - end - end -end diff --git a/spec/features/recapitulatif_page/show_page_spec.rb b/spec/features/recapitulatif_page/show_page_spec.rb deleted file mode 100644 index da9a6ffea..000000000 --- a/spec/features/recapitulatif_page/show_page_spec.rb +++ /dev/null @@ -1,59 +0,0 @@ -require 'spec_helper' - -feature 'Recapitulatif#Show Page' do - let(:dossier) { create(:dossier, :with_user, :with_procedure) } - let(:dossier_id) { dossier.id } - - before do - Capybara.current_session.driver.header('Referer', '/description') - login_as(dossier.user, :scope => :user) - visit "/users/dossiers/#{dossier_id}/recapitulatif" - end - - context 'sur la page recapitulative' do - scenario 'la section infos dossier est présente' do - expect(page).to have_selector('#infos_dossier') - end - - scenario 'le flux de commentaire est présent' do - expect(page).to have_selector('#commentaires_flux') - end - - scenario 'le numéro de dossier est présent' do - expect(page).to have_selector('#dossier_id') - expect(page).to have_content(dossier_id) - end - - context 'les liens de modifications' do - context 'lien description' do - scenario 'le lien vers description est présent' do - expect(page).to have_css('#maj_infos') - end - - scenario 'le lien vers description est correct' do - expect(page).to have_selector("a[id=maj_infos][href='/dossiers/#{dossier_id}/description?back_url=recapitulatif']") - end - end - end - - context 'visibilité Félicitation' do - scenario 'Est affiché quand l\'on vient de la page description hors modification' do - expect(page).to have_content('Félicitation') - end - - scenario 'N\'est pas affiché quand l\'on vient d\'une autre la page que description' do - Capybara.current_session.driver.header('Referer', '/') - visit "/users/dossiers/#{dossier_id}/recapitulatif" - - expect(page).to_not have_content('Félicitation') - end - - scenario 'N\'est pas affiché quand l\'on vient de la page description en modification' do - Capybara.current_session.driver.header('Referer', '/description?back_url=recapitulatif') - visit "/users/dossiers/#{dossier_id}/recapitulatif" - - expect(page).to_not have_content('Félicitation') - end - end - end -end diff --git a/spec/views/users/recapitulatif/show.html.haml_spec.rb b/spec/views/users/recapitulatif/show.html.haml_spec.rb index e65282685..3db5d2a0a 100644 --- a/spec/views/users/recapitulatif/show.html.haml_spec.rb +++ b/spec/views/users/recapitulatif/show.html.haml_spec.rb @@ -1,12 +1,61 @@ require 'spec_helper' -describe 'recapitulatif/show.html.haml', type: :view do - let(:dossier) { create(:dossier, :with_entreprise) } +describe 'users/recapitulatif/show.html.haml', type: :view do + let(:dossier) { create(:dossier, :with_entreprise, :with_procedure) } + let(:dossier_id) { dossier.id } + before do assign(:dossier, dossier.decorate) + assign(:procedure, dossier.procedure) assign(:commentaires, dossier.commentaires) render end - # it { expect(rendered).to have_content("Contacter l'administration") } - # it { expect(rendered).to include(dossier.mailto.gsub('&','&')) } + + context 'sur la rendered recapitulative' do + it 'la section infos dossier est présente' do + expect(rendered).to have_selector('#infos_dossier') + end + + it 'le flux de commentaire est présent' do + expect(rendered).to have_selector('#commentaires_flux') + end + + it 'le numéro de dossier est présent' do + expect(rendered).to have_selector('#dossier_id') + expect(rendered).to have_content(dossier_id) + end + + context 'les liens de modifications' do + context 'lien description' do + it 'le lien vers description est présent' do + expect(rendered).to have_css('#maj_infos') + end + + it 'le lien vers description est correct' do + expect(rendered).to have_selector("a[id=maj_infos][href='/users/dossiers/#{dossier_id}/description?back_url=recapitulatif']") + end + end + end + + # context 'visibilité Félicitation' do + # it 'Est affiché quand l\'on vient de la rendered description hors modification' do + # expect(rendered).to have_content('Félicitation') + # end + # + # it 'N\'est pas affiché quand l\'on vient d\'une autre la rendered que description' do + # Capybara.current_session.driver.header('Referer', '/') + # + # expect(rendered).to_not have_content('Félicitation') + # end + # + # it 'N\'est pas affiché quand l\'on vient de la rendered description en modification' do + # Capybara.current_session.driver.header('Referer', '/description?back_url=recapitulatif') + # + # expect(rendered).to_not have_content('Félicitation') + # end + # end + end + + + end