chore: minor changes

Author: not-matthias

.gitmodules

@@ -10,3 +10,9 @@
 [submodule "go-runner/testdata/projects/caddy"]
 	path = go-runner/testdata/projects/caddy
 	url = https://github.com/caddyserver/caddy
+[submodule "go-runner/testdata/projects/fuego"]
+	path = go-runner/testdata/projects/fuego
+	url = https://github.com/go-fuego/fuego
+[submodule "go-runner/testdata/projects/hugo"]
+	path = go-runner/testdata/projects/hugo
+	url = https://github.com/gohugoio/hugo/

example/fib_test.go

@@ -3,14 +3,13 @@ package example
 import "testing"
 
 func BenchmarkFibonacci10(b *testing.B) {
-	for i := 0; i < b.N; i++ {
-		fibonacci(10)
-	}
+	b.Run("fibonacci(10)", func(b *testing.B) {
+		b.Run("fibonacci(10)", func(b *testing.B) {
+			for i := 0; i < b.N; i++ {
+				fibonacci(10)
+			}
+		})
 
-	b.Run("fibonacci(40)", func(b *testing.B) {
-		for i := 0; i < b.N; i++ {
-			fibonacci(10)
-		}
 	})
 }
 

go-runner/src/builder/discovery.rs

@@ -241,6 +241,7 @@ fn discover_benchmarks(packages: Vec<GoPackage>) -> anyhow::Result<HashMap<Strin
             .map(has_test_files)
             .unwrap_or_default();
         if !has_tests {
+            debug!("Skipping package without test files: {}", package.name);
             continue;
         }
 
@@ -251,18 +252,30 @@ fn discover_benchmarks(packages: Vec<GoPackage>) -> anyhow::Result<HashMap<Strin
             .map(has_test_imports)
             .unwrap_or_default();
         if !has_test_imports {
+            debug!("Skipping package without test imports: {}", package.name);
             continue;
         }
 
         // Only include test executables, since we want to generate them manually.
         // Example format: `local.dev/example-complex [local.dev/example-complex.test]`
         if !package.import_path.ends_with(".test]") {
+            debug!("Skipping package without test executable: {}", package.name);
             continue;
         }
 
         // Skip packages that don't have benchmarks
-        let benchmarks = package.benchmarks()?;
+        let benchmarks = match package.benchmarks() {
+            Ok(benchmarks) => benchmarks,
+            Err(e) => {
+                warn!(
+                    "Failed to get benchmarks for package {}: {}",
+                    package.name, e
+                );
+                continue;
+            }
+        };
         if benchmarks.is_empty() {
+            debug!("Skipping package without benchmarks: {}", package.name);
             continue;
         }
 

go-runner/src/integration_tests.rs

@@ -27,16 +27,17 @@ fn assert_benchmarks_created(binary_dir: &Path, n: usize) {
 }
 
 #[rstest]
-// #[case("caddy")]
-#[case("fzf")]
-#[case("opentelemetry-go")]
-#[case("golang-benchmarks")]
+// #[case::caddy("caddy")]
+// #[case::hugo("hugo")]
+#[case::fzf("fzf")]
+#[case::opentelemetry_go("opentelemetry-go")]
+#[case::golang_benchmarks("golang-benchmarks")]
+#[case::fuego("fuego")]
 fn test_build_and_run(#[case] project_name: &str) {
     let temp_dir = setup_test_project(project_name).unwrap();
 
     let binary_dir = temp_dir.path().join(".codspeed").join("walltime");
     let binaries = crate::build_benchmarks(temp_dir.path(), &binary_dir).unwrap();
-    assert!(!binaries.is_empty(), "No benchmark binaries were created");
     assert_benchmarks_created(&binary_dir, binaries.len());
 
     // Mutex to prevent concurrent tests from interfering with CODSPEED_PROFILE_FOLDER env var

go-runner/testdata/projects/cli-runtime

@@ -0,0 +1 @@
+Subproject commit 4ef9d6338b9285c7945a2d6bd04a2d28781fa175