Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Feature Request] Add check warning against mocking record types #4388

Open
ralstonba opened this issue May 4, 2024 · 1 comment
Open

[Feature Request] Add check warning against mocking record types #4388

ralstonba opened this issue May 4, 2024 · 1 comment

Comments

@ralstonba
Copy link

ralstonba commented May 4, 2024 •
edited
Loading

Records are typically used as simple wrappers of data without any methods or complex behavior requiring external dependencies or expensive IO operations. As such, they should (almost) never be mocked, and instead a real instance should be used instead.
@ralstonba
Copy link
Author

ralstonba commented May 4, 2024 •
edited
Loading

I believe this PR addresses the FR: #4384

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ralstonba