Skip to content

Commit

Permalink
fix: api tests need to use latest deno
Browse files Browse the repository at this point in the history
  • Loading branch information
sprutton1 committed Jan 17, 2025
1 parent 1df917b commit 8cb8f47
Show file tree
Hide file tree
Showing 3 changed files with 82 additions and 11 deletions.
16 changes: 7 additions & 9 deletions .github/workflows/run-api-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,20 +73,18 @@ jobs:
uses: actions/checkout@v4

- name: Install Deno
uses: denoland/setup-deno@v1
with:
deno-version: v1.x
uses: denoland/setup-deno@v2

- name: Run the deno exec with retry
run: |
cd bin/si-api-test
echo "Running test ${{ matrix.tests.name }} with index: ${{ matrix.tests.index }}"
# Split the workspace IDs into an array
workspace_ids="${{ vars.API_TEST_WORKSPACE_IDS }}"
IFS=',' read -r -a workspace_array <<< "$workspace_ids"
# Pick the correct workspace ID based on the index
workspace_id="${workspace_array[${{ matrix.tests.index }}]}"
Expand All @@ -96,7 +94,7 @@ jobs:
n=0
max_retries=5
exit_code=0
until [ $n -ge $max_retries ]
do
unset exit_code || echo "exit_code not set"
Expand All @@ -107,7 +105,7 @@ jobs:
--userId ${{ secrets.API_TEST_EMAIL }} \
--password ${{ secrets.API_TEST_PASSWORD }} \
--tests ${{ matrix.tests.name }} || exit_code=$?
# Check the exit code
if [ -z "$exit_code" ]; then
echo "Deno task succeeded [ or the orchestration failed for a totally non-valid reason ]!"
Expand Down Expand Up @@ -152,7 +150,7 @@ jobs:
uses: actions/download-artifact@v4
with:
path: artifacts
- run: |
- run: |
has_artifacts=false
# Check for marker files
for marker in artifacts/*/failure-marker; do
Expand All @@ -179,7 +177,7 @@ jobs:
\"tags\": [
\"service:github\"
]
}"
}"
fi
- run: |
# Always send the Internal Slack Notification if failure detected, regardless of error source
Expand Down
2 changes: 1 addition & 1 deletion deno.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"workspace": ["./bin/lang-js", "./lib/ts-lib-deno"],
"workspace": ["./bin/lang-js", "./bin/si-api-test", "./lib/ts-lib-deno"],
"imports": {
"@cliffy/command": "jsr:@cliffy/command@^1.0.0-rc.7",
"@deno/emit": "jsr:@deno/emit@^0.46.0"
Expand Down
75 changes: 74 additions & 1 deletion deno.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 8cb8f47

Please sign in to comment.