Fix tests.

.github/workflows/rust.yml

@@ -26,10 +26,12 @@ jobs:
       - run: |
           cargo install grcov cargo-nextest
           rustup component add llvm-tools-preview
+          export RUSTFLAGS="-Cinstrument-coverage"
       - name: Install
         run: cargo install --all-features --path monoruby
       - name: Run tests
         run: |
+          cargo build
           LLVM_PROFILE_FILE="monoruby-%p-%m.profraw" cargo test
           LLVM_PROFILE_FILE="monoruby-bin-%p-%m.profraw" monoruby test.rb
           LLVM_PROFILE_FILE="monoruby-stdin-%p-%m.profraw" monoruby < test.rb

monoruby/Cargo.toml

@@ -33,7 +33,7 @@ perf = []
 emit-cfg = ["dump-bc"]
 no-gems = []
 no-startup = ["no-gems"]
-test = ["no-gems", "emit-asm", "emit-bc", "jit-log", "jit-debug", "deopt", "gc-log", "profile", "perf"]
+# test = ["no-gems", "emit-asm", "emit-bc", "jit-log", "jit-debug", "deopt", "gc-log", "profile", "perf"]
 
 [dependencies]
 clap = { version = "4.5.21", features = ["derive"] }

monoruby/src/builtins/array.rs

@@ -1734,8 +1734,9 @@ fn insert(_: &mut Executor, _: &mut Globals, lfp: Lfp) -> Result<Value> {
     Ok(lfp.self_val())
 }
 
-mod test {
-    use super::tests::*;
+#[cfg(test)]
+mod tests {
+    use crate::tests::*;
 
     #[test]
     fn array_new() {

monoruby/src/builtins/binding.rs

@@ -23,7 +23,8 @@ fn local_variables(_: &mut Executor, globals: &mut Globals, lfp: Lfp) -> Result<
     Ok(Value::array_from_vec(v))
 }
 
-mod test {
+#[cfg(test)]
+mod tests {
     use crate::tests::*;
 
     #[test]

monoruby/src/builtins/class.rs

@@ -174,7 +174,8 @@ extern "C" fn check_initializer(globals: &mut Globals, receiver: Value) -> Optio
     globals.check_method(receiver, IdentId::INITIALIZE)
 }
 
-mod test {
+#[cfg(test)]
+mod tests {
     use super::tests::*;
 
     #[test]

monoruby/src/builtins/dir.rs

@@ -169,7 +169,8 @@ fn home(_: &mut Executor, _: &mut Globals, _: Lfp) -> Result<Value> {
     Ok(Value::string(home.to_string_lossy().to_string()))
 }
 
-mod test {
+#[cfg(test)]
+mod tests {
     use super::tests::*;
 
     #[test]

monoruby/src/builtins/math.rs

@@ -161,8 +161,9 @@ extern "C" fn extern_sin(f: f64) -> f64 {
     f.sin()
 }
 
-mod test {
-    use super::tests::*;
+#[cfg(test)]
+mod tests {
+    use crate::tests::*;
 
     #[test]
     #[ignore]

monoruby/src/builtins/numeric/complex.rs

@@ -120,7 +120,8 @@ fn rect(_: &mut Executor, _: &mut Globals, lfp: Lfp) -> Result<Value> {
     Ok(Value::array2(r.get(), i.get()))
 }
 
-mod test {
+#[cfg(test)]
+mod tests {
     use crate::tests::*;
 
     #[test]

monoruby/src/builtins/numeric/float.rs

@@ -241,7 +241,8 @@ fn abs(_vm: &mut Executor, _globals: &mut Globals, lfp: Lfp) -> Result<Value> {
     Ok(Value::float(f.abs()))
 }
 
-mod test {
+#[cfg(test)]
+mod tests {
     use crate::tests::*;
 
     #[test]

monoruby/src/builtins/numeric/integer.rs

@@ -513,8 +513,9 @@ fn zero_(_vm: &mut Executor, _globals: &mut Globals, lfp: Lfp) -> Result<Value>
     Ok(Value::bool(b))
 }
 
-mod test {
-    use super::tests::*;
+#[cfg(test)]
+mod tests {
+    use crate::tests::*;
 
     #[test]
     fn times() {

monoruby/src/builtins/object/send.rs

@@ -68,8 +68,9 @@ fn object_send_inner(
     ir.reg2acc(bb, GP::Rax, dst);
 }
 
-mod test {
-    use super::tests::*;
+#[cfg(test)]
+mod tests {
+    use crate::tests::*;
 
     #[test]
     fn object_send() {

monoruby/src/builtins/process.rs

@@ -110,8 +110,9 @@ fn clock_gettime(_vm: &mut Executor, globals: &mut Globals, lfp: Lfp) -> Result<
     })
 }
 
-mod test {
-    use super::tests::*;
+#[cfg(test)]
+mod tests {
+    use crate::tests::*;
 
     #[test]
     fn process() {

monoruby/src/builtins/range.rs

@@ -213,8 +213,9 @@ fn toa(_vm: &mut Executor, _globals: &mut Globals, lfp: Lfp) -> Result<Value> {
     }
 }
 
-mod test {
-    use super::tests::*;
+#[cfg(test)]
+mod tests {
+    use crate::tests::*;
 
     #[test]
     fn range() {

monoruby/src/builtins/regexp.rs

@@ -183,7 +183,8 @@ fn match_(vm: &mut Executor, _globals: &mut Globals, lfp: Lfp) -> Result<Value>
     ))
 }
 
-mod test {
+#[cfg(test)]
+mod tests {
     use crate::tests::*;
     #[test]
     fn regexp_last_match() {

monoruby/src/builtins/string.rs

@@ -2135,8 +2135,9 @@ fn valid_encoding(_: &mut Executor, _: &mut Globals, lfp: Lfp) -> Result<Value>
     Ok(Value::bool(lfp.self_val().as_bytes().valid()))
 }
 
-mod test {
-    use super::tests::*;
+#[cfg(test)]
+mod tests {
+    use crate::tests::*;
 
     #[test]
     fn string_empty() {

monoruby/src/builtins/struct_class.rs

@@ -141,6 +141,7 @@ fn members(_vm: &mut Executor, globals: &mut Globals, lfp: Lfp) -> Result<Value>
     Ok(members)
 }
 
+#[cfg(test)]
 mod tests {
     use crate::tests::*;
 

monoruby/src/builtins/symbol.rs

@@ -54,7 +54,8 @@ fn ne(_: &mut Executor, _: &mut Globals, lfp: Lfp) -> Result<Value> {
     }
 }
 
-mod test {
+#[cfg(test)]
+mod tests {
     use crate::tests::*;
     #[test]
     fn symbol_cmp() {

monoruby/src/builtins/time.rs

@@ -352,8 +352,9 @@ impl TimeInner {
     }
 }
 
-mod test {
-    use super::tests::*;
+#[cfg(test)]
+mod tests {
+    use crate::tests::*;
 
     #[test]
     fn test_time() {

monoruby/src/value/rvalue/hash.rs

@@ -467,7 +467,8 @@ impl HashContent {
     }
 }
 
-mod test {
+#[cfg(test)]
+mod tests {
     use super::*;
 
     #[test]