Merge pull request asdf-vm#556 from klundberg/fix-plugin-test-issues

Stratus3D · web-flow · commit 16e8d3425056 · 2019-08-13T09:44:28.000-04:00
Fix for asdf plugin-test when installed with brew, and a minor logic fix to a warning
diff --git a/lib/commands/plugin-test.sh b/lib/commands/plugin-test.sh
@@ -39,7 +39,9 @@ plugin_test_command() {
     fi
 
     TEST_DIR=$(mktemp -dt asdf.XXXX)
-    git clone "$ASDF_DIR/.git" "$TEST_DIR"
+    cp -R "$ASDF_DIR/bin" "$TEST_DIR"
+    cp -R "$ASDF_DIR/lib" "$TEST_DIR"
+    cp "$ASDF_DIR/asdf.sh" "$TEST_DIR"
 
     plugin_test() {
         export ASDF_DIR=$TEST_DIR
@@ -72,8 +74,8 @@ plugin_test_command() {
                 fail_test "$plugin_name/bin/list-all does not set GitHub Authorization token"
             fi
 
-            # test for most common token names we have on plugins
-            if [ -z "$OAUTH_TOKEN" ] || [ -z "$GITHUB_API_TOKEN" ] ; then
+            # test for most common token names we have on plugins. If both are empty show this warning
+            if [ -z "$OAUTH_TOKEN" ] && [ -z "$GITHUB_API_TOKEN" ] ; then
                 echo "$plugin_name/bin/list-all is using GitHub API, just be sure you provide an API Authorization token"
                 echo "via your travis settings. This is the current rate_limit:"
                 echo
