RF: populate-intended-for command uses dashes (-)

...rather than underscores (`_`) for consistency with other commands

Author: pvelasco

heudiconv/cli/run.py

@@ -136,6 +136,7 @@ def get_parser():
             'heuristics', 'heuristic-info',
             'ls', 'populate-templates',
             'sanitize-jsons', 'treat-jsons',
+            'populate-intended-for'
         ),
         help='Custom action to be performed on provided files instead of '
              'regular operation.')

heudiconv/main.py

@@ -108,7 +108,7 @@ def process_extra_commands(outdir, command, files, dicom_dir_template,
         ensure_heuristic_arg(heuristic)
         from .utils import get_heuristic_description
         print(get_heuristic_description(heuristic, full=True))
-    elif command == 'populate_intended_for':
+    elif command == 'populate-intended-for':
         for subj in subjs:
             session_path = op.join(outdir, 'sub-' + subj)
             if session:

heudiconv/tests/test_main.py

@@ -315,7 +315,7 @@ def test_populate_intended_for(session, expected_folder, capfd):
     # that don't exist, and check that the output is the expected.
     bids_folder = expected_folder.split('sub-')[0]
     subjects = ['1', '2']
-    process_extra_commands(bids_folder, 'populate_intended_for', [], '',
+    process_extra_commands(bids_folder, 'populate-intended-for', [], '',
                            '', session, subjects, None)
     captured_output = capfd.readouterr().err
     for s in subjects: