fixup: pre-commit config

Author: not-matthias

.pre-commit-config.yaml

@@ -6,6 +6,7 @@ repos:
       - id: end-of-file-fixer
       - id: check-yaml
       - id: check-json
+        exclude: ^go-runner/testdata/
       - id: check-merge-conflict
       - id: check-added-large-files
 
@@ -14,7 +15,24 @@ repos:
     hooks:
       - id: fmt
         args: [--all, --]
+        files: ^go-runner/
+        pass_filenames: false
+        additional_dependencies: []
+        entry: bash -c 'cd go-runner && cargo fmt --all'
+        language: system
+
       - id: cargo-check
         args: [--all-targets]
+        files: ^go-runner/
+        pass_filenames: false
+        additional_dependencies: []
+        entry: bash -c 'cd go-runner && cargo check --all-targets'
+        language: system
+
       - id: clippy
         args: [--all-targets, --, -D, warnings]
+        files: ^go-runner/
+        pass_filenames: false
+        additional_dependencies: []
+        entry: bash -c 'cd go-runner && cargo clippy --all-targets -- -D warnings'
+        language: system

go-runner/.gitignore

@@ -1,3 +1,3 @@
 build-*/
 target/
-.codspeed
+.codspeed

go-runner/src/builder/discovery.rs

@@ -2,8 +2,8 @@
 
 use std::{collections::HashMap, path::Path, process::Command};
 
-use crate::prelude::*;
 use crate::builder::verifier;
+use crate::prelude::*;
 use serde::{Deserialize, Serialize};
 
 #[derive(Debug, Clone, Serialize, Deserialize)]
@@ -283,16 +283,15 @@ mod tests {
 
     #[test]
     fn test_parse_go_list_output() {
-        let content = include_str!("tests/go_list_output.json");
-        let packages: Vec<GoPackage> =
-            GoPackage::from_go_list_output(content).expect("Failed to parse JSON");
+        let content = include_str!("../../testdata/builder/go_list_output.json");
+        let packages: Vec<GoPackage> = GoPackage::from_go_list_output(content).unwrap();
 
         insta::assert_json_snapshot!(packages);
     }
 
     #[test]
     fn test_discover_benchmarks() {
-        let content = include_str!("tests/multiple-pkgs.json");
+        let content = include_str!("../../testdata/builder/multiple-pkgs.json");
         let packages: Vec<GoPackage> = GoPackage::from_go_list_output(content).unwrap();
         let benchmark_packages = discover_benchmarks(packages).unwrap();
 
@@ -303,12 +302,12 @@ mod tests {
 
     #[test]
     fn test_benchmarks_for_package() {
-        let content = include_str!("tests/go_list_output.json");
+        let content = include_str!("../../testdata/builder/go_list_output.json");
         let packages: Vec<GoPackage> = GoPackage::from_go_list_output(content).unwrap();
         let benchmark_packages = discover_benchmarks(packages).unwrap();
         let package = benchmark_packages
             .get("local.dev/example-complex/internal/config [local.dev/example-complex/internal/config.test]")
-            .expect("Package not found");
+            .unwrap();
 
         let mut benches = package.benchmarks().unwrap();
         benches.sort_by_cached_key(|b| b.name.clone());

go-runner/src/builder/patcher.rs

@@ -23,9 +23,10 @@ pub fn patch_imports<P: AsRef<Path>>(
         let patched_content = patch_go_source(&content)?;
 
         if patched_content != content {
-            fs::write(&go_file, patched_content).context(
-                format!("Failed to write patched Go file: {}", go_file.display())
-            )?;
+            fs::write(&go_file, patched_content).context(format!(
+                "Failed to write patched Go file: {}",
+                go_file.display()
+            ))?;
 
             debug!("Patched imports in: {}", go_file.display());
             patched_files += 1;
@@ -156,9 +157,9 @@ import (
 
 import (
     "fmt"
-    
+
     "testing"
-    
+
     "strings"
 )
 "#;

go-runner/src/builder/snapshots/go_runner__builder__patcher__tests__import_with_extra_whitespace.snap

@@ -6,8 +6,8 @@ package main
 
 import (
     "fmt"
-    
+
     testing "github.com/AvalancheHQ/codspeed-go/compat/testing"
-    
+
     "strings"
 )

go-runner/src/builder/template.go

@@ -7,7 +7,7 @@ import (
     "reflect"
     codspeed "github.com/AvalancheHQ/codspeed-go/compat/testing"
     codspeed_testing "github.com/AvalancheHQ/codspeed-go/testing/testing"
-    
+
     // Import parent package containing the benchmarks
 {{#each benchmarks}}
     {{import_alias}} "{{module_path}}"
@@ -83,4 +83,4 @@ func main() {
 
     m := codspeed_testing.MainStart(simpleDeps{}, tests, benchmarks, fuzzTargets, examples)
     m.Run()
-}
+}

go-runner/src/builder/tests/go_list_output.json renamed to go-runner/testdata/builder/go_list_output.json

@@ -920,4 +920,4 @@
         "fmt",
         "local.dev/codspeed-go/pkg/compat/testing"
     ]
-}
+}

go-runner/src/builder/tests/hugo.json renamed to go-runner/testdata/builder/hugo.json

@@ -362,4 +362,4 @@
     "TestImports": [
         "github.com/AvalancheHQ/codspeed-go/compat/testing"
     ]
-}
+}