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

[API] Add missing Admin Orders API contract tests #15907

Merged

Conversation

Rafikooo
Copy link
Contributor

Q A
Branch? 1.13
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
License MIT

We aim to achieve maximum API test coverage before the Sylius 1.13 release.

This PR enhances testing by introducing some utility functions

@Rafikooo Rafikooo added the API APIs related issues and PRs. label Feb 27, 2024
@Rafikooo Rafikooo requested a review from a team as a code owner February 27, 2024 13:24
Copy link

github-actions bot commented Feb 27, 2024

Bunnyshell Preview Environment deleted

Available commands:

  • /bns:deploy to redeploy the environment

tests/Api/Admin/OrdersTest.php Outdated Show resolved Hide resolved
tests/Api/Admin/OrdersTest.php Outdated Show resolved Hide resolved
tests/Api/Admin/OrdersTest.php Outdated Show resolved Hide resolved
@Rafikooo Rafikooo force-pushed the SYL-3349/admin-api-endpoints-improvements branch from 364312c to 49f5a17 Compare February 28, 2024 13:23
@NoResponseMate NoResponseMate merged commit 3826ebb into Sylius:1.13 Feb 28, 2024
28 checks passed
@NoResponseMate
Copy link
Contributor

Thank you, @Rafikooo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API APIs related issues and PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants