summaryrefslogtreecommitdiff
path: root/spec/requests/api/merge_requests_spec.rb
Commit message (Collapse)AuthorAgeFilesLines
...
* Refactor API classes. So api classes like Gitlab::Issues become API::IssuesDmitriy Zaporozhets2013-05-141-1/+1
|
* fixed test using repo with commits but old factoryDmitriy Zaporozhets2013-04-011-1/+1
|
* API: merge request documentation updated, added return codes to functionsSebastian Ziebell2013-02-271-1/+6
| | | | | | The API documentation of merge requests contains info to status codes for all functions. Required arguments are now checked in the merge requests API functions and a `400 Bad Request` error is returned if they are not given.
* Merge branch 'master' into fixes/apiSebastian Ziebell2013-02-201-0/+17
|\ | | | | | | | | Conflicts: spec/requests/api/projects_spec.rb
| * Merge Request uses StateMachine nowAndrew8xx82013-02-181-0/+17
| |
* | API: extracted helper method to provide 400 bad request error with descriptionSebastian Ziebell2013-02-131-0/+5
| | | | | | | | | | | | Extracted a method for 400 error (Bad request) and adjusted code accordingly. The name of the missing attribute is used to show which one was missing from the request. It is used to give an appropriate message in the json response.
* | Test added to check creation of note to a non-existent MR via APISebastian Ziebell2013-02-061-0/+5
| |
* | Creating MR comment without a note returns status code 400 (Bad request)Sebastian Ziebell2013-02-061-0/+5
| | | | | | | | | | | | Creating a comment to an existing merge request via API without providing a note returns a status code 400 now, suggesting a bad request. The reason for this is the resource itself (MR) exists but the required property is not set.
* | Creating or updating a MR returns more informative status codes.Sebastian Ziebell2013-02-061-0/+30
|/ | | | | | | | | Using the API library to create or update a merge request at the moment a 404 error is returned. This is fine when the merge request in question does not exist, but does not provide good information that for example a required attribute is missing. A status code of 400 (Bad request) is returned when creating or updating a merge request when either `source_branch` or `target_branch` is missing. A status code of 409 is returned when `source_branch` and `target_branch` are the same. Tests are added for these cases.
* Rspec fixesDmitriy Zaporozhets2013-01-041-1/+1
|
* Fix api specs. Use id instead pathDmitriy Zaporozhets2013-01-021-6/+6
|
* Fix few bugs and tests after refactoring ownership logicDmitriy Zaporozhets2013-01-021-1/+1
|
* Fixed some tests and snippet colorizeDmitriy Zaporozhets2012-11-231-6/+6
|
* Remove backward compatibility of factories.Vincent Bonmalais2012-11-131-8/+8
|
* API: fix after reviewValeriy Sizov2012-10-251-6/+6
|
* API: tests for merge_requestsValeriy Sizov2012-10-251-0/+62