aboutsummaryrefslogtreecommitdiffhomepage
path: root/tests/Wallabag/ApiBundle/Controller
Commit message (Collapse)AuthorAgeFilesLines
* tests: fix broken session clients for rest entry deletion testsKevin Decherf2019-02-201-4/+13
| | | | Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* EntryRestController: add support of expect parameter to delete actionKevin Decherf2019-02-201-5/+16
| | | | | | | | | | | The expect parameter enables an application to request the whole entry or only the id when requesting its deletion. `expects` defaults to `entry` to prevent any API breakage. Fixes #3711 Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* tests: create entry for testDeleteEntry, fix missing idKevin Decherf2019-02-171-7/+29
| | | | | | | | | | | When using the entity manager to retrieve an already stored entry, the id disapears from $entry after the first delete call. This leads to testing a nonexistent endpoint (api/entries/.json) during the second delete call. This change now creates an entry specifically for the test. Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* Avoid error when a bad `order` parameter is givenJeremy Benoist2019-01-141-0/+9
| | | | Only allowed parameter are asc & desc
* Merge pull request #3831 from wallabag/fix/api-bad-client-idJérémy Benoist2019-01-101-0/+14
|\ | | | | Cast client id to avoid PG error
| * Cast client id to avoid PG errorJeremy Benoist2019-01-091-0/+14
| | | | | | | | If someone send a malformated client_id when trying to authenticate using the API we got a 500 if wallabag use postgres because the request send a string instead of an integer.
* | TagRestController: fix test for tag without entriesKevin Decherf2018-12-301-2/+12
| | | | | | | | | | | | | | | | As the deletion now requires that at least one entry for the user must be linked to the given tag, we fix the test testDeleteUserTag by linking it to an entry. Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* | TagRestController: add tests to ensure that other user's tags are unreachableKevin Decherf2018-12-301-0/+32
|/ | | | Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* php-cs-fixerKevin Decherf2018-09-052-17/+17
| | | | Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* Fix testsJeremy Benoist2018-06-061-17/+17
|
* Fixed testsNicolas Lœuillet2018-01-121-1/+1
|
* api: copy entry object before sending, to keep idKevin Decherf2017-12-171-0/+1
| | | | | | Workaround for https://github.com/wallabag/android-app/issues/646 Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* Fix empty title and domain_name when exception is thrown during fetchKevin Decherf2017-12-131-0/+2
| | | | | | | | | | Add a new helper to set a default title when it's empty: 1/ use basename part of entry's path, if any 2/ or use domain name Fixes #2053 Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* CSJeremy Benoist2017-11-211-1/+1
|
* Fix phpcsKevin Decherf2017-11-191-17/+17
| | | | Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* Improve EntryRestControllerTest for origin_urlKevin Decherf2017-11-191-0/+55
| | | | | | | | Ensure that origin_url is initially null Ensure patching entry with origin_url='' Ensure patching entry with origin_url=null Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* Add some tests for OriginUrl in EntryRestControllerKevin Decherf2017-11-191-0/+73
| | | | Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
* CSJeremy Benoist2017-10-121-1/+1
|
* Fix #3361 add testSébastien Viande2017-10-121-0/+7
|
* Multiple tag search was broken from APIJeremy Benoist2017-09-061-0/+3
| | | | | | | | | | First, the setParameter() were done on the same parameter which in fact just duplicated the condition in the SQL query (like `where t.label = 'test' and t.label = 'test'`. Changed the parameter doesn't help because the query was then wrong. Changing the way to match associated tags for an entry and it worked.
* Set a starred_at field when an entry is starred.François D2017-08-251-0/+5
| | | | | | | | This date is used to sort starred entries. Can not use Entry::timestamps method otherwise starred_at will be updated each time entry is updated. Add an updateStar method into Entry class A migration script has been added in order to set starred_at field.
* Add a test for updatePublishedAtJeremy Benoist2017-07-241-0/+34
| | | | | | | To avoid error when a content is re-submitted and it previously add a published date. Also, fix the `testPostSameEntry`
* Revert client_credentials grant typesJeremy Benoist2017-07-081-21/+1
|
* CSJeremy Benoist2017-07-031-15/+14
|
* Fix PATCH methodJeremy Benoist2017-07-031-10/+58
| | | | | The PATCH method for the entry should only update what user sent to us and not the whole entry as it was before. Also, sending tags when patching an entry will now remove all current tags & assocatied new ones.
* Fix testsJeremy Benoist2017-07-031-12/+12
|
* Add a real configuration for CS-FixerJeremy Benoist2017-07-015-208/+208
|
* Return null instead of falseJeremy Benoist2017-06-281-3/+3
|
* Avoid BC on exists endpointJeremy Benoist2017-06-231-4/+37
|
* Add client_credentials oAuth2 auth methodThomas Citharel2017-06-141-3/+24
| | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* Add public filter/field in the APIJeremy Benoist2017-06-101-0/+52
| | | | | | Listing entries can now be filtered by “public”. Creating or patching an entry can now set is to public or remove the public. Entry response now include “is_public” boolean field
* Fix testJeremy Benoist2017-06-081-4/+4
| | | | | de_DE is not valid language. Zataz doesn’t send a valid language in their content (they use `fr-FR`).
* Add ability to name the clientJeremy Benoist2017-06-081-1/+2
|
* Create a client when creating a user using the apiJeremy Benoist2017-06-071-0/+12
| | | | | | | | While creating a new user using the API, we also create a new client for the current user. So the app which just create the user can use its newly created client to configure the app. That new client is only return after creating the user. When calling the endpoint /api/user to get user information, the new client information won’t be return.
* Merge pull request #3181 from wallabag/api-content-patchJérémy Benoist2017-06-071-0/+11
|\ | | | | Add ability to patch an entry with more fields
| * Add ability to patch an entry with more fieldsJeremy Benoist2017-06-021-0/+11
| | | | | | | | | | | | | | | | | | Like when we create an entry, we can now patch an entry with new fields: - content - language - preview_picture - published_at - authors
* | Return 201 on user creationJeremy Benoist2017-06-021-2/+2
| |
* | API user creation behing a toggleJeremy Benoist2017-06-022-21/+85
|/ | | | | | | | I've added a toggle feature (in internal settings) so that user api creation can be disabled while form registration still can be enabled. Also, the /api/user endpoint shouldn't require authentication. Even if we check the authentication when sending a GET request, to retrieve current user information. I've moved all the internal settings definition to config to avoid duplicated place to define them. I don't know why we didn't did that earlier.
* Merge pull request #3137 from aaa2000/isolated-testsJérémy Benoist2017-06-013-41/+69
|\ | | | | Isolated tests
| * Isolated testsadev2017-05-313-41/+69
| | | | | | | | Use https://github.com/dmaicher/doctrine-test-bundle to have test isolation.
* | Add support for authorsJeremy Benoist2017-05-311-0/+4
| |
* | Allow other fields to be send using APIJeremy Benoist2017-05-311-0/+6
|/ | | | | | | | | | | | | | | | Entry API can now have these new fields: - content - language - preview_picture - published_at Re-use the ContentProxy to be able to do the same using the web UI (in the future). htmLawed is used to clean stuff from content, I hope it’ll be enough to avoid security breach. Lower content validation when we want to update an entry with content already defined. Before, language & content_type were required. If there weren’t provided, we re-fetched the content using graby. I think these fields aren’t required for an entry to be created. So I removed them. Which means some import from the v1 export won’t be re-fetched since they provide content, url & title. Also, remove liberation link from Readability import to avoid overlaping import (from wallabag v1, which had the same link)
* Avoid side effect on other testsJeremy Benoist2017-05-301-1/+13
|
* Re-use `NewUserType` to validate registrationJeremy Benoist2017-05-301-0/+98
| | | | | | The only ugly things is how we handle error by generating the view and then parse the content to retrieve all errors… Fix exposition fields in User entity
* Merge remote-tracking branch 'origin/master' into 2.3Jeremy Benoist2017-05-191-0/+34
|\
| * Merge pull request #3095 from aaa2000/api-error-on-fail-fetch-contentThomas Citharel2017-05-101-0/+34
| |\ | | | | | | Create a new entry via API even when its content can't be retrieved
| | * Remove the created entry to avoid side effects on other testsadev2017-05-091-8/+18
| | |
| | * Create a new entry via API even when its content can't be retrievedadev2017-05-081-0/+24
| | | | | | | | | | | | Fix #2988
* | | Retrieve tag / tags value from query or requestJeremy Benoist2017-05-091-4/+26
| | | | | | | | | | | | | | | It allows to request to delete a tag using query string instead of body parameter (which seems to be the standard). Instead of breaking the previous behavior, I used a generic way to retrieve parameter (which looks into request attributes, query parameters and request parameters)
* | | Merge remote-tracking branch 'origin/master' into 2.3Jeremy Benoist2017-05-091-0/+16
|\| |