summaryrefslogtreecommitdiff
path: root/spec/models/ci
Commit message (Collapse)AuthorAgeFilesLines
* Fix broken tests17662-rename-buildsFilipa Lacerda2017-02-031-2/+2
| | | | | | | | | | | | | | | | | | | | Rename Build to Job Replace "Builds" by "Jobs" and fix broken specs Replace "Builds" by "Jobs" Fix broken spinach test Fix broken test Remove `˙` at the beginning of the file Fix broken spinach test Fix broken tests Changes after review
* Merge branch '20248-add-coverage-regex-in-job-yaml' into 'master'Douwe Maan2017-02-021-0/+41
|\ | | | | | | | | | | | | Add ability to define a coverage regex in the .gitlab-ci.yml Closes #20428 See merge request !7447
| * Make more code improvements around the '/' stripping logicLeandro Camargo2017-01-251-2/+2
| |
| * Force coverage value to always be surrounded by '/'Leandro Camargo2017-01-251-4/+5
| |
| * Improve specs styles/organization and add some more specsLeandro Camargo2017-01-251-6/+7
| |
| * Comply to more requirements and requests made in the code reviewLeandro Camargo2017-01-251-3/+8
| |
| * Simplify coverage setting and comply to some requests in code reviewLeandro Camargo2017-01-251-3/+4
| |
| * Add specs to cover the implemented feature and fix a small bugLeandro Camargo2017-01-251-0/+33
| |
* | Use `:empty_project` where possible in model specsrs-empty_project-modelsRobert Speicher2017-01-263-9/+8
|/
* Merge branch 'feature/success-warning-icons-in-stages-builds' into 'master' Kamil Trzciński2017-01-232-39/+136
|\ | | | | | | | | | | | | Use a warning icon for a stage with allowed to fail builds Closes #21948 See merge request !8503
| * Fix warnings argument memoization in CI/CD stagefeature/success-warning-icons-in-stages-buildsGrzegorz Bizon2017-01-191-5/+19
| |
| * Improve readability of specs for pipeline stagesGrzegorz Bizon2017-01-181-39/+64
| |
| * Pass memoizable warnings attribute to stage objectGrzegorz Bizon2017-01-181-5/+15
| |
| * Add specs for two new methods defined in stage classGrzegorz Bizon2017-01-111-0/+48
| |
* | Fix specsKamil Trzcinski2017-01-191-5/+28
| |
* | Add missing build_specKamil Trzcinski2017-01-191-0/+10
| |
* | Merge remote-tracking branch 'origin/master' into 21698-redis-runner-last-buildKamil Trzcinski2017-01-193-12/+1485
|\ \
| * \ Merge branch 'master' into fix/keep-artifacts-button-visibilityfix/keep-artifacts-button-visibilityGrzegorz Bizon2017-01-162-3/+1335
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | * master: (579 commits) Conflicts: spec/models/build_spec.rb
| | * | Fix Rubocop offense in build specsfix/merge-separate-build-spec-filesGrzegorz Bizon2017-01-131-1/+0
| | | |
| | * | Merge build specs into file that has valid locationGrzegorz Bizon2017-01-131-3/+1276
| | |/
| | * fix pipelines/index.html.haml merge conflictRegis2017-01-022-0/+39
| | |\ | | |/ | |/|
| | * Add missing specs for new methods in pipeline classGrzegorz Bizon2016-12-271-0/+42
| | |
| * | Merge branch 'fix-latest-pipeine-ordering-again' into 'master' Kamil Trzciński2016-12-261-0/+13
| |\ \ | | | | | | | | | | | | | | | | Order only for latest_successful_for See merge request !8301
| | * | Add a test for latest_successful_forfix-latest-pipeine-ordering-againLin Jen-Shin2016-12-261-0/+13
| | |/
| | * Revert "Merge branch 'fix-latest-pipeine-ordering' into 'master'"Grzegorz Bizon2016-12-241-6/+8
| | | | | | | | | This reverts merge request !8286
| | * Update description stating that ordering is importantLin Jen-Shin2016-12-231-2/+2
| | | | | | | | | | | | Feedback from Grzegorz
| | * Ci::Pipeline.latest order by id DESCLin Jen-Shin2016-12-231-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The name latest implies that it's reverse chronological, and we did expect it that way. https://gitlab.com/gitlab-org/gitlab-ce/issues/25993#note_20429761 >>> ok, I think markglenfletchera is correct in https://gitlab.com/gitlab-com/support-forum/issues/1394#note_20399939 that `Project#latest_successful_builds_for` is giving oldest pipeline rather than latest pipeline. This is a ~regression introduced by !7333 where `order(id: :desc)` was removed causing this. The offending change was: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333/diffs#b22732e5f39e176c7c719fe485847d0fb0564275_92_108 The confusion was caused by the `latest` name implication, which actually didn't order anything, and I think we should add `order(id: :desc)` to `Ci::Pipeline.latest` otherwise it's confusing that it's not actually ordered. >>> Closes #25993
| * | Add more storage statisticsMarkus Koller2016-12-211-0/+26
| |/ | | | | | | | | | | | | | | | | | | | | | | | | This adds counters for build artifacts and LFS objects, and moves the preexisting repository_size and commit_count from the projects table into a new project_statistics table. The counters are displayed in the administration area for projects and groups, and also available through the API for admins (on */all) and normal users (on */owned) The statistics are updated through ProjectCacheWorker, which can now do more granular updates with the new :statistics argument.
| * Fix stage and pipeline specs and rubocop offenses19703-direct-link-pipelinesGrzegorz Bizon2016-12-202-4/+4
| |
| * Fix broken testFilipa Lacerda2016-12-201-1/+1
| | | | | | | | Changes after review
| * Improve specsKamil Trzcinski2016-12-202-7/+13
| |
| * Add tests for stage API endpointKamil Trzcinski2016-12-202-0/+31
| |
| * Add explicit status test, feedback:show-commit-status-from-latest-pipelineLin Jen-Shin2016-12-151-0/+2
| | | | | | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20058993 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20059060 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20059357
| * Use described_class and update descriptionLin Jen-Shin2016-12-151-9/+9
| | | | | | | | | | | | | | | | | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20059124 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20059187 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20059322
| * Introduce latest_status and add a few testsLin Jen-Shin2016-12-141-0/+57
| | | | | | | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20003268
| * Fix detailed status specs for pipeline stage modelGrzegorz Bizon2016-12-121-1/+3
| |
| * Fix pipeline specs for detailed statusesGrzegorz Bizon2016-12-121-8/+12
| | | | | | | | [ci skip]
* | WIP: Add tests and make sure that headers are set21698-redis-runner-last-buildLin Jen-Shin2017-01-041-0/+33
|/ | | | | | | | | | * We realized that headers were not set whenever we give 204 because `render_api_error!` doesn't preserve the headers. * We also realized that `update_runner_info` would be called in POST /builds/register every time therefore runner is updated every time, ticking the queue, making this last_update didn't work very well, and the test would be failing due to that.
* Merge branch 'pipeline-stage' into 'master' Rémy Coutable2016-12-082-9/+179
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | Refine pipeline stages ## What does this MR do? Introduces a concept of `Ci::Stage` to make it easier to have detailed statuses. ## Why was this MR needed? This is needed to simplify the handling of `Ci::Statuses` and make the `Stage` actual concept in code: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7889 See merge request !7927
| * Added Ci::Stage specsKamil Trzcinski2016-12-062-0/+137
| |
| * Fix handling of allowed to failure jobsKamil Trzcinski2016-12-061-9/+42
| |
* | Merge branch '25209-improve-length-validators' into 'master' Rémy Coutable2016-12-071-0/+7
|\ \ | |/ |/| | | | | | | | | Use :maximum instead of :within for length validators with a 0..N range Closes #25209 See merge request !7894
| * Use :maximum instead of :within for length validators with a 0..N range25209-improve-length-validatorsRémy Coutable2016-12-061-0/+7
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Expose pipeline detailed status using status factoryGrzegorz Bizon2016-12-051-0/+70
|/
* Only include EmailHelpers in mailer specs and specs using themrs-include-emailhelpersRobert Speicher2016-12-011-0/+2
|
* Merge branch 'fix-cancelling-pipelines' into 'master' Kamil Trzciński2016-11-251-0/+154
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve how we could cancel pipelines: Improve how we could cancel pipelines: * Introduce `HasStatus.cancelable` which we might be able to cancel * Cancel and check upon `cancelable` * Also cancel on `CommitStatus` rather than just `Ci::Build` Fixes #23635 Fixes #17845 See merge request !7508
| * Expand the loop and reduce overlapped conditionsLin Jen-Shin2016-11-221-12/+30
| | | | | | | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7508#note_18794681
| * Passing a user to retry_failed in testsLin Jen-Shin2016-11-221-3/+3
| | | | | | | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7508#note_18794547
| * Test against all possible cases, feedback:Lin Jen-Shin2016-11-221-14/+17
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7508#note_18755739
| * Merge remote-tracking branch 'upstream/master' into fix-cancelling-pipelinesLin Jen-Shin2016-11-211-0/+6
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upstream/master: (302 commits) Fix a wrong "The build for this merge request failed" message Add documentation about todos and failed builds Add Changelog entry for failed builds todos fix Do not create TODO when build is allowed to fail Fix 404 on some group pages when name contains dot Grapify the users API Remove duplicate sidekiq throttling parameters Fix regression in Merge request form Fix wrong link Fix test Fix broken test Changes for stop url to path fix typo in gitlab_flow.md ('munch'->'much') Fix spec Backport some changes done from Time Tracking feature in EE. Use build instead create in BroadcastMessage model spec Try to fix tests Remove unnecessary self from user model Expose stop_path for environment to not construct that in frontend Bring back the `commit_url` as it's used by CycleAnalytics ...
View Lorry Controller Status