summaryrefslogtreecommitdiff
path: root/spec/requests
Commit message (Collapse)AuthorAgeFilesLines
* Improve commit status API specsfix/commit-status-api-improvementGrzegorz Bizon2016-03-011-30/+19
|
* Improve commit status API specs (refactoring)Grzegorz Bizon2016-02-291-41/+76
|
* Return empty array when commit has no statuses in APIGrzegorz Bizon2016-02-291-37/+69
| | | | | | | This makes API endpoint for Commit Statuses return empty array instead of 404 when commit exists, but has no statuses. Closes #3080
* Fix ci_build trace traitKamil Trzcinski2016-02-192-3/+3
|
* Fix specsKamil Trzcinski2016-02-191-59/+76
|
* Introduce API for serving the artifacts archiveKamil Trzcinski2016-02-191-0/+25
|
* Record user that erased a build through APIGrzegorz Bizon2016-02-191-0/+5
|
* Use POST method instead of DELETE when erasing a buildGrzegorz Bizon2016-02-191-3/+3
| | | | | Discussion: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/2560#note_3742042
* Fix name of build erasable, remove superfluous method from itGrzegorz Bizon2016-02-191-2/+2
|
* Move build eraseable API to proper API contextGrzegorz Bizon2016-02-192-12/+25
|
* Do not allow to modify build if it has been erasedGrzegorz Bizon2016-02-191-21/+35
|
* Use DELETE verb for erasing a build contentGrzegorz Bizon2016-02-191-0/+18
| | | | Also added API specs for that
* Refactorize `ci_runner` factory and `let` definitions in runners API specTomasz Maczukin2016-02-191-16/+14
|
* Remove unnecessary parametersTomasz Maczukin2016-02-191-5/+5
|
* Modify expectations for update runner featureTomasz Maczukin2016-02-191-4/+4
|
* Change `.map{...}.inject{...}` to `any?{...}` for searching shared runnersTomasz Maczukin2016-02-191-5/+5
|
* Reorganize `let` statements in spec/requests/api/runners_spec.rbTomasz Maczukin2016-02-191-12/+29
|
* Move :runner_id param to POST body when enabling specific runner in projectTomasz Maczukin2016-02-191-8/+14
|
* Add some modifications to spec/requests/api/runners_spec.rbTomasz Maczukin2016-02-191-88/+112
|
* Fix runners filteringTomasz Maczukin2016-02-191-2/+2
|
* Fix runners API specTomasz Maczukin2016-02-191-11/+11
|
* Split `/runners` entrypoint to `/runners` and `/runners/all`Tomasz Maczukin2016-02-191-11/+43
|
* Fix runners filtering in APITomasz Maczukin2016-02-191-17/+24
|
* Add feature to enable/disable runner in projectTomasz Maczukin2016-02-191-0/+102
|
* Add basic runners management APITomasz Maczukin2016-02-191-0/+278
| | | | | | | - add feature to list runners - add feature to show runners details - add feature to delete runner - add feature to update runner
* Merge branch 'master' of https://gitlab.com/gitlab-org/gitlab-ce into ↵Jacob Vosmaer2016-02-175-94/+143
|\ | | | | | | git-archive-refactor
| * Merge branch 'skakirill/gitlab-ce-api-merge-request-setup-milestone' into ↵Douwe Maan2016-02-121-3/+12
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' API: Allow to set or update a merge-request's milestone _Originally opened at !2107 by @ skakirill._ - - - See merge request !2755
| | * Add a spec for setting/updating a MR's milestone via the APIskakirill/gitlab-ce-api-merge-request-setup-milestoneRémy Coutable2016-02-091-3/+12
| | |
| * | Merge branch 'Schniz/gitlab-ce-feature/merge-request-closes-issues-in-api' ↵Rémy Coutable2016-02-111-0/+22
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Add merge_requests/:merge_request_id/closes_issues _Originally opened at !2660 by @Schniz._ - - - Added `/api/v3/projects/:id/merge_requests/:merge_request_id/closes_issues` route in the API for getting the issues that should be closed when a merge request merged successfully See merge request !2779
| | * | Pass current_user to merge_request.closes_issues in API ans streamline MR ↵Schniz/gitlab-ce-feature/merge-request-closes-issues-in-apiRémy Coutable2016-02-111-12/+8
| | | | | | | | | | | | | | | | API specs
| | * | Added '/api/v3/projects/:id/merge_requests/:merge_request_id/closes_issues' ↵Gal Schlezinger2016-02-111-0/+26
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | route in the API Added some documentation for it Added to changelog Added curl example and an attribute table Moved the api route definition from "lib/api/issues.rb" to "lib/api/merge_requests.rb" Fixed the attributes and changed the documentation to be at "merge_requests.md" too Changed generic titles to more specific titles added an underscore added tests. it depends on a newer version of gitlab-test project I'm doing a since I need to add a branch to the `gitlab-test` repo removed the before using 'iid' instead of 'id' in the description to reference the issues. that makes the tests pass Removed the 'closes-issues' key from test_env. so it should pass the tests Moved the two initializations to the describe block Changed the changelog
| * | Merge remote-tracking branch 'dev/master' into 'master'Robert Speicher2016-02-092-22/+34
| |\ \
| | * | Fix commit status testsKamil Trzcinski2016-02-061-18/+30
| | | |
| | * | Merge remote-tracking branch 'origin/master' into ci-permissionsKamil Trzcinski2016-02-041-3/+34
| | |\ \ | | | |/ | | | | | | | | | | | | # Conflicts: # app/views/projects/builds/index.html.haml
| | * | Make the CI permission model simplerKamil Trzcinski2016-02-021-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This MR simplifies CI permission model: - read_build: allows to read a list of builds, artifacts and trace - update_build: allows to cancel and retry builds - create_build: allows to create builds from gitlab-ci.yml (not yet implemented) - admin_build: allows to manage triggers, runners and variables - read_commit_status: allows to read a list of commit statuses (including the overall of builds) - create_commit_status: allows to create a new commit status using API Remove all extra methods to manage permission. Made all controllers to use explicitly the new permissions.
| * | | Merge branch 'develop7/add-merge_status-to-merge-request-in-api' into 'master' Douwe Maan2016-02-091-0/+1
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Expose MergeRequest#merge_status _Originally opened at !2478 by @develop7._ - - - This enables API users to see whether a merge request can be merged without conflicts. See merge request !2756
| | * | | Add an API spec for MergeRequest#merge_status and properly document this ↵develop7/add-merge_status-to-merge-request-in-apiRémy Coutable2016-02-081-0/+1
| | | |/ | | |/| | | | | | | | | resource
| * | | Extract shared context level up in build specsci/improve-ci-build-apiGrzegorz Bizon2016-02-041-56/+17
| | | | | | | | | | | | | | | | Also improve performance of specs by joining some of examples.
| * | | Improve CI API specs related to operations on buildGrzegorz Bizon2016-02-041-21/+34
| |/ / | | | | | | | | | | | | Conflicts: spec/factories/ci/builds.rb
| * | Merge branch 'support-akismet' into 'master' Robert Speicher2016-02-031-0/+31
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support Akismet spam checking for creation of issues via API Currently any spam detected by Akismet for non-members via API will be logged in a separate table in the admin page. Closes #5612 See merge request !2266
| | * | Fix rubocop offensesDouglas Barbosa Alexandre2016-02-021-5/+9
| | | |
| | * | Refactor spam filtering on issues APIDouglas Barbosa Alexandre2016-02-021-3/+8
| | | |
| | * | Support Akismet spam checking for creation of issues via APIStan Hu2016-02-021-0/+22
| | |/ | | | | | | | | | | | | | | | | | | Currently any spam detected by Akismet by non-members via API will be logged in a separate table in the admin page. Closes #5612
| * | Add a spec for the API pagination Link headerRémy Coutable2016-02-021-3/+3
| |/
* | Fix APIJacob Vosmaer2016-02-111-3/+10
|/
* Improve consistency and duplication for Merge Request APIimprove-mr-apiDmitriy Zaporozhets2016-01-281-31/+31
| | | | | | | * Follow REST for merge request API route * Remove repeating comments API for MR Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix CI runner version not being properly updated when asking for a buildfix-ci-runners-version-updateKamil Trzcinski2016-01-272-0/+29
| | | | | | Due to broken implementation of attribute_for_keys the runner information was not updated correctly. This MR adds test to check that such scenario will never happen again.
* Add specs. Adjust filter.Josh Frye2016-01-191-0/+23
|
* Merge branch 'ci/build_dependencies' into 'master' Douwe Maan2016-01-151-0/+12
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let the CI runner know about builds that this build depends on This allows us to implement artifacts passing: runner will download artifacts from all prior builds. It will happen automatically, and always, as long as artifacts are enabled. ## The changes: This MR exposes list of prior builds in CI::API::Builds. **The API response when asking for builds** ```json { "id": 48584, "ref": "0.1.1", "tag": true, "sha": "d63117656af6ff57d99e50cc270f854691f335ad", "status": "success", "name": "pages", "token": "9dd60b4f1a439d1765357446c1084c", "stage": "test", "project_id": 479, "project_name": "test", "commands": "echo commands", "repo_url": "http://gitlab-ci-token:token@gitlab.example/group/test.git", "before_sha": "0000000000000000000000000000000000000000", "allow_git_fetch": false, "options": { "image": "docker:image", "artifacts": { "paths": [ "public" ] }, "cache": { "paths": [ "vendor" ] } }, "timeout": 3600, "variables": [ { "key": "CI_BUILD_TAG", "value": "0.1.1", "public": true } ], "dependencies": { "builds": [ { "id": 48584, "ref": "0.1.1", "tag": true, "sha": "d63117656af6ff57d99e50cc270f854691f335ad", "status": "success", "name": "build", "token": "9dd60b4f1a439d1765357446c1084c", "stage": "build", "project_id": 479, "project_name": "test", "artifacts_file": { "filename": "artifacts.zip", "size": 0 } } ] } } ``` ## How it will work? **Example:** ``` build: type: build script: - echo TEST > test_file artifacts: untracked: true rspec: type: test script: - test-my-project staging: type: deploy script: - scp test_file root@server.com: ``` **The flow:** 1. We run `build`. The `build` creates a file `test_file`. This file gets archived and send us build artifacts. 2. We run `rspec`. The `rspec` downloads build artifacts from `build`. Uses the `test_file`. 3. We run `staging`. The `staging` downloads build artifacts from `build` and `rspec`, but since the `rspec` doesn't have build artifacts we skip that build. Deploys the `test_file`. This partially implements the https://gitlab.com/gitlab-org/gitlab-ce/issues/3423. In the next release we will introduce option to configure what artifacts are received. /cc @grzesiek @DouweM @sytse @rspeicher See merge request !2437
| * Change dependencies.builds to depends_on_buildsKamil Trzcinski2016-01-151-2/+2
| |