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
Ignore limit if no order exists when building subquery #16008
Conversation
5492061
to
69fe85b
Compare
if (in_array($result->name, ['mariano', 'lary'])) { | ||
$this->assertNotEmpty($result->articles); | ||
} else { | ||
$this->assertEmpty($result->articles); |
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.
This depends on the db since there is no order. Hard to force a result in the middle of these unit tests.
This doesn't fail without the fix. Not sure I can create a scenario that always tests this. |
->contain('Articles') | ||
->first(); | ||
|
||
if (in_array($result->name, ['mariano', 'lary'])) { |
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.
Should this be larry not lary
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.
Good catch. Still can't get it to fail in unit test.
69fe85b
to
bb54185
Compare
bb54185
to
8881d96
Compare
Any ideas for consistent unit tests or should we just commit and know it works? |
Fixes #14480