barrucadu / barrucadu/resolved

Purify the recursive resolver

Open
#85 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
25
Forks
0
PR merge metrics
No merged PRs in 30d

Description

The recursive resolver is awkward to test because it mixes network calls with business logic. So I just don't test it right now. I test the things it uses, but not it directly. This is a shame, as the recursive resolver is nontrivial.

Purifying it somehow - either making it return some state object and handling the network calls & iteration from a very small function outside it which can easily be eyeballed, or by passing in some "network handle" which can be mocked (ew) - would let me directly test it.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the recursive resolver and reading how it currently combines network calls, business logic, and iteration. Compare the proposed state-object boundary with the injectable network-handle approach, then add direct tests that do not require real network calls. Done means the resolver's nontrivial behavior is directly testable while network handling remains in a small, understandable function.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
networking
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.