summaryrefslogtreecommitdiff
path: root/spec/models
Commit message (Collapse)AuthorAgeFilesLines
* Invalidate stored service password if the endpoint URL is changedreset_service_passwordValery Sizov2015-10-123-0/+135
|
* Merge remote-tracking branch 'public/project-find-with-namespace-performance'Dmitriy Zaporozhets2015-10-081-0/+189
|\ | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * Revamp finding projects by namespacesYorick Peterse2015-10-081-15/+28
| | | | | | | | | | | | | | By using a JOIN we can remove the need for using 2 separate queries to find a project by its namespace. Combined with an index (only needed for PostgreSQL) this reduces the query time from ~245 ms (~520 ms for the first call) down to roughly 10 ms (~15 ms for the first call).
| * Added concern for case-insensitive WHERE queriesYorick Peterse2015-10-071-0/+176
| | | | | | | | | | | | On PostgreSQL these queries use LOWER(...) to compare columns and values. For MySQL a regular WHERE is performed as MySQL is already case-insensitive.
* | Merge remote-tracking branch 'public/trending-projects-performance'Dmitriy Zaporozhets2015-10-081-0/+38
|\ \ | |/ |/|
| * Revamp trending projects queryYorick Peterse2015-10-061-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes the query to use a COUNT nested in an INNER JOIN, instead of a COUNT plus a GROUP BY. There are two reasons for this: 1. Using a COUNT in an INNER JOIN can be quite a bit faster. 2. The use of a GROUP BY means that method calls such as "any?" (and everything else that calls "count") operate on a Hash that counts the amount of notes on a per project basis, instead of just counting the total amount of projects. The query has been moved into Project.trending as its logic is simple enough. As a result of this testing the TrendingProjectsFinder class simply involves testing if the right methods are called, removing the need for setting up database records.
* | Fix routing in CI mailerDmitriy Zaporozhets2015-10-071-2/+1
|/ | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix GitLabCiService and remove ci_yaml_file from CI push dataKamil Trzcinski2015-10-051-4/+4
|
* Add stage testsKamil Trzcinski2015-10-051-0/+34
|
* Fix rest of testsKamil Trzcinski2015-10-052-126/+54
|
* Fix build pipeliningKamil Trzcinski2015-10-051-0/+55
|
* Fix next round of testsKamil Trzcinski2015-10-055-25/+17
|
* Make commit_spec runKamil Trzcinski2015-10-053-97/+76
|
* Merge remote-tracking branch 'upstream/master'Guilherme Garnier2015-10-034-23/+33
|\
| * Merge branch 'access_level_badge_bug' into 'master' Dmitriy Zaporozhets2015-10-021-0/+12
| |\ | | | | | | | | | | | | | | | | | | Fix: Wrong access level badge on MR comments https://gitlab.com/gitlab-org/gitlab-ce/issues/2654 See merge request !1501
| | * Wrong access level badge on MR commentsaccess_level_badge_bugValery Sizov2015-10-021-0/+12
| | |
| * | Merge branch 'rs-throttle-reset' into 'master' Douwe Maan2015-10-021-0/+20
| |\ \ | | |/ | |/| | | | | | | | | | | | | Throttle "Forgot your password?" emails Addresses internal https://dev.gitlab.org/gitlab/gitlabhq/issues/2611 See merge request !1476
| | * Conform to spec guidelines that only exist in my headrs-throttle-resetRobert Speicher2015-10-021-1/+1
| | | | | | | | | | | | [ci skip]
| | * Add User#recently_sent_password_reset?Robert Speicher2015-10-011-0/+20
| | |
| * | Fix testsDmitriy Zaporozhets2015-10-021-21/+0
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Prevent creating 2 Ci::Project entities when enable CIDmitriy Zaporozhets2015-10-021-2/+1
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | Fix rubocop warnings in spec/modelsGuilherme Garnier2015-10-0311-19/+14
|/ /
* | Merge branch 'disable-report-button-if-already-reported' into 'master' Douwe Maan2015-10-012-0/+13
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disable the "Report abuse" button if a user has already been reported Hello, I've implemented the feature request #2330. Here is what it looks like: ![report-abuse-button-disabled](https://gitlab.com/rymai/gitlab-ce/uploads/420d51906eac1c40c50701a0a340474f/report-abuse-button-disabled.png) I hope that's an acceptable solution. cc @DouweM See merge request !1456
| * | Streamline the "Report button"rymai/gitlab-ce-disable-report-button-if-already-reportedRémy Coutable2015-09-292-1/+13
| | | | | | | | | | | | | | | | | | This simplifies the "Report button" to not use open a dropdown and adds a tooltip on this button. This also removes an extra spec and adds missing specs.
| * | Disable the "Report abuse" button if a user has already been reportedRémy Coutable2015-09-291-0/+1
| | |
* | | Merge branch 'ci-fixes' into 'master' Kamil Trzciński2015-10-011-0/+11
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CI: Fix clone url and fix project token removal - Fixes broken CI clone url on dev.gitlab.org - Fixes regression in CI token removal /cc @vsizov @dzaporozhets See merge request !1481
| * | | Fix: CI token removal regression from build traceci-fixesKamil Trzcinski2015-10-011-0/+11
| | |/ | |/|
* | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ceDouwe Maan2015-10-011-8/+63
|\ \ \ | |/ /
| * | Delegate ci_project parameters to projectsKamil Trzcinski2015-09-301-8/+63
| | | | | | | | | | | | | | | | | | | | | - It delegates name, path, gitlab_url, ssh_url_to_repo - Remove ability to set this parameters using CI API This fixes GitLab project rename, namespace change, repository rename, etc.
* | | Merge branch 'master' into flevour/gitlab-ce-project-path-insensitive-lookupDouwe Maan2015-10-0116-118/+107
|\ \ \ | |/ /
| * | Make ensure_gitlab_ci_project return ci_project or create a new oneci-commits-to-projectsKamil Trzcinski2015-09-292-5/+5
| | |
| * | Fix testsKamil Trzcinski2015-09-291-3/+5
| | |
| * | Rename commits to ci_commitsKamil Trzcinski2015-09-291-2/+2
| | |
| * | Fix testsKamil Trzcinski2015-09-291-7/+4
| | |
| * | Fix testsKamil Trzcinski2015-09-297-37/+46
| | |
| * | WIPKamil Trzcinski2015-09-297-24/+32
| |/
| * Merge branch 'remove-routes-helper' into 'master' Dmitriy Zaporozhets2015-09-241-2/+2
| |\ | | | | | | | | | | | | | | | | | | | | | | | | Fix Slack notification URL and remove the usage of Ci::RoutesHelper Fix for #2644 /cc @dzaporozhets @rspeicher See merge request !1391
| | * Fix GitlabCiService specsremove-routes-helperKamil Trzcinski2015-09-221-2/+2
| | |
| * | Enable CI for gitlab when .gitlab-ci.yml is pushedDmitriy Zaporozhets2015-09-231-0/+10
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Show CI status on all pages where commits list is renderedDmitriy Zaporozhets2015-09-231-0/+10
| |/ | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * Ensure raise_error expectations provide argumentsrs-fix-spec-warningsRobert Speicher2015-09-212-2/+4
| |
| * Merge branch 'jwt' into 'master' Douwe Maan2015-09-201-3/+0
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed format validation **What does this MR do?** Removals format validation for drone token **Are there points in the code the reviewer needs to double check?** Nothing **Why was this MR needed?** because drone migrated to JWT tokens and JWT token doesn't valid with current format ( i don't know how many symbols JWT can use for this ) See merge request !1359
| | * Removed format validationKirilll Zaitsev2015-09-191-3/+0
| | |
| * | Devise 3.5.x deprecates `confirm!` in favor of `confirm`Robert Speicher2015-09-191-1/+1
| |/
| * remove API calls from CE to CIci_closer_integrationValery Sizov2015-09-181-33/+4
| |
| * Fix CI status in the MR pageValery Sizov2015-09-182-22/+5
| |
* | Make Project#find_with_namespace case-insensitiveFrancesco Levorato2015-09-181-0/+1
|/
* add repository field to issue hook dataJungkook Park2015-09-171-0/+16
| | | | | | add a test for to_hook_data of issue model update CHANGELOG
* Make rubocop happyKamil Trzcinski2015-09-161-1/+1
|
* Fix CI testsKamil Trzcinski2015-09-161-2/+2
|
View Lorry Controller Status