From 09cda6fb7ba114257442e915fb76d2b568a557a2 Mon Sep 17 00:00:00 2001
From: desbonne <manuel.desbonnet@hp.com>
Date: Mon, 10 Sep 2012 12:58:35 +0100
Subject: [PATCH] Seems like test mode requires output_dir. Adding check
 (should output_dir be changed to a positional arg instead of switch?)

---
 jenkins-jobs | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/jenkins-jobs b/jenkins-jobs
index 8bd3a680..e0af354b 100755
--- a/jenkins-jobs
+++ b/jenkins-jobs
@@ -4,6 +4,7 @@ import jenkins_jobs.builder
 import argparse
 import ConfigParser
 import logging
+import sys
 
 def main():
     parser = argparse.ArgumentParser()
@@ -53,6 +54,9 @@ def main():
                           options.name))
         builder.update_job(options.path, options.name)
     elif options.command == 'test':
+        if(not options.output_dir):
+            logger.error("You must specify an output dir (-o) in test mode")
+            sys.exit(1)
         builder.update_job(options.path, options.name,
                            output_dir=options.output_dir)