big_code_analysis/metrics/

nom.rs

// Per-language metric and AST modules deliberately consume the macro-
// generated tree-sitter token enums via `use crate::*` and `use Foo::*`
// inside match expressions — explicit imports would list dozens of
// variants per arm and obscure the per-language token sets that are the
// point of these files. Allowed at the module level rather than per
// function so the per-language impl blocks stay readable.
#![allow(clippy::wildcard_imports, clippy::enum_glob_use)]
// Metric counts (token, function, branch, argument, etc.) are stored as
// `usize` and crossed with `f64` averages, ratios, and Halstead scores
// across the cyclomatic / MI / Halstead computations. The `usize as f64`
// and `f64 as usize` casts are intentional and snapshot-anchored — every
// site is bounded by the count it came from. Allowing the lints at the
// module level keeps the metric arithmetic legible.
#![allow(
    clippy::cast_precision_loss,
    clippy::cast_possible_truncation,
    clippy::cast_sign_loss
)]

use std::fmt;

use crate::checker::Checker;
use crate::macros::implement_metric_trait;

use crate::*;

/// The `Nom` metric suite.
#[derive(Clone, Debug, PartialEq)]
#[non_exhaustive]
pub struct Stats {
    functions: usize,
    closures: usize,
    functions_sum: usize,
    closures_sum: usize,
    functions_min: usize,
    functions_max: usize,
    closures_min: usize,
    closures_max: usize,
    space_count: usize,
}

impl Default for Stats {
    fn default() -> Self {
        Self {
            functions: 0,
            closures: 0,
            functions_sum: 0,
            closures_sum: 0,
            functions_min: usize::MAX,
            functions_max: 0,
            closures_min: usize::MAX,
            closures_max: 0,
            space_count: 1,
        }
    }
}

impl fmt::Display for Stats {
    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
        write!(
            f,
            "functions: {}, \
             closures: {}, \
             functions_average: {}, \
             closures_average: {}, \
             total: {}, \
             average: {}, \
             functions_min: {}, \
             functions_max: {}, \
             closures_min: {}, \
             closures_max: {}",
            self.functions_sum(),
            self.closures_sum(),
            self.functions_average(),
            self.closures_average(),
            self.total(),
            self.average(),
            self.functions_min(),
            self.functions_max(),
            self.closures_min(),
            self.closures_max(),
        )
    }
}

impl Stats {
    /// Merges a second `Nom` metric suite into the first one
    pub fn merge(&mut self, other: &Stats) {
        self.functions_min = self.functions_min.min(other.functions_min);
        self.functions_max = self.functions_max.max(other.functions_max);
        self.closures_min = self.closures_min.min(other.closures_min);
        self.closures_max = self.closures_max.max(other.closures_max);
        self.functions_sum += other.functions_sum;
        self.closures_sum += other.closures_sum;
        self.space_count += other.space_count;
    }

    /// Counts the number of function definitions in a scope
    #[inline]
    #[must_use]
    pub fn functions(&self) -> u64 {
        // Only function definitions are considered, not general declarations
        self.functions as u64
    }

    /// Counts the number of closures in a scope
    #[inline]
    #[must_use]
    pub fn closures(&self) -> u64 {
        self.closures as u64
    }

    /// Return the sum metric for functions
    #[inline]
    #[must_use]
    pub fn functions_sum(&self) -> u64 {
        // Only function definitions are considered, not general declarations
        self.functions_sum as u64
    }

    /// Return the sum metric for closures
    #[inline]
    #[must_use]
    pub fn closures_sum(&self) -> u64 {
        self.closures_sum as u64
    }

    /// Returns the average number of function definitions over all spaces.
    ///
    /// `nom`'s averages are **per space** (`space_count`). The `.max(1)`
    /// guard keeps a space-less aggregate from dividing by zero
    /// (`space_count` defaults to 1 and only grows, so today it is a
    /// no-op, but the guard removes the reliance on that invariant — cf.
    /// the divisor guards added for #428).
    #[inline]
    #[must_use]
    pub fn functions_average(&self) -> f64 {
        crate::metrics::average(self.functions_sum() as f64, self.space_count)
    }

    /// Returns the average number of closures over all spaces.
    #[inline]
    #[must_use]
    pub fn closures_average(&self) -> f64 {
        crate::metrics::average(self.closures_sum() as f64, self.space_count)
    }

    /// Returns the average number of function definitions and closures over all spaces.
    #[inline]
    #[must_use]
    pub fn average(&self) -> f64 {
        crate::metrics::average(self.total() as f64, self.space_count)
    }

    /// Counts the number of function definitions in a scope.
    ///
    /// Collapses the `usize::MAX` sentinel that `Stats::default()` plants
    /// into `functions_min` to `0.0`, so a never-observed space
    /// serializes to a meaningful number rather than `1.8446744e19`.
    #[inline]
    #[must_use]
    pub fn functions_min(&self) -> u64 {
        // Only function definitions are considered, not general declarations
        if self.functions_min == usize::MAX {
            0
        } else {
            self.functions_min as u64
        }
    }

    /// Counts the number of closures in a scope.
    ///
    /// Same `usize::MAX` sentinel collapse as `functions_min`.
    #[inline]
    #[must_use]
    pub fn closures_min(&self) -> u64 {
        if self.closures_min == usize::MAX {
            0
        } else {
            self.closures_min as u64
        }
    }
    /// Counts the number of function definitions in a scope
    #[inline]
    #[must_use]
    pub fn functions_max(&self) -> u64 {
        // Only function definitions are considered, not general declarations
        self.functions_max as u64
    }

    /// Counts the number of closures in a scope
    #[inline]
    #[must_use]
    pub fn closures_max(&self) -> u64 {
        self.closures_max as u64
    }
    /// Returns the total number of function definitions and
    /// closures in a scope
    #[inline]
    #[must_use]
    pub fn total(&self) -> u64 {
        self.functions_sum() + self.closures_sum()
    }
    #[inline]
    pub(crate) fn compute_sum(&mut self) {
        self.functions_sum += self.functions;
        self.closures_sum += self.closures;
    }
    #[inline]
    pub(crate) fn compute_minmax(&mut self) {
        self.functions_min = self.functions_min.min(self.functions);
        self.functions_max = self.functions_max.max(self.functions);
        self.closures_min = self.closures_min.min(self.closures);
        self.closures_max = self.closures_max.max(self.closures);
        self.compute_sum();
    }
}

#[doc(hidden)]
/// Per-language counting of methods (functions + closures).
pub(crate) trait Nom
where
    Self: Checker,
{
    /// Walk `node` and update `stats` with this metric for the language
    /// implementing the trait.
    ///
    /// Uses the source-aware [`Checker::is_func_with_code`] rather than the
    /// byte-less `is_func`. For every grammar that exposes a syntactic
    /// function-definition node, `is_func_with_code` forwards to `is_func`,
    /// so the count is unchanged. Elixir is the exception: its `def`/`defp`/
    /// `defmacro`/`defmacrop` declarations are ordinary `Call` nodes that
    /// only a source-text lookup can recognise, so its `is_func_with_code`
    /// override now classifies them as functions instead of leaving
    /// `functions_sum` permanently at 0 (#696).
    ///
    /// `ancestors` is the chain the walker descended through; Elixir's
    /// `is_func_with_code` needs it to spot a `def` inside a `quote`
    /// template in `O(1)` per step rather than `O(depth)` (#1084).
    fn compute<'a>(node: &Node<'a>, code: &[u8], ancestors: Ancestors<'a, '_>, stats: &mut Stats) {
        if Self::is_func_with_code(node, code, ancestors) {
            stats.functions += 1;
            return;
        }
        if Self::is_closure(node, ancestors) {
            stats.closures += 1;
        }
    }
}

implement_metric_trait!(
    [Nom],
    PythonCode,
    MozjsCode,
    JavascriptCode,
    TypescriptCode,
    TsxCode,
    CppCode,
    MozcppCode,
    CCode,
    ObjcCode,
    RustCode,
    PreprocCode,
    CcommentCode,
    JavaCode,
    KotlinCode,
    GoCode,
    PerlCode,
    BashCode,
    LuaCode,
    TclCode,
    PhpCode,
    CsharpCode,
    ElixirCode,
    RubyCode,
    GroovyCode,
    IrulesCode
);

#[cfg(test)]
#[allow(
    clippy::float_cmp,
    clippy::cast_precision_loss,
    clippy::cast_possible_truncation,
    clippy::cast_sign_loss,
    clippy::similar_names,
    clippy::doc_markdown,
    clippy::needless_raw_string_hashes,
    clippy::too_many_lines
)]
mod tests {
    use crate::test_support::check_metrics_only_shim;

    use super::*;

    check_metrics_only_shim!(check_metrics, Nom);
    // The C# indexer / property tests (#464, #472) assert that `nom`'s
    // function count agrees with `npm`'s method count for the same
    // member, so they need both families computed.
    check_metrics_only_shim!(check_nom_and_npm, Nom, Npm);

    /// Regression for #227: a `Stats::default()` that never sees an
    /// observation must not leak the `usize::MAX` sentinel for
    /// `functions_min` or `closures_min`. Both getters collapse the
    /// sentinel to `0.0` so JSON never emits `1.8446744e19`.
    #[test]
    fn nom_empty_file_min_is_zero() {
        let stats = Stats::default();
        assert_eq!(stats.functions_min(), 0);
        assert_eq!(stats.closures_min(), 0);
    }

    #[cfg(feature = "python")]
    #[test]
    fn python_nom() {
        check_metrics::<PythonParser>(
            "def a():
                 pass
             def b():
                 pass
             def c():
                 pass
             x = lambda a : a + 42",
            "foo.py",
            |metric| {
                // Number of spaces = 4
                // The `lambda` is detected as a closure via
                // `PythonCode::is_closure` (widened to accept both
                // aliased kind_ids in #419); pin the count explicitly
                // so a regression in the predicate fails loudly.
                assert_eq!(metric.nom.closures_sum(), 1);
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 3,
                  "closures": 1,
                  "functions_average": 0.75,
                  "closures_average": 0.25,
                  "total": 4,
                  "average": 1.0,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "rust")]
    #[test]
    fn rust_nom() {
        check_metrics::<RustParser>(
            "mod A { fn foo() {}}
             mod B { fn foo() {}}
             let closure = |i: i32| -> i32 { i + 42 };",
            "foo.rs",
            |metric| {
                // Number of spaces = 4
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 1,
                  "functions_average": 0.5,
                  "closures_average": 0.25,
                  "total": 3,
                  "average": 0.75,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "c")]
    #[test]
    fn c_nom() {
        check_metrics::<CParser>(
            "int foo();

             int foo() {
                 return 0;
             }",
            "foo.c",
            |metric| {
                // Number of spaces = 2
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "cpp")]
    #[test]
    fn cpp_nom() {
        check_metrics::<CppParser>(
            "struct A {
                 void foo(int) {}
                 void foo(double) {}
             };
             int b = [](int x) -> int { return x + 42; };",
            "foo.cpp",
            |metric| {
                // Number of spaces = 4
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 1,
                  "functions_average": 0.5,
                  "closures_average": 0.25,
                  "total": 3,
                  "average": 0.75,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    /// Free functions and member functions both surface as
    /// `Cpp::FunctionDefinition` and count toward `functions`.  Member
    /// functions are nested inside a struct/class space; the count is on
    /// the function-definition node itself, not on the enclosing scope.
    #[cfg(feature = "cpp")]
    #[test]
    fn cpp_free_and_member_functions() {
        check_metrics::<CppParser>(
            "int free_fn(int x) { return x; }
             struct S {
                 int member_fn(int x) { return x + 1; }
             };",
            "foo.cpp",
            |metric| {
                // 2 functions: `free_fn`, `S::member_fn`.
                let s = &metric.nom;
                assert_eq!(s.functions_sum(), 2);
                assert_eq!(s.closures_sum(), 0);
                assert_eq!(s.total(), 2);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    /// `static` member functions still surface as `Cpp::FunctionDefinition`
    /// — the `static` keyword is a storage-class specifier, not a separate
    /// node kind — so they are counted just like non-static members.
    #[cfg(feature = "cpp")]
    #[test]
    fn cpp_static_member_function() {
        check_metrics::<CppParser>(
            "struct S {
                 static int factory(int x) { return x; }
                 int method(int x) { return x + 1; }
             };",
            "foo.cpp",
            |metric| {
                let s = &metric.nom;
                assert_eq!(s.functions_sum(), 2);
                assert_eq!(s.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    /// Constructor and destructor definitions surface as
    /// `Cpp::FunctionDefinition` nodes with a `function_declarator` whose
    /// identifier is the class name (ctor) or `~ClassName` (dtor).  Both
    /// count as functions.
    #[cfg(feature = "cpp")]
    #[test]
    fn cpp_constructor_and_destructor() {
        check_metrics::<CppParser>(
            "struct S {
                 S() {}
                 ~S() {}
                 int method() { return 0; }
             };",
            "foo.cpp",
            |metric| {
                let s = &metric.nom;
                // 3 functions: S(), ~S(), method.
                assert_eq!(s.functions_sum(), 3);
                assert_eq!(s.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    /// Operator overloads surface as `FunctionDefinition` whose declarator
    /// has an `OperatorName` identifier (`operator+`, `operator==`).  Both
    /// inline overloads count toward `functions`.
    #[cfg(feature = "cpp")]
    #[test]
    fn cpp_operator_overloads() {
        check_metrics::<CppParser>(
            "struct V {
                 int x;
                 V operator+(const V& o) const { return V{x + o.x}; }
                 bool operator==(const V& o) const { return x == o.x; }
             };",
            "foo.cpp",
            |metric| {
                let s = &metric.nom;
                assert_eq!(s.functions_sum(), 2);
                assert_eq!(s.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    /// Function-template definition counts as a single function — the
    /// `template<>` prefix wraps a `FunctionDefinition` and does not
    /// produce additional function-definition nodes.
    #[cfg(feature = "cpp")]
    #[test]
    fn cpp_function_template() {
        check_metrics::<CppParser>(
            "template<typename T>
             T identity(T x) { return x; }",
            "foo.cpp",
            |metric| {
                let s = &metric.nom;
                assert_eq!(s.functions_sum(), 1);
                assert_eq!(s.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    /// Class-template member functions defined in-line each count as one
    /// function.  The `template<>` head wraps the class, and the methods
    /// inside it surface as ordinary `FunctionDefinition` nodes.
    #[cfg(feature = "cpp")]
    #[test]
    fn cpp_class_template_members() {
        check_metrics::<CppParser>(
            "template<typename T>
             struct Box {
                 T value;
                 T get() const { return value; }
                 void set(T v) { value = v; }
             };",
            "foo.cpp",
            |metric| {
                let s = &metric.nom;
                assert_eq!(s.functions_sum(), 2);
                assert_eq!(s.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    /// Lambdas inside a function body count as `closures`, not as
    /// `functions` — Cpp::LambdaExpression is the closure kind.  The
    /// enclosing function adds 1 to `functions`; each lambda adds 1 to
    /// `closures`.
    #[cfg(feature = "cpp")]
    #[test]
    fn cpp_lambdas_inside_function() {
        check_metrics::<CppParser>(
            "int run() {
                 auto add = [](int a, int b) { return a + b; };
                 auto mul = [](int a, int b) { return a * b; };
                 return add(1, 2) + mul(3, 4);
             }",
            "foo.cpp",
            |metric| {
                let s = &metric.nom;
                // 1 enclosing function + 2 closures.
                assert_eq!(s.functions_sum(), 1);
                assert_eq!(s.closures_sum(), 2);
                assert_eq!(s.total(), 3);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_nom() {
        check_metrics::<JavascriptParser>(
            "function f(a, b) {
                 function foo(a) {
                     return a;
                 }
                 var bar = (function () {
                     var counter = 0;
                     return function () {
                         counter += 1;
                         return counter
                     }
                 })();
                 return bar(foo(a), a);
             }",
            "foo.js",
            |metric| {
                // Number of spaces = 5.
                // functions: f, foo
                // closures:  the IIFE assigned to `bar`, and the
                //            function it returns
                //
                // The IIFE moved to the closure side in #1188. It used to
                // count as the function `bar` purely because its *result*
                // was bound to a name: written `(function(){…})()` with
                // the assignment dropped, the identical construct was
                // already a closure. Both spellings are closures now.
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 2,
                  "functions_average": 0.4,
                  "closures_average": 0.4,
                  "total": 4,
                  "average": 0.8,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    /// The two anonymous-function forms must be classified by the same
    /// rule (#1188).
    ///
    /// `check_if_func!` and `check_if_arrow_func!` walk the ancestor
    /// chain to decide whether an expression is *bound to a name* or used
    /// positionally, and they used to disagree in three ways. Each case
    /// below is one measured divergence; the function-expression and
    /// arrow spellings must now agree.
    ///
    /// Uses `metrics_verbatim` rather than the `check_metrics` shim,
    /// whose bare-`fn` callback cannot capture the case's label or its
    /// expectation.
    #[cfg(any(feature = "javascript", feature = "mozjs", feature = "typescript"))]
    fn check_js_binding_site_parity(lang: crate::LANG) {
        let split = |source: &str| {
            let m = crate::test_support::metrics_verbatim(
                lang,
                source.as_bytes(),
                crate::MetricsOptions::default(),
            );
            (m.nom.functions_sum(), m.nom.closures_sum())
        };

        // (label, function-expression form, arrow form, is it a function?)
        let cases: &[(&str, &str, &str, bool)] = &[
            // An IIFE is the same construct whether or not its *result*
            // is bound. It used to be a function in the bound spelling
            // only, because `$stop` was `Arguments` for functions and
            // `CallExpression` for arrows — different tree levels, and an
            // IIFE's chain carries no `arguments` node at all.
            (
                "bare IIFE",
                "(function (c) { return c; })(1);",
                "(() => 1)(1);",
                false,
            ),
            (
                "bound IIFE",
                "const v = (function (c) { return c; })(1);",
                "const v = (() => 1)(1);",
                false,
            ),
            // A pair with a *non-identifier* key. The arrow reached the
            // right verdict for a bare key only, through `$extra`'s
            // `property_identifier` sibling, which a string key does not
            // provide; `Pair` was in the function `$up` alone.
            (
                "string-keyed pair",
                "({ \"k\": function () { return 1; } });",
                "({ \"k\": () => 1 });",
                true,
            ),
            // The mirror: `has_sibling(PropertyIdentifier)` was in the
            // arrow `$extra` alone, so a class field initialiser was a
            // function as an arrow and a closure as a function expression.
            (
                "class field initialiser",
                "class C { p = function () { return 1; }; }",
                "class C { p = () => 1; }",
                true,
            ),
            // The three field-name spellings `$extra`'s
            // `property_identifier` sibling does *not* cover. The first
            // fix reached the identifier case only, so these stayed
            // closures as arrows while their `function` spellings were
            // already functions — the same divergence, surviving in the
            // shapes the test did not exercise. `p = …` above is the one
            // spelling that worked before the field-definition kind
            // joined both `$up` lists.
            (
                "computed-key class field",
                "class C { [\"k\"] = function () { return 1; }; }",
                "class C { [\"k\"] = () => 1; }",
                true,
            ),
            (
                "string-named class field",
                "class C { \"s\" = function () { return 1; }; }",
                "class C { \"s\" = () => 1; }",
                true,
            ),
            (
                "private class field",
                "class C { #p = function () { return 1; }; }",
                "class C { #p = () => 1; }",
                true,
            ),
            // Plain positional callbacks, unchanged by all of the above.
            (
                "positional callback",
                "run(function () { return 1; });",
                "run(() => 1);",
                false,
            ),
            // A positional callback that is the *object* of a member
            // expression. The first fix for the class-field divergence
            // used `has_sibling(PropertyIdentifier)`, which tests every
            // sibling rather than a binding position — so `.bind(this)`
            // supplied the sibling and made this a function while the
            // arrow spelling (which needs parentheses, interposing a
            // node) stayed a closure. That re-created the very func/arrow
            // divergence this test exists to forbid, in a new shape, and
            // it moved real pdf.js corpus counts before review caught it.
            // The binding site is now named structurally, by kind.
            (
                "member-expression object",
                "run(function () { return 1; }.bind(this));",
                "run((() => 1).bind(this));",
                false,
            ),
        ];

        for &(label, func_form, arrow_form, expected) in cases {
            for (form, source) in [("function", func_form), ("arrow", arrow_form)] {
                let (functions, closures) = split(source);
                assert_eq!(
                    functions == 1,
                    expected,
                    "{lang:?} {label} / {form}: expected a {}, got functions {functions} closures {closures}",
                    if expected { "function" } else { "closure" },
                );
            }
        }

        // The divergence that must **survive**: for a function expression
        // an `identifier` child is its optional name, so
        // `run(function g(){})` is a function; for an arrow it is the
        // un-parenthesised parameter, so `run(x => x)` must stay a
        // closure. Unifying that one would reclassify the commonest
        // callback shape in any JS corpus. Asserted in both directions so
        // a future "unify the last one too" cannot pass quietly.
        assert_eq!(
            split("run(function g() { return 1; });").0,
            1,
            "{lang:?}: a named function expression carries its own name",
        );
        assert_eq!(
            split("run(x => x);").1,
            1,
            "{lang:?}: an arrow's identifier child is its parameter, not a name",
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_binding_site_parity() {
        check_js_binding_site_parity(crate::LANG::Javascript);
    }

    #[cfg(feature = "mozjs")]
    #[test]
    fn mozjs_binding_site_parity() {
        check_js_binding_site_parity(crate::LANG::Mozjs);
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn typescript_binding_site_parity() {
        check_js_binding_site_parity(crate::LANG::Typescript);
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn tsx_binding_site_parity() {
        check_js_binding_site_parity(crate::LANG::Tsx);
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_call_nom() {
        check_metrics::<JavascriptParser>(
            "add_task(async function test_safe_mode() {
                 gAppInfo.inSafeMode = true;
             });",
            "foo.js",
            |metric| {
                // Number of spaces = 2
                // functions: test_safe_mode
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_assignment_nom() {
        check_metrics::<JavascriptParser>(
            "AnimationTest.prototype.enableDisplay = function(element) {};",
            "foo.js",
            |metric| {
                // Number of spaces = 2
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_labeled_nom() {
        check_metrics::<JavascriptParser>(
            "toJSON: function() {
                 return this.inspect(true);
             }",
            "foo.js",
            |metric| {
                // Number of spaces = 2
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_labeled_arrow_nom() {
        check_metrics::<JavascriptParser>(
            "const dimConverters = {
                pt: x => x,
             };",
            "foo.js",
            |metric| {
                // Number of spaces = 2
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_pair_nom() {
        check_metrics::<JavascriptParser>(
            "return {
                 initialize: function(object) {
                     this._object = object.toObject();
                 },
             }",
            "foo.js",
            |metric| {
                // Number of spaces = 2
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(any(feature = "javascript", feature = "mozjs", feature = "typescript"))]
    fn check_returned_object_arrow_nom<T: MetricSuite>(file_name: &str) {
        check_metrics::<T>(
            "function f() { return { foo: x => x }; }",
            file_name,
            |metric| {
                insta::allow_duplicates! {
                    insta::assert_json_snapshot!(
                        metric.nom,
                        @r#"
                    {
                      "functions": 2,
                      "closures": 0,
                      "functions_average": 0.6666666666666666,
                      "closures_average": 0.0,
                      "total": 2,
                      "average": 0.6666666666666666,
                      "functions_min": 0,
                      "functions_max": 1,
                      "closures_min": 0,
                      "closures_max": 0
                    }
                    "#
                    );
                }
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_returned_object_arrow_nom() {
        check_returned_object_arrow_nom::<JavascriptParser>("foo.js");
    }

    #[cfg(feature = "mozjs")]
    #[test]
    fn mozjs_returned_object_arrow_nom() {
        check_returned_object_arrow_nom::<MozjsParser>("foo.js");
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn typescript_returned_object_arrow_nom() {
        check_returned_object_arrow_nom::<TypescriptParser>("foo.ts");
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn tsx_returned_object_arrow_nom() {
        check_returned_object_arrow_nom::<TsxParser>("foo.tsx");
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_unnamed_nom() {
        check_metrics::<JavascriptParser>(
            "Ajax.getTransport = Try.these(
                 function() {
                     return function(){ return new XMLHttpRequest()}
                 }
             );",
            "foo.js",
            |metric| {
                // Number of spaces = 3
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 0,
                  "closures": 2,
                  "functions_average": 0.0,
                  "closures_average": 0.6666666666666666,
                  "total": 2,
                  "average": 0.6666666666666666,
                  "functions_min": 0,
                  "functions_max": 0,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_arrow_nom() {
        check_metrics::<JavascriptParser>(
            "var materials = [\"Hydrogen\"];
             materials.map(material => material.length);
             let add = (a, b)  => a + b;",
            "foo.js",
            |metric| {
                // Number of spaces = 3
                // Functions: add
                // Closures: material.map
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 1,
                  "functions_average": 0.3333333333333333,
                  "closures_average": 0.3333333333333333,
                  "total": 2,
                  "average": 0.6666666666666666,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_arrow_assignment_nom() {
        check_metrics::<JavascriptParser>("sink.onPull = () => { };", "foo.js", |metric| {
            // Number of spaces = 2
            insta::assert_json_snapshot!(
                metric.nom,
                @r#"
            {
              "functions": 1,
              "closures": 0,
              "functions_average": 0.5,
              "closures_average": 0.0,
              "total": 1,
              "average": 0.5,
              "functions_min": 0,
              "functions_max": 1,
              "closures_min": 0,
              "closures_max": 0
            }
            "#
            );
        });
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_arrow_new_nom() {
        check_metrics::<JavascriptParser>(
            "const response = new Promise(resolve => channel.port1.onmessage = resolve);",
            "foo.js",
            |metric| {
                // Number of spaces = 2
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 0,
                  "closures": 1,
                  "functions_average": 0.0,
                  "closures_average": 0.5,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 0,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "javascript")]
    #[test]
    fn javascript_arrow_call_nom() {
        check_metrics::<JavascriptParser>(
            "let notDisabled = TestUtils.waitForCondition(
                 () => !backbutton.hasAttribute(\"disabled\")
             );",
            "foo.js",
            |metric| {
                // Number of spaces = 2
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 0,
                  "closures": 1,
                  "functions_average": 0.0,
                  "closures_average": 0.5,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 0,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "java")]
    #[test]
    fn java_nom() {
        check_metrics::<JavaParser>(
            "class A {
                public void foo(){
                    return;
                }
                public void bar(){
                    return;
                }
            }",
            "foo.java",
            |metric| {
                // Number of spaces = 4
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 2,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "csharp")]
    #[test]
    fn csharp_nom() {
        check_metrics::<CsharpParser>(
            "class A {
                public void Foo() {
                    return;
                }
                public void Bar() {
                    return;
                }
                public int X { get; set; }
                public void Outer() {
                    void Local() { return; }
                    Local();
                }
            }",
            "foo.cs",
            |metric| {
                // Methods: Foo, Bar, Outer (=3 explicit)
                // Plus accessors `get`, `set` on X = 2 more functions
                // Plus `Local` local function = 1 more
                // Total functions = 6
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 6,
                  "closures": 0,
                  "functions_average": 0.75,
                  "closures_average": 0.0,
                  "total": 6,
                  "average": 0.75,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "csharp")]
    #[test]
    fn csharp_closure_nom() {
        check_metrics::<CsharpParser>(
            "class A {
                public void Run() {
                    System.Func<int, int> f = x => x + 1;
                    System.Action g = delegate(int y) { System.Console.WriteLine(y); };
                }
            }",
            "foo.cs",
            |metric| {
                // 1 method (Run), 1 lambda, 1 anonymous_method = 1 func + 2 closures.
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 2,
                  "functions_average": 0.2,
                  "closures_average": 0.4,
                  "total": 3,
                  "average": 0.6,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "csharp")]
    #[test]
    fn csharp_indexer_nom() {
        // A bodied indexer defines two callable accessors (`get`, `set`).
        // Before #464 the `indexer_declaration` node itself ALSO opened a
        // function space, triple-counting the indexer as 3 functions. The
        // correct count is 2 — the accessor count — matching the npm path
        // (`csharp_count_member`) which reports `class_methods == 2`.
        check_nom_and_npm::<CsharpParser>(
            "class A {
                private int[] _d;
                public int this[int i] { get => _d[i]; set => _d[i] = value; }
            }",
            "foo.cs",
            |metric| {
                // expected: get + set accessors = 2 functions; the
                // IndexerDeclaration node no longer opens its own space.
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.npm.class_nm_sum(), 2);
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 2,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "csharp")]
    #[test]
    fn csharp_expression_bodied_indexer_nom() {
        // An expression-bodied indexer (`this[int i] => _d[i];`) has NO
        // `accessor_declaration` child — it defines a single implicit
        // getter. Removing the IndexerDeclaration entry from is_func /
        // is_func_space outright would drop this to 0; the #464 fix gates
        // the entry on the absence of accessors so this form still counts
        // as 1, matching the npm `.max(1)` fallback.
        check_nom_and_npm::<CsharpParser>(
            "class A {
                private int[] _d;
                public int this[int i] => _d[i];
            }",
            "foo.cs",
            |metric| {
                // expected: one implicit getter, no accessor nodes => 1.
                assert_eq!(metric.nom.functions_sum(), 1);
                assert_eq!(metric.npm.class_nm_sum(), 1);
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 0,
                  "functions_average": 0.3333333333333333,
                  "closures_average": 0.0,
                  "total": 1,
                  "average": 0.3333333333333333,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "csharp")]
    #[test]
    fn csharp_property_nom() {
        // A bodied property (`int W { get => _w; set => _w = value; }`)
        // defines two callable accessors. The `property_declaration` node
        // must NOT open its own space on top of them, else it double-counts
        // (the property analogue of #464). The correct count is 2 — the
        // accessor count — matching the npm path which reports 2.
        check_nom_and_npm::<CsharpParser>(
            "class A {
                private int _w;
                public int W { get => _w; set => _w = value; }
            }",
            "foo.cs",
            |metric| {
                // expected: get + set accessors = 2 functions; the
                // PropertyDeclaration node defers and opens no space (#472).
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.npm.class_nm_sum(), 2);
            },
        );
    }

    #[cfg(feature = "csharp")]
    #[test]
    fn csharp_auto_property_nom() {
        // An auto-property (`int Y { get; set; }`) still has two
        // `accessor_declaration` children, so it defers to them exactly like
        // a bodied property — the #472 gate must not change this count.
        check_nom_and_npm::<CsharpParser>(
            "class A {
                public int Y { get; set; }
            }",
            "foo.cs",
            |metric| {
                // expected: get + set accessors = 2 functions, unchanged.
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.npm.class_nm_sum(), 2);
            },
        );
    }

    #[cfg(feature = "csharp")]
    #[test]
    fn csharp_expression_bodied_property_nom() {
        // An expression-bodied property (`int W => _w;`) has NO
        // `accessor_declaration` child — it defines a single implicit
        // getter via an `arrow_expression_clause`. With no accessor to
        // defer to, the `property_declaration` opened no space at all
        // before #472 (0 functions). The fix gates the entry on the
        // absence of accessors so it counts as 1, matching the npm
        // `.max(1)` fallback.
        check_nom_and_npm::<CsharpParser>(
            "class A {
                private int _w;
                public int W => _w;
            }",
            "foo.cs",
            |metric| {
                // expected: one implicit getter, no accessor nodes => 1.
                assert_eq!(metric.nom.functions_sum(), 1);
                assert_eq!(metric.npm.class_nm_sum(), 1);
            },
        );
    }

    #[cfg(feature = "go")]
    #[test]
    fn go_top_level_funcs() {
        check_metrics::<GoParser>(
            "package main
            func a() {}
            func b() {}
            func c() {}",
            "foo.go",
            |metric| {
                // Number of spaces = 4 (file unit + 3 funcs).
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 3,
                  "closures": 0,
                  "functions_average": 0.75,
                  "closures_average": 0.0,
                  "total": 3,
                  "average": 0.75,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "go")]
    #[test]
    fn go_method_declaration() {
        check_metrics::<GoParser>(
            "package main
            type T struct{}
            func (r *T) M() {}",
            "foo.go",
            |metric| {
                // method_declaration is counted as a function.
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "go")]
    #[test]
    fn go_func_literal_is_closure() {
        check_metrics::<GoParser>(
            "package main
            var f = func() {}",
            "foo.go",
            |metric| {
                // func_literal increments closure count, not function count.
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 0,
                  "closures": 1,
                  "functions_average": 0.0,
                  "closures_average": 0.5,
                  "total": 1,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 0,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "go")]
    #[test]
    fn go_nested_closures() {
        check_metrics::<GoParser>(
            "package main
            func f() {
                inner := func() {
                    deeper := func() {}
                    _ = deeper
                }
                _ = inner
            }",
            "foo.go",
            |metric| {
                // 1 function (f) + 2 closures (inner, deeper).
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 1,
                  "closures": 2,
                  "functions_average": 0.25,
                  "closures_average": 0.5,
                  "total": 3,
                  "average": 0.75,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    /// Regression for #1160: `compact_constructor_declaration` was absent
    /// from `JavaCode::is_func`, so a record's compact constructor was
    /// invisible to every metric keyed on that predicate — `nom` included.
    ///
    /// The delegating `R(int a)` is the control: it is an ordinary
    /// `constructor_declaration` and was always counted, so a fixture with
    /// only the compact form could not distinguish "counted once" from
    /// "counted as the other constructor".
    #[cfg(feature = "java")]
    #[test]
    fn java_record_compact_constructor_counts_as_a_function() {
        check_metrics::<JavaParser>(
            "record R(int a, int b) {
                 R { if (a < 0) { throw new IllegalArgumentException(); } }
                 R(int a) { this(a, 0); }
                 int sum() { return a + b; }
             }",
            "R.java",
            |metric| {
                // compact constructor + delegating constructor + sum().
                // Pre-fix this reported 2.
                assert_eq!(metric.nom.functions_sum(), 3);
                assert_eq!(metric.nom.closures_sum(), 0);
            },
        );
    }

    #[cfg(feature = "java")]
    #[test]
    fn java_closure_nom() {
        check_metrics::<JavaParser>(
            "interface printable{
                void print();
              }

              interface IntFunc {
                int func(int n);
              }

              class Printer implements printable{
                public void print(){System.out.println(\"Hello\");}

                public static void main(String args[]){
                  Printer  obj = new Printer();
                  obj.print();
                  IntFunc meaning = (i) -> i + 42;
                  int i = meaning.func(1);
                }
              }",
            "foo.java",
            |metric| {
                // Number of spaces = 8
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 4,
                  "closures": 1,
                  "functions_average": 0.5,
                  "closures_average": 0.125,
                  "total": 5,
                  "average": 0.625,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "groovy")]
    #[test]
    fn groovy_nom() {
        check_metrics::<GroovyParser>(
            "class Printer {
                void print() {
                    println 'hello'
                }
                static void main(String[] args) {
                    def p = new Printer()
                    p.print()
                    def doubler = { x -> x * 2 }
                    int r = doubler(21)
                }
            }",
            "foo.groovy",
            |metric| {
                // Two methods declared. The dekobon grammar parses
                // method bodies as `block`, distinct from `closure`, so
                // the explicit `doubler = { x -> x * 2 }` literal is the
                // only closure here (unlike the prior amaanq grammar
                // which mis-parsed each method body as a `closure`
                // node too).
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 1);
            },
        );
    }

    #[cfg(feature = "groovy")]
    #[test]
    fn groovy_nom_function_definition() {
        // `def foo() {}` at top level uses `function_definition`, not
        // `method_declaration`. Nom must count it as a function.
        check_metrics::<GroovyParser>(
            "def greet(name) {
                println(name)
            }
            greet('world')",
            "foo.groovy",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 1);
            },
        );
    }

    #[cfg(feature = "perl")]
    #[test]
    fn perl_nom() {
        check_metrics::<PerlParser>(
            "sub a { 1 }
             sub b { 2 }
             my $c = sub { 3 };
             sub outer {
                 my $inner = sub { 4 };
                 return $inner;
             }",
            "foo.pl",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 3,
                  "closures": 2,
                  "functions_average": 0.5,
                  "closures_average": 0.3333333333333333,
                  "total": 5,
                  "average": 0.8333333333333334,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn tsx_named_and_arrow_functions() {
        check_metrics::<TsxParser>(
            "function greet(name: string): string {
                 return `Hello, ${name}`;
             }
             const add = (a: number, b: number) => a + b;
             const log = () => { console.log('done'); };",
            "foo.tsx",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 3,
                  "closures": 0,
                  "functions_average": 0.75,
                  "closures_average": 0.0,
                  "total": 3,
                  "average": 0.75,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn typescript_named_arrow_and_class_methods() {
        check_metrics::<TypescriptParser>(
            "function compute(x: number): number {
                 return x * 2;
             }
             const double = (n: number): number => n * 2;
             class Calculator {
                 add(a: number, b: number): number {
                     return a + b;
                 }
             }",
            "foo.ts",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 3,
                  "closures": 0,
                  "functions_average": 0.6,
                  "closures_average": 0.0,
                  "total": 3,
                  "average": 0.6,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "mozjs")]
    #[test]
    fn mozjs_nom() {
        check_metrics::<MozjsParser>(
            "function f(a, b) {
                 function foo(a) {
                     return a;
                 }
                 var bar = (function () {
                     var counter = 0;
                     return function () {
                         counter += 1;
                         return counter
                     }
                 })();
                 return bar(foo(a), a);
             }",
            "foo.js",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 2,
                  "functions_average": 0.4,
                  "closures_average": 0.4,
                  "total": 4,
                  "average": 0.8,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "mozjs")]
    #[test]
    fn mozjs_arrow_and_method() {
        check_metrics::<MozjsParser>(
            "let add = (a, b) => a + b;
             class Counter {
                 increment() {
                     this.count++;
                 }
             }",
            "foo.js",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 2,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "kotlin")]
    #[test]
    fn kotlin_nom_class_with_methods() {
        check_metrics::<KotlinParser>(
            "class Calculator {
                fun add(a: Int, b: Int): Int {
                    return a + b
                }
                fun subtract(a: Int, b: Int): Int {
                    return a - b
                }
            }",
            "foo.kt",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 2,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "lua")]
    #[test]
    fn lua_nom() {
        check_metrics::<LuaParser>(
            "function greet(name)
  return \"hello \" .. name
end

local add = function(a, b)
  return a + b
end

local function outer()
  local inner = function()
    return 42
  end
  return inner()
end",
            "foo.lua",
            |metric| {
                // 2 named functions (greet, outer), 2 closures (add, inner)
                insta::assert_json_snapshot!(metric.nom, @r#"
                {
                  "functions": 2,
                  "closures": 2,
                  "functions_average": 0.4,
                  "closures_average": 0.4,
                  "total": 4,
                  "average": 0.8,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#);
            },
        );
    }

    #[cfg(feature = "bash")]
    #[test]
    fn bash_nom() {
        check_metrics::<BashParser>(
            "#!/bin/bash
foo() {
    echo 'hello'
}
bar() {
    echo 'world'
}
foo
bar",
            "foo.sh",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 0,
                  "functions_average": 0.6666666666666666,
                  "closures_average": 0.0,
                  "total": 2,
                  "average": 0.6666666666666666,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "tcl")]
    #[test]
    fn tcl_nom() {
        check_metrics::<TclParser>(
            "proc foo {a} { puts $a }
proc bar {x y} { puts $x }
foo 1
bar 2 3",
            "foo.tcl",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "tcl")]
    #[test]
    fn tcl_nested_nom() {
        check_metrics::<TclParser>(
            "proc outer {a} {
    proc inner {x} { puts $x }
    inner $a
}",
            "foo.tcl",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn typescript_class_methods() {
        check_metrics::<TypescriptParser>(
            "class Calc {
             add(a: number, b: number): number { return a + b; }
             sub(a: number, b: number): number { return a - b; }
         }",
            "foo.ts",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn typescript_arrow_and_function() {
        check_metrics::<TypescriptParser>(
            "function f(): number { return 1; }
         const g = (): number => 2;
         const h = (x: number): number => x * 2;",
            "foo.ts",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 3);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn tsx_class_methods() {
        check_metrics::<TsxParser>(
            "class Calc {
             add(a: number, b: number): number { return a + b; }
             sub(a: number, b: number): number { return a - b; }
         }",
            "foo.tsx",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "typescript")]
    #[test]
    fn tsx_arrow_and_function() {
        check_metrics::<TsxParser>(
            "function f(): number { return 1; }
         const g = (): number => 2;
         const h = (x: number): number => x * 2;",
            "foo.tsx",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 3);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "bash")]
    #[test]
    fn bash_multiple_functions_nom() {
        check_metrics::<BashParser>(
            "#!/bin/bash
f() {
    echo hello
}
g() {
    echo world
}",
            "foo.sh",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "bash")]
    #[test]
    fn bash_nested_functions_nom() {
        check_metrics::<BashParser>(
            "#!/bin/bash
outer() {
    inner() {
        echo inner
    }
    inner
}",
            "foo.sh",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "mozjs")]
    #[test]
    fn mozjs_nested_function_nom() {
        check_metrics::<MozjsParser>(
            "function outer() {
             function inner() {
                 return 1;
             }
             return inner();
         }",
            "foo.js",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "mozjs")]
    #[test]
    fn mozjs_class_methods_nom() {
        check_metrics::<MozjsParser>(
            "class Calc {
             add(a, b) { return a + b; }
             sub(a, b) { return a - b; }
         }",
            "foo.js",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "mozjs")]
    #[test]
    fn mozjs_iife_nom() {
        check_metrics::<MozjsParser>(
            "(function() {
             var x = 1;
             return x;
         })();",
            "foo.js",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 0);
                assert_eq!(metric.nom.closures_sum(), 1);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "kotlin")]
    #[test]
    fn kotlin_class_methods_nom() {
        check_metrics::<KotlinParser>(
            "class Calc {
             fun add(a: Int, b: Int): Int = a + b
             fun sub(a: Int, b: Int): Int = a - b
         }",
            "foo.kt",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 0);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "kotlin")]
    #[test]
    fn kotlin_lambda_nom() {
        check_metrics::<KotlinParser>(
            "fun f(list: List<Int>): Int {
             val double = { x: Int -> x * 2 }
             return list.sumOf(double)
         }",
            "foo.kt",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 1);
                assert_eq!(metric.nom.closures_sum(), 1);
                insta::assert_json_snapshot!(metric.nom);
            },
        );
    }

    #[cfg(feature = "php")]
    #[test]
    fn php_nom() {
        // Top-level function + 2 methods inside a class + 1 anonymous +
        // 1 arrow = 3 functions, 2 closures.
        check_metrics::<PhpParser>(
            "<?php
            function top(): void {}
            class A {
                public function m1(): void {}
                public function m2(): int {
                    $f = function () { return 1; };
                    $g = fn () => 2;
                    return $f() + $g();
                }
            }",
            "foo.php",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 3,
                  "closures": 2,
                  "functions_average": 0.42857142857142855,
                  "closures_average": 0.2857142857142857,
                  "total": 5,
                  "average": 0.7142857142857143,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "php")]
    #[test]
    fn php_nom_anonymous_class() {
        // Methods inside `new class { … }` count toward the closure-style
        // space mechanism: anonymous_class is its own space and its
        // method_declaration children are counted as functions.
        check_metrics::<PhpParser>(
            "<?php
            function f(): object {
                return new class {
                    public function inner(): int { return 1; }
                };
            }",
            "foo.php",
            |metric| {
                insta::assert_json_snapshot!(
                    metric.nom,
                    @r#"
                {
                  "functions": 2,
                  "closures": 0,
                  "functions_average": 0.5,
                  "closures_average": 0.0,
                  "total": 2,
                  "average": 0.5,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 0
                }
                "#
                );
            },
        );
    }

    #[cfg(feature = "php")]
    #[test]
    fn php_function_after_leading_backslash_group_use_is_counted() {
        // tree-sitter-php 0.24.2 parsed `use \Foo\{…};` as ERROR and the
        // recovery swallowed the function after it, so `u` scored nom 0.
        // 0.25.1 accepts the leading backslash; the function is counted.
        check_metrics::<PhpParser>(
            "<?php\nuse \\Foo\\{Bar, Baz};\nfunction u() { return 1; }\n",
            "foo.php",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 1);
            },
        );
    }

    // Elixir's `def`/`defp`/`defmacro`/`defmacrop` declarations surface
    // as `Call` nodes whose target is an `Identifier`, so the byte-less
    // `is_func` cannot see them. Since #696, `Nom::compute` consults the
    // source-aware `is_func_with_code`, so the three named declarations
    // (`public_fn`, `private_fn`, `with_anon`) now count as FUNCTIONS and
    // the two `fn x -> … end` literals count as CLOSURES — the same split
    // every other language already produced. `functions_sum` was pinned at
    // 0 before the fix (the bug this test now guards against regressing).
    #[cfg(feature = "elixir")]
    #[test]
    fn elixir_nom_counts_def_as_functions_and_fn_as_closures() {
        check_metrics::<ElixirParser>(
            "defmodule Foo do\n  def public_fn(x), do: x + 1\n  defp private_fn(x), do: x - 1\n  def with_anon do\n    inc = fn x -> x + 1 end\n    dec = fn x -> x - 1 end\n    {inc, dec}\n  end\nend\n",
            "foo.ex",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 3);
                assert_eq!(metric.nom.closures_sum(), 2);
                assert_eq!(metric.nom.total(), 5);
            },
        );
    }

    #[test]
    fn stats_display_commas_between_all_fields() {
        let stats = Stats {
            functions: 0,
            closures: 0,
            functions_sum: 3,
            closures_sum: 1,
            functions_min: 0,
            functions_max: 2,
            closures_min: 0,
            closures_max: 1,
            space_count: 2,
        };
        let formatted = format!("{stats}");

        // Every adjacent pair of labels must appear with ", " between the
        // previous field's value and the next label.
        let expected_fragments = [
            "functions: 3, closures: 1",
            "closures: 1, functions_average:",
            "functions_average: 1.5, closures_average:",
            "closures_average: 0.5, total:",
            "total: 4, average:",
            "average: 2, functions_min:",
            "functions_min: 0, functions_max:",
            "functions_max: 2, closures_min:",
            "closures_min: 0, closures_max:",
        ];

        for fragment in expected_fragments {
            assert!(
                formatted.contains(fragment),
                "missing fragment {fragment:?} in: {formatted}"
            );
        }
    }

    #[cfg(feature = "ruby")]
    #[test]
    fn ruby_nom() {
        // expected: total = 4 (2 methods `add`/`mul` + 1 singleton
        // method `self.factory` + 1 block argument to `each`).
        // `functions` counts only the named `Method` / `SingletonMethod`
        // forms (3); `closures` counts `Block` / `DoBlock` / `Lambda`
        // (1).
        check_metrics::<RubyParser>(
            "class C\n  def add(a, b)\n    a + b\n  end\n  def mul(a, b)\n    a * b\n  end\n  def self.factory\n    new\n  end\nend\n\n[1, 2, 3].each { |x| puts x }\n",
            "foo.rb",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 3);
                assert_eq!(metric.nom.closures_sum(), 1);
                assert_eq!(metric.nom.total(), 4);
            },
        );
    }

    #[cfg(feature = "ruby")]
    #[test]
    fn ruby_stabby_lambda_single_closure() {
        // A stabby lambda `->(z) { … }` parses as a `Lambda` node that
        // contains a `Block` for its body. `is_closure` must count the
        // pair as ONE closure, not two (#465). Revert-verified: counting
        // the inner `Block` again yields closures_sum == 2.0.
        check_metrics::<RubyParser>("f = ->(z) { z + 1 }\n", "stabby.rb", |metric| {
            assert_eq!(metric.nom.functions_sum(), 0);
            assert_eq!(metric.nom.closures_sum(), 1);
        });
    }

    #[cfg(feature = "ruby")]
    #[test]
    fn ruby_stabby_lambda_multi_statement_single_closure() {
        // A multi-statement body does not change the structure: still one
        // `Lambda` wrapping one `Block`, so still one closure.
        check_metrics::<RubyParser>(
            "f = ->(z) {\n  y = z + 1\n  y * 2\n}\n",
            "stabby_multi.rb",
            |metric| {
                assert_eq!(metric.nom.closures_sum(), 1);
            },
        );
    }

    #[cfg(feature = "ruby")]
    #[test]
    fn ruby_stabby_lambda_do_block_single_closure() {
        // The `do … end` body form of a stabby lambda parses as a `Lambda`
        // wrapping a `DoBlock`; both must collapse to one closure.
        check_metrics::<RubyParser>("f = ->(z) do\n  z + 1\nend\n", "stabby_do.rb", |metric| {
            assert_eq!(metric.nom.closures_sum(), 1);
        });
    }

    #[cfg(feature = "ruby")]
    #[test]
    fn ruby_keyword_lambda_single_closure() {
        // The keyword forms `lambda { }` / `proc { }` parse as a `Call`
        // carrying a `Block` argument (the parent is a `Call`, not a
        // `Lambda`), so they must still count exactly one closure. Guards
        // against the #465 fix regressing the keyword form to zero.
        check_metrics::<RubyParser>(
            "g = lambda { |z| z + 1 }\nh = proc { |z| z + 1 }\n",
            "keyword.rb",
            |metric| {
                assert_eq!(metric.nom.closures_sum(), 2);
            },
        );
    }

    /// iRules counts `when` event handlers and `proc` definitions as
    /// functions; the language has no closures. A file with two handlers
    /// and one proc reports three functions, zero closures. Confirms the
    /// handlers-as-functions decision end to end. (`try`'s `on` / `trap`
    /// handlers are branch points, not functions — issue #1266.)
    #[cfg(feature = "irules")]
    #[test]
    fn irules_nom_handlers_and_procs() {
        check_metrics::<IrulesParser>(
            "when CLIENT_ACCEPTED {
    log local0. \"connected\"
}
when HTTP_REQUEST {
    log local0. [HTTP::uri]
}
proc helper { x } {
    return $x
}
",
            "foo.irule",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 3);
                assert_eq!(metric.nom.closures_sum(), 0);
                assert_eq!(metric.nom.total(), 3);
            },
        );
    }

    /// iRules `try` handlers (`on error` / `trap`) are error-handler
    /// clauses, not functions: a proc containing both must report exactly
    /// one function (issue #1266). Before the fix each handler's dedicated
    /// `on_handler` / `trap_handler` node was classified as a function
    /// space, so this fixture reported three.
    #[cfg(feature = "irules")]
    #[test]
    fn irules_try_handlers_are_not_functions() {
        check_metrics::<IrulesParser>(
            "proc f {} {
    try {
        risky
    } on error {msg} {
        puts $msg
    } trap {POSIX} {msg} {
        puts $msg
    }
}
",
            "foo.irule",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 1);
                assert_eq!(metric.nom.closures_sum(), 0);
                assert_eq!(metric.nom.total(), 1);
            },
        );
    }

    /// Objective-C `@implementation` with two `method_definition`s and a
    /// `block_literal`: the two methods are functions, the block is a
    /// closure (it does not open its own space), so functions = 2,
    /// closures = 1, total = 3.
    #[cfg(feature = "objc")]
    #[test]
    fn objc_nom() {
        check_metrics::<ObjcParser>(
            "@implementation Foo
- (void)one {
    void (^blk)(void) = ^{
        [self two];
    };
    blk();
}
- (void)two {
    [self one];
}
@end
",
            "foo.m",
            |metric| {
                assert_eq!(metric.nom.functions_sum(), 2);
                assert_eq!(metric.nom.closures_sum(), 1);
                assert_eq!(metric.nom.total(), 3);
                insta::assert_json_snapshot!(metric.nom, @r#"
                {
                  "functions": 2,
                  "closures": 1,
                  "functions_average": 0.5,
                  "closures_average": 0.25,
                  "total": 3,
                  "average": 0.75,
                  "functions_min": 0,
                  "functions_max": 1,
                  "closures_min": 0,
                  "closures_max": 1
                }
                "#);
            },
        );
    }
}