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

try newer versions of drupal/civi in tests #504

Closed

Conversation

jackrabbithanna
Copy link
Collaborator

Overview

Test 10.4, Civi 5.81
11.1 civi master branch
PHP 8.3 / 8.4

@civibot civibot bot added the 4.0.x label Dec 5, 2024
@jackrabbithanna
Copy link
Collaborator Author

This PR to civicrm core broke tests badly: civicrm/civicrm-core#31046
Kernel tests now fail when tried when CiviCRM Core is 5.78 or above

This file adds an event listener https://github.com/civicrm/civicrm-core/blob/master/setup/plugins/installDatabase/SyncUsers.civi-setup.php
and runs the synchonizeusers() method.
But in the Kernel tests, the Drupal modules are not installed yet when CiviCRM is setup.

This will require major rework of setup() in Kernel tests, and it may not be possible actually.
We may delete them, or possibly invest in converting them to FunctionalJavascript tests.

I have no idea why synchronizeUsers() needs to be called on setup, as there are no contacts yet on install. Boooo

@puresyntax71 puresyntax71 mentioned this pull request Jan 8, 2025
@jackrabbithanna
Copy link
Collaborator Author

Closing in favor of #510 after several other test updates in other PRs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant