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

Fix tests for Craft 4-version of the plugin #263

Merged
merged 6 commits into from
Aug 24, 2023

Conversation

qrazi
Copy link

@qrazi qrazi commented Aug 9, 2023

Two notable things about these fixes:

  • I bumped the minimum PHP version in composer.json to 8.0.2, as that is also required by the minimal Craft CMS version 4.3.5
  • I marked one test as incomplete, as I'm not sure the expected result is still expected in Craft 4?

janhenckens and others added 6 commits June 9, 2023 12:34
- Add required property when creating Field (EventHandlersTest, SearchableBehaviorTest)
- Mark failing test as incomplete for now (EventHandlersTest)
- Set config value to test environment (ScoutUtilityTest.php:it_returns_html)
- Pass Queue instance from Application instead of new instance to prevent error "Queue must be an application component." (ImportIndexTest.php:it_calls_update_on_the_engine)

Refers to studioespresso#226
…DB user as expected by test config and GH Actions

Refers to studioespresso#226
@janhenckens
Copy link
Member

Hey @qrazi, thanks for this and for #264!
I'll have a look at the incomplete test and then merged both.

@janhenckens janhenckens self-requested a review August 10, 2023 18:32
@janhenckens janhenckens self-assigned this Aug 10, 2023
@janhenckens janhenckens merged commit a8713a1 into studioespresso:develop Aug 24, 2023
4 checks passed
@qrazi qrazi deleted the fix_tests branch August 27, 2023 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants