[8.x] Add model support for database assertions #37459
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey Guys!
A good thing when working with Eloquent, you never ever have to care about the table name.
But when it comes to testing there are some nice assertions
assertDatabase*()
Currently, I have to "hardcode" the table name in my tests (which makes it hard to rename a model and its table name later)
$this->assertDatabaseCount('users', 5);
Wouldnt it be great to just pass the model?
This PR adds exactly this!
I know we could also resolve the connection from the model but this could be done in a follow-up PR.
Cheers Adrian