Also ensure correctness of membership_fee_paid_on field

diff --git a/features/pre-commit-hook.feature b/features/pre-commit-hook.feature
index cbcbcfa517e241f1a94de4580d5c1c8b8b2eaa6d..8f59ee37464ea918b2b91f06d1c843a90336a870 100644 (file)
--- a/features/pre-commit-hook.feature
+++ b/features/pre-commit-hook.feature
@@ -45,6 +45,16 @@ Fonctionnalité: pre-commit hook Git
     Et que je fais un `commit` du nouveau fichier
     Alors je dois voir comme erreur "pas le bon format"
 
     Et que je fais un `commit` du nouveau fichier
     Alors je dois voir comme erreur "pas le bon format"
 

+  Scénario: Commit d'une fiche avec une date de cotisation incensée
+    Lorsque j'ajoute une fiche avec comme date de cotisation "2011-99-01"
+    Et que je fais un `commit` du nouveau fichier
+    Alors je dois voir comme erreur "pas le bon format"
+
+  Scénario: Commit d'une fiche avec une mauvaise date de cotisation
+    Lorsque j'ajoute une fiche avec comme date de cotisation "janvier 2013"
+    Et que je fais un `commit` du nouveau fichier
+    Alors je dois voir comme erreur "pas le bon format"
+

   Scénario: Suppression du nom sur une fiche existante
     Lorsque je supprime le nom sur une fiche existante
     Et que je fais un `commit` du nouveau fichier
   Scénario: Suppression du nom sur une fiche existante
     Lorsque je supprime le nom sur une fiche existante
     Et que je fais un `commit` du nouveau fichier

diff --git a/features/pre-receive-hook.feature b/features/pre-receive-hook.feature
index 70507596b8356af77c2a642a37fa6a56a2d687ef..aa7f405bb31b556b288abd58ec5ba753d1b5d9bb 100644 (file)
--- a/features/pre-receive-hook.feature
+++ b/features/pre-receive-hook.feature
@@ -45,6 +45,16 @@ Fonctionnalité: pre-receive hook Git
     Et que je pousse la modification
     Alors je dois voir comme erreur "pas le bon format"
 
     Et que je pousse la modification
     Alors je dois voir comme erreur "pas le bon format"
 

+  Scénario: Commit d'une fiche avec une date de cotisation incensée
+    Lorsque j'ajoute une fiche avec comme date de cotisation "2011-99-01"
+    Et que je pousse la modification
+    Alors je dois voir comme erreur "pas le bon format"
+
+  Scénario: Commit d'une fiche avec une mauvaise date de cotisation
+    Lorsque j'ajoute une fiche avec comme date de cotisation "janvier 2013"
+    Et que je pousse la modification
+    Alors je dois voir comme erreur "pas le bon format"
+

   Scénario: Suppression du nom sur une fiche existante
     Lorsque je supprime le nom sur une fiche existante
     Et que je pousse la modification
   Scénario: Suppression du nom sur une fiche existante
     Lorsque je supprime le nom sur une fiche existante
     Et que je pousse la modification

diff --git a/features/step_definitions/subscriptions.rb b/features/step_definitions/subscriptions.rb
index 7b70bd0d35fa7e576187b8c5539ffb6f9df60b93..e8481f6ede6300bcfc2afca024d924f82d48f908 100644 (file)
--- a/features/step_definitions/subscriptions.rb
+++ b/features/step_definitions/subscriptions.rb
@@ -50,6 +50,20 @@ When /^j'ajoute une fiche sans nom$/ do
   EOF
 end
 
   EOF
 end
 

+When /^j'ajoute une fiche avec comme date de cotisation "([^"]*)"$/ do |date|
+  @file = subscription_filename_for_id(new_id)
+  write_file @file, <<-EOF.gsub(/^    /, '')
+    ---
+    name: J. Example
+    email: joe@example.org
+    address: |
+      21 Jump Street
+      42000 Synthé
+    membership_fee_paid_on: #{date}
+    ---
+  EOF
+end
+

 When /^je supprime le nom sur une fiche existante$/ do
   @file = subscription_filename_for_id(1)
   run_simple "sed -e '/^name:/d' -i #{@file}"
 When /^je supprime le nom sur une fiche existante$/ do
   @file = subscription_filename_for_id(1)
   run_simple "sed -e '/^name:/d' -i #{@file}"

diff --git a/lib/nos_oignons/subscriptions.rb b/lib/nos_oignons/subscriptions.rb
index eef7946117c184b7329c06c48c229b1fff11bf93..2c8585e5f0caef627d6e6ea74e41d4a98865917f 100644 (file)
--- a/lib/nos_oignons/subscriptions.rb
+++ b/lib/nos_oignons/subscriptions.rb
@@ -15,6 +15,9 @@ def is_valid_subscription?(content)
   ['name', 'email'].each do |key|
     return false unless data.include?(key)
   end
   ['name', 'email'].each do |key|
     return false unless data.include?(key)
   end

+  if data.include?('membership_fee_paid_on')
+    return false unless data.is_a?(Date)
+  end

   true
 end
 
   true
 end