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

test: test that an options request on root endpoint is successfull #434

Closed
wants to merge 1 commit into from

Conversation

BacLuc
Copy link

@BacLuc BacLuc commented Aug 1, 2024

Related to api-platform/core#6455 It is more or less the same test as in
https://github.com/ecamp/ecamp3/blob/3cd279fd1c1367287ea989f38f7238f93026e4b4/api/tests/Api/RootTest.php

Q A
Branch? main
Tickets
License MIT
Doc PR

This is a test that tests the OPTIONS request on the root endpoint.
I did not know where to add this in https://github.com/api-platform/core
So i added the test here that a possible issue is detected a little earlier.

@vincentchalamon
Copy link
Contributor

Hi @BacLuc,

Thank you for such contribution. However, even if the tests are green, I don't see the point of such test in this project as it's a feature covered by Nelmio.

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