Merge pull request #128 from mbland/reorder-scripts-paths

path: Search `_GO_SCRIPTS_DIR` before plugin dirs

Author: mbland

lib/internal/path

@@ -13,7 +13,7 @@ if [[ "${#_GO_SEARCH_PATHS[@]}" -eq 0 ]]; then
       unset '_GO_PLUGINS_PATHS'
     fi
   fi
-  _GO_SEARCH_PATHS+=("${_GO_PLUGINS_PATHS[@]}" "$_GO_SCRIPTS_DIR")
+  _GO_SEARCH_PATHS+=("$_GO_SCRIPTS_DIR" "${_GO_PLUGINS_PATHS[@]}")
 fi
 
 _@go.list_available_commands() {

tests/path/init-constants.bats

@@ -39,8 +39,8 @@ teardown() {
 
   local expected_paths=(
     "$_GO_ROOTDIR/libexec"
-    "${plugin_bindirs[@]}"
-    "$TEST_GO_SCRIPTS_DIR")
+    "$TEST_GO_SCRIPTS_DIR"
+    "${plugin_bindirs[@]}")
 
   assert_line_equals 0 "_GO_PLUGINS_DIR: $TEST_GO_PLUGINS_DIR"
   assert_line_equals 1 \

tests/vars.bats

@@ -86,10 +86,10 @@ quotify_expected() {
     "[2]=\"$TEST_GO_PLUGINS_DIR/plugin2/bin\"")
   local search_paths=("[0]=\"$TEST_GO_ROOTDIR/bin\""
     "[1]=\"$_GO_CORE_DIR/libexec\""
-    "[2]=\"$TEST_GO_PLUGINS_DIR/plugin0/bin\""
-    "[3]=\"$TEST_GO_PLUGINS_DIR/plugin1/bin\""
-    "[4]=\"$TEST_GO_PLUGINS_DIR/plugin2/bin\""
-    "[5]=\"$TEST_GO_SCRIPTS_DIR\"")
+    "[2]=\"$TEST_GO_SCRIPTS_DIR\""
+    "[3]=\"$TEST_GO_PLUGINS_DIR/plugin0/bin\""
+    "[4]=\"$TEST_GO_PLUGINS_DIR/plugin1/bin\""
+    "[5]=\"$TEST_GO_PLUGINS_DIR/plugin2/bin\"")
 
   # Note that the `format` module imports `strings` and `validation`.
   local expected_modules=('[0]="module_0"'