Fix Update Testing Data Workflow (#339)

* unpin awscli

* Update CHANGELOG.md

* Added conditional download block to tests workflow

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* refactored pr tests to a dedicated workflow to streamline data download

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

.github/workflows/deploy_pr_tests.yaml

@@ -0,0 +1,21 @@
+name: Deploy PR Tests
+
+on:
+  workflow_dispatch:
+  pull_request:
+
+concurrency:
+    group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+    cancel-in-progress: true
+
+jobs:
+    update-testing-data:
+        uses: ./.github/workflows/update-testing-data.yml
+
+    run-tests:
+        needs: update-testing-data
+        uses: ./.github/workflows/run-tests.yml
+
+    dev-tests:
+        needs: update-testing-data
+        uses: ./.github/workflows/dev-testing.yaml

.github/workflows/dev-testing.yaml

@@ -1,11 +1,7 @@
 name: Dev Testing
 on:
-  pull_request:
   workflow_dispatch:
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
-  cancel-in-progress: true
+  workflow_call:
 
 jobs:
   run:

.github/workflows/run-tests.yml

@@ -1,13 +1,8 @@
 name: Full Tests
 on:
-  pull_request:
   workflow_dispatch:
   workflow_call:
 
-concurrency:
-  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
-  cancel-in-progress: true
-
 jobs:
   run:
     name: Full tests on ${{ matrix.os }} with Python ${{ matrix.python-version }}

.github/workflows/update-testing-data.yml

@@ -38,7 +38,7 @@ jobs:
       - if: steps.cache-ophys-datasets.outputs.cache-hit == false
         name: Install and configure AWS CLI
         run: |
-          pip install awscli==1.29.56
+          pip install awscli
           aws configure set aws_access_key_id ${{ secrets.AWS_ACCESS_KEY_ID }}
           aws configure set aws_secret_access_key ${{ secrets.AWS_SECRET_ACCESS_KEY }}
       - if: steps.cache-ophys-datasets.outputs.cache-hit == false

CHANGELOG.md

@@ -15,6 +15,7 @@
 * Remove unnecessary `scipy` import error handling: [#315](https://github.com/catalystneuro/roiextractors/pull/315)
 * Fixed the typing returned by the `InscopixImagingExtractor.get_dtype` method: [#326](https://github.com/catalystneuro/roiextractors/pull/326)
 * Detect Changelog Updates was moved to its own dedicated workflow to avoid daily testing failures: [#336](https://github.com/catalystneuro/roiextractors/pull/336)
+* Fixed the Update Testing Data Workflow by unpinning the version of awscli: [#339](https://github.com/catalystneuro/roiextractors/pull/339)
 
 ### Improvements