-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Add navigation witness #99
Conversation
16b06e5
to
8f86552
Compare
Podfile.lock
Outdated
- Tempura (from `.`) | ||
- Tempura (~> 6.0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[i] Maybe let's have TempuraTesting point to the local Tempura in the Podfile, and check that everything is set up correctly in the podspec
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
8f86552
to
bb1a98c
Compare
// MARK: - Wrapped | ||
|
||
/// Wraps any object into a reference-type object | ||
public class Wrapped<T> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[i] If you look in Ramen there is a Box
type that does this but it also has dynamic dispatch to the underlying value (which is kinda cool to have). I'd put that here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replaced Wrapped
with ad hoc NavigationRequests
type for improved ergonomic
…nd Hide dispatchable
…n-live implementations
Why
Expose the navigation as a protocol witness to facilitate testing of the app logic
Changes
Add NavigationWitness struct
Tasks