From 4de7c981f35b15360ae5a0adaf8d65460495e4d7 Mon Sep 17 00:00:00 2001
From: Ahmed AbouZaid <6760103+aabouzaid@users.noreply.github.com>
Date: Sun, 22 Sep 2024 09:39:33 +0200
Subject: [PATCH] fix: use macos compatible sed syntax (#63)

Signed-off-by: Ahmed AbouZaid <6760103+aabouzaid@users.noreply.github.com>
---
 .github/workflows/build.yml  | 54 ++++++++++++++++++++++++++++++++----
 cli/asdf-plugin-manager.sh   |  4 ++-
 test/.plugin-versions        |  2 +-
 test/.plugin-versions.update |  2 ++
 4 files changed, 54 insertions(+), 8 deletions(-)
 create mode 100644 test/.plugin-versions.update

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 9cb1d18..7a64830 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -28,16 +28,58 @@ jobs:
           asdf plugin list --urls --refs
           asdf global asdf-plugin-manager latest
           asdf plugin remove asdf-plugin-manager
-      - name: Test asdf-plugin-manager cli
+      - name: Setup asdf-plugin-manager cli
         run: |
           cp -a cli/asdf-plugin-manager.sh /usr/local/bin/asdf-plugin-manager
-          cd test
+      - name: Install | Test asdf-plugin-manager cli
+        working-directory: test
+        env:
+          ASDF_PLUGIN_MANAGER_PLUGIN_VERSIONS_FILENAME: ".plugin-versions"
+        run: |
+          set -euox pipefail
           asdf-plugin-manager list
           asdf-plugin-manager add-all
           asdf plugin list --refs
-      - name: Validate installed plugins
+          # Validate.
+          # I should install the managed plugin with the correct git ref.
+          PLUGIN_GIT_REF="$(grep -oE "[^ ]\w{6}$" "${ASDF_PLUGIN_MANAGER_PLUGIN_VERSIONS_FILENAME}")"
+          asdf plugin list --refs | grep "${PLUGIN_GIT_REF}"
+          echo "[Passed] The plugin git ref in ${ASDF_PLUGIN_MANAGER_PLUGIN_VERSIONS_FILENAME} matches the installed one."
+      - name: Update | Test asdf-plugin-manager cli
+        working-directory: test
+        env:
+          ASDF_PLUGIN_MANAGER_PLUGIN_VERSIONS_FILENAME: ".plugin-versions.update"
+        run: |
+          set -euox pipefail
+          asdf plugin list --refs
+          asdf-plugin-manager update-all
+          # Validate.
+          # It should get the latest commit in the plugin repo.
+          PLUGIN_GIT_URL="$(awk '/^venom/ {print $2}' "${ASDF_PLUGIN_MANAGER_PLUGIN_VERSIONS_FILENAME}")"
+          PLUGIN_GIT_REF_UPDATED="$(git ls-remote "${PLUGIN_GIT_URL}" HEAD | head -c 7)"
+          asdf plugin list --refs | grep "${PLUGIN_GIT_REF_UPDATED}"
+          echo "[Passed] The plugin git ref in ${ASDF_PLUGIN_MANAGER_PLUGIN_VERSIONS_FILENAME} updated to latest."
+      - name: Export | Test asdf-plugin-manager cli
+        working-directory: test
+        run: |
+          set -euox pipefail
+          asdf plugin add jq
+          asdf plugin list --refs --urls |
+            awk '{OFS = ","; print $1,$2,$4}' | tee asdf-plugin.csv
+          asdf-plugin-manager export |
+            awk '{OFS = ","; print $1,$2,$3}' | tee asdf-plugin-manager.csv
+          # Validate.
+          # It should export all installed plugins.
+          diff asdf-plugin.csv asdf-plugin-manager.csv
+          echo "[Passed] All installed plugins have been exported."
+      - name: Remove | Test asdf-plugin-manager cli
+        working-directory: test
         run: |
           set -euox pipefail
-          PLUGIN_GIT_REF=$(grep -oE "[^ ]\w{6}$" test/.plugin-versions)
-          asdf plugin list --refs | grep "${PLUGIN_GIT_REF}" &&
-          echo "[Passed] The plugin git ref in test/.plugin-versions matches the installed one."
+          asdf plugin add jq
+          asdf plugin list
+          asdf-plugin-manager remove-all
+          # Validate.
+          # It should only remove the managed plugins.
+          asdf plugin list | grep "jq"
+          echo "[Passed] All managed plugins have been removed."
diff --git a/cli/asdf-plugin-manager.sh b/cli/asdf-plugin-manager.sh
index 0dc4906..90c6902 100755
--- a/cli/asdf-plugin-manager.sh
+++ b/cli/asdf-plugin-manager.sh
@@ -125,7 +125,9 @@ update_plugins() {
         fi
 
         echo "[INFO] Updating git-ref in plugin version file: $(print_plugin_versions_filename)"
-        sed -i "/^\b${plugin_name}\b/ s/${plugin_ref_managed}/${plugin_ref_after_update}/" "$(print_plugin_versions_filename)"
+        cat "$(print_plugin_versions_filename)" |
+            sed "/^\b${plugin_name}\b/ s/${plugin_ref_managed}/${plugin_ref_after_update}/" |
+            tee "$(print_plugin_versions_filename)"
 
         echo '!!!'
         echo "[CAUTION] Please review the changes since last update:"
diff --git a/test/.plugin-versions b/test/.plugin-versions
index 72e8cc6..27ee6ae 100644
--- a/test/.plugin-versions
+++ b/test/.plugin-versions
@@ -1,2 +1,2 @@
 # name git-url git-ref
-venom https://github.com/aabouzaid/asdf-venom.git b0d00d6
+venom https://github.com/aabouzaid/asdf-venom.git dea0c86
diff --git a/test/.plugin-versions.update b/test/.plugin-versions.update
new file mode 100644
index 0000000..72e8cc6
--- /dev/null
+++ b/test/.plugin-versions.update
@@ -0,0 +1,2 @@
+# name git-url git-ref
+venom https://github.com/aabouzaid/asdf-venom.git b0d00d6