test: fix vacuous assertions in audit analyzer tests (#885)

same total_tokens value in the implementation, making the original
  comparison self-referential and unable to catch regressions; also
  pin turns == 2 from the fixture instead of just > 0.

  with assertions on actual noop context and reason values — extract_noops
  unconditionally sets tool: None, so the original assertion could never
  fail; new assertions verify context routing and the description→reason
  fallback for the third noop entry.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: github-actions[bot]

src/audit/analyzers/missing.rs

@@ -215,7 +215,12 @@ mod tests {
         assert_eq!(noops.len(), 3);
         assert_eq!(missing_tools[0].tool.as_deref(), Some("bash"));
         assert_eq!(missing_tools[1].tool.as_deref(), Some("python"));
-        assert!(noops.iter().all(|report| report.tool.is_none()));
+        assert_eq!(noops[0].context.as_deref(), Some("ctx-4"));
+        assert_eq!(noops[0].reason.as_deref(), Some("nothing to do"));
+        assert_eq!(noops[1].context.as_deref(), Some("ctx-5"));
+        assert_eq!(noops[2].context.as_deref(), Some("ctx-6"));
+        // third noop has no "reason" field — description fallback must apply
+        assert_eq!(noops[2].reason.as_deref(), Some("skipped"));
     }
 
     #[tokio::test]

src/audit/analyzers/otel.rs

@@ -158,12 +158,8 @@ mod tests {
 
         let analysis = analyze_otel(temp_dir.path()).await.unwrap();
 
-        assert!(analysis.metrics.token_usage > 0);
-        assert_eq!(
-            analysis.metrics.effective_tokens,
-            analysis.metrics.token_usage
-        );
-        assert!(analysis.metrics.turns > 0);
+        assert_eq!(analysis.metrics.token_usage, 33185);
+        assert_eq!(analysis.metrics.turns, 2);
         assert!(analysis.engine_config.is_none());
         assert!(analysis.aw_info.is_none());
     }