Skip to content

Conversation

@gproly
Copy link
Contributor

@gproly gproly commented Feb 12, 2025

Description

The code line: \Illuminate\Foundation\Testing\RefreshDatabaseState::$migrated = true; is no longer valid in Laravel 11.

Solution

The RefreshDatabase trait is removed so that the migrations are not re-executed in each test.

Related Tickets & Packages

https://processmaker.atlassian.net/browse/FOUR-21803

ci:connector-idp:feature/FOUR-20332
ci:package-actions-by-email:feature/FOUR-20332
ci:package-advanced-user-manager:feature/FOUR-20332
ci:package-ai:feature/FOUR-20332
ci:package-analytics-reporting:feature/FOUR-20332
ci:package-api-testing:feature/FOUR-20332
ci:package-auth:feature/FOUR-20332
ci:package-collections:feature/FOUR-20332
ci:package-data-sources:feature/FOUR-20332
ci:package-decision-engine:feature/FOUR-20332
ci:package-pm-blocks:feature/FOUR-20332
ci:package-projects:feature/FOUR-20332
ci:package-savedsearch:feature/FOUR-20332
ci:package-sentry:feature/FOUR-20332
ci:package-testing:feature/FOUR-20332
ci:package-translations:feature/FOUR-20332
ci:package-vocabularies:feature/FOUR-20332
ci:package-process-documenter:feature/FOUR-20332

ci:k8s-branch:fix-for-phpunit-10
...

## Description
The code line: `\Illuminate\Foundation\Testing\RefreshDatabaseState::$migrated = true;` is no longer valid in Laravel 11.

## Solution
The `RefreshDatabase` trait is removed so that the migrations are not re-executed in each test.

## Related Tickets & Packages
https://processmaker.atlassian.net/browse/FOUR-21803
@processmaker-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

@gproly gproly requested a review from nolanpro February 12, 2025 17:03
@nolanpro nolanpro merged commit 7b5b29a into feature/FOUR-20332 Feb 12, 2025
9 of 10 checks passed
@nolanpro nolanpro deleted the observation/FOUR-21803 branch February 12, 2025 17:14
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.

3 participants