More updates - still untested

broadinstitute · Nov 14, 2024 · cc0f636 · cc0f636
1 parent f9e373e
commit cc0f636
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/scripts/firecloud_api/UpdateTestInputs.py b/scripts/firecloud_api/UpdateTestInputs.py
@@ -1,7 +1,6 @@
 import argparse
 import json
 import os
-import datetime
 
 
 def update_test_inputs(inputs_json, truth_path, results_path, update_truth):
@@ -31,7 +30,6 @@ def update_test_inputs(inputs_json, truth_path, results_path, update_truth):
         test_inputs[new_key] = test_inputs.pop(key)
 
     # Add the truth_path and results_path to the test inputs JSON
-    current_time = datetime.now().strftime("%Y-%m-%d_%H-%M-%S")
     test_inputs[f"{test_name}.results_path"] = f"{results_path}/{sample_name}/"
     test_inputs[f"{test_name}.truth_path"] = f"{truth_path}/{sample_name}/"
     test_inputs[f"{test_name}.update_truth"] = update_truth

diff --git a/scripts/firecloud_api/firecloud_api.py b/scripts/firecloud_api/firecloud_api.py
@@ -173,7 +173,7 @@ def upload_test_inputs(self, pipeline_name, test_inputs):
     parser.add_argument('--submission_data_file', help='Path to submission data JSON file (required for submit)')
     parser.add_argument('--test_input_file', help='Path to test inputs JSON file (required for upload_test_inputs)')
 
-args = parser.parse_args()
+    args = parser.parse_args()
 
     # Initialize the FirecloudAPI instance with provided arguments
     firecloud_api = FirecloudAPI(args.token, args.namespace, args.workspace)
@@ -217,3 +217,5 @@ def upload_test_inputs(self, pipeline_name, test_inputs):
         if not all([args.pipeline_name, args.test_input_file]):
             print("For 'upload_test_inputs', --pipeline_name and --test_input_file are required.", file=sys.stderr)
         else:
+            success = firecloud_api.upload_test_inputs(args.pipeline_name, args.test_input_file)
+            print(success)