summaryrefslogtreecommitdiff
path: root/spec/lib
Commit message (Collapse)AuthorAgeFilesLines
* Fix specslabel-tooltipRémy Coutable2016-03-181-1/+1
|
* Return an empty Array when there aren't lines to parse.Rubén Dávila2016-03-151-0/+5
|
* Merge branch 'rs-issue-14220' into 'master' Douwe Maan2016-03-151-0/+20
|\ | | | | | | | | | | | | | | | | | | Be more intelligent about sanitizing links with unsafe protocols This prevents false matches on relative links like `[database](database.md)`. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/14220 See merge request !3210
| * Be more intelligent about sanitizing links with unsafe protocolsrs-issue-14220Robert Speicher2016-03-141-0/+20
| | | | | | | | | | | | | | This prevents false matches on relative links like `[database](database.md)`. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/14220
* | Merge branch 'gitlab-ci-yaml-updates' into 'master' Douwe Maan2016-03-141-1/+83
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | New CI YAML features This introduces a couple of small `.gitlab-ci.yml` features: 1. Documentation for: Allow to use YAML anchors when parsing the `.gitlab-ci.yml`: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/2958 2. Ignore jobs that start with `.` 3. Allow to pass name of created artifacts archive in `.gitlab-ci.yml` 4. Allow to define on which builds the current one depends on These are really small changes so it makes not sense to create a separate merge requests for them. @axil Could you review the documentation part? The implementation on GitLab Runner side: https://gitlab.com/gitlab-org/gitlab-ci-multi-runner/merge_requests/113. Fixes: https://gitlab.com/gitlab-org/gitlab-ce/issues/13755 https://gitlab.com/gitlab-org/gitlab-ce/issues/14211 https://gitlab.com/gitlab-org/gitlab-ce/issues/3423 cc @grzesiek @axil @DouweM See merge request !3182
| * Merge branch 'master' into gitlab-ci-yaml-updatesAchilleas Pipinellis2016-03-124-38/+110
| |\
| * \ Merge remote-tracking branch 'origin/master' into gitlab-ci-yaml-updatesKamil Trzcinski2016-03-111-0/+39
| |\ \ | | | | | | | | | | | | | | | | # Conflicts: # spec/lib/ci/gitlab_ci_yaml_processor_spec.rb
| * | | Allow to define on which builds the current one depends onKamil Trzcinski2016-03-111-0/+45
| | | |
| * | | Allow to pass name of created artifacts archive in `.gitlab-ci.yml`Kamil Trzcinski2016-03-111-1/+9
| | | |
| * | | Describe special YAML features: the use of anchors and hidden jobsKamil Trzcinski2016-03-111-0/+29
| | | |
* | | | Revert "Revert "Merge branch 'support-go-subpackages' into 'master' ""Stan Hu2016-03-121-0/+30
| | | | | | | | | | | | This reverts commit 5a586f364c5d2b866d6a074eff7996f05585b7d5
* | | | Revert "Merge branch 'support-go-subpackages' into 'master' "Douwe Maan2016-03-121-30/+0
| | | | | | | | | | | | This reverts merge request !3191
* | | | Merge branch 'support-go-subpackages' into 'master' Douwe Maan2016-03-121-0/+30
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | Support Golang subpackage fetching Closes #13805 See merge request !3191
| * | | Add spec for go-import middlewareStan Hu2016-03-121-0/+30
| |/ /
* | | Merge branch 'fix-gh-pr-import' into 'master' Douwe Maan2016-03-121-36/+28
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix importing PR's from GitHub when the source repo was removed Closes #13847 Closes gitlab-com/support-forum#584 See merge request !3172
| * | | Fix importing PR's from GitHub when the source repo was removedDouglas Barbosa Alexandre2016-03-101-36/+28
| | | |
* | | | Fixed ProjectSearchResults spec to use a ProjectYorick Peterse2016-03-111-2/+2
| | | | | | | | | | | | | | | | | | | | This spec was still passing an ID to the #initialize method instead of a Project instance.
* | | | Refactor Gitlab::SnippetSearchResultsYorick Peterse2016-03-111-0/+25
| | | | | | | | | | | | | | | | This removes the need for plucking snippet IDs into memory.
* | | | Refactor Gitlab::SearchResultsYorick Peterse2016-03-111-0/+55
| |_|/ |/| | | | | | | | | | | | | | | | | Instead of plucking IDs this class now uses ActiveRecord::Relation objects. Plucking IDs is problematic as searching for projects can lead to a huge amount of IDs being loaded into memory only to be used as an argument for another query (instead of just using a sub-query).
* | | Merge branch 'gitlab-ci-yaml-alias' into 'master' Kamil Trzciński2016-03-111-0/+39
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support YAML alias/anchor usage in .gitlab-ci.yml This allows to reuse one job as a template for another one: ``` job1: &JOBTMPL script: execute-script-for-job job2: *JOBTMPL ``` This also helps to solve some of the issues in gitlab-org/gitlab-ci#342 See merge request !2958
| * | Support YAML alias/anchor usage in .gitlab-ci.ymlPascal Bach2016-03-081-0/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows to reuse one job as a template for another one: ``` job1: &JOBTMPL script: execute-script-for-job job2: *JOBTMPL ``` This also helps to solve some of the issues in #342 Signed-off-by: Pascal Bach <pascal.bach@siemens.com> Signed-off-by: Fabio Huser <fabio.huser@siemens.com>
* | | Improve test descriptionsJacob Vosmaer2016-03-101-4/+4
| | |
* | | Add tests for ExclusiveLeaseJacob Vosmaer2016-03-101-0/+21
| |/ |/|
* | Fix bug where Bitbucket `closed` issues were imported as `opened`iurisilvio/gitlab-ce-bitbucket_closedIuri de Silvio2016-03-071-0/+88
|/
* Merge branch 'feature/cross-project-labels' into 'master' Douwe Maan2016-03-071-1/+26
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add support for cross project references for labels ## Summary Support for cross project references for labels. ## Rationale 1. Cross project label references are currently not supported in GitLab 1. `to_reference` method signature in `Label` model breaks the abstraction introduced in `Referable`. `concerns/referable.rb: def to_reference(_from_project = nil)` Signatures: ``` label.rb: def to_reference(format = :id) commit_range.rb: def to_reference(from_project = nil) commit.rb: def to_reference(from_project = nil) external_issue.rb: def to_reference(_from_project = nil) group.rb: def to_reference(_from_project = nil) issue.rb: def to_reference(from_project = nil) merge_request.rb: def to_reference(from_project = nil) milestone.rb: def to_reference(from_project = nil) project.rb: def to_reference(_from_project = nil) snippet.rb: def to_reference(from_project = nil) user.rb: def to_reference(_from_project = nil) ``` This MR suggests using `def to_reference(from_project = nil, format: :id)` which makes use of keyword arguments and preserves abstract interface. 1. We need support for cross project label references when we want to move issue to another project It may happen that issue description, system notes or comments contain reference to label and this reference will be invalid after moving issue to another project and will not be displayed correctly unless we have support for cross project references. Merge request that needs this feature: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/2831 I think that cross project label references may be useful, (example: `Hey, see our issues for CI in GitLab CE! - gitab-org/gitlab-ce~"CI"`). cc @JobV @DouweM @rspeicher See merge request !2966
| * Change content of cross project labelGrzegorz Bizon2016-03-031-3/+3
| |
| * Improve cross project label look and feelGrzegorz Bizon2016-03-031-2/+8
| |
| * Add support for cross project references for labelsGrzegorz Bizon2016-03-031-0/+19
| |
| * Add support for keyword arguments in label reference methodGrzegorz Bizon2016-03-031-1/+1
| |
* | Merge branch 'rs-frontmatter-pre' into 'master' Robert Speicher2016-03-051-0/+53
|\ \ | | | | | | | | | | | | Properly display YAML front matter in Markdown See merge request !3072
| * | Add YamlFrontMatterFilter to the PreProcessPipelineRobert Speicher2016-03-041-0/+53
| |/ | | | | | | | | This filter will detect YAML Front Matter and convert it to an HTML table for prettier formatting.
* | Merge branch 'rs-wiki-pipeline-spec' into 'master' Douwe Maan2016-03-042-44/+60
|\ \ | | | | | | | | | | | | | | | | | | | | | Add a spec for WikiPipeline Removes the specs from GollumTagsFilter that were more like integration tests for the pipeline than unit tests of the filter. See merge request !3054
| * | Replace `[[_TOC_]]` tag even if `toc` result is blankrs-wiki-pipeline-specRobert Speicher2016-03-032-3/+3
| | |
| * | Add a spec for WikiPipelineRobert Speicher2016-03-012-44/+60
| | | | | | | | | | | | | | | Removes the specs from GollumTagsFilter that were more like integration tests for the pipeline than unit tests of the filter.
* | | Use Gitlab::Git::DiffCollectionsJacob Vosmaer2016-03-032-2/+2
| |/ |/|
* | Add FilterArray class to Banzairs-filter-arrayRobert Speicher2016-03-011-0/+39
|/
* Merge branch 'rs-wiki-toc' into 'master' Douwe Maan2016-03-011-0/+52
|\ | | | | | | | | | | | | Replace Gollum `[[_TOC_]]` tag with result of TableOfContentsFilter Closes #2494 See merge request !2952
| * Replace Gollum `[[_TOC_]]` tag with result of TableOfContentsFilterrs-wiki-tocRobert Speicher2016-02-231-0/+52
| | | | | | | | Closes #2494
* | Merge branch 'rs-data-links' into 'master' Douwe Maan2016-02-291-1/+15
|\ \ | |/ |/| | | | | | | | | | | | | Sanitize `data` and `vbscript` links Closes #13625 Closes https://dev.gitlab.org/gitlab/gitlabhq/issues/2660 See merge request !2905
| * Sanitize `vbscript:` linksrs-data-linksRobert Speicher2016-02-231-0/+7
| | | | | | | | Closes https://dev.gitlab.org/gitlab/gitlabhq/issues/2660
| * Sanitize `data:` linksRobert Speicher2016-02-211-1/+8
| | | | | | | | Closes #13625
* | Extend specs for summary build statusci/extend-summary-status-specsGrzegorz Bizon2016-02-201-21/+51
|/ | | | | | | Additional specs cover a case when one of the builds is running but allowed to fail. Previous implementation returned `running` status, which is wrong. This behavior has been changed in 8.5, this commit adds missing specs.
* Merge branch 'commit-status-fix' into 'master' Kamil Trzciński2016-02-191-25/+29
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix missing ignored? for GenericCommitStatus This is pretty important fix. We should not release 8.5 without this. It critical error when handling GenericCommitStatus: ``` undefined method `ignored?' for #<GenericCommitStatus:0x007fe1a2a20fe0> ``` cc @grzesiek See merge request !2899
| * Fix missing ignored? for GenericCommitStatuscommit-status-fixKamil Trzcinski2016-02-191-25/+29
| |
* | Merge branch 'saml-decoupling' into 'master'Patricio Cano2016-02-192-3/+294
|\ \ | |/ |/|
| * Make new `allow_single_sign_on` feature backwards compatiblePatricio Cano2016-02-182-3/+23
| |
| * Decouple SAML authentication from the default Omniauth logicPatricio Cano2016-02-182-3/+274
| |
* | Merge branch 'rs-emoji' into 'master' Douwe Maan2016-02-191-2/+2
|\ \ | |/ |/| | | | | | | | | | | | | Update Gemojione for new hotness :sparkles: Before | After ------ | ----- ![Screen_Shot_2016-02-12_at_3.38.43_PM](/uploads/299e5ffe38c91657085741261a678f81/Screen_Shot_2016-02-12_at_3.38.43_PM.png) | ![Screen_Shot_2016-02-12_at_3.38.09_PM](/uploads/ccee29719a6445a9efd2498391a7eea0/Screen_Shot_2016-02-12_at_3.38.09_PM.png) See merge request !2800
| * Fix specs expecting `emoji/` in image pathRobert Speicher2016-02-131-2/+2
| |
* | Revert "Merge branch 'saml-decoupling' into 'master' "Douwe Maan2016-02-182-274/+3
| | | | | | | | | | This reverts commit c04e22fba8d130a58f498ff48127712d7dae17ee, reversing changes made to 0feab326d52222dc0ab5bd0a6b15dab297f44aa9.