summaryrefslogtreecommitdiff
path: root/spec/models/commit_spec.rb
Commit message (Collapse)AuthorAgeFilesLines
* Fix an order dependency in a specNick Thomas2018-01-291-3/+3
|
* Migrate Commit#uri_type to Gitalyfeature/migrate-commit-uri-to-gitalyAhmad Sherif2018-01-151-7/+17
| | | | Closes gitaly#915
* Display related merge requests in commit detail pageHiroyuki Sato2018-01-121-0/+13
|
* Remove the Commit#tree methodremove-commit-treeJacob Vosmaer2018-01-041-1/+0
|
* Replace '.team << [user, role]' with 'add_role(user)' in specs36782-replace-team-user-role-with-add_role-user-in-specsblackst0ne2017-12-221-2/+2
|
* Load commit in batches for pipelines#indexZeger-Jan van de Weg2017-12-191-0/+39
| | | | | | | | | | Uses `list_commits_by_oid` on the CommitService, to request the needed commits for pipelines. These commits are needed to display the user that created the commit and the commit title. This includes fixes for tests failing that depended on the commit being `nil`. However, now these are batch loaded, this doesn't happen anymore and the commits are an instance of BatchLoader.
* Optimise getting the pipeline status of commitsci-pipeline-status-queryYorick Peterse2017-11-161-2/+9
| | | | | This adds an optimised way of getting the latest pipeline status for a list of Commit objects (or just a single one).
* Prepare Repository#merge for migration to Gitalyoperation-service-mergeJacob Vosmaer2017-09-151-6/+1
|
* Improve description of a Commit#cherry_pick_message UTSaverio Miroddi2017-08-221-2/+2
|
* Fix rubocop errors in spec/models/commit_spec.rbSaverio Miroddi2017-08-211-2/+0
|
* Add UT for cherry-pick of an existing merge that is not foundSaverio Miroddi2017-08-211-19/+30
|
* Reverse order of commits in MR cherry-pick messageSaverio Miroddi2017-08-211-2/+3
| | | | Also improved the UT for better documenting this change.
* Remove redundant statement part in a test suiteSaverio Miroddi2017-08-181-1/+1
|
* Correct the cherry-pick message for merge commitsSaverio Miroddi2017-08-171-8/+35
| | | | | The list of commits must be generated from the merge request, not from a diff of the branches.
* Extended UTs for the new cherry-pick message formatSaverio Miroddi2017-08-161-0/+24
|
* Incorporate Gitaly's CommitService.FindCommit RPCgitaly-find-commitAlejandro Rodríguez2017-08-071-1/+1
|
* Refactor Gitlab::Git::Commit to include a repositoryAlejandro Rodríguez2017-08-071-2/+0
|
* Change all `:empty_project` to `:project`rs-empty_project-defaultRobert Speicher2017-08-021-1/+1
|
* Rename many path_with_namespace -> full_pathGabriel Mazetto2017-08-011-1/+1
|
* Remove superfluous lib: true, type: redis, service: true, models: true, ↵Rémy Coutable2017-07-271-1/+1
| | | | | | services: true, no_db: true, api: true Signed-off-by: Rémy Coutable <remy@rymai.me>
* Use RequestStoreWrap for Commit#authorLin Jen-Shin2017-07-181-5/+3
| | | | | | | We also try to use instance variable to cache the result if RequestStore is not available, so we could keep the same logic, using the same cache key. Also introduce a way to specify method specific cache key
* Correct RSpec/SingleLineHook cop offensesRobert Speicher2017-06-141-3/+9
|
* Fix linting, route, and specsRémy Coutable2017-06-091-1/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fix spec failures and add a feature flag for the performance barRémy Coutable2017-06-091-1/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch '12614-fix-long-message' into 'master'Dmitriy Zaporozhets2017-06-081-2/+24
|\ | | | | | | | | | | | | Fix long urls in the title of commit Closes #12614 See merge request !10938
| * Fix long urls in the title of commitAlexander Randa2017-05-121-2/+24
| |
* | Add support for deltas_only under GitalyAhmad Sherif2017-05-131-15/+0
| | | | | | | | Closes gitaly#199
* | Remove deltas_only from DiffCollectionJacob Vosmaer2017-05-111-17/+4
| |
* | Re-enable Gitaly commit_raw_diff featureAhmad Sherif2017-05-071-28/+27
| |
* | Rename latest_pipeline to last_pipelinealways-show-latest-pipeline-in-commit-boxLin Jen-Shin2017-05-031-3/+3
|/ | | | | | | | As it might be confusing that it's not the same as Pipeline#latest Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11038#note_28677090
* 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
View Lorry Controller Status