Add test for ClaudeInvocation

[dersam]

Closes #734

[dersam]

• Add test for cog chat config #763
• Add tests for chat sessions #762
• Add test for Roast::Cogs::Chat::Output #761
• Add tests for Cogs::Chat::Input #760
• Add test for ClaudeInvocation #759 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[mathiusj]

it seems a bit awkward there could even be the slightest possibility that failed? and completed? could be both true or false together, at the same time. This might or might not benefit this class's use case, but I'm curious if using an enum for these statuses could benefit here :)

[dersam]

I don't have a strong opinion about this. I'd still like to have the predicate methods (more readable), the state enum would be an implementation detail.

[dersam]

Merge activity

• Jan 30, 6:27 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jan 30, 6:27 PM UTC: @dersam merged this pull request with Graphite.