thoughtbot / thoughtbot/factory_bot
Pass context to initialize_with
Nobody has claimed this yet.
- Dominant language
- Ruby
- Stars
- 8.2k
- Forks
- 2.6k
- PR merge metrics
- No merged PRs in 30d
Description
This might be a duplicate of #1639.
Giving the context to initialize_with would give access to the transient attributes when building the object. This way we can make decisions while building the object.
We'll need to make sure we don't break existing, clever code in the process, e.g.
initialize_with { _1 }
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the initialize_with implementation and its related tests; first review how the block is currently invoked and how transient attributes are built. Check whether #1639 supersedes this issue, then preserve existing no-argument block behavior while making context available during object construction. Done means existing initialize_with usage still works and context can be used to make construction decisions.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ruby
- Domain
- testing
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100