handle errors in dep, correctly catch missing url in version check (#65)

* handle errors in dep, correctly catch missing url in version check

replace console.error with throw to handle dep error failures
in main try/catch to mark job as failed in one place

* add test action, prevent failure on empty options

* fix conflict resolution

Author: piotrkwiecinski

.github/workflows/test.yml

@@ -0,0 +1,17 @@
+name: 'build-test'
+on: # rebuild any PRs and main branch changes
+  pull_request:
+  push:
+    branches:
+      - master
+      - 'releases/*'
+
+jobs:
+  test: # make sure the action works on a clean machine without building
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: ./
+        with:
+          dep: list
+          deployer-version: "7.3.0"

index.js

@@ -94,8 +94,8 @@ async function dep() {
         break
       }
     }
-    if (url === null) {
-      console.error(`The version "${version}"" does not exist in the "https://deployer.org/manifest.json" file."`)
+    if (typeof url === 'undefined') {
+      core.setFailed(`The version "${version}"" does not exist in the "https://deployer.org/manifest.json" file."`)
     } else {
       console.log(`Downloading "${url}".`)
       await $`curl -LO ${url}`