Merge branch '51-execute-only-command-line-option-does-not-work' into 'master'

Resolve "--execute-only command line option does not work"

Closes #51

This was just a small issue, which I have solved. I also added a new test case for the `--execute-only` command line option.

See merge request !51

bob/bio/base/test/test_scripts.py

@@ -253,6 +253,33 @@ def test_verify_missing():
     shutil.rmtree(test_dir)
 
 
+def test_verify_execute_only():
+  test_dir = tempfile.mkdtemp(prefix='bobtest_')
+  # define dummy parameters
+  parameters = [
+      '-d', 'dummy',
+      '-p', 'dummy',
+      '-e', 'dummy',
+      '-a', 'dummy',
+      '--zt-norm',
+      '--allow-missing-files',
+      '-vs', 'test_missing',
+      '--temp-directory', test_dir,
+      '--result-directory', test_dir,
+      '--preferred-package', 'bob.bio.base',
+      '--imports', 'bob.bio.base.test.dummy',
+      '--execute-only', 'preprocessing', 'score-computation',
+      '--dry-run'
+  ]
+
+  try:
+    from bob.bio.base.script.verify import main
+    main(parameters)
+  finally:
+    if os.path.exists(test_dir):
+      shutil.rmtree(test_dir)
+
+
 def test_internal_raises():
   test_dir = tempfile.mkdtemp(prefix='bobtest_')
   # define dummy parameters

bob/bio/base/tools/command_line.py

@@ -271,7 +271,7 @@ def initialize(parsers, command_line_parameters = None, skips = []):
   if skips is not None and args.execute_only is not None:
     for skip in skips:
       if skip not in args.execute_only:
-        setattr("args", "skip_%s" % skip.replace("-", "_"),  True)
+        setattr(args, "skip_%s" % skip.replace("-", "_"),  True)
 
   if args.parallel is not None:
     args.grid = bob.bio.base.grid.Grid("local", number_of_parallel_processes = args.parallel)