Skip to content

FOUR-21803 Fix failing unit tests#7998

Merged
nolanpro merged 2 commits intofeature/FOUR-20332from
observation/FOUR-21803
Feb 7, 2025
Merged

FOUR-21803 Fix failing unit tests#7998
nolanpro merged 2 commits intofeature/FOUR-20332from
observation/FOUR-21803

Conversation

@gproly
Copy link
Copy Markdown
Contributor

@gproly gproly commented Feb 6, 2025

Description

Tests\Feature\Api\PerformanceModelsTest::testFactories
The data provider specified for Tests\Feature\Api\PerformanceModelsTest::testFactories is invalid Empty data set provided by data provider

Solution

This is a PHPUnit issue; the test is being fixed.

Related Tickets & Packages

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

## Description
Tests\Feature\Api\PerformanceModelsTest::testFactories
The data provider specified for Tests\Feature\Api\PerformanceModelsTest::testFactories is invalid
Empty data set provided by data provider

## Solution
This is a PHPUnit issue; the test is being fixed.

## Related Tickets & Packages
https://processmaker.atlassian.net/browse/FOUR-21803
## Description
1) Tests\Feature\Api\PerformanceRoutesTest::testRoutesSpeed
The data provider specified for Tests\Feature\Api\PerformanceRoutesTest::testRoutesSpeed is invalid
Undefined constant Tests\Feature\Api\PerformanceRoutesTest::endpoints

/Users/user/srv/http/processmaker4/tests/Feature/Api/PerformanceRoutesTest.php:94

## Solution
This is a PHPUnit issue; the test is being fixed.

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

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 CarliPinell February 7, 2025 13:08
@nolanpro nolanpro merged commit d894c6d into feature/FOUR-20332 Feb 7, 2025
7 of 8 checks passed
@nolanpro nolanpro deleted the observation/FOUR-21803 branch February 7, 2025 16:09
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.

2 participants