summaryrefslogtreecommitdiff
path: root/spec/models/commit_spec.rb
Commit message (Collapse)AuthorAgeFilesLines
* Setup and run a Gitaly server for testing if GitalyClient is enabledgitaly-testingAlejandro Rodríguez2017-04-111-27/+28
|
* Merge branch 'feature/use-gitaly-for-commit-show' into 'master'Robert Speicher2017-03-171-0/+28
|\ | | | | | | | | Use Gitaly for CommitController#show See merge request !9629
| * Use Gitaly for CommitController#showfeature/use-gitaly-for-commit-showAhmad Sherif2017-03-151-0/+28
| |
* | Adds pipeline mini-graph to system information box in Commit ViewFilipa Lacerda2017-03-161-0/+19
|/
* Use `:empty_project` where possible in model specsrs-empty_project-modelsRobert Speicher2017-01-261-7/+7
|
* Allows to search within project by commit's hashYarNayar2017-01-241-0/+18
| | | | Was proposed in #24833
* Mark MR as WIP when pushing WIP commitsjurre2017-01-191-0/+28
|
* 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-1/+1
| | | | | | | | | | 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-2/+2
| | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_20003268
* Merge remote-tracking branch 'upstream/master' into ↵Lin Jen-Shin2016-12-141-9/+33
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | show-commit-status-from-latest-pipeline * upstream/master: (557 commits) Fix wrong error message expectation in API::Commits spec Move admin settings spinach feature to rspec Encode when migrating ProcessCommitWorker jobs Prevent overflow with vertical scroll when we have space to show content Make rubocop happy API: Ability to cherry-pick a commit Be smarter when finding a sudoed user in API::Helpers Backport hooks on group policies for the EE-specific implementation API: Ability to get group's project in simple representation Add AddLowerPathIndexToRoutes to setup_postgresql.rake For single line git commit messages, the close quote should be on the same line as the open quote added border-radius and padding to labels Allow all alphanumeric characters in file names (!8002) Add failing test for #20190 Don't allow blank MR titles in API Replace static fixture for awards_handler_spec (!7661) Crontab typo '* */6' -> '0 */6' (4x/day not 1x-per-min-for-1h 4x/day) Fix test Tweak style and add back wording Clean up commit copy to clipboard and make consistent ...
| * Merge branch 'jej-23867-use-mr-finder-instead-of-access-check' into 'security'Douwe Maan2016-12-081-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace MR access checks with use of MergeRequestsFinder Split from !2024 to partially solve https://gitlab.com/gitlab-org/gitlab-ce/issues/23867 :warning: - Potentially untested :bomb: - No test coverage :traffic_light: - Test coverage of some sort exists (a test failed when error raised) :vertical_traffic_light: - Test coverage of return value (a test failed when nil used) :white_check_mark: - Permissions check tested - [x] :bomb: app/finders/notes_finder.rb:17 - [x] :warning: app/views/layouts/nav/_project.html.haml:80 [`.count`] - [x] :bomb: app/controllers/concerns/creates_commit.rb:84 - [x] :traffic_light: app/controllers/projects/commits_controller.rb:24 - [x] :traffic_light: app/controllers/projects/compare_controller.rb:56 - [x] :vertical_traffic_light: app/controllers/projects/discussions_controller.rb:29 - [x] :white_check_mark: app/controllers/projects/todos_controller.rb:27 - [x] :vertical_traffic_light: app/models/commit.rb:268 - [x] :white_check_mark: lib/gitlab/search_results.rb:71 - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_267_266 Memoize ` merged_merge_request(current_user)` - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_248_247 Expected side effect for `merged_merge_request!`, consider `skip_authorization: true`. - [x] https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2024/diffs#d1c10892daedb4d4dd3d4b12b6d071091eea83df_269_269 Scary use of unchecked `merged_merge_request?` See merge request !2033
| * Add shorthand support to gitlab markdown referencesOswaldo Ferreira2016-12-021-4/+10
| |
| * Pass commit data to ProcessCommitWorkerprocess-commit-worker-improvementsYorick Peterse2016-12-011-0/+17
| | | | | | | | | | | | | | | | | | | | | | By passing commit data to this worker we remove the need for querying the Git repository for every job. This in turn reduces the time spent processing each job. The migration included migrates jobs from the old format to the new format. For this to work properly it requires downtime as otherwise workers may start producing errors until they're using a newer version of the worker code.
* | Avoid using random in the tests, feedback:Lin Jen-Shin2016-11-241-2/+2
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_18860042
* | Fix test description to mention latest pipeline,Lin Jen-Shin2016-11-231-2/+2
| | | | | | | | | | feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7333#note_18819886
* | Still use compound pipeline status, but group byLin Jen-Shin2016-11-211-4/+4
| | | | | | | | | | | | ref and sha so that it would show latest pipeline if ref and sha are both specified, otherwise still the same as before.
* | Show commit status from latest pipelineLin Jen-Shin2016-11-081-16/+11
|/ | | | | | Rather than compound status from all pipelines. Closes #20560
* There's no such methodLin Jen-Shin2016-11-041-4/+0
|
* Merge status_for and status, feedback:Lin Jen-Shin2016-11-031-34/+36
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7034#note_17742297
* Add tests for Commit#status and Commit#status_for, feedback:Lin Jen-Shin2016-10-281-1/+44
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7034#note_17543036
* refactors tests because of gitlab-test repository changes20708-new-branch-is-immediatelly-tagged-as-mergedtiagonbotelho2016-10-111-4/+4
|
* Expand commit message width in repo viewClement Ho2016-08-021-0/+21
|
* Add specs for caching commit authorStan Hu2016-07-271-0/+20
|
* Ensure relative paths for video are rewritten as we do for images20189-markdown-video-doesn-t-work-when-the-referenced-video-file-is-in-same-repoRémy Coutable2016-07-261-0/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Optimize Banzai::Filter::RelativeLinkFilterAlejandro Rodríguez2016-06-211-0/+12
| | | | | | | | | | | | A lot of git operations were being repeated, for example, to build a url you would ask if the path was a Tree, which would call a recursive routine in Gitlab::Git::Tree#where, then ask if the path was a Blob, which would call a recursive routine at Gitlab::Git::Blob#find, making reference to the same git objects several times. Now we call Rugged::Tree#path, which allows us to determine the type of the path in one pass. Some other minor improvement added, like saving commonly used references instead of calculating them each time.
* Refactor ParticipableYorick Peterse2016-06-011-1/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are several changes to this module: 1. The use of an explicit stack in Participable#participants 2. Proc behaviour has been changed 3. Batch permissions checking == Explicit Stack Participable#participants no longer uses recursion to process "self" and all child objects, instead it uses an Array and processes objects in breadth-first order. This allows us to for example create a single Gitlab::ReferenceExtractor instance and pass this to any Procs. Re-using a ReferenceExtractor removes the need for running potentially many SQL queries every time a Proc is called on a new object. == Proc Behaviour Changed Previously a Proc in Participable was expected to return an Array of User instances. This has been changed and instead it's now expected that a Proc modifies the Gitlab::ReferenceExtractor passed to it. The return value of the Proc is ignored. == Permissions Checking The method Participable#participants uses Ability.users_that_can_read_project to check if the returned users have access to the project of "self" _without_ running multiple SQL queries for every user.
* Codestyle: make sure we have space around operatorsGabriel Mazetto2016-05-131-1/+1
|
* Rename CiStatus to StatusableKamil Trzcinski2016-04-161-0/+8
|
* Restrict access to references for confidential issuesDouglas Barbosa Alexandre2016-03-171-1/+12
|
* Ensure we don't check a commit's description for revert message if it has no ↵Rémy Coutable2016-02-221-0/+34
| | | | description
* Tag model specsDouwe Maan2015-12-091-1/+1
|
* Merge branch 'webhook_payload_with_changes' into 'master' Valery Sizov2015-12-071-0/+11
|\ | | | | | | | | | | | | Add added, modified and removed properties to commit object in webhook https://gitlab.com/gitlab-org/gitlab-ee/issues/20 See merge request !1988
| * fixes after reviewwebhook_payload_with_changesValery Sizov2015-12-071-1/+1
| |
| * fox specsValery Sizov2015-12-071-1/+1
| |
| * Add added, modified and removed properties to commit object in webhookValery Sizov2015-12-041-0/+11
| |
* | Render commit reference using short sha, but include full sha in comment.Douwe Maan2015-12-011-3/+14
| |
* | Fix specsDouwe Maan2015-11-301-10/+6
|/
* Fix mentionable specsDouwe Maan2015-10-121-2/+2
|
* Update mock and stub syntax for specsRobert Speicher2015-06-221-3/+5
|
* Add `to_reference` for models that support referencesRobert Speicher2015-05-261-2/+22
| | | | | Now there is a single source of information for which attribute a model uses to be referenced, and its special character.
* No longer needed to pass project argument to commit methods.Douwe Maan2015-04-241-2/+2
|
* Use project.commit convenience method.Douwe Maan2015-04-241-1/+1
|
* Update mentionable shared examples to be (a bit) more understandableRobert Speicher2015-04-161-1/+1
|
* Correct usage of `subject` in specsRobert Speicher2015-04-161-1/+2
|
* Don't use HTML ellipsis in EmailsOnPush subject truncated commit message.emailsonpush-hellipDouwe Maan2015-04-141-1/+1
|
* Updated rspec to rspec 3.x syntaxJeroen van Baarsen2015-02-121-21/+21
| | | | Signed-off-by: Jeroen van Baarsen <jeroenvanbaarsen@gmail.com>
* Allow commit messages to close several issues at once (thanks @123Haynesjubianchi2015-01-201-4/+0
| | | | for his work and help)
* Use 8chars short sha for commit in viewsDmitriy Zaporozhets2014-10-101-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Use short_id instead of [0..N] for short version of commit shaDmitriy Zaporozhets2014-10-101-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
View Lorry Controller Status