Merge pull request #214 from AlexVCaron/test/data

[IMPROV] Add error message to load_test_data when package name is wrong
scilus · Sep 12, 2024 · 6376c35 · 6376c35
2 parents ae6935a + d158932
commit 6376c35
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
@@ -163,13 +163,15 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - uses: actions/upload-artifact/merge@v4
+        continue-on-error: true
         with:
           name: nf-test-reports
           pattern: reports-*
           delete-merged: true
           retention-days: 10
           compression-level: 9
 
+
   confirm-pass:
     name: status
     runs-on: ubuntu-latest

diff --git a/subworkflows/nf-scil/load_test_data/main.nf b/subworkflows/nf-scil/load_test_data/main.nf
@@ -12,6 +12,10 @@ def fetch_archive ( name, destination, remote, database, data_identifiers ) {
 
     // Fetch file from remote if not present in cache
     def data_id = data_identifiers[name]
+    if ( !data_id ) {
+        error "Invalid test data identifier supplied: $name"
+    }
+
     def cache_entry = file("$cache_location/$data_id")
     if ( !cache_entry.exists() ) {
         def remote_entry = "${data_id[0..1]}/${data_id[2..-1]}"