summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Fix header link rendering when containing numbersfix-markdown-renderingYorick Peterse2016-04-061-0/+8
| | | | | | | | | | | This fixes the problem where Markdown such as: ### 31st Would get rendered as a link tag pointing to issue number 31 inside a header tag. See gitlab-org/gitlab-ce#14936 for more information.
* Merge branch 'reorder-language' into 'master' Yorick Peterse2016-04-051-4/+14
|\ | | | | | | | | Update language after doing all other operations See merge request !3533
| * Update language only on HEAD of the repositoryreorder-languageKamil Trzcinski2016-04-051-4/+14
| |
* | Merge branch 'fix-gh-pr-import' into 'master' Yorick Peterse2016-04-052-0/+31
|\ \ | | | | | | | | | | | | Fix Importing repos from GHE doesn't work See merge request !3529
| * | Flush repository cache before import project datafix-gh-pr-importDouglas Barbosa Alexandre2016-04-042-0/+31
| | | | | | | | | | | | | | | GitHub Pull Requests importer handle with the repository while importing data, we need to make sure that the cached values are valid.
* | | Wrap images in discussions and wikis with a link to the image source using ↵connorshea2016-04-042-1/+25
|/ / | | | | | | | | | | | | | | ImageLinkFilter. Resolves #14411. See merge request !3464
* | Reload forked_project_link projectsYorick Peterse2016-04-041-0/+5
| | | | | | | | | | | | | | This fixes ./spec/services/projects/unlink_fork_service_spec.rb which somehow started failing on the master branch. It certainly isn't a very elegant solution but seems to be the easiest/best way of solving this problem for the time being.
* | Merge branch 'fix_remove_fork_link' into 'master' Douwe Maan2016-04-041-0/+32
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove fork link closes all merge requests opened on source project Currently, if you: - create a fork - open a merge request on the source project - remove the fork link The created MR cannot be closed. With this MR, all pending MR is closed when the fork link is removed. See merge request !3189
| * | Fix rubocop in unlink fork service specsBaldinof2016-04-031-2/+2
| | |
| * | Move unlink fork logic to a serviceBaldinof2016-03-222-19/+32
| | |
| * | Merge branch 'master' into fix_remove_fork_linkBaldinof2016-03-1435-458/+789
| |\ \
| * | | Close merge requests when removing fork relationBaldinof2016-03-111-0/+19
| | | |
* | | | Merge branch 'update_repository_from_ee' into 'master' Douwe Maan2016-04-042-4/+23
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Migrate Repository#local_branches from gitlab-ee. REF: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/249 This will help us to avoid posible merge conflicts when merging gitlab-ce to gitlab-ee See merge request !3496
| * | | | Migrate Repository#local_branches from gitlab-ee.update_repository_from_eeRubén Dávila2016-04-012-4/+23
| | | | | | | | | | | | | | | | | | | | | | | | | This will help us to avoid posible merge conflicts when merging gitlab-ce to gitlab-ee
* | | | | Fix creation of merge requests for orphaned branchesStan Hu2016-04-042-0/+29
| | | | | | | | | | | | | | | | | | | | Closes #14875
* | | | | Merge branch 'add-ability-to-archive-a-project-via-api-14296' into 'master' Douwe Maan2016-04-031-0/+72
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | Add endpoints for archiving and unarchiving Closes #14296 See merge request !3372
| * | | | Merge branch 'master' into add-ability-to-archive-a-project-via-api-14296add-ability-to-archive-a-project-via-api-14296Zeger-Jan van de Weg2016-03-2912-54/+457
| |\ \ \ \
| * \ \ \ \ Merge branch 'master' into add-ability-to-archive-a-project-via-api-14296Zeger-Jan van de Weg2016-03-242-27/+53
| |\ \ \ \ \
| * | | | | | PUT becomes POST on archiving endpointsZeger-Jan van de Weg2016-03-241-29/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also the specs have a minor improvement. Mainly the access right spec. Changes are reflected in the docs
| * | | | | | Add endpoints for archiving and unarchivingZeger-Jan van de Weg2016-03-231-0/+72
| | | | | | |
* | | | | | | Merge branch 'routing' into 'master' Robert Speicher2016-04-014-4/+4
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added & use Gitlab::Routing for URL helpers Extracted from !3389 See merge request !3486
| * | | | | | | Added & use Gitlab::Routing for URL helpersroutingYorick Peterse2016-04-014-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rails' "url_helpers" method creates an anonymous Module (which a bunch of methods) on every call. By caching the output of this method in a dedicated method we can shave off about 10 seconds of loading time for an issue with around 200 comments.
* | | | | | | | Merge branch 'fix/fogbugz-import' into 'master' Robert Speicher2016-04-011-0/+24
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | spec and fix for fogbugz lonely user problem Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/14766 I encountered this issue while manually testing all import types for https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3066 This is really due to a horrible API ``` { 'people' => { 'person' => array_of_people_or_single_person_as_hash } } ``` See merge request !3457
| * | | | | | | | updated specfix/fogbugz-importJames Lopez2016-03-311-1/+1
| | | | | | | | |
| * | | | | | | | spec and fix for fogbugz lonely user problemJames Lopez2016-03-301-0/+24
| | | | | | | | |
* | | | | | | | | Merge branch 'error-when-ldap-doesn-t-supply-an-email-address-14559' into ↵Rémy Coutable2016-04-011-0/+7
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Allow temp-oauth-email as notification email Closes #14559 /cc @dblessing @rymai See merge request !3477
| * | | | | | | | | Allow temp-oauth-email as notification emailerror-when-ldap-doesn-t-supply-an-email-address-14559Zeger-Jan van de Weg2016-03-311-0/+7
| | | | | | | | | |
* | | | | | | | | | Merge branch ↵Douwe Maan2016-04-011-8/+35
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'snippets-with-comments-cause-a-500-when-they-show-up-in-search-results-14764' into 'master' Fix Error 500 when searching for a comment in a project snippet Closes #14764. /cc @stanhu, and thank you for the spec! ;) See merge request !3468
| * | | | | | | | | | Fix view of notes in search results when noteable is a snippetsnippets-with-comments-cause-a-500-when-they-show-up-in-search-results-14764Rémy Coutable2016-03-311-8/+35
| | |_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, streamline the view. Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | | | | | Merge branch 'fix-mentions-on-confidential-issues-for-non-members' into ↵Douwe Maan2016-04-011-21/+56
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Mentions on confidential issues doesn't create todos for non-members Closes #14569 See merge request !3374
| * | | | | | | | | | Mentions on confidential issues doesn't create todos for non-membersfix-mentions-on-confidential-issues-for-non-membersDouglas Barbosa Alexandre2016-03-301-21/+56
| | | | | | | | | | |
* | | | | | | | | | | Merge branch 'fix/issue-move-rewrite-uploads' into 'master' Douwe Maan2016-04-014-4/+112
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rewrite uploads when moving issue to another project Closes #14531 See merge request !3382
| * | | | | | | | | | | Fix rubocop offenses in upload rewriter specsfix/issue-move-rewrite-uploadsGrzegorz Bizon2016-03-311-1/+1
| | | | | | | | | | | |
| * | | | | | | | | | | Merge branch 'master' into fix/issue-move-rewrite-uploadsGrzegorz Bizon2016-03-3114-79/+345
| |\ \ \ \ \ \ \ \ \ \ \ | | | |/ / / / / / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (27 commits) Fix commit comment alignment minor cleanup in system_hook_spec Pre-calculate Emoji digests Clear .todo listener Change window.location to use turbolinks Make entire todo row clickable Add 8.6.2 CHANGELOG items Ensure uploads dir exists when running backup specs Move CarrierWave test env config to separate file Remove console logs Off the event initially Collapsed sidebar opens over instead of pushing content. Sidebar collapse update issue User selection from collapsed sidebar Add json response for user avatar in merge request Make changed values visible in minimized sidebar. Fixed MergeRequestController spec We need `sha` reference from `diff_base_commit` to generate the diff Use `diff_base_commit` instead of `target_branch` to generate diffs Isolate CarrierWave uploads in test enviroment ...
| * | | | | | | | | | | Improve specs for issue move serviceGrzegorz Bizon2016-03-311-2/+2
| | | | | | | | | | | |
| * | | | | | | | | | | Extend specs for GFM uploads rewriterGrzegorz Bizon2016-03-312-11/+32
| | | | | | | | | | | |
| * | | | | | | | | | | Remove reduntant `move_to_store` overrideGrzegorz Bizon2016-03-301-4/+4
| | | | | | | | | | | |
| * | | | | | | | | | | Add support for not Active Record based factoriesGrzegorz Bizon2016-03-301-4/+12
| | | | | | | | | | | |
| * | | | | | | | | | | Refactor uploads rewriter used when moving issueGrzegorz Bizon2016-03-302-9/+4
| | | | | | | | | | | |
| * | | | | | | | | | | Get FileUploader into test harness using factoryGrzegorz Bizon2016-03-302-4/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This attempts to get CarrierWave's uploader - `FileUploader` into test harness using a factory. that makes it easier to build an instance of it. Along with !3435 it may be easier to use uploaders in tests
| * | | | | | | | | | | Add uploads rewriter and use it when moving issueGrzegorz Bizon2016-03-302-0/+68
| | | | | | | | | | | |
* | | | | | | | | | | | Remove "Congratulations!" tweet button on newly-created project.connorshea2016-03-311-1/+0
| |_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I’ve removed everything related to the feature based on this commit: ce08f919f34fd8849834365 Resolves #10857.
* | | | | | | | | | | Merge branch 'clean-up-user-controller-spec' into 'master' Robert Speicher2016-03-311-12/+3
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Minor clean up on admin/users_controller_spec Extracted from !3439 See merge request !3460
| * | | | | | | | | | | Minor clean up on admin/users_controller_specZeger-Jan van de Weg2016-03-311-12/+3
| | | | | | | | | | | |
* | | | | | | | | | | | Exclude projects pending deletion from all resultsZeger-Jan van de Weg2016-03-311-0/+9
| |_|_|_|/ / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge branch 'fix-todos' into 'master' Rémy Coutable2016-03-311-0/+1
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Destroy related todos when an Issue/MR is deleted Closes #14550 Closes #14598 See merge request !3376
| * | | | | | | | | | Destroy related todos when an issue is deletedDouglas Barbosa Alexandre2016-03-301-0/+1
| | |/ / / / / / / / | |/| | | | | | | |
* | | | | | | | | | Merge branch 'reorder-project-expire-cache' into 'master' Yorick Peterse2016-03-301-1/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Expire the exists cache before deletion to ensure project dir actually exists See merge request !3413
| * | | | | | | | | | Expire the exists cache before deletion to ensure project dir actually existsStan Hu2016-03-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #14659
* | | | | | | | | | | Merge branch 'check-project-repo-exists' into 'master' Yorick Peterse2016-03-301-0/+27
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check if repo exists before attempting to update cache info See merge request !3414