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"
 
+  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

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"
 
+  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

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
 
+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}"

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
+  if data.include?('membership_fee_paid_on')
+    return false unless data.is_a?(Date)
+  end
   true
 end