-
Notifications
You must be signed in to change notification settings - Fork 10
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 805 response body expects "bad request" #104
Comments
benjifs
added a commit
to benjifs/micropub
that referenced
this issue
Oct 11, 2021
[micropub.rocks](https://micropub.rocks) ### Creating Posts (Form-Encoded) ✅ | 100 | Create an h-entry post (form-encoded) ✅ | 101 | Create an h-entry post with multiple categories (form-encoded) ✅ | 104 | Create an h-entry with a photo referenced by URL (form-encoded) ✅ | 107 | Create an h-entry post with one category (form-encoded) ### Creating Posts (JSON) ✅ | 200 | Create an h-entry post (JSON) ✅ | 201 | Create an h-entry post with multiple categories (JSON) ➖ | 202 | Create an h-entry with HTML content (JSON) ✅ | 203 | Create an h-entry with a photo referenced by URL (JSON) ➖ | 204 | Create an h-entry post with a nested object (JSON) ✅ | 205 | Create an h-entry post with a photo with alt text (JSON) ✅ | 206 | Create an h-entry with multiple photos referenced by URL (JSON) ### Creating Posts (Multipart) ✅ | 300 | Create an h-entry with a photo (multipart) ✅ | 301 | Create an h-entry with two photos (multipart) ### Updates ✅ | 400 | Replace a property ✅ | 401 | Add a value to an existing property ✅ | 402 | Add a value to a non-existent property ✅ | 403 | Remove a value from a property ✅ | 404 | Remove a property ✅ | 405 | Reject the request if operation is not an array ### Deletes ✅ | 500 | Delete a post (form-encoded) ✅ | 501 | Delete a post (JSON) ➖ | 502 | Undelete a post (form-encoded) ➖ | 503 | Undelete a post (JSON) ### Query ✅ | 600 | Configuration Query ✅ | 601 | Syndication Endpoint Query ✅ | 602 | Source Query (All Properties) ✅ | 603 | Source Query (Specific Properties) ### Media Endpoint ✔️ | 700 | Upload a jpg to the Media Endpoint ✔️ | 701 | Upload a png to the Media Endpoint ✔️ | 702 | Upload a gif to the Media Endpoint ### Authentication ✅ | 800 | Accept access token in HTTP header ✅ | 801 | Accept access token in POST body⚠️ | 802 | Does not store access token property ✅ | 803 | Rejects unauthenticated requests⚠️ | 804 | Rejects unauthorized access tokens⚠️ | 805 | Rejects multiple authentication methods ✔️ - 700, 701, 702 work but show as failed because of [Issue #102](aaronpk/micropub.rocks#102)⚠️ - 802 [Issue #103](aaronpk/micropub.rocks#103)⚠️ - 804 [Issue #101](aaronpk/micropub.rocks#101)⚠️ - 805 [Issue #104](aaronpk/micropub.rocks#104) ➖ - Unsupported
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Should the response body be checked here?
micropub.rocks/views/server-tests/805.php
Lines 52 to 54 in eeac57a
In Test 405 it only checks for the status code when receiving a
400
.If the response body is checked, based on the spec it should be returning:
The text was updated successfully, but these errors were encountered: