8000 Introduce `assertQueryCount()` by derrabus · Pull Request #9423 · doctrine/orm · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Introduce assertQueryCount() #9423

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
Merged
merged 1 commit into from
Jan 24, 2022
Merged

Introduce assertQueryCount() #9423

merged 1 commit into from
Jan 24, 2022

Conversation

derrabus
Copy link
Member

This PR introduces a new assertion assertQueryCount() to make it easier in tests to check how many queries have ben submitted to the database.

@derrabus derrabus added this to the 2.12.0 milestone Jan 24, 2022
@derrabus derrabus requested a review from greg0ire January 24, 2022 00:28
greg0ire
greg0ire previously approved these changes Jan 24, 2022
View reviewed changes
Copy link
Member
@greg0ire greg0ire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a vendor/bin/phpcbf but looks good 👍

@derrabus derrabus force-pushed the improvement/assert-query-count branch from e0ee32e to 6d1b1f6 Compare January 24, 2022 08:13
@derrabus derrabus force-pushed the improvement/assert-query-count branch from 6d1b1f6 to 5fc74d8 Compare January 24, 2022 08:16
@derrabus derrabus merged commit f5be418 into doctrine:2.12.x Jan 24, 2022
@derrabus derrabus deleted the improvement/assert-query-count branch January 24, 2022 08:39
derrabus added a commit to derrabus/orm that referenced this pull request Jan 24, 2022
@derrabus
Merge branch '2.12.x' into 3.0.x
36dcf15 
* 2.12.x:
  Introduce assertQueryCount (doctrine#9423)
n-e-m-a-nj-a pushed a commit to n-e-m-a-nj-a/orm that referenced this pull request Mar 29, 2022
@derrabus @n-e-m-a-nj-a
Introduce assertQueryCount (doctrine#9423)
1055669
@github-actions github-actions bot mentioned this pull request Apr 19, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@greg0ire greg0ire greg0ire left review comments

Assignees
No one assigned
Labels
Test Suite
Projects
None yet
Milestone
2.12.0
Development

Successfully merging this pull request may close these issues.
2 participants
@derrabus @greg0ire
0