adherent 0.3.4 → 0.3.5

data/spec/features/adhesions_spec.rb

@@ -1,23 +1,17 @@
 # coding utf-8
 
 require 'rails_helper'
-require 'support/fixtures'
+
 
 RSpec.configure do |c|
   #  c.filter = {wip:true}
 end
 
 describe 'ADHESIONS', :type => :feature do  
-  include Fixtures 
-  
+  fixtures :all
   
   before(:each) do
-    create_members(1) 
-    @member= @members.first
-  end
-   
-  after(:each) do
-    Adherent::Member.delete_all
+    @member= adherent_members(:Durand)
   end
    
   describe 'création d une adhésion' do

data/spec/features/coord_spec.rb

@@ -1,5 +1,4 @@
 require 'rails_helper'
-require 'support/fixtures'
 
 
 RSpec.configure do |c|
@@ -7,12 +6,11 @@ RSpec.configure do |c|
 end
 
 describe 'COORDONNEE', :type => :feature do
-   include Fixtures 
+   fixtures :all
   
   
-   before(:each) do
-     create_members(1)
-     @member = @members.first
+   before(:each) do 
+     @member = Adherent::Member.first
    end
    
    describe 'création de coordonnées' do

data/spec/features/js_spec.rb

@@ -2,7 +2,7 @@
 
 require 'rails_helper'
 require 'database_cleaner' 
-require 'support/fixtures'
+
 
 RSpec.configure do |config|
   config.use_transactional_fixtures = false
@@ -27,16 +27,15 @@ end
 module Adherent
 
 describe 'javascript requests', :type => :feature do
-  include Fixtures
+  fixtures :all
   
-  before(:each) do
-    create_members
-    @member = @members.first
-    @domid = "#member_#{@member.id}" 
-  end
+ 
   
   describe 'delete member', js:true do 
-    it 'supprimer un membre le supprime' do 
+    
+    it 'supprimer un membre sans payment le supprime'do 
+      @member = adherent_members(:Durand)
+      @domid = "#member_#{@member.id}" 
       visit adherent.members_path
       within(@domid) do 
          click_link 'Supprimer'   
@@ -44,14 +43,15 @@ describe 'javascript requests', :type => :feature do
       alert = page.driver.browser.switch_to.alert
       alert.accept
       sleep 1
-      expect(Adherent::Member.count).to eq(4) 
+      expect(Adherent::Member.count).to eq(2) 
     end
   end
   
   describe 'delete adhesions', js:true do
     
-    it 'supprimer une adhésion dans la liste la supprime' do
-      adh = @member.next_adhesion.save
+    it 'supprimer une adhésion dans la liste la supprime'  do
+      @member = adherent_members(:Dupont)
+      nb_adhs = @member.adhesions.count
       adh = @member.adhesions.last
       adh_id = "#adhesion_#{adh.id}"
       visit adherent.member_adhesions_path(@member)
@@ -61,20 +61,22 @@ describe 'javascript requests', :type => :feature do
       alert = page.driver.browser.switch_to.alert
       alert.accept
       sleep 1
-      expect(Adherent::Adhesion.count).to eq(0)
+      expect(@member.adhesions(true).count).to eq(nb_adhs - 1)
     end
     
   end
   
-  describe 'delete payment', js:true do 
+  describe 'delete payment', js:true   do 
     it 'détruit le payment' do
-      @pay = @member.payments.create!(date:Date.today, amount:54.32, mode:'Chèque')
+      @member = adherent_members(:Dupont)
+      nb_pays = @member.payments.count
+      @pay = @member.payments.last
       visit adherent.member_payment_path(@member, @pay)
       click_link 'Supprimer'  
       alert = page.driver.browser.switch_to.alert
       alert.accept
       sleep 1
-      expect(Adherent::Payment.count).to eq(0)
+      expect(@member.payments(true).count).to eq(nb_pays - 1)
     end
   end
    

data/spec/features/members_spec.rb

@@ -1,85 +1,65 @@
 # coding utf-8
 
 require 'rails_helper'
-require 'support/fixtures'
+
 
 RSpec.configure do |c|
 #  c.filter = {wip:true}
 end
 
 describe 'afficher tous les membres', :type => :feature do
-   include Fixtures 
-  
-  
-   before(:each) do
-     create_members
-   end
+  fixtures :all
   
   it 'affiche la liste' do
     visit adherent.members_path
     expect(page).to have_selector('h3', text:'Liste des membres')
   end
   
-  it 'affiche le tableau des membres' do
+  it 'affiche le tableau des membres'  do
     visit adherent.members_path
-    expect(page.all('table > tbody > tr').size).to eq(5)
+    expect(page.all('table > tbody > tr').size).to eq(Adherent::Member.count)
   end
   
   describe 'vérification des liens' do
     
     before(:each) do 
       visit adherent.members_path
+      @m = Adherent::Member.first
     end
     
-    it 'cliquer sur le lien détail mène à la vue new coordonnées' do
-      first_id  = Adherent::Member.first.id
-      within(:css, 'table tbody tr:first') do
-        page.find("#coord_member_#{first_id}").click  
-      end
-      expect(page.find('h3')).to have_content 'Saisie des coordonnées de le prénom NOM_0' 
+    it 'sans coordonnées, renvoie vers la saisie des coordonnées' do
+      page.find("#coord_member_#{@m.id}").click   
+      expect(page.find('h3')).to have_content "Saisie des coordonnées de #{@m.to_s}"
       
     end
     
-    it 'si le membre a des coordonnées le détail les affiche' do
-      first = Adherent::Member.first
-      first.create_coord(city:'Marseille')
-      within(:css, 'table tbody tr:first') do
-        page.find("#coord_member_#{first.id}").click  
-      end
-      expect(page.find('h3')).to have_content 'Fiche coordonnées le prénom NOM_0' 
+    it 'avec, les affiche'  do 
+      m = adherent_members(:Dupont)
+      page.find("#coord_member_#{m.id}").click  
+      expect(page.find('h3')).to have_content "Saisie des coordonnées de #{m.to_s}"
     end
     
-    it 'adhesion renvoie vers nouvelle adhésion' do
-      first_id  = Adherent::Member.first.id
-      within(:css, 'table tbody tr:first') do
-        page.find("#adhesion_member_#{first_id}").click  
-      end
-      expect(page.find('h3')).to have_content 'Renouvellement ou nouvelle adhésion pour le prénom NOM_0' 
+    it 'adhesion renvoie vers nouvelle adhésion'   do
+      m = adherent_members(:Durand)
+      page.find("#adhesion_member_#{m.id}").click  
+      expect(page.find('h3')).to have_content "Renouvellement ou nouvelle adhésion pour #{m.to_s}"
     end
     
     it 'ou vers la liste des adhésions' do
-      first = Adherent::Member.first
-      first.next_adhesion.save
-      within(:css, 'table tbody tr:first') do
-        page.find("#adhesion_member_#{first.id}").click  
-      end
-      expect(page.find('h3')).to have_content 'Historique des adhésions pour le prénom NOM_0' 
+      m = adherent_members(:Dupont)
+      page.find("#adhesion_member_#{m.id}").click  
+      expect(page.find('h3')).to have_content "Historique des adhésions pour #{m.to_s}" 
     end
     
     it 'payement renvoie vers la vue index des payment' do
-      first_id  = Adherent::Member.first.id
-      within(:css, 'table tbody tr:first') do
-        page.find("#payment_member_#{first_id}").click  
-      end
-      expect(page.find('h3')).to have_content 'Historique des paiements reçus de le prénom NOM_0'
+      m = adherent_members(:Dupont)
+      page.find("#payment_member_#{m.id}").click  
+      expect(page.find('h3')).to have_content "Historique des paiements reçus de #{m.to_s}"
       
     end
     
-    it 'payement renvoie vers la vue index des payment' do  
-      first_id  = Adherent::Member.first.id
-      within(:css, 'table tbody tr:first') do
-        page.find("#edit_member_#{first_id}").click  
-      end
+    it 'on peut éditer le membre' do  
+      page.find("#edit_member_#{@m.id}").click  
       expect(page.find('h3')).to have_content 'Modification membre'
       
     end
@@ -92,7 +72,7 @@ describe 'afficher tous les membres', :type => :feature do
      
   end
   
-  describe 'create_members', wip:true do
+  describe 'create_members'  do
     it 'remplir le form crée un membre et renvoie sur la page new_coord' do
       visit adherent.new_member_path
       fill_in 'Nom', with:'James'
@@ -101,7 +81,7 @@ describe 'afficher tous les membres', :type => :feature do
       expect {click_button 'Créer le membre'}.to change {Adherent::Member.count}.by(1)
       expect(page.find('h3').text).to eq('Saisie des coordonnées de Jessie JAMES')
     end
-    
+     
     
   end
   

data/spec/features/payments_spec.rb

@@ -1,25 +1,14 @@
 # coding utf-8
 
 require 'rails_helper'
-require 'support/fixtures'
-
-RSpec.configure do |c|
-#  c.filter = {wip:true}
-end
 
 describe 'PAYMENTS', :type => :feature do
-   include Fixtures 
+   fixtures :all
    
    before(:each) do
-     create_members(1) 
-     @member= @members.first
-     @member.next_adhesion.save  
+     @member= adherent_members(:Dupont)
    end
    
-  after(:each) do
-    Adherent::Member.delete_all
-  end
-   
    describe 'création d un payment' do
      
      it 'la page new payment affiche un form' do
@@ -37,38 +26,30 @@ describe 'PAYMENTS', :type => :feature do
        expect(page.find('h3').text).to eq("Historique des paiements reçus de #{@member.to_s}") 
      end
     
-    
    end
    
   describe 'la table des payments pour un membre' do
     
-    before(:each) do
-      
-      @member.payments.create!(:date=>Date.today, :mode=>'CB', amount:876.54) 
-      
+    before(:each) do      
       visit adherent.member_payments_path @member
     end
     
-    it 'affiche les paiements' do
-      
-      expect(page.find('h3').text).to eq("Historique des paiements reçus de #{@member.to_s}")
+    it 'affiche une table des paiements' do
+      expect(page.find('h3').text).to eq("Historique des paiements reçus de #{@member.to_s}") 
       expect(page.all('table').size).to eq(1) 
     end
     
     describe 'les icones d action' do
       
-      it 'cliquer sur imputation conduit à la vue new_règlement' do
-        
+      it 'cliquer sur imputation conduit à la vue new_règlement' do 
         click_link 'Imputation'
-        expect(page.find('h3').text).to eq("Imputation du montant restant (876,54 €) payé par #{@member.to_s}")  
-        
+        expect(page.find('h3').text).to eq("Imputation du montant restant (15,00 €) payé par #{@member.to_s}")  
       end
       
-      
     end
   end
   
-  describe 'le formulaire pour un nouveau paiement' do
+  describe 'le formulaire pour un nouveau paiement'do
     
     before(:each) do
       visit adherent.new_member_payment_path @member
@@ -79,21 +60,18 @@ describe 'PAYMENTS', :type => :feature do
     end
     
     it 'le montant doit être en format anglais' do
-      expect(page.find('input#payment_amount').value).to eq('0.00')
+      
+      expect(page.find('input#payment_amount').value).to eq(@member.unpaid_amount.to_s)
     end
     
   end
   
-  describe 'l affichage d un paiement' do
+  describe 'l affichage d un paiement'  do
     before(:each) do
-      @payment = create_payment(@member)
+      @payment = adherent_payments(:pay_1)
       visit adherent.member_payment_path @member, @payment
     end
     
-    after(:each) do
-      Adherent::Payment.delete_all
-    end
-    
     it 'affiche le titre' do
       expect(page.find('h3').text).to eq("Détail d'un paiement")
     end
@@ -105,10 +83,8 @@ describe 'PAYMENTS', :type => :feature do
     end
     
     it 'comprend une partie reçu pour impression' do
-      expect(page.find('.print').text).to have_content('Le trésorier')
-    end 
+      expect(page.find('.print').text).to have_content('Le trésorier') 
+    end
   end
-   
-   
-   
+  
 end

data/spec/features/reglements_spec.rb

@@ -1,14 +1,9 @@
 # coding utf-8
 
 require 'rails_helper'
-require 'support/fixtures'
-
-RSpec.configure do |c|
- #   c.filter = {wip:true}
-end
 
 describe 'REGLEMENT', :type => :feature do 
-  include Fixtures 
+  fixtures :all
   
   def create_payment(montant)
     visit adherent.new_member_payment_path(@member)
@@ -20,15 +15,13 @@ describe 'REGLEMENT', :type => :feature do
   
   
   before(:each) do
-    create_members(3)
-    @member= @members.first
-    @member.next_adhesion(50).save!
+    @member= adherent_members(:Dupont)
   end
    
   describe 'un membre avec une adhésion effectue un paiement du montant exact' do
      
     it 'créer le payment change le nombre de payments' do
-      expect {create_payment(50)}.to change {@member.payments.count}.by(1)
+      expect {create_payment(26.66)}.to change {@member.payments(true).count}.by(1)
     end
     
     it 'créer le payment crée un règlement' do
@@ -42,10 +35,10 @@ describe 'REGLEMENT', :type => :feature do
     
   end
   
-  describe 'le membre effectue un payment d un montant supérieur' do
+  describe 'le membre effectue un payment d un montant supérieur'  do
     
     before(:each) do
-      create_payment(66)
+      create_payment(66.66) 
     end
     
     it 'l adhésion du membre est réglée' do
@@ -53,15 +46,15 @@ describe 'REGLEMENT', :type => :feature do
     end
     
     it 'il reste un montant à imputer' do
-      expect(@member.payments.last.non_impute).to eq(16) 
+      expect(@member.payments.last.non_impute).to eq(40) 
     end
     
   end
   
-   describe 'le membre effectue un payment d un montant inférieur' do
+   describe 'le membre effectue un payment d un montant inférieur'  do
     
     before(:each) do
-      create_payment(33)
+      create_payment(16.66)
     end
     
     it 'l adhésion du membre est réglée' do
@@ -69,20 +62,17 @@ describe 'REGLEMENT', :type => :feature do
     end
     
     it 'il reste un montant à imputer' do
-      expect(@member.unpaid_amount).to eq(17)
+      expect(@member.unpaid_amount).to eq(10)
     end
     
   end
   
   describe 'imputation sur d autres adhésions' do
-    
    
     before(:each) do
-      create_payment(80)
+      create_payment(46.66)
       @pay = @member.payments.last
-      @member2 = @members.second
-      @member2.next_adhesion(49).save
-      @members.third.next_adhesion(27).save
+      @nb_regs = Adherent::Reglement.count
       visit adherent.new_payment_reglement_path(@pay)
     end
    
@@ -90,7 +80,7 @@ describe 'REGLEMENT', :type => :feature do
     # l'être pour les 30 restants sur les deux autres
     it 'on vérifie' do
       expect(@member.unpaid_amount).to eq(0)
-      expect(@member.payments.last.non_impute).to eq(30)
+      expect(@member.payments.last.non_impute).to eq(20)
     end
     
     it 'la page new_reglement affiche un form' do
@@ -98,7 +88,7 @@ describe 'REGLEMENT', :type => :feature do
     end
     
     it 'le montant affiché est le montant restant' do
-      expect(page.find('#reglement_amount').value).to eq('30,00 €')
+      expect(page.find('#reglement_amount').value).to eq('20,00 €')
     end
     
     it 'le select propose les deux adhesions' do
@@ -108,35 +98,39 @@ describe 'REGLEMENT', :type => :feature do
     describe 'on choisit une option et on valide' do
       
       before(:each) do
-        select "#{@member2.to_s} - 49,00 €", :from=>'Adhésion'
+        select "#{adherent_members(:Fidele).to_s} - 24,00 €", :from=>'Adhésion'
         click_button 'Enregistrer' 
       end
       
+#      it 'on regarde' , focus:true  do
+#        visit adherent.new_payment_reglement_path(@pay)
+#        expect(page).to have_content('Bonjour')
+#      end
+      
       it 'on revient à la page index des payements' do
         expect(page.find('h3').text).to eq("Historique des paiements reçus de #{@member.to_s}")
       end
       
       it 'il y a maintenant 2 reglements' do
-        regs = Adherent::Reglement.all
-        expect(regs.size).to eq(2)        
+        expect(Adherent::Reglement.count).to eq(@nb_regs + 1)        
       end
       
-      it 'le premier appartenant à member et pour 50 €' do
+      it 'le premier appartenant à member et pour 26.66 €' do
         arf = Adherent::Reglement.first
         expect(arf.payment_id).to eq(@pay.id)
         expect(arf.adhesion.member).to eq(@member)
-        expect(arf.amount).to eq(50)
+        expect(arf.amount).to eq(26.66)
       end
       
       it 'le deuxième règlement' do
         arl = Adherent::Reglement.last
         expect(arl.payment_id).to eq(@pay.id)
-        expect(arl.adhesion.member).to eq(@member2)
-        expect(arl.amount).to eq(30)
+        expect(arl.adhesion.member).to eq(adherent_members(:Fidele))
+        expect(arl.amount).to eq(20)
       end
       
       it 'le membre2 ne doit plus que 19 €' do
-        expect(@member2.unpaid_amount).to eq(19)
+        expect(adherent_members(:Fidele).unpaid_amount).to eq(30)
       end
       
       

data/spec/models/adherent/adhesion_spec.rb

@@ -1,21 +1,20 @@
 # coding utf-8
 
 require 'rails_helper'
-require 'support/fixtures'
+
 
 RSpec.configure do |c|
   # c.filter = {wip:true}
 end
 
 describe 'Adhesion', :type => :model do
-  include Fixtures 
+  
+  
+  fixtures :all
   
   
   before(:each) do
-     @adh = Adherent::Adhesion.new(:from_date=>Date.today,
-      :to_date=>(Date.today.years_since(1)-1),
-      :amount=>19.27)
-    @adh.member_id = 1
+     @adh = adherent_adhesions(:adh_1)
   end
   
   describe 'validations' do
@@ -92,7 +91,7 @@ describe 'Adhesion', :type => :model do
     
     it 'due est la différence entre amount et received' do
       allow(@adh).to receive(:received).and_return 19
-      expect(@adh.due).to eq(0.27)
+      expect(@adh.due).to eq(7.66)
     end
     
     it 'is_paid indique si due vaut zero' do
@@ -109,42 +108,23 @@ describe 'Adhesion', :type => :model do
   end
   
   # on utilise des modèles réels car on veut tester la requête proprement dite
-  describe 'Ahesion.unpaid' , wip:true do
-    
+  describe 'Ahesion.unpaid' do
     
     before(:each) do
-      create_members(2)
-      @m1 = @members.first; @m2 = @members.last
-     @a1 =  @m1.adhesions.create!(amount:100, from_date:'01/08/2013', to_date:'31/08/2013')
-     @a2 =  @m2.adhesions.create!(amount:50, from_date:'01/08/2013', to_date:'31/08/2013')
-     @m2.payments.create!(date:Date.today, amount:40, mode:'CB')
-      
-    end
-    
-    after(:each) do
-      Adherent::Member.delete_all
+      @m3 = adherent_members(:Fidele)
+      @nb_unpaid = Adherent::Adhesion.unpaid.to_a.size
     end
     
-    it 'requete renvoyant les adhesions impayées' do
-       expect(Adherent::Adhesion.unpaid.to_a.size).to eq(2)
-       expect(Adherent::Adhesion.unpaid.first.reglements_amount.to_i).to eq(0)
-       expect(Adherent::Adhesion.unpaid.last.reglements_amount.to_i).to eq(40)
+    it 'sait calculer le montant total du'  do
+      expect(Adherent::Adhesion.unpaid.to_a.sum(&:amount)).to eq(76.66)
     end
-    
-    it 'on rajoute un payment' do
-      @m1.payments.create!(date:Date.today, amount:100, mode:'CB')
-      expect(Adherent::Adhesion.unpaid.to_a.size).to eq(1)
-      expect(Adherent::Adhesion.unpaid.first).to eq(@a2)
-      expect(Adherent::Adhesion.unpaid.first.reglements_amount.to_i).to eq(40)
-    end
-    
-    it 'on paye la dernière' do
-      @m2.payments.create!(date:Date.today, amount:10, mode:'CB')
-      @m1.payments.create!(date:Date.today, amount:100, mode:'CB')
-      expect(Adherent::Adhesion.unpaid.to_a.size).to eq(0)
+        
+    it 'un paiement suffisant pour une adhésion règle une adhésion' do
+       @m3.payments.create!(date:Date.today, amount:40, mode:'CB')
+       expect(Adherent::Adhesion.unpaid.to_a.size).to eq(@nb_unpaid - 1)
+       expect(Adherent::Adhesion.unpaid.to_a.sum(&:amount)).to eq(52.66) # celle de 24
     end
-    
-    
+     
   end
   
   describe 'next adh values' do
@@ -191,8 +171,6 @@ describe 'Adhesion', :type => :model do
   
   describe 'add_reglement' do
     
-    before(:each) {@adh.save!}
-    
     it 'le montant imputé est plafonné' do
       @adh.add_reglement(1, 50)
       expect(@adh).to be_is_paid
@@ -201,7 +179,7 @@ describe 'Adhesion', :type => :model do
     it 'le montant imputé est celui du payment' do
       @adh.add_reglement(1, 10)
       expect(@adh).not_to be_is_paid
-      expect(@adh.due).to eq(9.27)
+      expect(@adh.due).to eq(16.66)
     end
     
     it 'add_reglement renvoie un règlement' do

data/spec/models/adherent/coord_spec.rb

@@ -1,10 +1,12 @@
-# coding utf-8
+# coding utf-8 
 
 require 'rails_helper'
 
 describe 'Coord', :type => :model do 
+  fixtures :all
+  
   before(:each) do
-    @m = double(Adherent::Member)
+    @m = adherent_members(:Dupont) 
   end
   
   it 'les coordonnées sont rattachées à un membre' do
@@ -15,12 +17,13 @@ describe 'Coord', :type => :model do
   end
   
   it 'coord est dépendant du membre' do
-    m = Adherent::Member.new(number:'Adh1', name:'James', forname:'Jessie')
-    m.organism_id = 1
-    m.save
-    @c = m.create_coord(city:'Lille', zip:59000)
-    expect(Adherent::Coord.count).to eq(1)
-    m.destroy
-    expect(Adherent::Coord.count).to eq(0)
+    expect{@m.create_coord(city:'Lille', zip:59000)}. 
+      to change{Adherent::Coord.count}.by 1
+  end
+  
+  it 'détruire le membre détruit ses coordonnées' do
+    m = adherent_members(:Durand)
+    expect{adherent_members(:Durand).destroy}.
+      to change{Adherent::Coord.count}.by -1
   end
 end