Merge pull request #46 from JuliaSprenger/fix/cli

Fix missing cli default value and add tests

Author: JuliaSprenger

redcap_bridge/cli.py

@@ -35,6 +35,9 @@ def main(command_line=None):
     if args.debug:
         print("debug: " + str(args))
     if args.command == 'download':
+        if not args.format:
+            args.format = ['csv']
+
         download_records(args.destination[0], args.config_json[0], format=args.format[0],
                          compressed=bool(args.compressed))
 

redcap_bridge/test_redcap/test_cli.py

@@ -0,0 +1,52 @@
+import subprocess
+import pathlib
+
+from redcap_bridge.server_interface import upload_datadict, upload_records
+from redcap_bridge.test_redcap.test_utils import (test_directory, initialize_test_dir)
+from redcap_bridge.test_redcap.test_server_interface import SERVER_CONFIG_YAML
+
+project_dir = test_directory / 'testfiles' / 'TestProject'
+
+
+def test_installed(initialize_test_dir):
+    """
+    Check that RedCapBridge was installed successfully
+    """
+    result = subprocess.run(['RedCapBridge', '--help'], stdout=subprocess.PIPE)
+    assert 'usage:' in str(result.stdout)
+
+
+def test_download(initialize_test_dir):
+    """
+    Check that download option works for Test Project
+    """
+
+    # Set up project on server
+    metadata_csv = test_directory / 'testfiles' / 'metadata.csv'
+    upload_datadict(metadata_csv, SERVER_CONFIG_YAML)
+    records_csv = test_directory / 'testfiles' / 'record.csv'
+    upload_records(records_csv, SERVER_CONFIG_YAML)
+
+    output_file = test_directory / 'cli_download_test.csv'
+
+    # download with default arguments
+    result = subprocess.run(['RedCapBridge', 'download', output_file, SERVER_CONFIG_YAML],
+                            stdout=subprocess.PIPE)
+    assert 'error' not in str(result.stdout)
+    assert output_file.exists()
+    output_file.unlink()
+
+    # download in compressed mode
+    result = subprocess.run(['RedCapBridge', 'download', '--compressed', output_file,
+                             SERVER_CONFIG_YAML],
+                            stdout=subprocess.PIPE)
+    assert 'error' not in str(result.stdout)
+    assert pathlib.Path(output_file).exists()
+    output_file.unlink()
+
+    # download with format argument
+    result = subprocess.run(['RedCapBridge', 'download', '--format', 'csv', output_file,
+                             SERVER_CONFIG_YAML],
+                            stdout=subprocess.PIPE)
+    assert 'error' not in str(result.stdout)
+    assert pathlib.Path(output_file).exists()