add validation presence user for dossier

app/controllers/users/dossiers_controller.rb

@@ -18,7 +18,7 @@ class Users::DossiersController < UsersController
     procedure = Procedure.find(params['procedure_id'])
     @etablissement = Etablissement.new(SIADE::EtablissementAdapter.new(siret).to_params)
     @entreprise = Entreprise.new(SIADE::EntrepriseAdapter.new(siren).to_params)
-    @dossier = Dossier.create
+    @dossier = Dossier.create(user: current_user)
     @dossier.draft!
 
     @dossier.procedure = procedure

app/models/dossier.rb

@@ -28,7 +28,7 @@ class Dossier < ActiveRecord::Base
   validates :montant_projet, presence: true, allow_blank: false, allow_nil: true
   validates :montant_aide_demande, presence: true, allow_blank: false, allow_nil: true
   validates :date_previsionnelle, presence: true, allow_blank: false,  unless: Proc.new { description.nil? }
-
+  validates :user, presence: true
 
   def retrieve_piece_justificative_by_type(type)
     pieces_justificatives.where(type_de_piece_justificative_id: type).last

spec/controllers/backoffice/commentaires_controller_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Backoffice::CommentairesController, type: :controller do
-  let(:dossier) { create(:dossier) }
+  let(:dossier) { create(:dossier, :with_user) }
   let(:dossier_id) { dossier.id }
   let(:email_commentaire) { 'test@test.com' }
   let(:texte_commentaire) { 'Commentaire de test' }

spec/controllers/backoffice/dossiers_controller_spec.rb

@@ -1,7 +1,7 @@
 require 'rails_helper'
 
 describe Backoffice::DossiersController, type: :controller do
-  let(:dossier) { create(:dossier, :with_entreprise) }
+  let(:dossier) { create(:dossier, :with_entreprise, :with_user) }
   let(:dossier_id) { dossier.id }
   let(:bad_dossier_id) { Dossier.count + 10 }
   let(:gestionnaire) { create(:gestionnaire) }

spec/controllers/description_controller_spec.rb

@@ -1,7 +1,8 @@
 require 'spec_helper'
 
 describe Users::DescriptionController, type: :controller do
-  let(:dossier) { create(:dossier, :with_procedure) }
+  let(:user) { create(:user) }
+  let(:dossier) { create(:dossier, :with_procedure, user: user) }
   let(:dossier_id) { dossier.id }
   let(:bad_dossier_id) { Dossier.count + 10 }
 

spec/controllers/users/dossiers_controller_spec.rb

@@ -1,6 +1,7 @@
 require 'spec_helper'
 
 describe Users::DossiersController, type: :controller do
+  let(:user) { create(:user) }
   describe '.index' do
     subject { get :index }
     context 'when user is not logged in' do
@@ -14,7 +15,7 @@ describe Users::DossiersController, type: :controller do
       it { is_expected.to have_http_status(:success) }
     end
   end
-    let(:dossier) { create(:dossier, :with_entreprise, :with_procedure) }
+  let(:dossier) { create(:dossier, :with_entreprise, :with_procedure, user: user) }
   let(:procedure) { create(:procedure) }
   let(:dossier_id) { dossier.id }
   let(:siret_not_found) { 999_999_999_999 }
@@ -51,10 +52,11 @@ describe Users::DossiersController, type: :controller do
     end
 
     describe 'professionnel fills form' do
+      let(:user) { create(:user) }
       context 'when pro_dossier_id is empty' do
         context 'with valid siret ' do
           before do
-            sign_in create(:user)
+            sign_in user
           end
 
           subject  { post :create, siret: siret, pro_dossier_id: '', procedure_id: Procedure.last }
@@ -69,9 +71,15 @@ describe Users::DossiersController, type: :controller do
           end
 
           it 'links entreprise to dossier' do
+            subject
             expect(Entreprise.last.dossier).to eq(Dossier.last)
           end
 
+          it "links dossier to user" do
+            subject
+            expect(Dossier.last.user).to eq(user)
+          end
+
           it 'creates etablissement for dossier' do
             expect { subject }.to change { Etablissement.count }.by(1)
           end

spec/decorators/dossier_decorator_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe DossierDecorator do
-  let(:dossier) { create(:dossier) }
+  let(:dossier) { create(:dossier, :with_user) }
   subject { dossier.decorate }
   describe 'last_update' do
     subject { Timecop.freeze(Time.new(2015, 12, 24, 14, 10)) { super().last_update } }

spec/features/backoffice/add_commentaire_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 feature 'add commentaire on backoffice' do
-  let(:dossier) { create(:dossier, :with_entreprise, :with_procedure) }
+  let(:dossier) { create(:dossier, :with_entreprise, :with_procedure, :with_user) }
   let(:dossier_id) { dossier.id }
   let!(:commentaire) { create(:commentaire, dossier: dossier, email: 'toto@toto.com') }
   let(:email_commentaire) { 'test@test.com' }

spec/features/backoffice/connection_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 feature 'when gestionnaire come to /backoffice and is not authenticated' do
   let(:procedure) { create(:procedure) }
-  let!(:dossier) { create(:dossier, procedure: procedure) }
+  let!(:dossier) { create(:dossier, :with_user, procedure: procedure) }
   before do
     visit backoffice_path
   end

spec/features/backoffice/display_dossier_page_spec.rb

@@ -1,7 +1,8 @@
 require 'spec_helper'
 
 feature 'Admin/Dossier#Show Page' do
-  let!(:dossier) { create(:dossier, :with_entreprise, :with_procedure) }
+  let(:user) { create(:user) }
+  let!(:dossier) { create(:dossier, :with_entreprise, :with_procedure, user: user) }
   let(:dossier_id) { dossier.id }
 
   before do

spec/features/backoffice/navigate_to_dossier_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 feature 'on backoffice page' do
   let(:procedure) { create(:procedure) }
-  let!(:dossier) { create(:dossier, :with_entreprise, procedure: procedure) }
+  let!(:dossier) { create(:dossier, :with_user, :with_entreprise, procedure: procedure) }
   before do
     visit backoffice_path
   end

spec/features/datepicker_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 feature 'On the description page' do
-  let!(:dossier) { create(:dossier, :with_entreprise, :with_procedure) }
+  let!(:dossier) { create(:dossier, :with_entreprise, :with_procedure, :with_user) }
   before do
     visit users_dossier_description_path dossier
   end

spec/features/description_page/show_page_spec.rb

@@ -1,7 +1,8 @@
 require 'spec_helper'
 
 feature 'Description#Show Page' do
-  let(:dossier) { create(:dossier, :with_procedure) }
+  let(:user) { create(:user) }
+  let(:dossier) { create(:dossier, :with_procedure, user: user) }
   let(:dossier_id) { dossier.id }
 
   before do
@@ -89,7 +90,8 @@ feature 'Description#Show Page' do
              description: 'Description de test',
              montant_projet: 12_000,
              montant_aide_demande: 3000,
-             date_previsionnelle: '20/01/2016')
+             date_previsionnelle: '20/01/2016',
+             user: user)
     end
 
     scenario 'Nom du projet' do

spec/features/description_page/upload_piece_justificative_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 feature 'user is on description page' do
-  let(:dossier) { create(:dossier, :with_entreprise, :with_procedure) }
+  let(:dossier) { create(:dossier, :with_entreprise, :with_procedure, :with_user) }
   before do
     visit users_dossier_description_path dossier
   end

spec/features/users/list_dossiers_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 feature 'user access to the list of his dossier' do
 
   let(:user) { create(:user) }
-  let!(:dossier1) { create(:dossier) }
+  let!(:dossier1) { create(:dossier, :with_user) }
   before do
     visit root_path
     page.find_by_id('user_email').set user.email

spec/models/dossier_spec.rb

@@ -1,6 +1,7 @@
 require 'spec_helper'
 
 describe Dossier do
+  let(:user) { create(:user) }
   describe 'database columns' do
     it { is_expected.to have_db_column(:description) }
     it { is_expected.to have_db_column(:autorisation_donnees) }
@@ -54,7 +55,7 @@ describe Dossier do
   end
 
   describe 'methods' do
-    let(:dossier) { create(:dossier, :with_entreprise, :with_procedure) }
+    let(:dossier) { create(:dossier, :with_entreprise, :with_procedure, user: user) }
 
     let(:entreprise) { dossier.entreprise }
     let(:etablissement) { dossier.etablissement }
@@ -71,7 +72,7 @@ describe Dossier do
 
     describe 'creation' do
       it 'create default cerfa' do
-        expect { described_class.create }.to change { Cerfa.count }.by(1)
+        expect { described_class.create(user: user) }.to change { Cerfa.count }.by(1)
       end
 
       it 'link cerfa to dossier' do
@@ -94,7 +95,7 @@ describe Dossier do
 
     describe '#build_default_pieces_justificatives' do
       context 'when dossier is linked to a procedure' do
-        let(:dossier) { create(:dossier, :with_procedure) }
+        let(:dossier) { create(:dossier, :with_procedure, user: user) }
         it 'build all pieces justificatives needed' do
           expect(dossier.pieces_justificatives.count).to eq(2)
         end
@@ -102,7 +103,7 @@ describe Dossier do
     end
 
     describe '#save' do
-      subject { create(:dossier, procedure_id: nil) }
+      subject { create(:dossier, procedure_id: nil, user: user) }
       context 'when is linked to a procedure' do
         it 'creates default pieces justificatives' do
           expect(subject).to receive(:build_default_pieces_justificatives)

spec/views/backoffice/index.html.haml_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe 'backoffice/index.html.haml', type: :view do
   let!(:procedure) { create(:procedure) }
-  let!(:decorated_dossier) { create(:dossier, procedure: procedure).decorate }
+  let!(:decorated_dossier) { create(:dossier, :with_user, procedure: procedure).decorate }
   before do
     assign(:dossiers, [decorated_dossier])
     render

spec/views/dossiers/show.html.haml_spec.rb

@@ -1,7 +1,8 @@
 require 'spec_helper'
 
 describe 'dossiers/show.html.haml', type: :view do
-  let(:dossier) { create(:dossier, :with_entreprise) }
+  let(:user) { create(:user) }
+  let(:dossier) { create(:dossier, :with_entreprise, user: user) }
 
   before do
     assign(:dossier, dossier)

spec/views/users/dossiers/index.html.haml_spec.rb

@@ -3,8 +3,8 @@ require 'spec_helper'
 describe 'users/dossiers/index.html.haml', type: :view do
 
   describe 'list dossiers' do
-    let(:dossier1) { create(:dossier).decorate }
-    let(:dossier2) { create(:dossier).decorate }
+    let(:dossier1) { create(:dossier, :with_user).decorate }
+    let(:dossier2) { create(:dossier, :with_user).decorate }
     before do
       assign(:dossiers, [dossier1, dossier2])
       render