Always ask before saving/pushing Cobertura reports, in case the tests failed

joaquincasares · joaquincasares · commit 11e5edadd01b · 2013-03-29T17:09:48.000-05:00
diff --git a/testing/bin/coverage b/testing/bin/coverage
@@ -128,11 +128,16 @@ def main():
         # Run against the entire integration suite
         execute('%s' % cobertura_build_command)
 
-        # Save out cobertura site files
-        save_cobertura_site()
+        try:
+            if raw_input("Save Cobertura Report? [y/N] ").lower() == 'y':
 
-        # Perhaps move cobertura site files to a central location
-        maybe_upload_cobertura_site()
+                # Save out cobertura site files
+                save_cobertura_site()
+
+                # Perhaps move cobertura site files to a central location
+                maybe_upload_cobertura_site()
+        except KeyboardInterrupt:
+            print
 
     # Optionally, open coverage report when done building
     if read_config('general', 'open_cobertura_site_after_build', 'boolean'):
