Next step in pre-commit hook implementation

diff --git a/bin/pre-commit-hook b/bin/pre-commit-hook
index aa60ccbe37c0832d938113d5625fec358131ee06..713cb6fa33c1b98a7e615f6aa6e71b4eebec9c18 100755 (executable)
--- a/bin/pre-commit-hook
+++ b/bin/pre-commit-hook
@@ -16,7 +16,9 @@ else
 end
 
 def is_valid_subscription?(content)
-  content.length != 0 && YAML.load(content)
+  content.length != 0 &&
+    content.start_with?("---\n") &&
+    YAML.load(content)
 end
 
 def is_valid_subscription_file?(file)

diff --git a/features/pre-commit-hook.feature b/features/pre-commit-hook.feature
index a03fff0273cf7ce3158c058e5458dd86d66d5074..dff8a4bf0f18328cb3eeb9d7fb6e183dd1b0bce3 100644 (file)
--- a/features/pre-commit-hook.feature
+++ b/features/pre-commit-hook.feature
@@ -19,3 +19,8 @@ Fonctionnalité: pre-commit hook Git
     Lorsque j'ajoute une fiche vide
     Et que je fais un `commit` du nouveau fichier
     Alors je dois voir comme erreur "pas le bon format"
+
+  Scénario: Commit d'un fichier sans YAML
+    Lorsque j'ajoute une fiche sans YAML
+    Et que je fais un `commit` du nouveau fichier
+    Alors je dois voir comme erreur "pas le bon format"

diff --git a/features/step_definitions/subscriptions.rb b/features/step_definitions/subscriptions.rb
index 76a4a1a1b179ab179a361c5f3f8d30e3bf4e9ce2..8d3a754929bfbdd48d261d88e8595896e24135b6 100644 (file)
--- a/features/step_definitions/subscriptions.rb
+++ b/features/step_definitions/subscriptions.rb
@@ -7,3 +7,10 @@ When /^j'ajoute une fiche vide$/ do
   @file = subscription_filename_for_id(new_id)
   write_file @file, ''
 end
+
+When /^j'ajoute une fiche sans YAML$/ do
+  @file = subscription_filename_for_id(new_id)
+  write_file @file, <<-EOF.gsub(/^    /, '')
+    Ceci est une fiche sans YAML.
+  EOF
+end