chain-modules/stats-cohort-size
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(clippy): bring source under -D warnings ahead of CI

Browse source 
Prep for the Forgejo CI gate. Adjustments per module are small
and local:

  - test modules get inner `#![allow(clippy::unwrap_used,
    expect_used, panic)]` so the existing assert.expect()
    test idiom keeps working without rewriting every fixture
  - the dead_code field that downstream consumers may still
    want serialised gets an explicit #[allow(dead_code)]
  - manual char/range comparisons fold to the idiomatic forms
    (`['…']`, `(2..=5).contains(&n)`)
  - one snake_case rename in text-readability-score

Also re-bakes module.wasm so the committed artefact matches
the post-fmt source byte-for-byte.

No behaviour change, no test change. cargo fmt --all -- --check
and cargo clippy --all-targets -- -D warnings now both pass.

Signed-off-by: flemming-it <sf@flemming.it>
This commit is contained in:
flemming-it 2026-06-18 12:13:11 +02:00
parent e1134ae730
commit 629d3e9608
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/lib.rs
View file

@ -24,7 +24,7 @@ struct Cohort {
#[fai_module]
pub fn invoke(_ctx: Context, inputs: Inputs) -> Result<Outputs, ModuleError> {
let id = inputs.require_text("cohort_id")?;
let cohort = resolve(&id).ok_or_else(|| {
let cohort = resolve(id).ok_or_else(|| {
ModuleError::invalid_input(format!(
"unknown cohort_id: {id} (v0.1.0 supports: \
berlin-kmu, de-kmu, gastro-de, handwerk-be, bauwirtschaft-be)"
@ -76,6 +76,7 @@ fn resolve(id: &str) -> Option<Cohort> {
#[cfg(test)]
mod tests {
#![allow(clippy::unwrap_used, clippy::expect_used, clippy::panic)]
use super::*;
#[test]