fix(ruby): ensure ruby-build bin exists before trying to use (#2913)

Fixes #2817

.github/workflows/test.yml

@@ -17,7 +17,7 @@ env:
   MISE_TRUSTED_CONFIG_PATHS: ${{ github.workspace }}
   MISE_EXPERIMENTAL: 1
   RUST_BACKTRACE: 1
-  GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+  GITHUB_TOKEN: ${{ secrets.RTX_GITHUB_TOKEN || secrets.GITHUB_TOKEN }}
   NPM_CONFIG_FUND: false
 
 permissions:

e2e/cli/test_current

@@ -0,0 +1,8 @@
+#!/usr/bin/env bash
+
+mise i dummy@3 tiny@2
+mise use dummy@3
+
+assert "mise current" "dummy 3"
+assert "mise current dummy" "3"
+assert "mise current tiny" ""

e2e/plugins/core/test_ruby_ls_remote

@@ -0,0 +1,6 @@
+#!/usr/bin/env bash
+# Install and build ruby using ruby-build
+
+assert_contains "mise ls-remote ruby@3" "3.0.0"
+assert_contains "mise ls-remote ruby@truffleruby-24" "truffleruby-24.0.0"
+assert_contains "mise ls-remote ruby@truffleruby+graalvm-24" "truffleruby+graalvm-24.0.0"

src/cli/current.rs

@@ -121,40 +121,3 @@ static AFTER_LONG_HELP: &str = color_print::cstr!(
     3.11.0 3.10.0
 "#
 );
-
-#[cfg(test)]
-mod tests {
-    use crate::test::reset;
-    use std::env;
-    use test_log::test;
-
-    #[test]
-    fn test_current() {
-        reset();
-        assert_cli_snapshot!("current", @r###"
-        tiny 3.1.0
-        dummy ref:master
-        "###);
-    }
-
-    #[test]
-    fn test_current_with_runtimes() {
-        reset();
-        assert_cli_snapshot!("current", "tiny", @"3.1.0");
-    }
-
-    #[test]
-    fn test_current_missing() {
-        reset();
-        assert_cli!("uninstall", "--all", "dummy");
-
-        env::set_var("MISE_DUMMY_VERSION", "1.1.0");
-        assert_cli_snapshot!("current", @r###"
-        dummy 1.1.0
-        tiny 3.1.0
-        mise dummy@1.1.0 is specified in MISE_DUMMY_VERSION=1.1.0, but not installed
-        "###);
-
-        env::remove_var("MISE_DUMMY_VERSION");
-    }
-}

src/plugins/core/ruby.rs

@@ -93,7 +93,7 @@ impl RubyPlugin {
     }
     fn update_ruby_build(&self) -> Result<()> {
         let _lock = self.lock_build_tool();
-        if self.ruby_build_path().exists() {
+        if self.ruby_build_bin().exists() {
             let cur = self.ruby_build_version()?;
             let latest = self.latest_ruby_build_version();
             match (cur, latest) {
@@ -435,30 +435,6 @@ mod tests {
     use indoc::indoc;
     use pretty_assertions::assert_eq;
 
-    #[test]
-    fn test_list_versions_matching() {
-        reset();
-        let plugin = RubyPlugin::new();
-        assert!(
-            !plugin.list_versions_matching("3").unwrap().is_empty(),
-            "versions for 3 should not be empty"
-        );
-        assert!(
-            !plugin
-                .list_versions_matching("truffleruby-24")
-                .unwrap()
-                .is_empty(),
-            "versions for truffleruby-24 should not be empty"
-        );
-        assert!(
-            !plugin
-                .list_versions_matching("truffleruby+graalvm-24")
-                .unwrap()
-                .is_empty(),
-            "versions for truffleruby+graalvm-24 should not be empty"
-        );
-    }
-
     #[test]
     fn test_parse_gemfile() {
         reset();

src/versions_host.rs

@@ -6,7 +6,10 @@ use crate::{backend, http, registry};
 use url::Url;
 
 pub fn list_versions(ba: &BackendArg) -> eyre::Result<Option<Vec<String>>> {
-    if !SETTINGS.use_versions_host || ba.short.contains(':') {
+    if !SETTINGS.use_versions_host
+        || ba.short.contains(':')
+        || !REGISTRY.contains_key(ba.short.as_str())
+    {
         return Ok(None);
     }
     // ensure that we're using a default shorthand plugin