Permalink
Browse files

Fix broken retcode (was already returning 0)

  • Loading branch information...
cykl committed Jan 6, 2014
1 parent cc167d2 commit 13a503eba7da8d6604cecc1e64c19e04a3aac6ff
Showing with 2 additions and 2 deletions.
  1. +1 −1 bintest/infoqscraper/presentation/test_download.py
  2. +1 −1 infoqscraper/main.py
@@ -81,7 +81,7 @@ def test_download_output_file_already_exist(self):
open(output_path, 'w').close()
with self.assertRaises(subprocess.CalledProcessError) as cm:
self.run_cmd(self.default_cmd + [short_presentation_id, "-o", output_path])
- self.assertEquals(cm.exception.returncode, 1)
+ self.assertEquals(cm.exception.returncode, 2)
self.assertTrue(os.path.exists(output_path))
self.assertTrue(os.stat(output_path).st_size == 0)
shutil.rmtree(tmp_dir)
View
@@ -369,7 +369,7 @@ def main():
module = module_class()
try:
- module.main(infoq_client, args.module_args)
+ return module.main(infoq_client, args.module_args)
except (ArgumentError, CommandError) as e:
return warn(e)

0 comments on commit 13a503e

Please sign in to comment.