Fix for asdf plugin-test when installed with brew, and a minor logic fix to a warning

Kevin Lundberg · Kevin Lundberg · commit c77ee57fe2cb · 2019-07-30T23:33:07.000-04:00
diff --git a/lib/commands/plugin-test.sh b/lib/commands/plugin-test.sh
@@ -39,7 +39,7 @@ plugin_test_command() {
     fi
 
     TEST_DIR=$(mktemp -dt asdf.XXXX)
-    git clone "$ASDF_DIR/.git" "$TEST_DIR"
+    cp -R "$ASDF_DIR/" "$TEST_DIR"
 
     plugin_test() {
         export ASDF_DIR=$TEST_DIR
@@ -72,8 +72,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
