summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Fix shoulda-matchers requireRobert Speicher2015-06-101-0/+1
|
* Memoize result of JSON.parse in json_responseRobert Speicher2015-06-101-1/+1
| | | | | This might see a minor speedup in test cases that call this method many times.
* Add spec/support files for WebMock and test coverageRobert Speicher2015-06-103-12/+12
|
* "expect { }.not_to raise_error" no longer takes a specific classRobert Speicher2015-06-101-6/+2
|
* ensure_length_of -> validate_length_ofRobert Speicher2015-06-106-11/+11
|
* Correctly remove already-mentioned commits for cross referencesrs-issue-1778Robert Speicher2015-06-081-2/+19
| | | | Fixes #1778
* Merge branch 'cernvcs/gitlab-ce-feature/auto_link_ldap_omniauth'Douwe Maan2015-06-051-12/+154
|\
| * Add option to automatically link omniauth and LDAP identitiesAlex Lossent2015-06-031-12/+154
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Until now, a user needed to first sign in with his LDAP identity and then manually link his/her account with an omniauth identity from their profile. Only when this is done can the user authenticate with the omniauth provider and at the same time benefit from the LDAP integration (HTTPS authentication with LDAP username/password and in EE: LDAP groups, SSH keys etc.). This feature automates the process by looking up a corresponding LDAP person when a user connects with omniauth for the first time and then automatically linking the LDAP and omniauth identities (of course, like the existing allow_single_sign_on setting, this is meant to be used with trusted omniauth providers). The result is identical to a manual account link. Add config initializers for other omniauth settings.
* | Merge branch 'maser/gitlab-ce-order-commit-comments-in-api' into 'master'Dmitriy Zaporozhets2015-06-051-1/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Order commit comments in API chronologically When fetching commit comments via API, the comments were not ordered, but just returned in the order Postgresql finds them. Now the API always returns comments in chronological order. Same as !628 but with CI See merge request !768
| * | Order commit comments in API chronologicallyMartin Luder2015-05-081-1/+2
| | | | | | | | | | | | | | | | | | When fetching commit comments via API, the comments were not ordered, but just returned in the order Postgresql finds them. Now the API always returns comments in chronological order.
* | | Merge branch 'rs-remove-guard' into 'master'Dmitriy Zaporozhets2015-06-041-1/+0
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove Guard None of the GitLab B.V. developers were using it. See internal https://dev.gitlab.org/gitlab/gitlabhq/issues/2360 See merge request !764
| * | | Remove unnecessary require from RepositoryCache specRobert Speicher2015-06-041-1/+0
| | | |
* | | | Merge branch 'fix-upgrader-script' into 'master'Dmitriy Zaporozhets2015-06-041-0/+15
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix upgrader script This is a fix for upgrader script not guessing the latest version correctly. Upgrader now handles versions where a version part (major/minor/patch) can have multi-digit number, also ensures that the latest version is chosen from git tags by converting tag to Gitlab::VersionInfo and than selecting the latest/greatest version. Fixes: #1476 See merge request !695
| * | | | Update mocking/stubbing syntax to the new RSpec 3 syntaxMartins Polakovs2015-05-301-1/+1
| | | | |
| * | | | Fix upgrader scriptMartins Polakovs2015-05-231-0/+15
| | | | |
* | | | | Merge branch 'make-namespaces-api-available-to-all-users' into 'master'Dmitriy Zaporozhets2015-06-042-1/+30
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make namespace API available to all users ### What does this MR do? This MR makes it possible for a user to query namespaces to which he/she has access. Also, it adds documentation for the existing API. ### Why was this MR needed? Even though the `groups` API exists, it might still be useful to have an endpoint that tells the namespace type (e.g. `user` vs. `group`), especially if a user has access to a number of different projects. ### What are the relevant issue numbers? Closes https://github.com/gitlabhq/gitlabhq/issues/9328 See merge request !708
| * | | | | Make namespace API available to all usersStan Hu2015-05-282-1/+30
| | | | | | | | | | | | | | | | | | | | | | | | Closes https://github.com/gitlabhq/gitlabhq/issues/9328
* | | | | | Remove show actions from Admin and Project DeployKeysRobert Speicher2015-06-031-1/+1
| |_|/ / / |/| | | |
* | | | | Merge branch 'fix-group-remove' into 'master'Dmitriy Zaporozhets2015-06-031-0/+44
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Group improvements * remove projects before removing group * execute all hooks/events from project destroy when group removed * log group create/remove * delay remove of namespace directory (to prevent NFS issues) Inspired by !759 See merge request !761
| * | | | | Skip repo removing whem remove user or groupDmitriy Zaporozhets2015-06-031-0/+44
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | Merge branch 'repo-remove' into fix-group-removeDmitriy Zaporozhets2015-06-033-14/+36
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: spec/features/projects_spec.rb
* | \ \ \ \ \ Merge branch 'repo-remove'Dmitriy Zaporozhets2015-06-033-14/+36
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | / | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: spec/features/projects_spec.rb
| * | | | | Fix tests and increase delay time before remove repositoryDmitriy Zaporozhets2015-06-032-16/+2
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | | Add tests for project destroy servicerepo-removeDmitriy Zaporozhets2015-06-031-0/+34
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | | Add JS specs for Array extensionsRobert Speicher2015-06-021-0/+12
| | | | | |
* | | | | | Move jQuery enable/disable extensions to extensions/jqueryRobert Speicher2015-06-022-9/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes redundant enableButton/disableButton extensions, and adds specs for the jQuery extensions.
* | | | | | Add ZenMode javascript specsRobert Speicher2015-06-022-0/+61
| |/ / / / |/| | | |
* | | | | Merge branch 'ignore-references' into 'master'Dmitriy Zaporozhets2015-06-022-6/+39
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't notify users mentioned in code blocks or blockquotes. cc @rspeicher See merge request !753
| * | | | | Fix spec.ignore-referencesDouwe Maan2015-06-021-6/+15
| | | | | |
| * | | | | Ignore references in blockquotes.Douwe Maan2015-06-021-0/+4
| | | | | |
| * | | | | Actually ignore references in code blocks etc.Douwe Maan2015-06-021-0/+20
| | | | | |
* | | | | | Further limit the limited whitelist for project/group descriptionsrs-more-nofollowRobert Speicher2015-06-021-2/+17
| | | | | |
* | | | | | Rename ReferenceFilterSpecHelper to FilterSpecHelperRobert Speicher2015-06-0215-61/+56
| | | | | | | | | | | | | | | | | | | | | | | | And make it more generalized for all filter specs.
* | | | | | Add feature specs for Project and Group description renderingRobert Speicher2015-06-023-12/+82
| | | | | |
* | | | | | Add a `pipeline` context option for SanitizationFilterRobert Speicher2015-06-021-0/+14
| |/ / / / |/| | | | | | | | | | | | | | | | | | | When this option is `:description`, we use a more restrictive whitelist. This is used for Project and Group description fields.
* | | | | GitLab CI service sends gitlab-ci.yml fileci_yml_fileValery Sizov2015-06-021-0/+15
|/ / / /
* | | | Merge branch 'web-editor-rugged' into 'master'Dmitriy Zaporozhets2015-06-021-46/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create and edit files in web editor via rugged - [x] create file via rugged - [x] update file via rugged - [x] remove file via rugged - [ ] fix tests - [x] remove satellites code - [x] create activity event for new/edit file via rugged - [x] base64 support Part of https://dev.gitlab.org/gitlab/gitlabhq/issues/2300 See merge request !751
| * | | | Remove unnecessary satellite files and add CHANGELOG itemDmitriy Zaporozhets2015-06-021-14/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | Remove file api tests which depend on old satellite logicDmitriy Zaporozhets2015-06-021-32/+3
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | Merge branch 'rs-issue-1721' into 'master'Douwe Maan2015-06-021-0/+6
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix link_to_gfm with only a reference having the incorrect link Before: ```ruby link_to_gfm("#4", "/foo") # => "<a href="/namespace/project/issues/4" class="gfm gfm-issue">#4</a>" link_to_gfm("See #4", "/foo") # => "<a href="/foo">See </a><a href="/namespace/project/issues/4" class="gfm gfm-issue">#4</a>" ``` After: ```ruby link_to_gfm("#4", "/foo") # => "<a href="/foo">#4</a>" link_to_gfm("See #4", "/foo") # => "<a href="/foo">See </a><a href="/namespace/project/issues/4" class="gfm gfm-issue">#4</a>" ``` Closes #192 Closes #1721 Closes [GitHub #7115](https://github.com/gitlabhq/gitlabhq/issues/7115) See merge request !739
| * | | | Fix link_to_gfm with only a reference having the incorrect linkrs-issue-1721Robert Speicher2015-05-291-0/+6
| | | | | | | | | | | | | | | | | | | | Closes #1721
* | | | | Disable changing of the source branch in merge request update APIStan Hu2015-05-291-2/+2
| | | | |
* | | | | Merge branch 'support-edit-target-branch-in-mr' into 'master'Dmitriy Zaporozhets2015-05-292-1/+24
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support editing target branch of merge request ### What does this MR do? This MR makes it possible to edit the target branch of a merge request and adds a system note when this happens. ### Why was this MR needed? Because lots of people requested this feature. :) ### Screenshots **Edit MR page** ![image](https://gitlab.com/gitlab-org/gitlab-ce/uploads/9b3d405bf7b5f945e35bae3534c2b67b/image.png) **New MR page** ![image](https://gitlab.com/gitlab-org/gitlab-ce/uploads/3657a2a9efad6d10e8470637d1166bdb/image.png) **System note** ![image](https://gitlab.com/gitlab-org/gitlab-ce/uploads/cc8066f3d3bdf09c0cce27193210567d/image.png) ### What are the relevant issue numbers? * Closes https://github.com/gitlabhq/gitlabhq/issues/7105 * See: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/130! See merge request !738
| * | | | | Support editing target branch of merge requestStan Hu2015-05-292-1/+24
| | |/ / / | |/| | | | | | | | | | | | | | | | | | Closes https://github.com/gitlabhq/gitlabhq/issues/7105 See: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/130
* | | | | Merge branch 'rs-teaspoon' into 'master'Dmitriy Zaporozhets2015-05-2912-89/+292
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add Teaspoon for Javascript testing Looking to expand our Javascript unit tests, and this gem makes things a bit better in that respect. See https://github.com/modeset/teaspoon See merge request !715
| * | | | | Stub ajax in JS specsRobert Speicher2015-05-282-2/+5
| | | | | |
| * | | | | Remove jasmine-fixture, use teaspoon fixturesRobert Speicher2015-05-288-58/+64
| | | | | |
| * | | | | teaspoon installRobert Speicher2015-05-282-0/+224
| | | | | |
| * | | | | Remove jasmine-rails; add teaspoonRobert Speicher2015-05-282-30/+0
| | |/ / / | |/| | |
* | | | | Add MergeRequests#commits action and routeRobert Speicher2015-05-291-12/+20
|/ / / / | | | | | | | | | | | | /:namespace_id/:project_id/merge_requests/:id/commits(.:format)