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.

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

Restore functional cache tests #8981

Merged

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Sep 6, 2021

When adding support for Doctrine cache 2, many functional tests that involve caching have been disabled. This PR enables them again, using a wrapped ArrayAdapter instead of ArrayCache.

I'm targeting 2.9.x here because I'm only changing tests and not touching any production code.

@derrabus derrabus force-pushed the improvement/restore-functional-cache-tests branch 3 times, most recently from a2a13b7 to 16b0b85 Compare September 6, 2021 15:58
Signed-off-by: Alexander M. Turek <me@derrabus.de>
@derrabus derrabus force-pushed the improvement/restore-functional-cache-tests branch from 16b0b85 to 2cb124c Compare September 6, 2021 16:08
@derrabus derrabus added this to the 2.9.6 milestone Sep 6, 2021
@derrabus derrabus merged commit 316ba5f into doctrine:2.9.x Sep 7, 2021
@derrabus derrabus deleted the improvement/restore-functional-cache-tests branch September 7, 2021 20:45
derrabus added a commit to derrabus/orm that referenced this pull request Sep 8, 2021
* 2.9.x:
  Restore functional cache tests (doctrine#8981)
  Fix English in `note`. (doctrine#8987)
  Remove detach deprecation entry in UPGRADE.md (doctrine#8978)
  Bump to PHPStan 0.12.98 and Psalm 4.10.0 (doctrine#8979)
derrabus added a commit to derrabus/orm that referenced this pull request Sep 8, 2021
* 2.9.x:
  Restore functional cache tests (doctrine#8981)
  Fix English in `note`. (doctrine#8987)
  Remove detach deprecation entry in UPGRADE.md (doctrine#8978)
  Bump to PHPStan 0.12.98 and Psalm 4.10.0 (doctrine#8979)

Signed-off-by: Alexander M. Turek <me@derrabus.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants