refactor: replace manual mocks with mockall in producer#3192
refactor: replace manual mocks with mockall in producer#3192raushan728 wants to merge 6 commits intoFuelLabs:masterfrom
Conversation
|
Thanks for the contribution! Before we can merge this, we need @raushan728 to sign the Fuel Labs Contributor License Agreement. |
PR SummaryStandardizes producer test infrastructure by replacing hand-rolled mocks with
Written by Cursor Bugbot for commit 041a442. This will update automatically on new commits. Configure here. |
xgreenx
left a comment
There was a problem hiding this comment.
Can you please apply nightly formatting?
|
Applied nightly formatting as requested. All imports are now in vertical layout format. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
|
Hey @raushan728 ! Thanks for the contribution! It's a little bit of an internal debate whether or not we should use It's not up to me alone, but that's my position. |
|
Thanks @MitchTurner. I'll wait for the team's final decision on this before making further changes. |
Migrated manual mock implementations in
producer/src/mocks.rstomockallto address internal TODOs and standardize testing infrastructure.