Repository navigation
test(resources): fill coverage gaps in resource snapshot (INV-06) - #129
Conversation
…Linux probes No-Test: test-only change, adds missing coverage for src-tauri/src/resources.rs Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7cace22511
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ); | ||
| } | ||
|
|
||
| #[test] |
There was a problem hiding this comment.
Sobald dieser Commit durch den red-first-Check läuft, wird er abgelehnt: src-tauri/src/resources.rs gilt gemäß tf_path_is_source in scripts/lib/test-first.sh:27-37 als trailerpflichtiger Quellcode, aber die Commitnachricht enthält nur den Aufzählungspunkt - Trailer: No-Test (...); tf_message_has_required_trailer akzeptiert in Zeile 56 ausschließlich eine Zeile, die direkt mit No-Test:, Test-First: oder Regression-For: beginnt, sodass scripts/ci/red-first.sh:148-153 mit Fehler beendet. Die Commitnachricht benötigt daher eine eigenständige Zeile wie No-Test: test-only coverage additions.
AGENTS.md reference: AGENTS.md:L155-L158
Useful? React with 👍 / 👎.
|
Queued — the merge queue status continues in this comment ↓. |
|
@Mergifyio queue |
Merge Queue Status
This pull request spent 42 minutes 54 seconds in the queue, including 10 minutes 12 seconds running CI. Required conditions to merge
|
Ich habe den Rust-Teil auf Testlücken untersucht und für
resources.rs(Ressourcenmessung, bisher nur ein Test) fünf fehlende Tests ergänzt. Es wurde kein Produktcode geändert, daher gibt es keinen echten Fehler zu beheben. Die übrigen Funde stehen unten in der Bestandsaufnahme als Vorschläge für eigene Pakete.Report
src-tauri/src/resources.rs(Testmodul): verschachtelte Verzeichnisse, leeres/fehlendes Verzeichnis (Nonestatt falscher 0), Symlinks werden nicht verfolgt (unix), Linux-Speicherwerte plausibel, Snapshot bei fehlendem Datenordner.cargo test --bin projecta resources::→ Exit 0 (6 passed);bash scripts/ci/gates.sh lane prepush→ Exit 0 (1661 Tests, 7 skipped).No-Test(reine Testergänzung, kein roter Test an der Merge-Base nötig).#[cfg(windows)]-Zweige vonresources.rs(Win32-Aufrufe) und der Symlink-Test (nur unix); Windows-Bahn läuft in der Merge-Queue.Bestandsaufnahme
src-tauri/src/resources.rs:128-130:page_size()ist fest 4096; auf Systemen mit 16K/64K-Seiten (aarch64) meldetprocess_ram_byteszu wenig. Vorschlag: eigenes Paket, Seitengröße persysconflesen.resources.rs:38-56:dir_sizezählt unlesbare Unterordner still als 0 (unwrap_or(0)), obwohl ein Fehler auf oberster EbeneNoneliefert; inkonsistent, Zahl kann zu klein sein. Vorschlag: Fehler durchreichen oder als „unvollständig“ markieren.resources.rs:disk_free_bytesundcpu_since_start_permilleliefern unter Linux immerNone; die Anzeige ist dort leer. Vorschlag:statvfsund/proc/self/statergänzen.src-tauri/src/workers/native_launch.rs,workers/development.rs: kein eigener Test im Modul (nur indirekt überworkers.rs-Tests); sicherheitsrelevanter Startpfad. Vorschlag: Paket mit Direkttests fürbind_candidate-Ablehnungen (leere Quelle,observedAtin der Zukunft, Rolle ohne Kandidat).workers/native_runner.rs(563 Zeilen, 2 Tests),workers/native_supervisor.rs(200 Zeilen, 1 Test),process_capture/windows_*.rs(nur Windows): dünne Abdeckung. Vorschlag: Fehlerpfade testen.store/development_launches.rs,native_completion.rs,development_budget.rs,discovery.rsohne Inline-Tests: werden über*_tests.rs-Dateien bzw. Store-Tests abgedeckt; nicht geprüft, ob jeder Fehlerpfad dabei ist. Nahtstellestore/, daher nur seriell.src-tauri/src/testutil.rs:294,374-460,606undtestgate.rs:632,675,732,1090,1507: viele zeitgesteuerte Tests (thread::sleep, 5 s Schlafzeiten). Das Repo hatflaky-test-detection.yml; kein Fund eines konkreten Flackerers. Vorschlag: bei Auffälligkeit auf Bedingungs-Polling umstellen.#[ignore]-Tests; keine Tests mitassert!(true)oderok() || err()-Tautologien; keine festen Ports in Bind-Aufrufen (nur URL-Strings).🤖 Generated with Claude Code