Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Refactor API classes. So api classes like Gitlab::Issues become API::Issues | Dmitriy Zaporozhets | 2013-05-14 | 1 | -1/+1 | |
| | ||||||
* | fixed test using repo with commits but old factory | Dmitriy Zaporozhets | 2013-04-01 | 1 | -1/+1 | |
| | ||||||
* | API: merge request documentation updated, added return codes to functions | Sebastian Ziebell | 2013-02-27 | 1 | -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/api | Sebastian Ziebell | 2013-02-20 | 1 | -0/+17 | |
|\ | | | | | | | | | Conflicts: spec/requests/api/projects_spec.rb | |||||
| * | Merge Request uses StateMachine now | Andrew8xx8 | 2013-02-18 | 1 | -0/+17 | |
| | | ||||||
* | | API: extracted helper method to provide 400 bad request error with description | Sebastian Ziebell | 2013-02-13 | 1 | -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 API | Sebastian Ziebell | 2013-02-06 | 1 | -0/+5 | |
| | | ||||||
* | | Creating MR comment without a note returns status code 400 (Bad request) | Sebastian Ziebell | 2013-02-06 | 1 | -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 Ziebell | 2013-02-06 | 1 | -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 fixes | Dmitriy Zaporozhets | 2013-01-04 | 1 | -1/+1 | |
| | ||||||
* | Fix api specs. Use id instead path | Dmitriy Zaporozhets | 2013-01-02 | 1 | -6/+6 | |
| | ||||||
* | Fix few bugs and tests after refactoring ownership logic | Dmitriy Zaporozhets | 2013-01-02 | 1 | -1/+1 | |
| | ||||||
* | Fixed some tests and snippet colorize | Dmitriy Zaporozhets | 2012-11-23 | 1 | -6/+6 | |
| | ||||||
* | Remove backward compatibility of factories. | Vincent Bonmalais | 2012-11-13 | 1 | -8/+8 | |
| | ||||||
* | API: fix after review | Valeriy Sizov | 2012-10-25 | 1 | -6/+6 | |
| | ||||||
* | API: tests for merge_requests | Valeriy Sizov | 2012-10-25 | 1 | -0/+62 | |