Skip to content

Create stubs instead of mocks#6564

Merged
derrabus merged 1 commit intodoctrine:3.9.xfrom
simPod:stub
Oct 21, 2024
Merged

Create stubs instead of mocks#6564
derrabus merged 1 commit intodoctrine:3.9.xfrom
simPod:stub

Conversation

@simPod
Copy link
Copy Markdown
Contributor

@simPod simPod commented Oct 21, 2024

Q A
Type improvement
Fixed issues

Summary

This is a followup of #6558, I have touched only what previous PR touched.

@greg0ire requested to use stubs where there are no expectations

image

@simPod simPod changed the title Fix typo in PostgreSql documentation reference Create stubs instead of mocks Oct 21, 2024
@simPod simPod marked this pull request as ready for review October 21, 2024 08:49
@greg0ire greg0ire added this to the 3.9.4 milestone Oct 21, 2024
@simPod
Copy link
Copy Markdown
Contributor Author

simPod commented Oct 21, 2024

btw any idea how to make codecov pass?

image

@greg0ire
Copy link
Copy Markdown
Member

I'd say get somebody to answer codecov/codecov-action#1580

@derrabus derrabus merged commit b87a89f into doctrine:3.9.x Oct 21, 2024
derrabus added a commit to derrabus/dbal that referenced this pull request Oct 21, 2024
* 3.9.x:
  Create stubs instead of mocks (doctrine#6564)
  test: remove ->expects(self::any())
  Fix typo in PostgreSql documentation reference
@simPod simPod deleted the stub branch October 21, 2024 11:28
derrabus added a commit that referenced this pull request Oct 21, 2024
* 4.2.x:
  Create stubs instead of mocks (#6564)
  [Bug] Query Cache mangled if saved by-reference (#6552)
  test: remove ->expects(self::any())
  Fix typo in PostgreSql documentation reference
  fix
  Acknowledge the existence of 3.10 (#6553)
  test: cover nested transactions
derrabus added a commit that referenced this pull request Oct 21, 2024
* 4.3.x:
  Create stubs instead of mocks (#6564)
  [Bug] Query Cache mangled if saved by-reference (#6552)
  test: remove ->expects(self::any())
  Fix typo in PostgreSql documentation reference
  fix
  Acknowledge the existence of 3.10 (#6553)
  Simplify tracking implicitly created indexes
  Remove handling unuique constraint column names as associative array (#6549)
  test: cover nested transactions
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants