summaryrefslogtreecommitdiff
path: root/spec/requests/api/issues_spec.rb
Commit message (Collapse)AuthorAgeFilesLines
* Adds updated_at filter to issues and merge_requests APIJacopo2018-03-051-0/+36
|
* Adjust the API issues move spec to use project ID 0Stan Hu2018-02-261-2/+2
| | | | Closes #43631
* Remove extraneous tests from Issues API specrs-over-testingRobert Speicher2018-02-191-25/+1
| | | | | | | | These were all testing an implementation detail of `Issues::UpdateService` which is already well-tested and doesn't need to be re-tested here. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/25201
* Fix order dependencies in some specsNick Thomas2018-02-161-1/+1
| | | | | | Our automatically-generated project paths are of the form project<N>. If a spec manually specifies a project path of that form, it may conflict with the automatically-generated paths in some circumstances.
* Fix spec failures in issues_spec.rbAhmad Sherif2018-01-261-2/+2
|
* [API] Fix creating issue when assignee_id is empty42025-fix-issue-apiValery Sizov2018-01-151-0/+9
| | | | see https://gitlab.com/gitlab-org/gitlab-ce/issues/42025
* API: get participants from merge_requests & issuesBrent Greeff2018-01-051-0/+12
|
* Replace '.team << [user, role]' with 'add_role(user)' in specs36782-replace-team-user-role-with-add_role-user-in-specsblackst0ne2017-12-221-4/+4
|
* Merge branch 'issue_30663' into 'security-10-2'Sean McGivern2017-12-081-0/+14
| | | | | | | | | Prevent creating issues through API without having permissions See merge request gitlab/gitlabhq!2225 (cherry picked from commit c298bbaa88883343dc9cbbb6abec0808fb3b546c) 915b97c5 Prevent creating issues through API without having permissions
* Refactor `have_http_status` into `have_gitlab_http_status` in the specsJacopo2017-10-201-80/+80
|
* Add 'closed_at' attribute to Issues APIVitaliy @blackst0ne Klachkov2017-09-161-1/+10
|
* Add my_reaction_emoji param to /issues APIHiroyuki Sato2017-09-051-0/+10
|
* Merge branch '28453-add-time-estimate-time-spent-to-api-issue-output' into ↵Robert Speicher2017-08-301-0/+12
|\ | | | | | | | | | | | | | | | | 'master' Add time stats to Issue and Merge Request API Closes #28453 See merge request !13335
| * Add missing N+1 test to issues specTravis Miller2017-08-291-0/+12
| |
* | Merge branch 'api-delete-respect-headers' into 'master'Sean McGivern2017-08-301-0/+4
|\ \ | | | | | | | | | | | | API: Respect the 'If-Unmodified-Since' for delete endpoints See merge request !9621
| * | Add tests for the unmodified headerRobert Schilling2017-08-281-0/+4
| |/
* | replace `is_spam?` with `spam?`Maxim Rydkin2017-08-291-2/+2
|/
* Don't include EmailHelpers manually, pick with rspecuse-rspec-support-helperLin Jen-Shin2017-08-031-3/+1
| | | | | | | | | `:mailer` is needed to pick it easily, while `type: :mailer` is needed for picking it automatically for tests located in spec/mailers/*_spec.rb It's a bit complicated in spec/services/notification_service_spec.rb but we'll leave it alone for now.
* Change all `:empty_project` to `:project`rs-empty_project-defaultRobert Speicher2017-08-021-7/+7
|
* Merge issuable "reopened" state into "opened"merge-issuable-reopened-into-opened-stateYorick Peterse2017-07-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Having two states that essentially mean the same thing is very much like having a boolean "true" and boolean "mostly-true": it's rather silly. This commit merges the "reopened" state into the "opened" state while taking care of system notes still showing messages along the lines of "Alice reopened this issue". A big benefit from having only two states (opened and closed) is that indexing and querying becomes simpler and more performant. For example, to get all the opened queries we no longer have to query both states: SELECT * FROM issues WHERE project_id = 2 AND state IN ('opened', 'reopened'); Instead we can query a single state directly, which can be much faster: SELECT * FROM issues WHERE project_id = 2 AND state = 'opened'; Further, only having two states makes indexing easier as we will only ever filter (and thus scan an index) using a single value. Partial indexes could help but aren't supported on MySQL, complicating the development process and not being helpful for MySQL.
* Merge branch 'tc-issue-api-assignee' into 'master'Sean McGivern2017-07-271-1/+36
|\ | | | | | | | | | | | | Add author_id & assignee_id param to /issues API Closes #29430 See merge request !13004
| * Allow query param scope for /issues API endpointtc-issue-api-assigneeToon Claes2017-07-241-4/+12
| |
| * Add author_id & assignee_id param to /issues APIToon Claes2017-07-241-0/+27
| | | | | | | | Allow issues filtering on `author_id` and `assignee_id`.
* | Resolve "More RESTful API: include resource URLs in responses"Oswaldo Ferreira2017-07-251-0/+13
|/
* Merge branch 'feature/user-agent-details-api' into 'master'Douwe Maan2017-07-071-0/+19
|\ | | | | | | | | | | | | Allow admins to retrieve user agent details for an issue or snippet Closes #29508 See merge request !12655
| * refactor code based on feedbackJames Lopez2017-07-061-1/+1
| |
| * add user agent details API endpoints to issues and snippetsJames Lopez2017-07-061-1/+1
| |
| * add issues spec and user agent details to entitiesJames Lopez2017-07-061-0/+19
| |
* | Remove CE from spec example descriptionsToon Claes2017-07-071-2/+2
|/ | | | They are confusing.
* fix for Follow-up from "Backport of Multiple Assignees featurefixes_for_multiple_issue_assigneesValery Sizov2017-05-111-1/+1
|
* Multiple issue assignee: CE restriction for multiple assigneesValery Sizov2017-05-051-0/+22
|
* [Multiple issue assignee]: reslving some conflictsValery Sizov2017-05-041-8/+0
|
* Backport of multiple_assignees_feature [ci skip]Valery Sizov2017-05-041-6/+70
|
* Use rspec-set to speed up examplesZeger-Jan van de Weg2017-04-261-22/+27
|
* Merge branch '26437-closed-by' into 'master' Rémy Coutable2017-04-261-0/+35
|\ | | | | | | | | | | | | Add issues/:iid/closed_by api endpoint Closes #26437 See merge request !10511
| * Add issues/:iid/closed_by api endpointmhasbini2017-04-121-0/+35
| |
* | Unnecessary "include WaitForAjax" and "include ApiHelpers"Jacopo2017-04-211-2/+1
|/ | | | | Removed all the unnecessary include of `WaitForAjax` and `ApiHelpers` in the specs. Removed unnecessary usage of `api:true`
* Merge branch 'update-droplab-to-webpack-version' into new-resolvable-discussionLuke "Jared" Bennett2017-04-061-241/+135
|\
| * Add "search" optional param and docs for V430195-document-search-param-on-apiOswaldo Ferreira2017-04-051-241/+135
| | | | | | | | Notice that this param is being supported since V3, but we have not added the proper docs for it
* | Fix some specsDouwe Maan2017-04-051-1/+1
|/
* Don't use FFaker in factories, use sequences insteadRémy Coutable2017-04-031-3/+3
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fix API group/issues default state filterAlexander Randa2017-03-281-6/+14
|
* Return 404 in project issues API endpoint when project cannot be foundAdam Niedzielski2017-03-201-0/+6
| | | | Closes #29631
* Put the use of "No Label" in the descriptiontc-clean-up-no-label-docToon Claes2017-03-161-0/+10
| | | | | To get the issues that have no label, the special keyword `No Label` can be used. This `No Label` label can be used like any other label.
* Rename variableBob Van Landuyt2017-03-131-2/+2
| | | | | merge_request_for_resolving_discussions -> merge_request_to_resolve_discussions_of
* Always require MR-iid for resolving discussionsBob Van Landuyt2017-03-131-0/+1
| | | | | And deduplicate the finding of MR's & discussions. Now the searching is done in the service, istead of the controller & the API.
* Delegate a single discussion to a new issueBob Van Landuyt2017-03-131-13/+17
| | | | | | Delegate a discussion in a merge request into a new issue. The discussion wil be marked as resolved and a system note will be added linking to the newly created issue.
* Implement review comments from @dbalexandre28251-mr-and-issue-iids-for-api-v4Timothy Andrew2017-03-071-0/+6
| | | | | | - Typo in docs - Newline between test/expectation in `api/issues_spec` - Use `find_by` instead of `reference_by` in the structure defining awardables
* API routes referencing a specific issue should use the issue `iid`Timothy Andrew2017-03-071-55/+95
| | | | | | - As opposed to the issue `id` that was previously being used. - This brings the API routes closer to the web interface's routes. - This is specific to API v4.
* Remove "subscribed" field from API responses returning list of issues or ↵api-drop-subscribedAdam Niedzielski2017-03-061-0/+7
| | | | merge requests
View Lorry Controller Status